test(migrations): drift-guard-test + plan-doc sync na idempotency-fix #99 — main is nu rood #100

Merged

janpeter merged 1 commit from fix/m17-deploy-fields-idempotent into main

2026-07-04 15:36:06 +02:00

janpeter commented

2026-07-04 15:30:22 +02:00

Owner

Context — kruisende fixes

De migratie-idempotency-fix voor de fase-4-blokkade (42701 op prod) is via #99 (server-Claude, 31310cb) al op main gemerged. Deze PR en #99 zijn parallel ontstaan; dit is het resterende delta (7f0e716).

Waarom deze PR nodig is: main staat rood

#99 repareerde alléén de migration.sql. De drift-guard-test __tests__/db/deploy-fields-migration.test.ts asserteert op main nog de kale ADD COLUMN-strings en faalt tegen de nieuwe IF NOT EXISTS-SQL. Deze PR:

repareert die asserts (IF NOT EXISTS-varianten) + voegt een index-idempotency-assert toe;
trekt plan-doc Task 1.3 (docs/plans/M17-deploy-job.md) gelijk met de definitieve SQL, zodat her-uitvoering van het plan de 42701-fout niet herintroduceert;
documenteert het dev-DB-neveneffect in de migratie-comment: migrate dev stopt op de checksum-wijziging → remedie prisma migrate reset + npm run seed (resolve --applied werkt niet); migrate deploy verifieert geen checksums, dus prod/Neon zijn veilig (geverifieerd op Prisma 7.8.0).

Verificatie

npm run verify groen op deze branch: 199 files / 1596 tests.
Adversarieel 3-lenzen-panel op de migratie-eindtoestand: prod out-of-order apply slaagt statement-voor-statement (constraint-herdefinitie strikte superset, IDEA_CHAT blijft geldig); verse-DB lexicografische volgorde maakt de M18-fore-runner een no-op; Prisma-tooling-gedrag geverifieerd tegen de geïnstalleerde engine.

Na merge

Fase 4 (ST-1460) opnieuw dispatchen naar scrum4me-server:claude — begint met update_scrum4me_web + migraties + seed.

🤖 Generated with Claude Code

## Context — kruisende fixes De migratie-idempotency-fix voor de fase-4-blokkade (42701 op prod) is via **#99** (server-Claude, `31310cb`) al op main gemerged. Deze PR en #99 zijn parallel ontstaan; dit is het resterende delta (`7f0e716`). ## Waarom deze PR nodig is: main staat rood #99 repareerde alléén de `migration.sql`. De drift-guard-test `__tests__/db/deploy-fields-migration.test.ts` asserteert op main nog de kale `ADD COLUMN`-strings en **faalt** tegen de nieuwe `IF NOT EXISTS`-SQL. Deze PR: - repareert die asserts (IF NOT EXISTS-varianten) + voegt een index-idempotency-assert toe; - trekt plan-doc Task 1.3 (`docs/plans/M17-deploy-job.md`) gelijk met de definitieve SQL, zodat her-uitvoering van het plan de 42701-fout niet herintroduceert; - documenteert het dev-DB-neveneffect in de migratie-comment: `migrate dev` stopt op de checksum-wijziging → remedie `prisma migrate reset` + `npm run seed` (`resolve --applied` werkt niet); `migrate deploy` verifieert geen checksums, dus prod/Neon zijn veilig (geverifieerd op Prisma 7.8.0). ## Verificatie - `npm run verify` groen op deze branch: 199 files / **1596 tests**. - Adversarieel 3-lenzen-panel op de migratie-eindtoestand: prod out-of-order apply slaagt statement-voor-statement (constraint-herdefinitie strikte superset, IDEA_CHAT blijft geldig); verse-DB lexicografische volgorde maakt de M18-fore-runner een no-op; Prisma-tooling-gedrag geverifieerd tegen de geïnstalleerde engine. ## Na merge Fase 4 (ST-1460) opnieuw dispatchen naar scrum4me-server:claude — begint met `update_scrum4me_web` + migraties + seed. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-07-04 15:30:22 +02:00

test(migrations): asserts + plan-doc gelijkgetrokken met idempotente add_deploy_fields

CI / Lint, Typecheck, Test & Build (pull_request) Successful in 3m37s

Details

CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped

Details

CI / Detect deploy-relevant changes (pull_request) Has been skipped

Details

CI / Deploy Preview (PR) (pull_request) Has been skipped

Details

CI / Deploy Production (main) (pull_request) Has been skipped

Details

7f0e716f59

Vervolg op 31310cb (server-fix): de drift-guard-test asserteerde nog de kale
ADD COLUMN-strings en faalde daardoor; nu asserts op de IF NOT EXISTS-varianten
+ een index-idempotency-assert. Plan-doc Task 1.3 (M17-deploy-job.md) citeert
de definitieve SQL zodat her-uitvoering de 42701-fout niet herintroduceert.
Migratie-comment documenteert het dev-DB-neveneffect (migrate dev stopt op de
checksum-wijziging; remedie = migrate reset + seed, resolve --applied werkt
niet; migrate deploy verifieert geen checksums — geverifieerd op Prisma 7.8.0).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

janpeter changed title from ~~fix(migrations): add_deploy_fields idempotent — deblokkeert M17 fase-4-rollout~~ to test(migrations): drift-guard-test + plan-doc sync na idempotency-fix #99 — main is nu rood

2026-07-04 15:33:12 +02:00

janpeter merged commit 010226504b into main

2026-07-04 15:36:06 +02:00

janpeter referenced this pull request from a commit

2026-07-04 15:36:08 +02:00

Merge pull request 'test(migrations): drift-guard-test + plan-doc sync na idempotency-fix #99 — main is nu rood' (#100) from fix/m17-deploy-fields-idempotent into main