Skip to content

feat(memory): split remote scans into ours and theirs - #156

Draft
patrick-ogrady wants to merge 1 commit into
mainfrom
prefer-our-memory
Draft

feat(memory): split remote scans into ours and theirs#156
patrick-ogrady wants to merge 1 commit into
mainfrom
prefer-our-memory

Conversation

@patrick-ogrady

@patrick-ogrady patrick-ogrady commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

Remote scans now retrieve the authenticated namespace and all other visible namespaces independently, then present both groups to the agent as ours and theirs. This prevents either source from crowding out the other while leaving the agent free to choose the strongest guidance.

  • make each store scan operate on one namespace scope and return a candidate vector; an empty vector means no match
  • issue own and others requests sequentially on one bookmark chain, with each response independently ranked and bounded
  • derive scan scopes from the authenticated server identity and validate response ownership and ordering
  • push scope filtering and a configurable per-scope result limit into the Cloudflare Worker
  • render only the current grouped scan shape in the TUI and update the memory documentation

Local scans continue to use the selected local backend and populate only ours. Remote failures remain all-or-error and never fall back locally.

@patrick-ogrady patrick-ogrady changed the title tune(memory): Return ours and theirs to agent feat(memory): split remote scans into ours and theirs Aug 16, 2026
@codspeed-hq

codspeed-hq Bot commented Aug 16, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 18 untouched benchmarks
⏩ 31 skipped benchmarks1


Comparing prefer-our-memory (6926bc8) with main (ee5b1ca)

Open in CodSpeed

Footnotes

  1. 31 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

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