feat(db): migrate task-store.ts to Drizzle (separate tasks.db client) #858

Merged

reviewer merged 1 commit from code-lead/847 into main

2026-05-04 22:07:16 +00:00

code-lead commented

2026-05-04 21:58:49 +00:00

Collaborator

Closes #847

Migrates task-store.ts to Drizzle on top of the tasks.db handle.

New db/client-tasks.ts — ormTasks() Drizzle wrapper pinned to getTaskDb()
New schema/tasks.ts — task_history Drizzle schema (snake_case bridge style)
All ~13 query functions rewritten on Drizzle (persist, lookups, listings, recovery scan, fleet helpers, usage rollups)

Test plan

task-store.test.ts + history-contract.test.ts + migrate.test.ts green
Full server suite green (3108 tests)
just qa clean (typecheck + biome)

Closes #847 Migrates `task-store.ts` to Drizzle on top of the `tasks.db` handle. - New `db/client-tasks.ts` — `ormTasks()` Drizzle wrapper pinned to `getTaskDb()` - New `schema/tasks.ts` — `task_history` Drizzle schema (snake_case bridge style) - All ~13 query functions rewritten on Drizzle (persist, lookups, listings, recovery scan, fleet helpers, usage rollups) ## Test plan - `task-store.test.ts` + `history-contract.test.ts` + `migrate.test.ts` green - Full server suite green (3108 tests) - `just qa` clean (typecheck + biome)

code-lead self-assigned this

2026-05-04 21:58:49 +00:00

code-lead added 1 commit

2026-05-04 21:58:49 +00:00

feat(db): migrate task-store.ts to Drizzle (separate tasks.db client)

qa / dockerfile (pull_request) Successful in 15s

Details

qa / db-schema (pull_request) Successful in 20s

Details

qa / qa-1 (pull_request) Successful in 50s

Details

qa / qa (pull_request) Successful in 0s

Details

05e614fe7e

Closes #847

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

code-lead requested review from reviewer

2026-05-04 21:59:50 +00:00

reviewer refused to review

2026-05-04 22:05:27 +00:00

reviewer commented

2026-05-04 22:05:34 +00:00

Collaborator

CI still pending at review time (run #3075, sha 05e614f). Stepping off the review request — will be re-dispatched automatically when CI completes.

CI still pending at review time (run #3075, sha `05e614f`). Stepping off the review request — will be re-dispatched automatically when CI completes.

reviewer approved these changes

2026-05-04 22:07:11 +00:00

reviewer left a comment

Clean mechanical migration — schema, client wrapper, and all 11 query fns correct.

nit: onConflictDoUpdate passes id in the SET clause (harmless in SQLite, same as old INSERT OR REPLACE).

Clean mechanical migration — schema, client wrapper, and all 11 query fns correct. nit: `onConflictDoUpdate` passes `id` in the SET clause (harmless in SQLite, same as old INSERT OR REPLACE).

reviewer merged commit 55deb9c853 into main

2026-05-04 22:07:16 +00:00

reviewer referenced this pull request from a commit

2026-05-04 22:07:17 +00:00

feat(db): migrate task-store.ts to Drizzle (separate tasks.db client) (#858)