feat(schema): support managed SQLite triggers #6

Open

toad wants to merge 3 commits from feat/sqlite-triggers into feat/sqlite-generated-columns

toad commented

2026-09-20 20:10:33 +00:00

Member

OpenSpec

Change: sqlite-triggers (spec-driven). Proposal, design, capability spec and tasks reviewed; all 7/7 tasks complete. OpenSpec artifacts remain local because repository ignores openspec/.

Scope

Generalize schema member collection for managed triggers alongside tables, views and indexes.
Persist deterministic trigger metadata for timing, event, optional UPDATE OF and WHEN, and ordered SQL body; expose fluent public builder and executable CREATE TRIGGER generator.
Cowboy replaces changed triggers, removes stale tracked triggers, restores unchanged triggers lost to table/view rebuilds after dependencies converge, and skips surviving unchanged triggers on restart.
README and authoritative docs/plans/full-sqlite.md describe API, migration behavior, trusted raw SQL, and Bun/D1 verification limits.

Checkpoints

4251843 schema model, public trigger builder, generator.
5160cd8 migration convergence and Bun lifecycle tests.
0cecb72 documentation and source plan.
All commits pushed to feat/sqlite-triggers; PR targets feat/sqlite-generated-columns.

Verification

openspec validate sqlite-triggers --strict passed; apply status 7/7.
Focused trigger and affected stacked-change tests passed; full bun test: 119 passed, 0 failed (354 assertions).
Read-only source and test type checks passed; bun run check passed during implementation.
bun run lint: 0 errors, 14 pre-existing warnings.
View-only trigger replacement and generated-column rebuild with trigger restoration probes passed.
Reviewed only diff against feat/sqlite-generated-columns, including design, documentation, regression risk and trusted raw-SQL handling; git diff --check passed. Working tree clean and local/remote HEAD match.

No live Cloudflare D1 trigger execution is claimed.

## OpenSpec Change: `sqlite-triggers` (spec-driven). Proposal, design, capability spec and tasks reviewed; all 7/7 tasks complete. OpenSpec artifacts remain local because repository ignores `openspec/`. ## Scope - Generalize schema member collection for managed triggers alongside tables, views and indexes. - Persist deterministic trigger metadata for timing, event, optional UPDATE OF and WHEN, and ordered SQL body; expose fluent public builder and executable CREATE TRIGGER generator. - Cowboy replaces changed triggers, removes stale tracked triggers, restores unchanged triggers lost to table/view rebuilds after dependencies converge, and skips surviving unchanged triggers on restart. - README and authoritative `docs/plans/full-sqlite.md` describe API, migration behavior, trusted raw SQL, and Bun/D1 verification limits. ## Checkpoints - `4251843` schema model, public trigger builder, generator. - `5160cd8` migration convergence and Bun lifecycle tests. - `0cecb72` documentation and source plan. All commits pushed to `feat/sqlite-triggers`; PR targets `feat/sqlite-generated-columns`. ## Verification - `openspec validate sqlite-triggers --strict` passed; apply status 7/7. - Focused trigger and affected stacked-change tests passed; full `bun test`: 119 passed, 0 failed (354 assertions). - Read-only source and test type checks passed; `bun run check` passed during implementation. - `bun run lint`: 0 errors, 14 pre-existing warnings. - View-only trigger replacement and generated-column rebuild with trigger restoration probes passed. - Reviewed only diff against `feat/sqlite-generated-columns`, including design, documentation, regression risk and trusted raw-SQL handling; `git diff --check` passed. Working tree clean and local/remote HEAD match. No live Cloudflare D1 trigger execution is claimed.

toad added 1 commit

2026-09-20 20:10:33 +00:00

feat(schema): add SQLite trigger builder and generator

Checks / checks (pull_request) Successful in 7s

Details

Checks / release (pull_request) Has been skipped

Details

425184379c

dan added 1 commit

2026-09-20 20:13:07 +00:00

feat(migration): converge managed SQLite triggers

Checks / checks (pull_request) Successful in 6s

Details

Checks / release (pull_request) Has been skipped

Details

5160cd8bf9

dan added 1 commit

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

docs: describe managed SQLite triggers

Checks / checks (pull_request) Successful in 7s

Details

Checks / release (pull_request) Has been skipped

Details

0cecb72e55

Checks / checks (pull_request) Successful in 7s

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-triggers:feat/sqlite-triggers

git switch feat/sqlite-triggers

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.