Skip to content

Commit 12b499d

Browse files
Thomas TupperThomas Tupper
authored andcommitted
docs: add plugin-v2 scope isolation fix plan and test criteria
1 parent 0a13f23 commit 12b499d

1 file changed

Lines changed: 21 additions & 0 deletions

File tree

SCOPE_ISOLATION_FIX_PLAN.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Scope Isolation Fix Plan (plugin-v2)
2+
3+
## Problem
4+
Session and long-term recalls can leak across scopes in some flows.
5+
6+
## Desired behavior
7+
- `scope=session` => query with `run_id` only
8+
- `scope=long-term` => query with `user_id` only
9+
- `scope=all` => merge session+long-term and de-dup deterministically
10+
11+
## Patch steps
12+
1. Enforce strict id routing in provider methods (`search`, `getAll`, `add` where applicable).
13+
2. Add guard rails that reject ambiguous scope+id combinations.
14+
3. Add smoke tests:
15+
- write session marker, verify absent from long-term search
16+
- write long-term marker, verify absent from session search
17+
- verify scope=all returns both
18+
19+
## Done criteria
20+
- Scope leakage tests pass twice in a row against live R720 API.
21+
- Dashboard scope views align with tool-level scope results.

0 commit comments

Comments
 (0)