Skip to content

Make localIndex setting public#312402

Merged
vijayupadya merged 3 commits intomainfrom
vijayu/localSetting
Apr 24, 2026
Merged

Make localIndex setting public#312402
vijayupadya merged 3 commits intomainfrom
vijayu/localSetting

Conversation

@vijayupadya
Copy link
Copy Markdown
Contributor

Promotes the local session indexing setting from a internal setting to a public experimental setting, making it accessible via VS Code Settings UI and ExP flights.

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 24, 2026 17:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Promotes the “local session indexing” feature flag from a team-internal setting to a public experimental setting so it can be configured via Settings UI and controlled via ExP.

Changes:

  • Replaces the team-internal config key with a new public experimental setting github.copilot.chat.localIndex.enabled.
  • Updates chronicle-related feature gating (intent availability, context keys, trackers/exporters) to use the new config key.
  • Adds the new setting to package.json contributions and updates localized strings.
Show a summary per file
File Description
extensions/copilot/src/platform/configuration/common/configurationService.ts Removes old team-internal key and introduces new public LocalIndexEnabled config key.
extensions/copilot/src/extension/intents/node/chronicleIntent.ts Gates chronicle intent availability on the new config key.
extensions/copilot/src/extension/contextKeys/vscode-node/contextKeys.contribution.ts Updates context key wiring to read the new config key.
extensions/copilot/src/extension/chronicle/vscode-node/sessionStoreTracker.ts Switches local tracking enablement check to the new config key.
extensions/copilot/src/extension/chronicle/vscode-node/remoteSessionExporter.ts Switches local-enabled check to the new config key for cloud export gating.
extensions/copilot/src/extension/chronicle/common/test/sessionIndexingPreference.spec.ts Updates mocked config key string to the new fully-qualified setting.
extensions/copilot/src/extension/chronicle/common/sessionIndexingPreference.ts Removes outdated doc comment referencing the old local-index setting path.
extensions/copilot/package.nls.json Updates chronicle descriptions and adds a new localized description for the new setting.
extensions/copilot/package.json Contributes the new public experimental setting to Settings UI / ExP (onExp).

Copilot's findings

  • Files reviewed: 9/9 changed files
  • Comments generated: 3

Comment thread extensions/copilot/package.nls.json
Comment thread extensions/copilot/package.nls.json Outdated
@vijayupadya vijayupadya marked this pull request as ready for review April 24, 2026 19:01
@vijayupadya vijayupadya merged commit 0e1f836 into main Apr 24, 2026
26 checks passed
@vijayupadya vijayupadya deleted the vijayu/localSetting branch April 24, 2026 19:01
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 24, 2026
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.

3 participants