Harden auto-recall with timeout and circuit breaker#254
Open
shardbigwhite-dotcom wants to merge 1 commit intoCortexReach:masterfrom
Open
Harden auto-recall with timeout and circuit breaker#254shardbigwhite-dotcom wants to merge 1 commit intoCortexReach:masterfrom
shardbigwhite-dotcom wants to merge 1 commit intoCortexReach:masterfrom
Conversation
Collaborator
|
Thanks for taking the time to contribute this! Really appreciate it. 🙏 There are merge conflicts with Let us know if you run into any issues! |
AliceLJY
added a commit
to AliceLJY/memory-lancedb-pro
that referenced
this pull request
Mar 23, 2026
…ntrol Adds a new `recallMode` configuration option that provides finer control over auto-recall behavior than the existing boolean `autoRecall` flag: - "full": existing behavior — inject matching memories into context - "summary": lightweight hint only — tells agent how many memories matched without consuming context budget; agent can call memory_recall on demand - "off": disable auto-recall entirely When `recallMode` is set, it takes precedence over `autoRecall`. When neither is set, auto-recall remains off by default (no breaking change). Motivation: in complex development sessions, auto-recall competes with compact/compression for limited context budget, reducing signal-to-noise ratio. Summary mode gives agents awareness of available memories without the context cost. See Anthropic's "Effective Context Engineering" and OpenAI's "Harness Engineering" for the underlying design principles. Related: CortexReach#307 (autoRecallExcludeAgents), CortexReach#254 (auto-recall circuit breaker) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This was referenced Mar 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
before_agent_startwith a small total timeoutWhy
This addresses the failure mode described in #253: when retrieval is unhealthy,
autoRecallshould degrade to "no memory recall" instead of making the request appear hung.Notes