fix(memory): exclude rejected approaches from native memory - #76
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
You have reached your Codex usage limits for security reviews. Please try again later. |
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (2 files)
Reviewed by gemini-3.7-flash · Input: 140K · Output: 18.4K · Cached: 762.5K |
Summary
Rust memory paths now fail closed around rejected approaches. Native state sync, rendering, and memory tools accept only explicit positive categories, so future negative categories cannot leak into agent memory by default.
This is the second PR in the anti-memory stack. It builds on the typed model in PR #75; PR #77 adds writers and PR #78 adds warning retrieval.
Design
Validation
bun run check:allStack created with GitHub Stacks CLI • Give Feedback 💬