Skip to content

test(core): add retrieval leakage regression suite#1038

Merged
kunickiaj merged 1 commit into
mainfrom
05-04-test_core_add_retrieval_leakage_regression_suite
May 5, 2026
Merged

test(core): add retrieval leakage regression suite#1038
kunickiaj merged 1 commit into
mainfrom
05-04-test_core_add_retrieval_leakage_regression_suite

Conversation

@kunickiaj
Copy link
Copy Markdown
Owner

@kunickiaj kunickiaj commented May 5, 2026

Description

Adds a single mixed-domain fixture (`seedMixedScopeFixture`) and uses it to drive leakage regression tests across every memory exposure path: store get/recent/search/timeline/explain, semanticSearch, pack, export, MCP direct reads, and viewer `/api/memory` / `/api/observations` / `/api/pack`. The fixture seeds three memories (local-default personal, authorized work scope, and an unauthorized OSS scope without local membership) and asserts that the unauthorized memory cannot reach any read surface.

Implements codemem-ov4g.5.6.

Type of Change

  • 🧪 Testing (test-only changes)

Testing

  • Relevant checks pass locally (`pnpm run tsc`, `pnpm run lint`, `pnpm run test`)
  • Added/updated tests for changes
  • Manually verified changes work as expected

Checklist

  • Code follows project style (`pnpm run lint` passes for touched files)
  • Self-review completed
  • Documentation updated (if needed)
  • No new warnings introduced

@kunickiaj kunickiaj marked this pull request as ready for review May 5, 2026 05:24
@kunickiaj kunickiaj changed the base branch from 05-04-fix_core_enforce_scope-aware_export_import to graphite-base/1038 May 5, 2026 14:36
@kunickiaj kunickiaj force-pushed the 05-04-test_core_add_retrieval_leakage_regression_suite branch from acc6db8 to f38975f Compare May 5, 2026 14:37
@kunickiaj kunickiaj force-pushed the graphite-base/1038 branch from 0806658 to 741f366 Compare May 5, 2026 14:37
@kunickiaj kunickiaj changed the base branch from graphite-base/1038 to 05-04-fix_core_enforce_scope-aware_export_import May 5, 2026 14:37
@kunickiaj kunickiaj force-pushed the 05-04-test_core_add_retrieval_leakage_regression_suite branch from f38975f to cd602a9 Compare May 5, 2026 14:46
@kunickiaj kunickiaj force-pushed the 05-04-fix_core_enforce_scope-aware_export_import branch from 741f366 to 754f71f Compare May 5, 2026 14:46
Copy link
Copy Markdown
Owner Author

kunickiaj commented May 5, 2026

Merge activity

  • May 5, 2:59 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 5, 3:21 PM UTC: Graphite rebased this pull request as part of a merge.
  • May 5, 3:22 PM UTC: @kunickiaj merged this pull request with Graphite.

@kunickiaj kunickiaj changed the base branch from 05-04-fix_core_enforce_scope-aware_export_import to graphite-base/1038 May 5, 2026 15:17
@kunickiaj kunickiaj changed the base branch from graphite-base/1038 to main May 5, 2026 15:19
@kunickiaj kunickiaj force-pushed the 05-04-test_core_add_retrieval_leakage_regression_suite branch from cd602a9 to e193195 Compare May 5, 2026 15:20
@kunickiaj kunickiaj merged commit 62fa668 into main May 5, 2026
8 checks passed
@kunickiaj kunickiaj deleted the 05-04-test_core_add_retrieval_leakage_regression_suite branch May 5, 2026 15:22
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