feat(mem0): add the memory port v2 store proven on live Mem0 - #553
Open
outof-place wants to merge 2 commits into
Open
outof-place wants to merge 2 commits into
outof-place wants to merge 2 commits into
Conversation
outof-place
added
the
changelog: skip
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
This review includes 23 billable files and costs up to $5.75. Or wait 53 minutes for your next included review. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (23)
Comment |
This branch was successfully deployed
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.
Stage 5 of the agent memory rebuild (plan:
docs/plans/2026-09-23-memory-package.md, D2, D5, D6, Q2, Q6, Q14). This adds a Mem0 store for memory port v2 and checks it against live Mem0. Nothing calls it yet; stage 6a wires it in. The v1 adapter is untouched.integrations/mem0/store.tsaddsmem0MemoryStore(ctx)andreadMem0Identity(ctx). The identity comes fromGET /v1/ping/and is the{orgId, projectId}a run pins (Q6).integrations/mem0/store-client.tsmakes every v3 call and maps statuses:key_rejected.upgrade_requiredarequota.rate_limited.timeout, and a connection that provably never opened isunreachable. All of these areunavailableand carry the status.rejected.heldis complete: it pages to 205 and beyond, and refuses if the listing changes mid-read. It is ordered by anaddedAtstamp.threshold0.pendingonly when Mem0 saysPENDING.delete_linked, sendsinclude_merged, and names every memory the cascade took. A cascade that reached beyond the subject is logged.*or%is refused and nothing is sent.store.test.ts: 130 tests in the package. They include the stage 2 conformance suite run on a fake, which was built from response bodies recorded live (test-fixtures/live-*, each with a source note).live-probe.ts,pnpm probe:live) used app_idai-workflow-probeand probe-only user ids. When it finished, both runs listed a count of 0.immutablememory returned 200 and changed the text, so new writes do not setimmutable.Gate: one review pass. One fixer addressed its two majors: forget missing merged and cascaded memories, and a post-send drop being reported as "nothing landed". The same fixer fixed the minors.
No user-visible change, so there is no changelog entry.
🤖 Generated with Claude Code