Conversation
added 6 commits
September 8, 2026 14:37
…ses needs-decision
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Intent
Le capitaine demande d enchaîner encore deux lots regroupés et précise : « là tu utilises gpt 5.6 ».
Ce lot regroupe trois défauts de la même famille :
fm-send --resolve-keyrefuse des clés que le repli reconnaît pourtant sur des lignesblocked:;resolved [key=...]: ...peut laisser la clé ouverte après un repli complet ;pending-reply-*ignorent silencieusement certaines résolutions manuelles pourtant explicites.Ce brief est en français, ta livraison ne l est pas.
What Changed
fm-send --resolve-key, and captain-held transfers.Risk Assessment
✅ Low: The changes are bounded, preserve reserved-key guards, and implement the accepted resolution, transfer, and reconciliation-routing decisions without additional substantiated defects.
Testing
Classifier, send, drain, transfer, and routing checks passed, with CLI evidence and a baseline regression reproduction. Broader lifecycle runs hit execution deadlines; focused affected selectors completed successfully. No source changes were needed.
Evidence: CLI decision closure, delivery, full replay, and rejected-resolution reconciliation
Source: CLI decision closure, delivery, full replay, and rejected-resolution reconciliation
Evidence: Rejected reserved close: baseline versus target behavior
Source: Rejected reserved close: baseline versus target behavior
Pipeline
Updates from git push no-mistakes
✅ **intent** - passed
✅ No issues found.
✅ **Rebase** - passed
✅ No issues found.
✅ **Review** - passed
✅ No issues found.
✅ **Test** - passed
✅ No issues found.
bash tests/fm-classify-decision-key.test.shbash tests/fm-classify-corr-token.test.shbash tests/fm-send-resolve-key.test.shbash tests/fm-wake-drain-open-decisions.test.shSelected captain-hold lifecycle tests: completion attestation, reserved-key transfer, and answer recording.Seven selected watcher tests covering span closure, decision-owned routing, rejected transfers/resolutions, and ordinary blocker routing.Ran the captain-hold lifecycle suite twice; execution deadlines interrupted broader cases. Re-ran the three change-relevant selectors successfully.Executed real fm-send and fm-wake-drain against an isolated fixture with stubbed terminal transport; captured inbox delivery, persisted resolution records, full replay, and loud rejection of unofficial reserved-key closes.Compared rejected-close classification at base d42016a and target: reproduced silent baseline behavior and verified actionable, decision-owned target behavior.Removed temporary fixtures and confirmedgit status --shortwas clean.✅ **Document** - passed
✅ No issues found.
✅ **Lint** - passed
✅ No issues found.
✅ **Push** - passed
✅ No issues found.