janpeter/scrum4me-shared
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(queue): de pariteitsgate bestaat nu — header bijgewerkt #41

Merged
janpeter merged 1 commit from docs/queue-identity-gate into main 2026-07-25 22:14:07 +02:00
Conversation 1 Commits 1 Files changed 1 +30 -18
janpeter commented 2026-07-25 22:09:30 +02:00
Owner
Copy link

Nasleep van de vijf PR's rond de gedeelde queue-identity, nu die allemaal gemerged zijn. Comment-only — bewezen met git diff … | grep '^[-+][^-+]' | grep -v '^[-+]//', die niets print.

Waarom

De header van lib/queue-identity.ts beschreef de situatie tijdens de uitrol. Nu die klaar is, klopten drie beweringen niet meer:

Stond er Is nu waar
NOG GEEN PARITEITSGATE De gate bestaat: Scrum4Me/__tests__/db/agent-message-queue-migration.test.ts toetst de type-, status- en source-CHECK tegen dit bestand
"drie exemplaren in totaal" Twee. scrum4me-workers is re-exports; alleen s4m-queue/src/types.ts blijft handkopie
CHECK-constraints alleen in Scrum4Me Ze staan op twee plekken — ook s4m-queue/migrations/001_init.sql, het testschema van de CLI

Wat er is toegevoegd

De twee grenzen van de gate, allebei bewust en allebei door review vastgesteld: servers en modellen vallen erbuiten omdat er geen CHECK is om tegen te toetsen, en de test leest één migratiebestand in plaats van de vouw van alle migraties — wie een tweede migratie schrijft die deze constraints via DROP/ADD CONSTRAINT wijzigt, moet die test meenemen.

Welke repo dit bestand echt gebruikt. Nagemeten: scrum4me-workers importeert het via lib/queue/types.ts. scrum4me-mcp pint de submodule wél maar heeft geen enkele import in src/ — die code komt met fase 2. Dat onderscheid staat er nu expliciet, in plaats van beide repos als consument op te voeren.

Verificatie

npm run verify groen: dep-gate OK: lib/ is dep-clean, tsc --noEmit schoon, 22 bestanden / 237 tests.

Elke bewering in de nieuwe tekst is vóór het schrijven getoetst tegen origin/main van de betrokken repo — niet tegen de verwachting. Dat is de les die dit blok vier reviewrondes heeft gekost.

🤖 Generated with Claude Code

Nasleep van de vijf PR's rond de gedeelde queue-identity, nu die allemaal gemerged zijn. **Comment-only** — bewezen met `git diff … | grep '^[-+][^-+]' | grep -v '^[-+]//'`, die niets print. ## Waarom De header van `lib/queue-identity.ts` beschreef de situatie tijdens de uitrol. Nu die klaar is, klopten drie beweringen niet meer: | Stond er | Is nu waar | |---|---| | `NOG GEEN PARITEITSGATE` | De gate bestaat: `Scrum4Me/__tests__/db/agent-message-queue-migration.test.ts` toetst de type-, status- en source-CHECK tegen dit bestand | | "drie exemplaren in totaal" | Twee. `scrum4me-workers` is re-exports; alleen `s4m-queue/src/types.ts` blijft handkopie | | CHECK-constraints alleen in Scrum4Me | Ze staan op twee plekken — ook `s4m-queue/migrations/001_init.sql`, het testschema van de CLI | ## Wat er is toegevoegd **De twee grenzen van de gate**, allebei bewust en allebei door review vastgesteld: servers en modellen vallen erbuiten omdat er geen CHECK is om tegen te toetsen, en de test leest één migratiebestand in plaats van de vouw van alle migraties — wie een tweede migratie schrijft die deze constraints via `DROP/ADD CONSTRAINT` wijzigt, moet die test meenemen. **Welke repo dit bestand echt gebruikt.** Nagemeten: `scrum4me-workers` importeert het via `lib/queue/types.ts`. `scrum4me-mcp` pint de submodule wél maar heeft geen enkele import in `src/` — die code komt met fase 2. Dat onderscheid staat er nu expliciet, in plaats van beide repos als consument op te voeren. ## Verificatie `npm run verify` groen: dep-gate `OK: lib/ is dep-clean`, `tsc --noEmit` schoon, 22 bestanden / 237 tests. Elke bewering in de nieuwe tekst is vóór het schrijven getoetst tegen `origin/main` van de betrokken repo — niet tegen de verwachting. Dat is de les die dit blok vier reviewrondes heeft gekost. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
docs(queue): de pariteitsgate bestaat nu — header bijgewerkt
All checks were successful
CI / Verify (pull_request) Successful in 17s
Details
fedb2eb844 
De rollout is klaar, dus drie beweringen in deze header waren niet langer waar:

- "NOG GEEN PARITEITSGATE" -- Scrum4Me/__tests__/db/agent-message-queue-
  migration.test.ts toetst sinds de merge de type-, status- en source-CHECK
  tegen dit bestand. De twee grenzen van die gate staan er nu bij: servers en
  modellen vallen erbuiten (geen CHECK), en de test leest een migratiebestand
  in plaats van de vouw van alle migraties.
- "drie exemplaren in totaal" -- het zijn er twee. scrum4me-workers is nu
  re-exports; alleen s4m-queue/src/types.ts blijft een handkopie.
- De CHECK-constraints staan niet alleen in Scrum4Me: s4m-queue/migrations/
  001_init.sql draagt dezelfde drie voor het testschema van de CLI.

Toegevoegd: welke repo dit bestand echt importeert (workers) en welke alleen
de submodule pint zonder code (mcp, tot fase 2).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-07-25 22:11:09 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen findings. De wijziging past alleen commentaar in lib/queue-identity.ts aan, houdt de gedeelde module dep-clean, en is consistent met de productdocs waarin scrum4me-shared als canonieke pure TypeScript/shared-schema bron wordt beschreven.

Toetsing

  • Codekwaliteit: akkoord; geen uitvoerbare code gewijzigd.
  • Architectuur-/patroon-conformiteit: akkoord; geen nieuwe runtime-, Prisma-, Next- of React-afhankelijkheden en geen wijziging aan shared API surface.
  • Tests: akkoord voor deze scope; documentatie-only wijziging, geen aanvullende test nodig.
  • Docs: akkoord; header verduidelijkt de actuele pariteitsgate en de bekende grenzen daarvan.
# Verdict: APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen findings. De wijziging past alleen commentaar in `lib/queue-identity.ts` aan, houdt de gedeelde module dep-clean, en is consistent met de productdocs waarin `scrum4me-shared` als canonieke pure TypeScript/shared-schema bron wordt beschreven. ## Toetsing - Codekwaliteit: akkoord; geen uitvoerbare code gewijzigd. - Architectuur-/patroon-conformiteit: akkoord; geen nieuwe runtime-, Prisma-, Next- of React-afhankelijkheden en geen wijziging aan shared API surface. - Tests: akkoord voor deze scope; documentatie-only wijziging, geen aanvullende test nodig. - Docs: akkoord; header verduidelijkt de actuele pariteitsgate en de bekende grenzen daarvan.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-shared!41
No description provided.