ref(memory)!: remove MemoryStore - #1651
Open
dcramer wants to merge 2 commits into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
dcramer
force-pushed
the
codex/1600-memory-viewer
branch
2 times, most recently
from
August 23, 2026 03:02
8c8345f to
8d124e1
Compare
dcramer
force-pushed
the
codex/1600-memory-store
branch
from
August 23, 2026 03:41
ef93bb5 to
bd83622
Compare
dcramer
changed the title
![sentry-warden[bot]](https://avatars.githubusercontent.com/in/2755838?s=60&v=4){kind=small}
dcramer
marked this pull request as ready for review
August 23, 2026 17:02
dcramer
force-pushed
the
codex/1600-memory-store
branch
from
August 23, 2026 20:17
6eb1f3a to
02aa5c1
Compare
dcramer
changed the title
dcramer
force-pushed
the
codex/1600-memory-store
branch
from
August 23, 2026 20:45
02aa5c1 to
71088bf
Compare
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.
Removes the published
MemoryStoreandcreateMemoryStoreAPI. Memory creation remains one function because it owns idempotency, duplicate handling, preference replacement, and embedding writes. Search and recall remain one function because they share text and vector ranking. Listing and removal now query Drizzle directly in their tool handlers.Expired-memory cleanup stays shared because creation, retrieval, and session processing all use the same bounded update and embedding deletion. User and Location attribution, linked Identity access, search order, and product responses stay unchanged.
Tests and evals use
@sentry/junior-memory/testingto create memory and read or clear test data. This test entry does not expose database tables or add names to the production package root.This is a breaking change only for code that imports the removed store API. The old names are not kept. The supported plugin and REST APIs do not change.
Refs #1600