-
Notifications
You must be signed in to change notification settings - Fork 1.6k
update memory tool context #3604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…rted repo path tests
…ring based on memory tool availability
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the Copilot Chat “memory” tool and related prompts to support a clearer memory model with user- and session-scoped storage, and adjusts Anthropic tool handling accordingly.
Changes:
- Add a
/memories/session/namespace (workspace/session-scoped) and route/memories/to global (user-scoped) storage. - Update agent prompts to include user + session memory context at chat start, and refresh memory tool instructions.
- Simplify Anthropic Messages API tool handling by removing special-casing for a built-in Anthropic memory tool and ensuring
memoryisn’t deferred when tool search is enabled.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/platform/networking/common/anthropic.ts | Marks memory as non-deferred for Anthropic tool-search scenarios. |
| src/platform/endpoint/node/messagesApi.ts | Removes built-in-memory injection/filtering; sends memory as a regular tool. |
| src/extension/tools/node/test/memoryTool.spec.tsx | Updates tests to use /memories/session/ paths and adds merged-root view test. |
| src/extension/tools/node/memoryTool.tsx | Implements user vs session scoping, merged root listing, and updated invocation messages. |
| src/extension/tools/node/memoryContextPrompt.tsx | Adds user memory content + session memory file listing into prompt context. |
| src/extension/tools/common/memoryCleanupService.ts | Expands isMemoryUri to recognize both workspace and global storage locations. |
| src/extension/prompts/node/agent/agentPrompt.tsx | Switches to new memory prompt components and threads sessionResource for session listing. |
| package.json | Updates tool description to document the new three-scope memory model. |
No description provided.