Feat/cli autogen - #95
Conversation
…race, scoped clearClients
…eAITokenUsageDataSchema
0xanshu/project
…update proto submodule
Greptile SummaryThis PR adds project-scoped onboarding, authentication, payments, and event querying. The main changes are:
Confidence Score: 3/5This PR needs a database migration and backfill before it is safe to merge. The main project-scoping paths are consistent, but the schema adds required
What T-Rex did
Important Files Changed
|
| projectId: uuid("project_id") | ||
| .references(() => projectsTable.id) | ||
| .notNull(), |
There was a problem hiding this comment.
Missing migration path
This PR adds required project_id foreign keys to existing tables, including users.project_id, without a migration or backfill in the changeset. Existing deployments already have rows in users, api_keys, event tables, and metadata; applying this schema as-is requires a non-null project value for every existing row, so the upgrade can fail until a project row is created and existing data is backfilled.
Artifacts
Repro: schema-upgrade harness that applies required users.project_id to historical rows
- Contains supporting evidence from the run (text/javascript; charset=utf-8).
Repro: equivalent SQL for the historical users row and required project_id addition
- Contains supporting evidence from the run (text/x-sql; charset=utf-8).
Repro: database error output showing NOT NULL project_id upgrade failure
- Keeps the command output available without making the summary code-heavy.
Repro: attempted repository Drizzle push path without available local Postgres
- Keeps the command output available without making the summary code-heavy.
No description provided.