Skip to content

Remove memory hooks from default agent behavior#104

Closed
wmeddie wants to merge 1 commit into
mainfrom
fix/remove-memory-hooks
Closed

Remove memory hooks from default agent behavior#104
wmeddie wants to merge 1 commit into
mainfrom
fix/remove-memory-hooks

Conversation

@wmeddie
Copy link
Copy Markdown
Member

@wmeddie wmeddie commented May 13, 2026

Summary

  • Remove memory hooks (recall/remember/consolidate) from default agent behavior — they made LLM calls before and after every message, confusing models with injected memory text in limited context
  • MemoryHooks struct, MCP tools (search_memory/save_memory), and all memory infrastructure remain available for user-configured use

Changes

  • HooksConfig::default() now returns empty vectors
  • Remove recall hook from conversation processor and task dispatcher
  • Remove remember hook from conversation processor and task dispatcher
  • Remove consolidation hook from reconciler idle loop
  • Remove has_recall_hook/has_remember_hook helper functions
  • Update test to assert hooks are empty by default

Memory hooks (recall/remember/consolidate) made LLM calls before and
after every agent message, confusing models with injected memory text
in limited context. Remove them from defaults.

- HooksConfig::default() now returns empty vectors
- Remove recall hook from conversation processor and task dispatcher
- Remove remember hook from conversation processor and task dispatcher
- Remove consolidation hook from reconciler idle loop
- Remove has_recall_hook/has_remember_hook helper functions
- MemoryHooks struct and MCP tools remain for user-configured use
@wmeddie wmeddie force-pushed the fix/remove-memory-hooks branch from adce62b to 5dd8896 Compare May 13, 2026 15:23
@wmeddie wmeddie closed this May 13, 2026
@wmeddie wmeddie deleted the fix/remove-memory-hooks branch May 13, 2026 15:36
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