Skip to content

injection guard: never arm on a quoted spill path; lift when the file is gone - #338

Merged
plur9 merged 1 commit into
mainfrom
fix/injection-guard-false-arm
Nov 14, 2026
Merged

plur9 merged 1 commit into
mainfrom
fix/injection-guard-false-arm

Conversation

@plur9

@plur9 plur9 commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

A session start returned normally, but an injected memory quoted the old overflow message and a placeholder spill path (<project>/tool-results/mcp-plur-plur_session_start-<ts>.txt). The guard armed on the quote and blocked every tool for the rest of the session, waiting for a file that never existed (reading a missing file errors, so clear never ran). Now: arm only on a spill file that exists; the check lifts a gate whose file is gone. Two tests written first and seen to fail; guard suites 117 passed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MrfsFw8X6iNzKLGMKxzKDR

…file is gone

On 2026-11-14 a session start returned normally, but an injected memory quoted
the old overflow message and a placeholder spill path. The guard armed on the
quote and blocked every tool (Bash, agent messages) for a file that never
existed; reading a missing file is an error, so the clear hook never ran.
Now it arms only on a spill file that exists, and the check lifts a gate whose
file is gone. Two tests written first and seen to fail; guard suites 117 passed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MrfsFw8X6iNzKLGMKxzKDR
@plur9
plur9 merged commit 697633a into main Nov 14, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant