Add Meta.table_name to all models missing it #72

Open
opened 2026-03-27 04:06:28 +00:00 by cal · 0 comments
Owner

Problem

Only 6 of ~25 model classes define table_name in their Meta class. All others rely on Peewee's default lowercasing. This violates the project's own documented convention ("always specify Meta.table_name to match PostgreSQL naming").

Any future class rename will silently break the table mapping.

Fix

Add explicit table_name to all model Meta classes in app/db_engine.py.

Severity

High — fragile mapping, violates project convention.

## Problem Only 6 of ~25 model classes define `table_name` in their `Meta` class. All others rely on Peewee's default lowercasing. This violates the project's own documented convention ("always specify `Meta.table_name` to match PostgreSQL naming"). Any future class rename will silently break the table mapping. ## Fix Add explicit `table_name` to all model `Meta` classes in `app/db_engine.py`. ## Severity High — fragile mapping, violates project convention.
Claude added the
ai-working
label 2026-03-27 09:31:13 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/major-domo-database#72
No description provided.