fix: preserve mutation outcomes and bound reference replies - #45
Merged
Merged
Conversation
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.
When a trash move completes near its request deadline, the Gateway can replace the known file location with a timeout. Reference queries also hide their existing Host limit from callers, and large replies have no final text budget.
This change:
Validation:
npm run checkpassed all 460 tests with no failures or skips, including fresh-process stdio, native builds and delivery verification.npm run test:error-contractspassed 17 scenarios. Focused regressions exercise real file mutations and cancellation, a real Roslyn Host with 120 references, escaped long previews, candidate identity and metadata overflow. Initial regressions failed on the old behavior and passed after the fixes.The output budget does not bound Roslyn search time or memory. Client cancellation/disconnection cannot guarantee delivery of the final mutation result. Installed Skills and an already-running MCP connection are not updated by rebuilding. This is author self-review; required PR-head CI remains the merge gate.