sigitex/outlaw

Watch

Fork 0

feat(query): support SQLite statement conflict policies #3

Open

toad wants to merge 5 commits from feat/sqlite-statement-conflicts into feat/sqlite-table-semantics

toad commented

2026-09-20 15:53:34 +00:00

Member

OpenSpec

Change: sqlite-statement-conflicts (spec-driven). 7/7 tasks complete.

Scope

Typed .or("ABORT" | "FAIL" | "IGNORE" | "REPLACE") on INSERT and UPDATE, compatible with RETURNING and UPDATE WHERE. OR ROLLBACK and UPSERT excluded.
SQL generators render policies before table targets while preserving no-policy statements.
Bun SQLite tests cover unique/NOT NULL conflict effects, RETURNING, type-chain inference and rollback rejection.
README documents use, SQLite limits, and Bun result behavior; authoritative full-SQLite plan updated. Change artifacts included.

Checkpoints

2d93d14 implementation and tests.
31155dc documentation and completed OpenSpec artifacts.
ec5db98 Merge parent branch transaction fix without history rewrite.
389ea02 Test statement conflict policies in nested transactions.

Verification

bun run check: passed after parent fix.
bun run lint: 0 errors, 14 existing warnings.
bun test: 107 passed, 0 failed.
openspec validate sqlite-statement-conflicts: passed; apply progress 7/7.
git diff --check: passed.

Cloudflare D1 behavior not independently exercised; SQLite runtime semantics tested with Bun.

## OpenSpec Change: `sqlite-statement-conflicts` (spec-driven). 7/7 tasks complete. ## Scope - Typed `.or("ABORT" | "FAIL" | "IGNORE" | "REPLACE")` on INSERT and UPDATE, compatible with RETURNING and UPDATE WHERE. OR ROLLBACK and UPSERT excluded. - SQL generators render policies before table targets while preserving no-policy statements. - Bun SQLite tests cover unique/NOT NULL conflict effects, RETURNING, type-chain inference and rollback rejection. - README documents use, SQLite limits, and Bun result behavior; authoritative full-SQLite plan updated. Change artifacts included. ## Checkpoints - `2d93d14` implementation and tests. - `31155dc` documentation and completed OpenSpec artifacts. - `ec5db98` Merge parent branch transaction fix without history rewrite. - `389ea02` Test statement conflict policies in nested transactions. ## Verification - `bun run check`: passed after parent fix. - `bun run lint`: 0 errors, 14 existing warnings. - `bun test`: 107 passed, 0 failed. - `openspec validate sqlite-statement-conflicts`: passed; apply progress 7/7. - `git diff --check`: passed. Cloudflare D1 behavior not independently exercised; SQLite runtime semantics tested with Bun.

toad added 1 commit

2026-09-20 15:53:34 +00:00

feat(query): support SQLite statement conflict policies

Checks / checks (pull_request) Successful in 7s

Details

Checks / release (pull_request) Has been skipped

Details

2d93d14689

dan added 1 commit

2026-09-20 15:54:22 +00:00

docs(query): document statement conflicts and complete spec

Checks / checks (pull_request) Successful in 6s

Details

Checks / release (pull_request) Has been skipped

Details

31155dc04f

dan added 2 commits

2026-09-20 17:02:56 +00:00

chore: merge table semantics transaction fix ec5db98840

test(query): cover conflict policies in nested transactions

Checks / checks (pull_request) Successful in 7s

Details

Checks / release (pull_request) Has been skipped

Details

389ea02871

dan added 1 commit

2026-09-20 19:30:14 +00:00

chore: gitignore update

Checks / checks (pull_request) Successful in 6s

Details

Checks / release (pull_request) Has been skipped

Details

08d1a9ca5a

Checks / checks (pull_request) Successful in 6s

Details

Checks / release (pull_request) Has been skipped

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feat/sqlite-statement-conflicts:feat/sqlite-statement-conflicts

git switch feat/sqlite-statement-conflicts

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.