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

test(worker-logs): tweede harness-fixture, een echte taakjob #275

Merged
janpeter merged 1 commit from test/harness-task-fixture into main 2026-09-29 21:44:38 +02:00
Conversation 1 Commits 1 Files changed 2 +289
janpeter commented 2026-09-29 21:10:25 +02:00
Owner
Copy link

M4 (agent-harness run-logging), Taak 11a / criterium 2: het run-log van een echte taakjob wordt de tweede parser-fixture. Plan: agent-harness docs/plans/M4-harness-run-logging.md, Taak 11.

Wat

  • Nieuw: test/fixtures/worker-logs/harness-task-impl.log.

    • Het echte run-log van TASK_IMPLEMENTATION-job cmun0mmuu000gue17ixshy7c4 op max2 (run 20260929T183419Z, T-45 / ST-014), gedispatcht met required_capability: 'local_llm'.
    • Byte-exact overgenomen: sha256 4a0be17a…feef, 127 KB, 61 regels.
    • harness check-run-logs vond er 0 geheimen in.

  • test/parse-worker-log.test.ts: 9 nieuwe tests. Ze pinnen:

    • de samenvatting: job, status success, model, beurten, 204 419 ms, exit 0;
    • de meta-regel worktree path=…;
    • de jobstappen in volgorde, tot en met job_status done pushed_at=ja;
    • de containerblokken prepare, gate en gate, met isError false, true en false, fullLength gelijk aan outputLength, en een verschil in ts gelijk aan durationMs;
    • de drie run_tests-regels;
    • de 14 modeltoolcalls met hun resultaten;
    • het result-event en de loop_end-regel;
    • de volledige volgorde van de 82 eventsoorten.

    De verwachtingen komen uit JSON.parse van de fixture-regels, niet uit de parser zelf.

De parser verandert niet: git diff -- lib/ is leeg. Elke verwachting was bij de eerste run groen tegen de ongewijzigde parser.

Verificatie

  • npm run typecheck groen.
  • test/parse-worker-log.test.ts 77/77 groen.
  • npm test: alleen de twee bekende macOS-platformbestanden falen, zoals op main.
  • Taakreview: spec-conform en approved (0 Critical/Important). De reviewer leidde elke waarde opnieuw af uit de ruwe regels, zonder de parser. Van 41 mutanten binnen de brief vielen er 39 om; de 2 overlevers zijn equivalent.

🤖 Generated with Claude Code

M4 (agent-harness run-logging), Taak 11a / criterium 2: het run-log van een echte taakjob wordt de tweede parser-fixture. Plan: agent-harness `docs/plans/M4-harness-run-logging.md`, Taak 11. ## Wat - **Nieuw:** `test/fixtures/worker-logs/harness-task-impl.log`. - Het echte run-log van TASK_IMPLEMENTATION-job `cmun0mmuu000gue17ixshy7c4` op max2 (run `20260929T183419Z`, T-45 / ST-014), gedispatcht met `required_capability: 'local_llm'`. - Byte-exact overgenomen: sha256 `4a0be17a…feef`, 127 KB, 61 regels. - `harness check-run-logs` vond er 0 geheimen in. - **`test/parse-worker-log.test.ts`:** 9 nieuwe tests. Ze pinnen: - de samenvatting: job, status success, model, beurten, 204 419 ms, exit 0; - de meta-regel `worktree path=…`; - de jobstappen in volgorde, tot en met `job_status done pushed_at=ja`; - de containerblokken prepare, gate en gate, met isError false, true en false, fullLength gelijk aan `outputLength`, en een verschil in `ts` gelijk aan `durationMs`; - de drie `run_tests`-regels; - de 14 modeltoolcalls met hun resultaten; - het result-event en de loop_end-regel; - de volledige volgorde van de 82 eventsoorten. De verwachtingen komen uit `JSON.parse` van de fixture-regels, niet uit de parser zelf. De parser verandert niet: `git diff -- lib/` is leeg. Elke verwachting was bij de eerste run groen tegen de ongewijzigde parser. ## Verificatie - `npm run typecheck` groen. - `test/parse-worker-log.test.ts` 77/77 groen. - `npm test`: alleen de twee bekende macOS-platformbestanden falen, zoals op main. - Taakreview: spec-conform en approved (0 Critical/Important). De reviewer leidde elke waarde opnieuw af uit de ruwe regels, zonder de parser. Van 41 mutanten binnen de brief vielen er 39 om; de 2 overlevers zijn equivalent. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
test(worker-logs): tweede harness-fixture, een echte taakjob
All checks were successful
CI / Select checks (pull_request) Successful in 19s
Details
CI / Ops-agent checks (pull_request) Successful in 37s
Details
CI / DB access operator (pull_request) Successful in 1m23s
Details
CI / Deploy artifact checks (pull_request) Successful in 20s
Details
CI / Docker image build (pull_request) Successful in 3m37s
Details
CI / Root app checks (pull_request) Successful in 7m18s
Details
CI / Mac foundation hermetic checks (pull_request) Successful in 5m0s
Details
CI / Required checks (pull_request) Successful in 22s
Details
5853749f42 
M4-criterium 2 vraagt het run-log van een echte taakjob als tweede fixture. Dit is dat
log: TASK_IMPLEMENTATION-job cmun0mmuu000gue17ixshy7c4 op max2, 61 regels en 127145 bytes,
byte-exact overgenomen. De eerste fixture is een idea-chat zonder containers en gates;
dit log heeft prepare, drie run_tests, twee gates (eerst rood, dan groen), twee TOOL_ERRORs
en drie door de schrijver afgekapte resultaten.

De nieuwe tests pinnen wat de parser ervan maakt, met verwachtingen uit JSON.parse van de
fixture-regels en niet uit de parser zelf:
- de samenvatting: job, model, 12 beurten, 204419 ms, exitcode 0, afgesloten;
- de volgorde van de eventsoorten over alle 82 events;
- de worktree-regel en de jobstappen als meta-events, in volgorde;
- prepare en de twee gates als tool-call plus tool-result (isError, fullLength, en het
  begin van de aanroep durationMs voor zijn resultaat);
- de drie run_tests-containers als raw-regel met exitcode en seconden;
- de 14 modeltoolaanroepen (ids, namen, argumenten) en hun resultaten;
- het done-resultaat, beide gates en het einde van de modellus.

De parser is niet gewijzigd en de bestaande tests zijn ongewijzigd.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
s4m-codex-reviewer approved these changes 2026-09-29 21:16:37 +02:00
s4m-codex-reviewer left a comment
Copy link

Verdict: APPROVED

Findings

  • Geen bevindingen met blocker-, error- of comment-severity.

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

De nieuwe fixture is gericht en de tests verifiëren de relevante parsercontracten: eventvolgorde, metadata, containers, tool-calls/-results, truncatie en eindstatus.

# Verdict: APPROVED ## Findings - Geen bevindingen met blocker-, error- of comment-severity. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De nieuwe fixture is gericht en de tests verifiëren de relevante parsercontracten: eventvolgorde, metadata, containers, tool-calls/-results, truncatie en eindstatus.
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!275
No description provided.