fix(prompts): bewaar verdict-trace op job.summary (review-done zonder summary)

janpeter commented

2026-06-10 19:54:20 +02:00

Owner

Canary A (Phase 3, SPEC_REVIEW) toonde dat de verdict-trace die submit_review op ClaudeJob.summary schrijft (SPEC_REVIEW REJECTED (7 findings): ...) direct daarna werd overschreven: stap 2 van het prompt-afsluitprotocol liet de runner update_job_status(done, summary) aanroepen, en die done-summary wint. Zelfde patroon bestond al bij PR_REVIEW (post_pr_review-trace, Phase 2 verdict-led summary).

Fix (prompt-only, 6 bestanden): stap 2 wordt update_job_status({ job_id, status: done }) zonder summary, in spec/task/pr review-prompts (claude- en codex-varianten). update_job_status laat job.summary bij weglating ongemoeid (spread-guard summary !== undefined), dus de trace van submit_review/post_pr_review blijft staan.

Veiligheid: review-jobs zijn source=MANUAL en passeren de done-gate zonder summary-eis (geen verify-gate, geen PLAN_CHAT-summary-eis) — broncheck op update-job-status.ts.

Verify: 750/750 tests, tsc clean. Canary B (TASK_REVIEW) valideert de trace live na rebuild.

🤖 Generated with Claude Code

Canary A (Phase 3, SPEC_REVIEW) toonde dat de verdict-trace die submit_review op ClaudeJob.summary schrijft (SPEC_REVIEW REJECTED (7 findings): ...) direct daarna werd overschreven: stap 2 van het prompt-afsluitprotocol liet de runner update_job_status(done, summary) aanroepen, en die done-summary wint. Zelfde patroon bestond al bij PR_REVIEW (post_pr_review-trace, Phase 2 verdict-led summary). **Fix (prompt-only, 6 bestanden):** stap 2 wordt update_job_status({ job_id, status: done }) zonder summary, in spec/task/pr review-prompts (claude- en codex-varianten). update_job_status laat job.summary bij weglating ongemoeid (spread-guard summary !== undefined), dus de trace van submit_review/post_pr_review blijft staan. **Veiligheid:** review-jobs zijn source=MANUAL en passeren de done-gate zonder summary-eis (geen verify-gate, geen PLAN_CHAT-summary-eis) — broncheck op update-job-status.ts. **Verify:** 750/750 tests, tsc clean. Canary B (TASK_REVIEW) valideert de trace live na rebuild. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 1 commit

2026-06-10 19:54:20 +02:00

fix(prompts): review-afsluiting done zónder summary — bewaar de verdict-trace a76953e3e1

Canary A (Phase 3) toonde dat de runner-done-call de verdict-trace op
ClaudeJob.summary overschrijft: submit_review (en post_pr_review) zetten
'<KIND> <verdict> (n findings): …' resp. 'PR review <event> …' op de job,
waarna stap 2 van het afsluitprotocol update_job_status(done, summary)
liet aanroepen — die laatste won. update_job_status laat summary bij
weglating ongemoeid, dus de fix is prompt-only: stap 2 wordt done zonder
summary, in alle zes review-prompts (spec/task/pr × claude/codex).
MANUAL-jobs passeren de done-gate zonder summary-eis (bron-check verified).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

janpeter merged commit 6a4cae58f7 into main

2026-06-10 20:12:52 +02:00

janpeter referenced this pull request from a commit

2026-06-10 20:12:54 +02:00

Merge pull request 'fix(prompts): bewaar verdict-trace op job.summary (review-done zonder summary)' (#48) from fix/review-done-preserve-summary-trace into main

Rows
Columns

fix(prompts): bewaar verdict-trace op job.summary (review-done zonder summary) #48