Repository navigation
Withdrawn - #94783
Closed
gabmfranco-ds wants to merge 1 commit into
Closed
Withdrawn#94783gabmfranco-ds wants to merge 1 commit into
gabmfranco-ds wants to merge 1 commit into
Conversation
… rewind leak fix, --resume @platform Measured on a live corpus with a 20-question recall harness: hit@3 went from 14/20 to 19/20. Four changes: 1. search_messages: a multi-term query that finds nothing under FTS5's implicit AND is retried with the same terms OR-joined (BM25 still ranks). Queries with explicit operators, phrases, prefix globs, or CJK are never rewritten. 2. session_search discover: two-pass scan — interactive sources first, demoted (cron) sources only top up the remaining window. Post-hoc demotion (NousResearch#19434) could not help when >_DISCOVER_SCAN_LIMIT cron rows filled the window before interactive hits were ever fetched. 3. get_messages_around / get_anchored_view: rewound rows (active=0, compacted=0) no longer surface through context windows and bookends. search_messages already hid them as hits, but the surrounding window leaked exactly the content the user's rewind removed. The anchor itself stays visible; compaction-archived rows (NousResearch#38763) stay visible. 4. CLI: --resume @telegram / @WhatsApp / @<platform> resumes that platform's most recent session in the terminal — the reverse direction of /handoff, following the --resume @claude/@codex precedent. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JUZHShW7jYpF4PxrjYLc4Q
alt-glitch
added
type/bug
The zero-result OR-relaxation part duplicates the active SessionDB retry in #88030. Please extract the distinct discover, rewind-window, and resume changes for independent review. |
gabmfranco-ds
changed the title
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.
Withdrawn.