Add a command to toggle TODO visibility - #6135

Adds a new VS Code command to quickly toggle the dart.showTodos setting from the command palette, avoiding manual edits to settings JSON when iterating on lints and diagnostics.

Changes:

• Register dart.toggleShowTodos and implement logic to flip dart.showTodos between enabled/disabled.
• Add test coverage for toggling behaviour across user/workspace scopes (including multi-root coverage).
• Contribute the new command to package.json so it’s discoverable in VS Code.

Comments suppressed due to low confidence (1)

src/extension/commands/analyzer.ts:64

• This command may select ConfigurationTarget.WorkspaceFolder, but config.setShowTodos() ultimately calls workspace.getConfiguration("dart") without a resource URI. VS Code requires a resource-scoped configuration to reliably update WorkspaceFolder settings, otherwise the update can fail or write to the wrong scope/folder. Consider using a resource-scoped configuration for both inspect() and update().

			let target = vs.ConfigurationTarget.Global;
			const configValue = vs.workspace.getConfiguration("dart").inspect(settingName);
			if (configValue?.workspaceFolderValue !== undefined) {
				target = vs.ConfigurationTarget.WorkspaceFolder;
			} else if (configValue?.workspaceValue !== undefined) {

• Files reviewed: 4/4 changed files
• Comments generated: 3
• Review effort level: Low

Comments suppressed due to low confidence (2)

src/test/multi_root/commands/analyzer.test.ts:23

• This multi-root test is not actually exercising the folder-level (WorkspaceFolder) configuration case:

• refresh() uses getConfiguration("dart") without a resource scope, so inspect() won’t reliably surface workspaceFolderValue.
• The test sets ConfigurationTarget.Workspace (workspace file), but the title says the setting is defined in folder settings.

Adjust the test to scope the configuration to helloWorldMainFile and use ConfigurationTarget.WorkspaceFolder, then assert workspaceFolderValue is what gets toggled.

	function refresh(): void {
		config = vs.workspace.getConfiguration("dart");
	}

	beforeEach(async () => {
		await activate(helloWorldMainFile);
		refresh();
		// Clear overrides to start from a clean state.
		await config.update(settingName, undefined, vs.ConfigurationTarget.Global);
		await config.update(settingName, undefined, vs.ConfigurationTarget.Workspace);
		refresh();
	});

	it("toggles in Workspace settings when defined in Folder settings (multi-root workspace)", async () => {
		await config.update(settingName, false, vs.ConfigurationTarget.Workspace);

src/extension/commands/analyzer.ts:69

• workspace.getConfiguration("dart") is unscoped, so inspect("showTodos") will not reliably populate workspaceFolderValue in multi-root workspaces and may update the wrong target. Also, config.setShowTodos(..., WorkspaceFolder) uses the global/unscoped Config wrapper which doesn’t know which folder to write to.

Use a resource-scoped configuration (for example the active editor’s URI) and perform the update via that scoped WorkspaceConfiguration so folder/workspace/global targeting works correctly.

			const currentValue = config.showTodos;
			const newValue = !currentValue;

			let target = vs.ConfigurationTarget.Global;
			const configValue = vs.workspace.getConfiguration("dart").inspect(settingName);

• Files reviewed: 4/4 changed files
• Comments generated: 0 new
• Review effort level: Low