test(mcp): prove tracedecay_source_edit_reconcile behavior - #1700
ScriptedAlchemy wants to merge 1 commit into
Conversation
Call the production MCP tool and assert retained-unknown refusal, rolled-back confirmation, committed confirmation, and exact retry against live file bytes. Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
|
|
Closing as superseded. A sibling change proving the same tool landed first as #1648. Evidence from the triage: git log origin/master --diff-filter=A -- mcp_handler_test/source_edit_reconcile_test.rs => f241ad3, carried by b33e5a1 'merge: PR #1648 test(mcp): prove tracedecay_source_edit_reconcile behavior'; same path added in both, 8 conflict hunks Reopen if the landed version misses an assertion this branch carries; name it and it can be ported as a small follow-up. |
Summary
tracedecay_source_edit_reconcilethrough the production MCP dispatch, not a schema pin or unit-level journal fixture.Motivation
The MCP suite covered source-edit apply and retry, but never called
tracedecay_source_edit_reconcile. Agents that seeEffectUnknownhave to conclude the retained effect from the live candidate file. This test is that journey.Changes
crates/tracedecay/tests/mcp_suite/mcp_handler_test/source_edit_reconcile_test.rsdrives the real tool. Publication is interrupted by making the candidate directory unwritable after the journal is durable, which is the productionEffectUnknownboundary. Assertions are the returned messages, receipt fields, and the literal file bytes.Test plan
cargo test -p tracedecay --features test-transport --test mcp_suite -- source_edit_reconcilereports a non-zero countChecklist
CHANGELOG.mdnot updated: test-only, no user-facing contract change.envfiles included