janpeter/Scrum4Me
0
0
Fork 0
Code Issues 3 Pull requests 1 Projects Releases Packages Wiki Activity Actions

docs(queue-dispatch): ISS-12 — afgemelde incarnatie + nooit gestarte poging sluit zelf af #306

Merged
janpeter merged 1 commit from docs/iss12-signed-off-unstarted into main 2026-10-05 12:37:07 +02:00
Conversation 0 Commits 1 Files changed 1 +21 -3
janpeter commented 2026-10-05 12:16:53 +02:00
Owner
Copy link

Runbook-update voor ISS-12. Hoort bij scrum4me-mcp PR #191: janpeter/scrum4me-mcp#191

  • §9.2 krijgt een derde zelf-afsluitend geval. Een supervisor die vóór de start is vervangen (afgemelde incarnatie, nooit gestarte poging) wordt door de service-tick gesloten:

    • FAILED met DISPATCH_INCARNATION_SIGNED_OFF_BEFORE_START;
    • of CANCELLED bij een vastgelopen annulering.

    De sectie bevat ook het permit-bewijs.

  • §9.1 krijgt de aanvulling dat er nog steeds geen automatische retry is: deze klasse sluit af, maar probeert niet opnieuw.

Pas mergen samen met of ná #191 (scrum4me-mcp), en alleen met GO van JP.

🤖 Generated with Claude Code

Runbook-update voor **ISS-12**. Hoort bij scrum4me-mcp PR #191: https://git.jp-visser.nl/janpeter/scrum4me-mcp/pulls/191 - §9.2 krijgt een derde zelf-afsluitend geval. Een supervisor die vóór de start is vervangen (afgemelde incarnatie, nooit gestarte poging) wordt door de service-tick gesloten: - `FAILED` met `DISPATCH_INCARNATION_SIGNED_OFF_BEFORE_START`; - of `CANCELLED` bij een vastgelopen annulering. De sectie bevat ook het permit-bewijs. - §9.1 krijgt de aanvulling dat er nog steeds geen automatische retry is: deze klasse sluit af, maar probeert niet opnieuw. Pas mergen samen met of ná #191 (scrum4me-mcp), en alleen met GO van JP. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
docs(queue-dispatch): ISS-12 — afgemelde incarnatie + nooit gestarte poging sluit zelf af
All checks were successful
CI / DB Access Policy Gate (pull_request) Successful in 2m32s
Details
CI / Lint, Typecheck, Test & Build (pull_request) Successful in 4m50s
Details
CI / DB Access Production Sentinel (pull_request) Successful in 1m23s
Details
CI / Deploy Manual (workflow_dispatch) (pull_request) Has been skipped
Details
CI / Queue Dispatch Guard Gate (pull_request) Successful in 2m55s
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
6227808e8b 
Runbook §9.1/9.2: de service-tick sluit een nooit gestarte poging van een
afgemelde supervisor-incarnatie zelf af (FAILED, of CANCELLED bij een
vastgelopen annulering). Bewijs: een startpermit bestaat alleen samen met
started_at, en een afgemelde incarnatie krijgt er nooit meer een.
Implementatie: scrum4me-mcp fix/iss12-signed-off-unstarted.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
s4m-codex-reviewer reviewed 2026-10-05 12:22:26 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: COMMENT

Findings:

  • minor — docs/runbooks/queue-dispatch.md:591 — De inleiding zegt nog “Er zijn twee situaties”, terwijl deze PR een derde zelf-afsluitend geval toevoegt en regel 629 terecht naar drie gevallen verwijst. Wijzig de inleiding naar “drie situaties”. Dit is niet blokkerend.

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

De toevoeging sluit aan bij de lifecycle- en stopbewijsafspraken: geen automatische retry, afsluiting uitsluitend van een afgemelde incarnatie zonder start, en behoud van UNCERTAIN voor actieve incarnaties. De statustransities, summary en bewijssoort komen overeen met de implementatie en regressietests van scrum4me-mcp PR #191 (inmiddels gemergd). Deze PR rondt uitsluitend de runbook-update af.

Verificatie op commit 6227808e8bd6f8447e6f36026ccfdd8dec85cf80: git diff --check geslaagd; node scripts/check-doc-links.mjs geslaagd (310 bestanden). Geen runtimecode gewijzigd; runtime-tests en typecheck niet uitgevoerd voor deze documentatie-review. De aparte merge-GO uit de PR-beschrijving blijft van toepassing.

## Verdict: COMMENT Findings: - **minor — docs/runbooks/queue-dispatch.md:591** — De inleiding zegt nog “Er zijn twee situaties”, terwijl deze PR een derde zelf-afsluitend geval toevoegt en regel 629 terecht naar drie gevallen verwijst. Wijzig de inleiding naar “drie situaties”. Dit is niet blokkerend. geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De toevoeging sluit aan bij de lifecycle- en stopbewijsafspraken: geen automatische retry, afsluiting uitsluitend van een afgemelde incarnatie zonder start, en behoud van `UNCERTAIN` voor actieve incarnaties. De statustransities, summary en bewijssoort komen overeen met de implementatie en regressietests van scrum4me-mcp PR #191 (inmiddels gemergd). Deze PR rondt uitsluitend de runbook-update af. Verificatie op commit `6227808e8bd6f8447e6f36026ccfdd8dec85cf80`: `git diff --check` geslaagd; `node scripts/check-doc-links.mjs` geslaagd (310 bestanden). Geen runtimecode gewijzigd; runtime-tests en typecheck niet uitgevoerd voor deze documentatie-review. De aparte merge-GO uit de PR-beschrijving blijft van toepassing.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
severity/s3

   
severity/s4
No labels
severity/s3
severity/s4
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!306
No description provided.