Cleanup: delete `db.ts`, lint-ban raw SQL outside DB layer, update docs #849

New issue

Closed

opened 2026-05-04 20:29:50 +00:00 by claude-desktop · 0 comments

claude-desktop commented

2026-05-04 20:29:50 +00:00

Collaborator

As a maintainer, I want the now-empty db.ts deleted, raw SQL lint-banned outside an explicit escape hatch, and the architecture docs updated, so that the migration is closed and the new pattern is enforced going forward.

Acceptance criteria

Deletion

db.ts reduced to barrel re-exports OR deleted entirely if no shared utilities remain (only getDb(), agentsDbPath(), resetDb() survive — relocate to db/client.ts).
All imports from database/db updated to per-aggregate files.

Lint

Biome rule (or custom grep CI step) bans .query( / .run( / .exec( calls outside infrastructure/database/ and outside files explicitly listed as raw-SQL escape hatches.
just qa enforces.

Docs

docs/database.md (created in foundation story) updated with final layout: one file per aggregate, schema/ folder, drizzle/ migrations, drift check.
CLAUDE.md "Tech stack" section mentions Drizzle.
docs/modules.md table updated with new file list.

Sanity

just qa green.
Service boots, processes a webhook end-to-end, all rows land correctly.

Out of scope

Further refactors in domain layer.

References

All prior stories in this milestone (#839–#848).

**As a** maintainer, **I want** the now-empty `db.ts` deleted, raw SQL lint-banned outside an explicit escape hatch, and the architecture docs updated, **so that** the migration is closed and the new pattern is enforced going forward. ## Acceptance criteria ### Deletion - [ ] `db.ts` reduced to barrel re-exports OR deleted entirely if no shared utilities remain (only `getDb()`, `agentsDbPath()`, `resetDb()` survive — relocate to `db/client.ts`). - [ ] All imports from `database/db` updated to per-aggregate files. ### Lint - [ ] Biome rule (or custom grep CI step) bans `.query(` / `.run(` / `.exec(` calls outside `infrastructure/database/` and outside files explicitly listed as raw-SQL escape hatches. - [ ] `just qa` enforces. ### Docs - [ ] `docs/database.md` (created in foundation story) updated with final layout: one file per aggregate, schema/ folder, drizzle/ migrations, drift check. - [ ] `CLAUDE.md` "Tech stack" section mentions Drizzle. - [ ] `docs/modules.md` table updated with new file list. ### Sanity - [ ] `just qa` green. - [ ] Service boots, processes a webhook end-to-end, all rows land correctly. ## Out of scope - Further refactors in domain layer. ## References - All prior stories in this milestone (#839–#848).

claude-desktop added the

area:database

type:user-story

labels

2026-05-04 20:30:11 +00:00

claude-desktop added this to the ORM migration (Drizzle) milestone

2026-05-04 20:30:19 +00:00

claude-desktop added a new dependency

2026-05-04 21:47:30 +00:00

#841 Migrate operator_sessions.ts to Drizzle

claude-desktop added a new dependency

2026-05-04 21:47:30 +00:00