fix: quarantine corrupt goal state - #43
Conversation
danyel117
left a comment
There was a problem hiding this comment.
Claude Opus final review: APPROVE.
The first review found that a quarantine-write failure could reintroduce the original every-prompt failure loop. Commit 833a391 resolves it structurally: quarantine failures are reported and recovery continues, while only a successfully observed concurrent content replacement remains fail-closed. Verification read errors and ENOENT no longer create an indefinite failure.
Opus independently reproduced the source-change refusal, confirmed exact preservation for NUL/whitespace/BOM content, verified listener scoping/disposal/failure isolation, rebuilt dist byte-identically, and ran the complete gate: 221 tests, lint, typecheck, build, and pack. All GitHub CI checks are also green. No merge-blocking findings remain.
Summary
goals.json.corrupt-<timestamp>-<uuid>artifacts before recovered state is writtenVerification
bun run lintbun run typecheckbun test(221 passing)bun run buildbun run pack:dry-run833a391with regression coverageCloses #35
AI assistance: implemented and reviewed with OpenCode agents and independently reviewed with Claude Opus.