No related issue: split out of #5191 after a repro pass on planning 0.46.0 (origin/main at 4d1d9d8).
Summary
Finding 16 of #5191, reported against planning 0.42.0: The user wrapped up without addressing unticked commitments or an unread Claude note, and the export then contradicted their intent.
Fix
Warn or gate Wrap up on open assumptions; coordinate with #3941.
Verification
Repro pass on origin/main, playwright-cli driving Chromium against a copy of plugins/planning/surface/, plus round.sh, watch.sh and curl. Verdict: reproduces.
Most of finding 16 is already fixed on origin/main (commitment checkboxes and "Confirm all (N)" on the wrap-up view, and an unanswered Claude reply as a Loose end). What remains: Wrap up sends unconditionally (plugins/planning/surface/index.html:1579); the row beside the button warns only "Understanding not confirmed yet." (:1039-1041); openAssumptions() (:541) is not consulted there. Overlaps open #3941 (finalize action that recaps and validates). The part about 21 commitments exported as unconfirmed named risks is on #4653.
Related
Refs #5191. #4653 tracks the commitment and export gaps behind findings 9, 17 and part of 18.
No related issue: split out of #5191 after a repro pass on planning 0.46.0 (origin/main at 4d1d9d8).
Summary
Finding 16 of #5191, reported against planning 0.42.0: The user wrapped up without addressing unticked commitments or an unread Claude note, and the export then contradicted their intent.
Fix
Warn or gate Wrap up on open assumptions; coordinate with #3941.
Verification
Repro pass on origin/main,
playwright-clidriving Chromium against a copy ofplugins/planning/surface/, plusround.sh,watch.shandcurl. Verdict: reproduces.Most of finding 16 is already fixed on origin/main (commitment checkboxes and "Confirm all (N)" on the wrap-up view, and an unanswered Claude reply as a Loose end). What remains: Wrap up sends unconditionally (
plugins/planning/surface/index.html:1579); the row beside the button warns only "Understanding not confirmed yet." (:1039-1041);openAssumptions()(:541) is not consulted there. Overlaps open #3941 (finalize action that recaps and validates). The part about 21 commitments exported as unconfirmed named risks is on #4653.Related
Refs #5191. #4653 tracks the commitment and export gaps behind findings 9, 17 and part of 18.