Skip to content

Commit d34ffb2

Browse files
authored
Revert "first cut - migrate settings marked with INTERNAL and adopt to advanc…" (#2021)
This reverts commit 7098ab5.
1 parent 89a2f98 commit d34ffb2

File tree

59 files changed

+376
-922
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

59 files changed

+376
-922
lines changed

package.json

Lines changed: 0 additions & 463 deletions
Large diffs are not rendered by default.

package.nls.json

Lines changed: 0 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,6 @@
160160
"github.copilot.config.debugTerminalCommandPatterns": "A list of commands for which the \"Debug Command\" quick fix action should be shown in the debug terminal.",
161161
"github.copilot.config.edits.suggestRelatedFilesFromGitHistory": "Whether to suggest related files from git history for the Copilot Edits working set.",
162162
"github.copilot.chat.edits.suggestRelatedFilesForTests": "Whether to suggest source files from test files for the Copilot Edits working set.",
163-
"github.copilot.config.suggestRelatedFilesFromGitHistory.useEmbeddings": "Use embeddings to suggest related files from git history.",
164163
"github.copilot.config.codeGeneration.instructions": "A set of instructions that will be added to Copilot requests that generate code.\nInstructions can come from: \n- a file in the workspace: `{ \"file\": \"fileName\" }`\n- text in natural language: `{ \"text\": \"Use underscore for field names.\" }`\n\nNote: Keep your instructions short and precise. Poor instructions can degrade Copilot's quality and performance.",
165164
"github.copilot.config.codeGeneration.instructions.deprecated": "Use instructions files instead. See https://aka.ms/vscode-ghcp-custom-instructions for more information.",
166165
"github.copilot.config.codeGeneration.useInstructionFiles": "Controls whether code instructions from `.github/copilot-instructions.md` are added to Copilot requests.\n\nNote: Keep your instructions short and precise. Poor instructions can degrade Copilot's quality and performance. [Learn more](https://aka.ms/github-copilot-custom-instructions) about customizing Copilot.",
@@ -362,54 +361,6 @@
362361
"github.copilot.config.tools.memory.enabled": "Enable memory tool to allow models to store and retrieve information across conversations. \n\n**Note**: This is an experimental feature.",
363362
"github.copilot.config.completionsFetcher": "Sets the fetcher used for the inline completions.",
364363
"github.copilot.config.nesFetcher": "Sets the fetcher used for the next edit suggestions.",
365-
"github.copilot.config.debug.overrideChatEngine": "Override the chat model. This allows you to test with different models.\n\n**Note**: This is an advanced debugging setting and should not be used while self-hosting as it may lead to a different experience compared to end-users.",
366-
"github.copilot.config.projectLabels.expanded": "Use the expanded format for project labels in prompts.",
367-
"github.copilot.config.projectLabels.chat": "Add project labels in chat requests.",
368-
"github.copilot.config.projectLabels.inline": "Add project labels in inline edit requests.",
369-
"github.copilot.config.workspace.maxLocalIndexSize": "Maximum size of the local workspace index.",
370-
"github.copilot.config.workspace.enableFullWorkspace": "Enable full workspace context analysis.",
371-
"github.copilot.config.workspace.enableCodeSearch": "Enable code search in workspace context.",
372-
"github.copilot.config.workspace.enableEmbeddingsSearch": "Enable embeddings-based search in workspace context.",
373-
"github.copilot.config.workspace.preferredEmbeddingsModel": "Preferred embeddings model for semantic search.",
374-
"github.copilot.config.workspace.prototypeAdoCodeSearchEndpointOverride": "Override endpoint for Azure DevOps code search prototype.",
375-
"github.copilot.config.feedback.onChange": "Enable feedback collection on configuration changes.",
376-
"github.copilot.config.review.intent": "Enable intent detection for code review.",
377-
"github.copilot.config.notebook.summaryExperimentEnabled": "Enable the notebook summary experiment.",
378-
"github.copilot.config.notebook.variableFilteringEnabled": "Enable filtering variables by cell document symbols.",
379-
"github.copilot.config.notebook.alternativeFormat": "Alternative document format for notebooks.",
380-
"github.copilot.config.notebook.alternativeNESFormat.enabled": "Enable alternative format for Next Edit Suggestions in notebooks.",
381-
"github.copilot.config.editSourceTracking.showDecorations": "Show decorations for edit source tracking.",
382-
"github.copilot.config.editSourceTracking.showStatusBar": "Show status bar item for edit source tracking.",
383-
"github.copilot.config.localWorkspaceRecording.enabled": "Enable local workspace recording for analysis.",
384-
"github.copilot.config.editRecording.enabled": "Enable edit recording for analysis.",
385-
"github.copilot.config.temporalContext.maxAge": "Maximum age (in editor changes) for temporal context.",
386-
"github.copilot.config.temporalContext.preferSameLang": "Prefer same language files in temporal context.",
387-
"github.copilot.config.codesearch.agent.enabled": "Enable code search capabilities in agent mode.",
388-
"github.copilot.config.agent.temperature": "Temperature setting for agent mode requests.",
389-
"github.copilot.config.instantApply.shortContextModelName": "Model name for short context instant apply.",
390-
"github.copilot.config.instantApply.shortContextLimit": "Token limit for short context instant apply.",
391-
"github.copilot.config.summarizeAgentConversationHistoryThreshold": "Threshold for summarizing agent conversation history.",
392-
"github.copilot.config.agentHistorySummarizationMode": "Mode for agent history summarization.",
393-
"github.copilot.config.agentHistorySummarizationWithPromptCache": "Use prompt caching for agent history summarization.",
394-
"github.copilot.config.agentHistorySummarizationForceGpt41": "Force GPT-4.1 for agent history summarization.",
395-
"github.copilot.config.useResponsesApiTruncation": "Use Responses API for truncation.",
396-
"github.copilot.config.enableReadFileV2": "Enable version 2 of the read file tool.",
397-
"github.copilot.config.enableAskAgent": "Enable the Ask agent for answering questions.",
398-
"github.copilot.config.omitBaseAgentInstructions": "Omit base agent instructions from prompts.",
399-
"github.copilot.config.promptFileContextProvider.enabled": "Enable prompt file context provider.",
400-
"github.copilot.config.tools.defaultToolsGrouped": "Group default tools in prompts.",
401-
"github.copilot.config.virtualTools.embeddingRanking": "Use embedding-based ranking for virtual tools.",
402-
"github.copilot.config.multiReplaceStringGrok.enabled": "Enable multi-replace string with Grok.",
403-
"github.copilot.config.claudeCode.enabled": "Enable Claude Code agent.",
404-
"github.copilot.config.claudeCode.debug": "Enable debug mode for Claude Code agent.",
405-
"github.copilot.config.copilotCLI.enabled": "Enable Copilot CLI integration.",
406-
"github.copilot.config.cli.isolation.enabled": "Enable CLI isolation for agent sessions.",
407-
"github.copilot.config.cli.mcp.enabled": "Enable Model Context Protocol (MCP) server for CLI.",
408-
"github.copilot.config.copilotCodingAgent.enabled": "Enable Copilot Coding Agent (cloud).",
409-
"github.copilot.config.gpt5AlternativePatch": "Enable GPT-5 alternative patch format.",
410-
"github.copilot.config.inlineEdits.triggerOnEditorChangeAfterSeconds": "Trigger inline edits after editor has been idle for this many seconds.",
411-
"github.copilot.config.inlineEdits.nextCursorPrediction.displayLine": "Display predicted cursor line for next edit suggestions.",
412-
"github.copilot.config.inlineEdits.nextCursorPrediction.currentFileMaxTokens": "Maximum tokens for current file in next cursor prediction.",
413364
"github.copilot.command.refreshAgentSessions": "Refresh Agent Sessions",
414365
"github.copilot.command.deleteAgentSession": "Delete Agent Session",
415366
"github.copilot.command.cli.sessions.resumeInTerminal": "Resume Agent Session in Terminal",

src/extension/agents/claude/node/claudeCodeAgent.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ export class ClaudeCodeSession extends Disposable {
238238
private async _startSession(token: vscode.CancellationToken): Promise<void> {
239239
// Build options for the Claude Code SDK
240240
// process.env.DEBUG = '1'; // debug messages from sdk.mjs
241-
const isDebugEnabled = this.configService.getConfig(ConfigKey.AdvancedExperimental.ClaudeCodeDebugEnabled);
241+
const isDebugEnabled = this.configService.getConfig(ConfigKey.Internal.ClaudeCodeDebugEnabled);
242242
this.logService.trace(`appRoot: ${this.envService.appRoot}`);
243243
const pathSep = isWindows ? ';' : ':';
244244
const options: Options = {

src/extension/agents/copilotcli/node/mcpHandler.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ export class CopilotCLIMCPHandler implements ICopilotCLIMCPHandler {
8989
) { }
9090

9191
public async loadMcpConfig(workingDirectory: string | undefined): Promise<Record<string, MCPServerConfig> | undefined> {
92-
if (!this.configurationService.getConfig(ConfigKey.AdvancedExperimental.CLIMCPServerEnabled)) {
92+
if (!this.configurationService.getConfig(ConfigKey.Internal.CLIMCPServerEnabled)) {
9393
return undefined;
9494
}
9595

src/extension/chatSessions/vscode-node/copilotCLIChatSessionsContribution.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ export class CopilotCLIChatSessionContentProvider implements vscode.ChatSessionC
246246
[MODELS_OPTION_ID]: _sessionModel.get(copilotcliSessionId)?.id ?? defaultModel.id,
247247
};
248248

249-
if (!existingSession && this.configurationService.getConfig(ConfigKey.AdvancedExperimental.CLIIsolationEnabled)) {
249+
if (!existingSession && this.configurationService.getConfig(ConfigKey.Internal.CLIIsolationEnabled)) {
250250
options[ISOLATION_OPTION_ID] = isolationEnabled ? 'enabled' : 'disabled';
251251
}
252252
const history = existingSession?.object?.getChatHistory() || [];

src/extension/chatSessions/vscode-node/copilotCLITerminalIntegration.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export class CopilotCLITerminalIntegration extends Disposable implements ICopilo
5959
}
6060

6161
private async initialize(): Promise<void> {
62-
const enabled = this.configurationService.getConfig(ConfigKey.AdvancedExperimental.CopilotCLIEnabled);
62+
const enabled = this.configurationService.getConfig(ConfigKey.Internal.CopilotCLIEnabled);
6363
if (!enabled) {
6464
return;
6565
}

src/extension/configuration/vscode-node/configurationMigration.ts

Lines changed: 23 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111

1212
import { ConfigurationTarget, Uri, window, workspace, WorkspaceFolder } from 'vscode';
13-
import { ConfigurationKeyValuePairs, ConfigurationMigration, ConfigurationMigrationRegistry, ConfigurationValue } from '../../../platform/configuration/common/configurationService';
13+
import { Emitter } from '../../../util/vs/base/common/event';
1414
import { DisposableStore, IDisposable } from '../../../util/vs/base/common/lifecycle';
1515
import { localize } from '../../../util/vs/nls';
1616
import { IExtensionContribution } from '../../common/contributions';
@@ -35,6 +35,28 @@ export const Extensions = {
3535
ConfigurationMigration: 'base.contributions.configuration.migration'
3636
};
3737

38+
export type ConfigurationValue = { value: any | undefined /* Remove */ };
39+
export type ConfigurationKeyValuePairs = [string, ConfigurationValue][];
40+
export type ConfigurationMigrationFn = (value: any) => ConfigurationValue | ConfigurationKeyValuePairs | Promise<ConfigurationValue | ConfigurationKeyValuePairs>;
41+
export type ConfigurationMigration = { key: string; migrateFn: ConfigurationMigrationFn };
42+
43+
export interface IConfigurationMigrationRegistry {
44+
registerConfigurationMigrations(configurationMigrations: ConfigurationMigration[]): void;
45+
}
46+
47+
class ConfigurationMigrationRegistryImpl implements IConfigurationMigrationRegistry {
48+
readonly migrations: ConfigurationMigration[] = [];
49+
50+
private readonly _onDidRegisterConfigurationMigrations = new Emitter<ConfigurationMigration[]>();
51+
readonly onDidRegisterConfigurationMigration = this._onDidRegisterConfigurationMigrations.event;
52+
53+
registerConfigurationMigrations(configurationMigrations: ConfigurationMigration[]): void {
54+
this.migrations.push(...configurationMigrations);
55+
}
56+
}
57+
58+
export const ConfigurationMigrationRegistry = new ConfigurationMigrationRegistryImpl();
59+
3860
export class ConfigurationMigrationContribution implements IExtensionContribution {
3961
private readonly _disposables = new DisposableStore();
4062

src/extension/context/node/resolvers/promptWorkspaceLabels.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ export class PromptWorkspaceLabels implements IPromptWorkspaceLabels {
5656
}
5757

5858
public async collectContext(): Promise<void> {
59-
const expandedLabels = this._configurationService.getExperimentBasedConfig(ConfigKey.AdvancedExperimentalExperiments.ProjectLabelsExpanded, this._experimentationService);
59+
const expandedLabels = this._configurationService.getExperimentBasedConfig(ConfigKey.Internal.ProjectLabelsExpanded, this._experimentationService);
6060
this.strategy = expandedLabels ? PromptWorkspaceLabelsStrategy.Expanded : PromptWorkspaceLabelsStrategy.Basic;
6161
await this.workspaceLabels.collectContext();
6262

src/extension/conversation/vscode-node/feedbackCollection.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ export function startFeedbackCollection(accessor: ServicesAccessor) {
2525
const instantiationService = accessor.get(IInstantiationService);
2626
const logService = accessor.get(ILogService);
2727
const disposables = new DisposableStore();
28-
const enabled = configurationService.getConfig(ConfigKey.AdvancedExperimental.FeedbackOnChange);
28+
const enabled = configurationService.getConfig(ConfigKey.Internal.FeedbackOnChange);
2929
if (!enabled) {
3030
return disposables;
3131
}

src/extension/inlineEdits/node/nextEditCache.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ export class NextEditCache extends Disposable {
6767
}
6868
// if editor-change triggering is allowed,
6969
// it means an edit in file A can result in a cached edit for file B to be less relevant than with the edits in file A included
70-
if (configService.getExperimentBasedConfig(ConfigKey.AdvancedExperimentalExperiments.InlineEditsTriggerOnEditorChangeAfterSeconds, expService) !== undefined) {
70+
if (configService.getExperimentBasedConfig(ConfigKey.Internal.InlineEditsTriggerOnEditorChangeAfterSeconds, expService) !== undefined) {
7171
for (const [k, v] of this._sharedCache.entries()) {
7272
if (v.docId !== doc.id) {
7373
this._sharedCache.deleteKey(k);

0 commit comments

Comments
 (0)