fix: preserve newlines and show metadata in auto-recall#602
Open
daniel-rudaev wants to merge 1 commit intoCortexReach:masterfrom
Open
fix: preserve newlines and show metadata in auto-recall#602daniel-rudaev wants to merge 1 commit intoCortexReach:masterfrom
daniel-rudaev wants to merge 1 commit intoCortexReach:masterfrom
Conversation
Two changes to improve auto-recall context quality: 1. sanitizeForContext: replace newlines with literal \n instead of collapsing to spaces. Preserves paragraph structure and meaning, especially important for non-Latin scripts (Hebrew, CJK) where line breaks carry semantic weight. 2. Auto-recall line format: show folder, date, and source from entry metadata instead of category:scope. Users store rich metadata via memory-pro import — the recall display should surface it. Before: - [other:global] all text on one line no structure After: - [Goals] 2024-05-30 (apple_notes) text with\npreserved structure Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
1 participant
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
Two changes to improve auto-recall context injection quality:
Preserve newlines in
sanitizeForContext— replace\r\nwith literal\ninstead of collapsing to a space. Line breaks carry semantic weight, especially in non-Latin scripts (Hebrew, CJK, Arabic) where a newline separates distinct thoughts. Collapsing to space merges them into an unreadable run-on.Show entry metadata in recall line format — display
folder,date, andsourcefrom entry metadata instead of the generic[category:scope]. Users who import memories viamemory-pro importwith rich metadata (folder organization, source tracking, timestamps) lose all of that context in the current display format. The agent can't tell where a recalled memory came from.Before:
After:
Context
Discovered while building an Apple Notes → memory-pro import pipeline. 1,800+ personal notes imported with folder, source, author, and date metadata. Auto-recall surfaced them with no attribution and destroyed the original formatting.
Test plan
\nin recalled text🤖 Generated with Claude Code