feat(docs): guard markdown writes against orphaned comments#747
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 11, 2026, 2:18 AM ET / 06:18 UTC. Summary Reproducibility: unclear. The review failed before ClawSweeper could establish a reproduction path. Review metrics: none identified. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Risk before merge
Maintainer options:
Next step before merge
Review detailsBest possible solution: Retry the Codex review after fixing the execution failure. Do we have a high-confidence way to reproduce the issue? Unclear. The review failed before ClawSweeper could establish a reproduction path. Is this the best way to solve the issue? Unclear. Retry the review first so ClawSweeper can evaluate the actual issue and fix direction. AGENTS.md: unclear because the file could not be read completely. Codex review notes: model internal, reasoning high; reviewed against 22c97dea8707. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
Summary
--check-orphanstodocs write --replace --markdowndocs comments locateentity and whitespace matching to identify currently anchored open comments whose quotes would disappearVerification
make ciclawdbot@gmail.com: preserving replacement succeeded; droppingDROP PHRASEreturned exit 11; JSON identified the exact comment; readback confirmed the blocked write left the document unchangedCloses #691