fix(mcp): reject stale AST coordinates after source changes - #2105
fix(mcp): reject stale AST coordinates after source changes#2105lorenzozanee wants to merge 2 commits into
Conversation
Signed-off-by: lorenzozanee <wyz0707@proton.me>
|
Thanks for opening this — it has been seen, and it is queued. This note is automated, but it is not a brush-off: it exists so you know where your PR stands instead of having to guess from silence. Current review status: working through a backlog. What that means for this PR, concretely:
Things that will genuinely speed it up whenever review does happen:
If this fixes a bug, a reproduction we can run is worth more than a description of the symptom. Thanks for contributing, and sorry in advance for the wait. |
Signed-off-by: lorenzozanee <wyz0707@proton.me>
|
Thank you for addressing stale AST coordinates and including coverage in tests/test_mcp.c. This is queued as a high-priority correctness fix. We need more time to review the source-change checks and their behavior across the snippet/outline paths before giving a decision. The review queue is currently full, so detailed feedback may take a little time. We are working through it carefully and appreciate the work you have put into supporting the project. Thank you for your patience. |
Fixes #1750. Source edits can invalidate AST line ranges, so this update uses the canonical file metadata on Windows before exposing stored coordinates, rejects stale snippet reads, and preserves raw search matches without stale AST attribution.