Verify Settings Auto-Refetch Compatibility for Experimental Configuration Values

[sandy081]

Ref - https://github.com/microsoft/vscode-internalbacklog/issues/5545

With the experimental configuration system migration from onExP tag to structured experiment objects, many settings now have mode:'startup' by default. Eg:

'chat.editRequests': {
			tags: ['experimental'],
			experiment: {
				mode: 'startup'
			}
		}

This issue tracks the review of settings that could potentially handle automatic configuration changes when experiment values are updated, and identifies candidates for enabling mode: auto.

Detecting Experiment-Driven Configuration Changes

Components can detect when a configuration change originates because of Default value has update by checking the source property in the configuration change event. When event.source === ConfigurationTarget.DEFAULT.

this.configurationService.onDidChangeConfiguration(event => {
    if (event.source === ConfigurationTarget.DEFAULT && event.affectsConfiguration('your.setting.key')) {
        // This change came from default value change
        // Handle dynamic configuration change
    }
});

Settings Categories to Review:

• Inline Suggest Settings (editorOptions.ts) - @hediet
  • editor.inlineSuggest.experimental.suppressInlineSuggestions
  • editor.inlineSuggest.experimental.triggerCommandOnProviderChange
• Quick Suggestions (editorOptions.ts) - @hediet
  • editor.quickSuggestionsDelay
• Chat Experience Settings (chat.contribution.ts) - @justschen / @connor4312 / @bhavyaus
  • chat.emptyChatState.enabled
  • ✅ chat.mcp.assisted.nuget.enabled
  • ✅chat.todoListTool.enabled
• Inline Chat Features (inlineChat.ts) - @amunger
  • inlineChat.notebookAgent
• Telemetry Settings (editTelemetry.contribution.ts) @hediet
  • telemetry.editStats.detailed.enabled
  • editor.aiStats.enabled

For each setting, evaluate:

1. Can the default setting value change dynamically without causing user experience disruptions?
2. What happens if the setting changes automatically while the application is running?

Next Steps:

1. Component owners review their assigned settings
2. Update mode: 'auto' for settings that can handle dynamic changes

[bpasero]

Very very cool!

@sandy081 just to clarify: if my experimental setting simply is read when applied, I do not need to react to the event right? I would just get the up-to-date value when reading it.