Gate chat.copilotMemory.enabled by editor preview policy

[Copilot]

• Extract isCopilotMemoryConfigEnabled helper to agentMemoryService.ts
• Update AgentMemoryService.isCAPIMemorySyncConfigEnabled to delegate to helper
• Update MemoryContextPrompt.render() to use helper
• Update MemoryInstructionsPrompt.render() to use helper
• Update MemoryTool to use helper
• Update ToolsContribution to use helper

Original prompt

The chat.copilotMemory.enabled setting should be handled by the preview policy, to be disabled when GH editor previews are disabled for the user.

The user has attached the following file paths as relevant context:

• .github/copilot-instructions.md
• .github/copilot-instructions.md
• AGENTS.md

Created from VS Code.

[bhavyaus]

@copilot

The isEditorPreviewFeaturesEnabled() check is now repeated in 5 places. Extract a shared helper in agentMemoryService.ts

export function isCopilotMemoryConfigEnabled(
    authenticationService: IAuthenticationService,
    configurationService: IConfigurationService,
    experimentationService: IExperimentationService
): boolean {
    if (!authenticationService.copilotToken?.isEditorPreviewFeaturesEnabled()) {
        return false;
    }
    return configurationService.getExperimentBasedConfig(ConfigKey.CopilotMemoryEnabled, experimentationService);
}

The 4 call sites (MemoryContextPrompt, MemoryInstructionsPrompt, MemoryTool, ToolsContribution) can then inject IAuthenticationService and call this helper instead of inlining the config check. AgentMemoryService.isCAPIMemorySyncConfigEnabled can delegate to it too.

This way if the gating logic changes, there's only one place to update.

[Copilot]

Pull request overview

This PR ensures the chat.copilotMemory.enabled enablement check is additionally gated by the organization “editor preview features” policy (derived from the Copilot token), so Copilot Memory is treated as disabled when preview features are not allowed.

Changes:

• Added a shared isCopilotMemoryConfigEnabled(...) helper that gates the config by copilotToken.isEditorPreviewFeaturesEnabled().
• Updated Copilot Memory call sites (tools + prompts + memory tool logic) to use the shared helper instead of reading the experiment-based config directly.
• Updated ToolsContribution wiring (including test simulation instantiation) to inject IAuthenticationService where needed.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
test/base/extHostContext/simulationExtHostToolsService.ts	Updates test harness instantiation of ToolsContribution to match the new constructor dependencies.
src/extension/tools/vscode-node/tools.ts	Uses the new helper when deciding whether to include local repo-scoped memories in the “show memories” command.
src/extension/tools/node/memoryTool.tsx	Uses the new helper to decide whether to expose local repo memory behavior vs CAPI-backed memory.
src/extension/tools/node/memoryContextPrompt.tsx	Gates prompt inclusion/fetching of repo memories using the helper (policy-aware enablement).
src/extension/tools/common/agentMemoryService.ts	Introduces isCopilotMemoryConfigEnabled(...) and routes AgentMemoryService enablement checks through it.