Skip to content

Fix local memory fallbacks and temporary topic scoring#32

Closed
elinawang626-design wants to merge 3 commits intoNevaMind-AI:mainfrom
elinawang626-design:codex/memory-fallback-fixes
Closed

Fix local memory fallbacks and temporary topic scoring#32
elinawang626-design wants to merge 3 commits intoNevaMind-AI:mainfrom
elinawang626-design:codex/memory-fallback-fixes

Conversation

@elinawang626-design
Copy link
Copy Markdown

Summary

This PR fixes several fallback inconsistencies around local memory and temporary-topic handling.

Changes

  • unify proactive memu_memory with the shared executor so proactive mode and normal chat use the same fallback behavior
  • require complete remote memU config before attempting remote retrieval
  • fall back to local controlled memory when remote retrieve returns a non-OK response or throws
  • fall back to local controlled memory when remote memorization crashes before creating a task
  • fall back to heuristic temporary-topic scoring when no Claude API key is configured
  • add regression tests for memory fallback behavior and heuristic temporary-topic transitions

Verification

  • npm run typecheck
  • npm test

Copy link
Copy Markdown
Author

Closing this in favor of #33, which contains the same bugfix scope rebased into a clean upstream-compatible PR with only the intended 6-file change set.

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