feat(dispatch): jobId in de pogingscontext (M41 increment 2) #176

Merged

janpeter merged 1 commit from feat/m41-reviewer-kind into main

2026-10-02 16:42:00 +02:00

janpeter commented

2026-10-02 16:32:49 +02:00

Owner

Samenvatting

M41 increment 2 (sprint S-2026-10-02-2, story ST-1615, taak T-1932).

De ExecutionContext die de supervisor bij een claim krijgt, bevat nu jobId:

jobroute: het id van de gebonden claude_jobs-rij;
hostroute: null.

De supervisor gebruikt dit om het run-log van een beheerde poging aan de job te koppelen in Worker Logs (scrum4me-docker, zie de bijbehorende PR).

Test

Eerst rood in attempts.integration: expected undefined to be '7727b96f-…' (jobroute) en expected undefined to be null (hostroute).
Groen: dispatch-integratiesuite 27 files / 279 tests op een wegwerp-pg17. npm test 2020 groen, typecheck en typecheck:tests schoon.

Merge-volgorde

Samen met de scrum4me-docker-PR van M41 increment 2. Het docker-transport leest jobId optioneel, dus de volgorde is niet kritiek; zonder deze PR schrijft de supervisor alleen geen run-log.

🤖 Generated with Claude Code

## Samenvatting M41 increment 2 (sprint S-2026-10-02-2, story ST-1615, taak T-1932). De `ExecutionContext` die de supervisor bij een claim krijgt, bevat nu `jobId`: - **jobroute:** het id van de gebonden `claude_jobs`-rij; - **hostroute:** `null`. De supervisor gebruikt dit om het run-log van een beheerde poging aan de job te koppelen in Worker Logs (scrum4me-docker, zie de bijbehorende PR). ## Test - **Eerst rood** in `attempts.integration`: `expected undefined to be '7727b96f-…'` (jobroute) en `expected undefined to be null` (hostroute). - **Groen:** dispatch-integratiesuite 27 files / 279 tests op een wegwerp-pg17. `npm test` 2020 groen, typecheck en typecheck:tests schoon. ## Merge-volgorde Samen met de scrum4me-docker-PR van M41 increment 2. Het docker-transport leest `jobId` optioneel, dus de volgorde is niet kritiek; zonder deze PR schrijft de supervisor alleen geen run-log. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-10-02 16:32:49 +02:00

feat(dispatch): jobId in de pogingscontext (M41 U1)

CI / Final merge attestation and immutable publication (pull_request) Has been skipped

Details

CI / PR candidate (never published) (pull_request) Successful in 5m18s

Details

3986d4c81a

De supervisor krijgt bij een claim op de jobroute het id van de gebonden
claude_jobs-rij (null op de hostroute), zodat hij het run-log van een
beheerde poging aan de job kan koppelen in Worker Logs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

s4m-codex-reviewer approved these changes

2026-10-02 16:38:11 +02:00

s4m-codex-reviewer left a comment

Verdict: APPROVED

Findings

Geen bevindingen met blokkerende of error-severity.

De wijziging voegt jobId consequent toe aan het uitvoeringscontextcontract, vult die vanuit de gebonden kandidaat en dekt zowel de job- als hostroute met integratietests. Product- en test-TypeScript-typechecks slagen. De relevante integratietest kon in deze runtime niet starten wegens een native Vitest-binding die vanaf /tmp niet geladen mag worden.

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

# Verdict: APPROVED ## Findings - Geen bevindingen met blokkerende of error-severity. De wijziging voegt `jobId` consequent toe aan het uitvoeringscontextcontract, vult die vanuit de gebonden kandidaat en dekt zowel de job- als hostroute met integratietests. Product- en test-TypeScript-typechecks slagen. De relevante integratietest kon in deze runtime niet starten wegens een native Vitest-binding die vanaf `/tmp` niet geladen mag worden. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

janpeter merged commit ef5ad873f4 into main

2026-10-02 16:42:00 +02:00

janpeter referenced this pull request from a commit

2026-10-02 16:42:01 +02:00

Merge pull request 'feat(dispatch): jobId in de pogingscontext (M41 increment 2)' (#176)