Skip to content

docs: note busy_timeout must precede schema init - #7

Closed
sara-lolatte wants to merge 2 commits into
dzhng:mainfrom
sara-lolatte:docs/concurrency-pragma-order
Closed

sara-lolatte wants to merge 2 commits into
dzhng:mainfrom
sara-lolatte:docs/concurrency-pragma-order

Conversation

@sara-lolatte

Copy link
Copy Markdown

Docs-only change. Expected path: risk:low → auto-merge when test is green, no human touch.

Documents the root cause behind the flaky db-busy-timeout test (fix in a separate PR since src/db.ts is a protected path).

sara-lolatte and others added 2 commits September 19, 2026 11:19
- scripts/risk-gate.ts classifies PRs low/medium/high by protected paths,
  size, deleted tests and secret patterns; unit-tested in test/risk-gate.test.ts
- risk-gate.yml labels the PR, posts a sticky verdict, enables auto-merge for
  low (and medium with ai-review:pass), and fails the check for high until a
  human approves
- claude-review.yml runs claude-code-action against .github/review/RUBRIC.md
  and docs/agent-learnings.md; posts a visible skip notice when no API key
- CODEOWNERS mirrors the protected paths as a GitHub-native backstop

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@sara-lolatte

Copy link
Copy Markdown
Author

Apologies — opened against upstream by mistake from a fork used for an exercise. Closing; not intended for this repo.

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