Skip to content

fix(memory): reject one-off/topic content in update_memory tool prompt #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
GurneeshBudhiraja merged 1 commit into from
Jul 15, 2026
Merged

fix(memory): reject one-off/topic content in update_memory tool prompt #8

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 14 additions & 1 deletion tools/user_notes_tool.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -740,7 +740,20 @@ def _check_user_notes_requirements() -> bool:
"on something, ALWAYS capture it. Do NOT save the topic of the current "
"conversation, what the user is working on or asking about right now, their "
"current goals/tasks, one-off request parameters, transient values, or small "
"talk — conversation content is not memory. Most turns need NO save, and no save "
"talk — conversation content is not memory. "
# Contrastive examples — mirror of MEMORY_CONTRASTIVE_EXAMPLES in the web app
# (lib/services/memory/memory-policy.ts). Naming the exact leak modes stops
# a one-off task instruction being saved as if it were a durable preference.
"Examples — do NOT save for: a question ('how does Q3 revenue compare by "
"region?'), or a one-off request scoped to the current task ('make this chart "
"blue', 'add the total row and include Q4', 'break revenue down by region for "
"this analysis') — transient even when phrased as commands with "
"'add/include/show'. DO save for a fact that outlives this task: a standing "
"preference ('always show revenue in millions'), a business definition/metric "
"('fiscal year ends in March'), a hard constraint, or an explicit correction "
"('actually, churn should exclude trials'). When unsure whether something is "
"standing or one-off, do NOT save it. "
"Most turns need NO save, and no save "
"means DO NOT CALL THIS TOOL — never call it with placeholder content like "
"'nothing' or 'none'; only call when you have a real fact to write. The current "
"memory is shown in your context under 'Project Memory (AI-maintained)'. Never "
Expand Down
Loading