charles/claude-hooks
1
0
Fork 0
Code Issues 10 Pull requests Projects Releases Packages 1 Wiki Activity Actions

feat(db): extract dead_letter_prs + watched_repos to Drizzle (#842) #860

Merged
reviewer merged 1 commit from code-lead/842 into main 2026-05-04 22:18:19 +00:00
Conversation 1 Commits 1 Files changed 24 +973 -463
code-lead commented 2026-05-04 22:15:20 +00:00
Collaborator
Copy link

Splits the small dead_letter_prs and watched_repos aggregates out of the db.ts god-file onto Drizzle (bridge style — snake_case TS field names mirror SQL column names).

Test plan

  • bun x turbo run typecheck clean
  • bun x @biomejs/biome@^2 check . no new errors
  • bun x turbo run test — 3124 pass / 0 fail (incl. new round-trip Drizzle ↔ raw)
  • All known callers updated

Closes #842

Splits the small `dead_letter_prs` and `watched_repos` aggregates out of the `db.ts` god-file onto Drizzle (bridge style — snake_case TS field names mirror SQL column names). ## Test plan - [x] `bun x turbo run typecheck` clean - [x] `bun x @biomejs/biome@^2 check .` no new errors - [x] `bun x turbo run test` — 3124 pass / 0 fail (incl. new round-trip Drizzle ↔ raw) - [x] All known callers updated Closes #842
feat(db): extract dead_letter_prs + watched_repos to Drizzle (#842)
All checks were successful
qa / dockerfile (pull_request) Successful in 14s
Details
qa / db-schema (pull_request) Successful in 33s
Details
qa / qa-1 (pull_request) Successful in 48s
Details
qa / qa (pull_request) Successful in 0s
Details
8a9c1a2662 
Splits the small dead-letter and watched-repo aggregates out of the
db.ts god-file onto Drizzle, keeping the bridge style (snake_case TS
field names == SQL column names). resetWatchdogTables stays in db.ts
since recoveries + counters are out of scope for this story.
reviewer approved these changes 2026-05-04 22:18:14 +00:00
reviewer left a comment
Copy link

Schemas correct, Drizzle bridge consistent, round-trip tests cover both aggregates. event-handlers.ts audited — imports only clearPrDependencies/getPrDependencies/getSettings, none of the moved fns. All callers updated, CI green.

Schemas correct, Drizzle bridge consistent, round-trip tests cover both aggregates. `event-handlers.ts` audited — imports only `clearPrDependencies`/`getPrDependencies`/`getSettings`, none of the moved fns. All callers updated, CI green.
Sign in to join this conversation.
Reviewers
No reviewers
reviewer
Labels
Clear labels   
area:agents
Agent types, pool scheduling, per-instance config

  
area:dashboard
Dashboard UI and observability surfaces

  
area:database
DB layer — schema, migrations, ORM, raw SQL

  
area:design
UI/UX mockup work — routes to designer agent

  
area:design-review
Design review dispatch — routes to design-reviewer agent

  
area:flows
Flow runner — YAML loader, executor, op registry, expression eval

  
area:infra
Deployment, isolation, containers, systemd units

  
area:meta
Tracking, scaffolding, project setup

  
area:security
Security — routes to reviewer-security (opus)

  
area:sessions
Session-id store, Claude SDK resume logic

  
area:webhook
Forgejo webhook routing and handlers

  
area:workdir
Clone cache, worktrees, git identity

  
security
Security-sensitive issue

  
type:bug
Bug

  
type:chore
Chore

  
type:meta
Tracking or decisions, not implementation work

  
type:user-story
User story

No labels
area:agents
area:dashboard
area:database
area:design
area:design-review
area:flows
area:infra
area:meta
area:security
area:sessions
area:webhook
area:workdir
security
type:bug
type:chore
type:meta
type:user-story
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
code-lead
2 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
charles/claude-hooks!860
No description provided.