You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR docs: plan proof output retention #199 implements the load-bearing Phase 1 path for task transcripts, but the review keeps several medium/low items open as follow-up rather than merge blockers.
Phase 0's consumer-inventory appendix did not land, so future edits still need a stable map of TaskState.resultText consumers.
The review calls out missing end-to-end stitched-prompt coverage, artifact byte-parity coverage, findings-dir parity coverage, noisy resume fallback diagnostics, and source/doc cleanup items.
Acceptance Criteria
Add or inline a durable consumer inventory for TaskState.resultText and transcript consumers under packages/proof/src/**.
Replace source-grep tests with behavioral tests where the review calls that out, especially downstream skip on BUDGET-EXCEEDED and resume/transcript consumer wiring.
Add artifact parity coverage for ${taskId}.md vs ${taskId}.stream.txt modulo the markdown metadata header.
Add diagnostics for missing registered transcript files after resume, so fallback to bounded resultText is visible to operators and/or downstream prompts.
Address the low-severity stale comments/docs noted in the review where they are adjacent to the hardening work.
This issue is linked back from docs/proposals/proof-output-retention-plan.md and, where relevant, docs/proposals/proof-output-retention-review.md so the repo artifacts and GitHub issues remain navigable in both directions.
Summary
Close the residual Phase 0/1 hardening gaps that were explicitly left outside the Phase 1 merge bar.
Source Context
Why This Exists
TaskState.resultTextconsumers.Acceptance Criteria
TaskState.resultTextand transcript consumers underpackages/proof/src/**.BUDGET-EXCEEDEDand resume/transcript consumer wiring.${taskId}.mdvs${taskId}.stream.txtmodulo the markdown metadata header.resultTextis visible to operators and/or downstream prompts.Verification
pnpm -F @flatbread/proof testpnpm verifyRelated Phase Issues
Artifact Linkage
This issue is linked back from
docs/proposals/proof-output-retention-plan.mdand, where relevant,docs/proposals/proof-output-retention-review.mdso the repo artifacts and GitHub issues remain navigable in both directions.