Skip to content

Unify SQLite busy baseline - #387

Merged
zccz14 merged 2 commits into
mainfrom
fix-sqlite-busy-baseline
May 4, 2026
Merged

zccz14 merged 2 commits into
mainfrom
fix-sqlite-busy-baseline

Conversation

@zccz14

@zccz14 zccz14 commented May 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Configure every openTaskDatabase() SQLite connection with the shared busy/WAL baseline: busy_timeout, WAL journal mode, synchronous=NORMAL, and foreign keys.
  • Remove the optimizer lane events local busy timeout because the shared database opener now owns that baseline.
  • Add a behavior test that opens a real task database and verifies the persisted PRAGMA behavior instead of source shape.

Tests

  • pnpm install --ignore-scripts
  • pnpm exec vitest run --config vitest.workspace.ts --project api modules/api/test/task-database.test.ts (RED: failed with busy_timeout 0 before implementation, then passed)
  • pnpm --filter ./modules/contract run build:dist (explicit prerequisite for package exports)
  • pnpm exec vitest run --config vitest.workspace.ts --project api modules/api/test/task-database.test.ts modules/api/test/optimizer-lane-events.test.ts modules/api/test/task-repository.test.ts modules/api/test/manager-state-repository.test.ts modules/api/test/coordinator-state-repository.test.ts modules/api/test/dimension-repository.test.ts modules/api/test/opencode-session-repository.test.ts
  • pnpm --filter ./modules/api run typecheck
  • pnpm --filter ./modules/api run test:lint
  • pnpm --filter ./modules/api run test:vitest

@zccz14
zccz14 enabled auto-merge (squash) May 4, 2026 05:29
@zccz14
zccz14 merged commit acf6b71 into main May 4, 2026
1 check passed
@zccz14
zccz14 deleted the fix-sqlite-busy-baseline branch May 4, 2026 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant