LearnNewsExamplesServices
Frontmatter
id10951
titleAdd team-private retrieval integration tests for Memory Core
stateClosed
labels
enhancementaitestingarchitecture
assigneesneo-gpt
createdAtMay 8, 2026, 12:48 PM
updatedAtMay 12, 2026, 4:09 AM
githubUrlhttps://github.com/neomjs/neo/issues/10951
authorneo-gpt
commentsCount0
parentIssue10945
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtMay 9, 2026, 10:59 PM

Add team-private retrieval integration tests for Memory Core

Closedenhancementaitestingarchitecture
neo-gpt
neo-gpt commented on May 8, 2026, 12:48 PM

Context

Parent epic #10945 needs actionable deployment-pipeline coverage for multi-user Memory Core. The current integration suite proves basic alice/bob Chroma isolation, but not the team/private retrieval policy surface that #10010/#10011 track.

Duplicate sweep before filing:

  • gh issue list --search "team private retrieval Memory Core Native Edge Graph" found adjacent epics #10016/#10691/#9999 and #10945, but no focused integration child.
  • KB ticket search for the full multi-user deployment integration framing returned no conceptual duplicate.
  • This ticket is the deployed-shape proof linked to the policy tickets, not a replacement for them.

The Problem

A multi-user Memory Core deployment needs more than per-user isolation. Team/private retrieval semantics must be proven through real MCP requests across the same deployed stack agents will use. Otherwise Chroma metadata filtering, session summary retrieval, and Native Edge Graph tenant isolation can drift separately.

The Architectural Reality

Relevant surfaces:

  • #10010 — team vs private context retrieval flag.
  • #10011 — Native Edge Graph tenant isolation semantics.
  • #10016 — Multi-Tenant Identity & Data Privacy.
  • Current integration fixture and mcpClient.mjs identity helpers.
  • Memory Core tools: raw memory queries, session memory/session summary reads, and graph/topology query paths.

The Fix

Add integration tests that seed memories and graph records under at least two users and one shared/team context, then verify retrieval behavior through real MCP calls.

The first pass should cover the smallest meaningful matrix:

  • private memory is visible only to its owner;
  • team/shared memory is visible to authorized team context;
  • unrelated user cannot read private/team data;
  • legacy/no-user data behavior is explicit and tested or deferred.

Contract Ledger Matrix

Target Surface Source of Authority Proposed Behavior Fallback Docs Evidence
Raw memory retrieval #10010, #10016 Honors private/team visibility under deployed MCP calls If team context is not implemented, test private isolation and mark team part blocked MultiTenant/SharedDeployment docs if touched Integration query matrix with owner/team/unrelated users
Session summaries #10010, #10945 Retrieval respects same identity policy as raw memory If summaries remain disabled in fixture, document dependency on #10948/#10813 MemoryCore docs if touched Summary read assertions or explicit blocked dependency
Native Edge Graph #10011, #10016 Graph query paths apply tenant/team constraints If graph RLS is not available, file precise substrate follow-up Graph/privacy docs Integration graph query assertions

Acceptance Criteria

  • Integration fixture can seed at least owner/private and unrelated-user Memory Core records.
  • Private retrieval is denied for unrelated user through real MCP calls.
  • Team/shared retrieval behavior is tested or explicitly blocked by a named missing substrate.
  • Native Edge Graph retrieval semantics are covered or explicitly linked to #10011 as a blocker.
  • Legacy/no-user data behavior is documented in the test or Related section.
  • PR body includes deliberate cross-tenant regression proof.

Out of Scope

  • Replacing #10010/#10011 policy work.
  • Designing a full organization/team administration UI.
  • Weakening default privacy semantics to make tests easier.

Related

Parent: #10945 Adjacent: #10010, #10011, #10016, #10691, #9999

Origin Session ID: c02fbf4e-870c-44c0-ba7e-e9ffacce094b

Retrieval Hint: query_raw_memories(query="team private retrieval Memory Core integration Native Edge Graph tenant isolation")

tobiu referenced in commit a77adf0 - "docs(agentos): v13 architectural path strategy document (#10957) (#10958) on May 8, 2026, 2:18 PM
tobiu closed this issue on May 9, 2026, 10:59 PM
tobiu referenced in commit 2591a76 - "feat(ai): add team-private retrieval integration proof (#10951) (#11055) on May 9, 2026, 10:59 PM