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

fix(worker-logs): fullLength van een harness-toolresultaat telt de foutcode-tag nooit mee #274

Merged
janpeter merged 1 commit from fix/harness-fulllength-fallback into main 2026-09-29 19:52:07 +02:00
Conversation 1 Commits 1 Files changed 2 +25 -4
janpeter commented 2026-09-29 19:34:14 +02:00
Owner
Copy link

Vervolg op #273: deze commit (7d1dcceed) werd gepusht terwijl #273 al gemerged werd (merge om 19:30:23, push een paar seconden later), en zit daardoor niet op main.

Wat

harness.tool_result zonder contentLength: fullLength is nu de lengte van de content in plaats van de body. Daardoor telt de tag [<errorCode>] nooit mee, en betekent fullLength op beide paden de lengte van de tooluitvoer, zoals spec §5.4 hem definieert (agent-harness docs/specs/2026-09-28-harness-run-logging-design.md, rij harness.tool_result). Het pad mét contentLength is ongewijzigd. Achtergrond: review 856 en de reactie daarop in #273.

De writer schrijft contentLength altijd, dus in de praktijk verandert er niets aan wat de dashboards tonen. Dit maakt alleen de betekenis eenduidig.

Verificatie

  • RED eerst: fullLength 17 tegen verwacht 4. Daarna test/parse-worker-log.test.ts 68/68 en npm run typecheck groen.
  • npm test: alleen de twee bekende macOS-platformbestanden falen (caddy-write-wrapper, db-access-policy-bundle-flow), zoals op main.
  • Nieuwe inline test voor errorCode + contentLength, zowel niet afgekapt als door de writer afgekapt. De fixture-test (call_uwrob4gs, 71) is ongewijzigd.
  • Scoped re-review: schoon.

Uitrol: geen aparte redeploy nodig. Dit gaat mee met de volgende redeploy_ops_dashboard.

🤖 Generated with Claude Code

Vervolg op #273: deze commit (`7d1dcceed`) werd gepusht terwijl #273 al gemerged werd (merge om 19:30:23, push een paar seconden later), en zit daardoor niet op main. ## Wat `harness.tool_result` zonder `contentLength`: `fullLength` is nu de lengte van de content in plaats van de body. Daardoor telt de tag `[<errorCode>] ` nooit mee, en betekent `fullLength` op beide paden de lengte van de tooluitvoer, zoals spec §5.4 hem definieert (agent-harness `docs/specs/2026-09-28-harness-run-logging-design.md`, rij `harness.tool_result`). Het pad mét `contentLength` is ongewijzigd. Achtergrond: review 856 en de reactie daarop in #273. De writer schrijft `contentLength` altijd, dus in de praktijk verandert er niets aan wat de dashboards tonen. Dit maakt alleen de betekenis eenduidig. ## Verificatie - RED eerst: `fullLength` 17 tegen verwacht 4. Daarna `test/parse-worker-log.test.ts` 68/68 en `npm run typecheck` groen. - `npm test`: alleen de twee bekende macOS-platformbestanden falen (`caddy-write-wrapper`, `db-access-policy-bundle-flow`), zoals op main. - Nieuwe inline test voor `errorCode` + `contentLength`, zowel niet afgekapt als door de writer afgekapt. De fixture-test (`call_uwrob4gs`, 71) is ongewijzigd. - Scoped re-review: schoon. Uitrol: geen aparte redeploy nodig. Dit gaat mee met de volgende `redeploy_ops_dashboard`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(worker-logs): fullLength van een harness-toolresultaat telt de foutcode-tag nooit mee
All checks were successful
CI / Select checks (pull_request) Successful in 24s
Details
CI / Ops-agent checks (pull_request) Successful in 42s
Details
CI / DB access operator (pull_request) Successful in 1m27s
Details
CI / Deploy artifact checks (pull_request) Successful in 39s
Details
CI / Docker image build (pull_request) Successful in 1m27s
Details
CI / Root app checks (pull_request) Successful in 8m34s
Details
CI / Mac foundation hermetic checks (pull_request) Successful in 2m19s
Details
CI / Required checks (pull_request) Successful in 21s
Details
7d1dcceedc 
Spec §5.4 definieert fullLength van harness.tool_result als contentLength, de lengte
van de volledige tooluitvoer. Het `[<errorCode>] `-voorvoegsel is een aantekening van
de parser en geen tooluitvoer. Met contentLength klopte dat al; zonder contentLength
viel fullLength terug op de lengte van de body en telde de tag mee (17 voor
`[TOOL_ERROR] boom`). De terugval is nu de lengte van de content zelf (4), zodat
fullLength altijd de lengte van de tooluitvoer betekent.

Het pad met contentLength is ongewijzigd en heeft nu een eigen test voor een resultaat
met foutcode, zowel niet afgekapt als door de schrijver afgekapt.

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

APPROVED

Findings

  • Geen findings.

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

De correctie houdt fullLength consistent met de contractbetekenis (de volledige tool-uitvoer) en dekt zowel resultaten met als zonder contentLength af.

## APPROVED ### Findings - Geen findings. Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De correctie houdt `fullLength` consistent met de contractbetekenis (de volledige tool-uitvoer) en dekt zowel resultaten met als zonder `contentLength` af.
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!274
No description provided.