Skip to content

fix: bind M5.2 reviewer digest in tool schema - #142

Merged
veil-chow-fyaic merged 1 commit into
mainfrom
fix/m52-review-digest-binding
Sep 2, 2026
Merged

veil-chow-fyaic merged 1 commit into
mainfrom
fix/m52-review-digest-binding

Conversation

@veil-chow-fyaic

Copy link
Copy Markdown
Contributor

Summary

  • replace model copying of findingDigest from the read-tool output with a coordinator-bound schema constant
  • keep exact reviewer path and counterexample validation against the detached checkout
  • retain action/lifecycle/Git/verifier digest bindings and wrong-content negative coverage
  • record attempt 13 and its selected-vs-completed diagnosis

Evidence

Attempt 13 emitted 5 selected actions but only 4 completed actions, proving the second reviewer callback was rejected before completion. The read callback completed. This removes the redundant copying obligation without weakening content verification.

Validation

  • focused M5.2 suite: 17/17
  • real-effects positive and wrong-finding negative both pass
  • documentation lint: 114 files, zero issues
  • CI runs the full 385-test suite

Advances #91.

@veil-chow-fyaic
veil-chow-fyaic merged commit 49230f3 into main Sep 2, 2026
2 checks passed
@veil-chow-fyaic
veil-chow-fyaic deleted the fix/m52-review-digest-binding branch September 2, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant