Skip to content

Fail open when memory recall task errors#1280

Open
JohnKruse wants to merge 1 commit intoagent0ai:mainfrom
JohnKruse:codex/memory-recall-fail-open
Open

Fail open when memory recall task errors#1280
JohnKruse wants to merge 1 commit intoagent0ai:mainfrom
JohnKruse:codex/memory-recall-fail-open

Conversation

@JohnKruse
Copy link

Summary

  • fail open when the async memory recall task errors
  • log a warning instead of aborting the request path
  • add a focused regression test for RecallWait

Why

Memory recall is non-essential. If embeddings or the recall backend fail, normal request handling should continue instead of failing the entire reply path.

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