janpeter/Ops-dashboard
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

test(worker-logs): dispatch-pool run-log (M41 increment 2) #279

Merged
janpeter merged 2 commits from feat/m41-dispatch-run-log into main 2026-10-02 16:55:25 +02:00
Conversation 1 Commits 2 Files changed 3 +121
janpeter commented 2026-10-02 16:32:50 +02:00
Owner
Copy link

Samenvatting

M41 increment 2 (sprint S-2026-10-02-2, story ST-1615, taak T-1936). Alleen tests: Worker Logs leest een run-log uit de nieuwe pool dispatch al correct in. Er is geen productiewijziging nodig.

  • Fixture: test/fixtures/worker-logs/dispatch-codex-reviewer.log (geschoond), met dezelfde metaregels als de supervisor van scrum4me-docker schrijft en daarna codex exec --json-events.
  • test/dispatch-run-log.test.ts (5 tests):
    • parseRunLog levert job-id, status success, exit 0 en de codex-events;
    • pool-discovery vindt dispatch/<slot>/runs/<ts>.log;
    • de run-id is dispatch/<slot>/<ts>;
    • ingest schrijft pool=dispatch en het job-id.
  • De test staat in de CI-groep base (scripts/ci/test-groups.json).

Test

  • Rood/groen: dit zijn karakteriseringstests en ze waren direct groen. Dat ze echt iets controleren is aangetoond door de tag tijdelijk te wijzigen: dan falen er 2.
  • Live bewijs: het run-log van de echte reviewproef (2026-10-02) werd door deze parser gelezen als een geslaagde run.

🤖 Generated with Claude Code

## Samenvatting M41 increment 2 (sprint S-2026-10-02-2, story ST-1615, taak T-1936). Alleen tests: Worker Logs leest een run-log uit de nieuwe pool `dispatch` al correct in. Er is geen productiewijziging nodig. - **Fixture:** `test/fixtures/worker-logs/dispatch-codex-reviewer.log` (geschoond), met dezelfde metaregels als de supervisor van scrum4me-docker schrijft en daarna `codex exec --json`-events. - **`test/dispatch-run-log.test.ts` (5 tests):** - `parseRunLog` levert job-id, status `success`, exit 0 en de codex-events; - pool-discovery vindt `dispatch/<slot>/runs/<ts>.log`; - de run-id is `dispatch/<slot>/<ts>`; - ingest schrijft `pool=dispatch` en het job-id. - De test staat in de CI-groep `base` (`scripts/ci/test-groups.json`). ## Test - **Rood/groen:** dit zijn karakteriseringstests en ze waren direct groen. Dat ze echt iets controleren is aangetoond door de tag tijdelijk te wijzigen: dan falen er 2. - **Live bewijs:** het run-log van de echte reviewproef (2026-10-02) werd door deze parser gelezen als een geslaagde run. 🤖 Generated with [Claude Code](https://claude.com/claude-code) 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
test(ci): dispatch-run-log-test in de base-groep (M41 U5)
All checks were successful
CI / Select checks (pull_request) Successful in 17s
Details
CI / Ops-agent checks (pull_request) Successful in 53s
Details
CI / DB access operator (pull_request) Successful in 1m40s
Details
CI / Deploy artifact checks (pull_request) Successful in 44s
Details
CI / Root app checks (pull_request) Successful in 9m4s
Details
CI / Docker image build (pull_request) Successful in 1m22s
Details
CI / Mac foundation hermetic checks (pull_request) Successful in 4m19s
Details
CI / Required checks (pull_request) Successful in 28s
Details
d0d144170e 
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-10-02 16:39:10 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Bevindingen

  • Geen blokkerende of error-severity bevindingen.
  • INFO — test/dispatch-run-log.test.ts:34 — de nieuwe fixture dekt de dispatch-pool end-to-end: discovery, Codex-eventparsing, run-id en database-ingestie.
  • INFO — scripts/ci/test-groups.json:20 — de test is opgenomen in de basale CI-testgroep.

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

Lokale uitvoering was niet mogelijk omdat de checkout geen geïnstalleerde ontwikkelafhankelijkheden bevatte (vitest en tsc ontbreken); de diff is wel statisch gecontroleerd.

# Verdict: APPROVED ## Bevindingen - Geen blokkerende of error-severity bevindingen. - INFO — `test/dispatch-run-log.test.ts:34` — de nieuwe fixture dekt de dispatch-pool end-to-end: discovery, Codex-eventparsing, run-id en database-ingestie. - INFO — `scripts/ci/test-groups.json:20` — de test is opgenomen in de basale CI-testgroep. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Lokale uitvoering was niet mogelijk omdat de checkout geen geïnstalleerde ontwikkelafhankelijkheden bevatte (`vitest` en `tsc` ontbreken); de diff is wel statisch gecontroleerd.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels   
severity/s2

   
severity/s3
No labels
severity/s2
severity/s3
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/Ops-dashboard!279
No description provided.