feat: add provider-level todo list toggle with advanced settings section

- Add todoListEnabled field to provider settings schema
- Implement migration logic for global to provider-level settings
- Create TodoListSettingsControl component for UI toggle
- Create AdvancedSettingsSection to group diff and todo list settings
- Update system prompt generation to respect todoListEnabled setting
- Update task creation logic to conditionally include update_todo_list tool
- Add comprehensive tests for tool inclusion/exclusion logic
- Add translation keys for new settings UI

This allows users to disable todo lists at the API profile level,
similar to how diff edits can be disabled, with both toggles
organized in a collapsible advanced settings section.

Author: roomote

packages/types/src/provider-settings.ts

@@ -56,6 +56,7 @@ export type ProviderSettingsEntry = z.infer<typeof providerSettingsEntrySchema>
 const baseProviderSettingsSchema = z.object({
 	includeMaxTokens: z.boolean().optional(),
 	diffEnabled: z.boolean().optional(),
+	todoListEnabled: z.boolean().optional(),
 	fuzzyMatchThreshold: z.number().optional(),
 	modelTemperature: z.number().nullish(),
 	rateLimitSeconds: z.number().optional(),

src/core/config/ProviderSettingsManager.ts

@@ -25,6 +25,7 @@ export const providerProfilesSchema = z.object({
 		.object({
 			rateLimitSecondsMigrated: z.boolean().optional(),
 			diffSettingsMigrated: z.boolean().optional(),
+			todoListSettingsMigrated: z.boolean().optional(),
 			openAiHeadersMigrated: z.boolean().optional(),
 		})
 		.optional(),
@@ -47,6 +48,7 @@ export class ProviderSettingsManager {
 		migrations: {
 			rateLimitSecondsMigrated: true, // Mark as migrated on fresh installs
 			diffSettingsMigrated: true, // Mark as migrated on fresh installs
+			todoListSettingsMigrated: true, // Mark as migrated on fresh installs
 			openAiHeadersMigrated: true, // Mark as migrated on fresh installs
 		},
 	}
@@ -112,6 +114,7 @@ export class ProviderSettingsManager {
 					providerProfiles.migrations = {
 						rateLimitSecondsMigrated: false,
 						diffSettingsMigrated: false,
+						todoListSettingsMigrated: false,
 						openAiHeadersMigrated: false,
 					} // Initialize with default values
 					isDirty = true
@@ -129,6 +132,12 @@ export class ProviderSettingsManager {
 					isDirty = true
 				}
 
+				if (!providerProfiles.migrations.todoListSettingsMigrated) {
+					await this.migrateTodoListSettings(providerProfiles)
+					providerProfiles.migrations.todoListSettingsMigrated = true
+					isDirty = true
+				}
+
 				if (!providerProfiles.migrations.openAiHeadersMigrated) {
 					await this.migrateOpenAiHeaders(providerProfiles)
 					providerProfiles.migrations.openAiHeadersMigrated = true
@@ -204,6 +213,31 @@ export class ProviderSettingsManager {
 		}
 	}
 
+	private async migrateTodoListSettings(providerProfiles: ProviderProfiles) {
+		try {
+			let todoListEnabled: boolean | undefined
+
+			try {
+				todoListEnabled = await this.context.globalState.get<boolean>("alwaysAllowUpdateTodoList")
+			} catch (error) {
+				console.error("[MigrateTodoListSettings] Error getting global todo list settings:", error)
+			}
+
+			if (todoListEnabled === undefined) {
+				// Failed to get the existing value, use the default.
+				todoListEnabled = true
+			}
+
+			for (const [_name, apiConfig] of Object.entries(providerProfiles.apiConfigs)) {
+				if (apiConfig.todoListEnabled === undefined) {
+					apiConfig.todoListEnabled = todoListEnabled
+				}
+			}
+		} catch (error) {
+			console.error(`[MigrateTodoListSettings] Failed to migrate todo list settings:`, error)
+		}
+	}
+
 	private async migrateOpenAiHeaders(providerProfiles: ProviderProfiles) {
 		try {
 			for (const [_name, apiConfig] of Object.entries(providerProfiles.apiConfigs)) {

src/core/config/__tests__/ProviderSettingsManager.spec.ts

@@ -66,6 +66,7 @@ describe("ProviderSettingsManager", () => {
 						rateLimitSecondsMigrated: true,
 						diffSettingsMigrated: true,
 						openAiHeadersMigrated: true,
+						todoListSettingsMigrated: true,
 					},
 				}),
 			)

src/core/prompts/system.ts

@@ -59,6 +59,7 @@ async function generatePrompt(
 	partialReadsEnabled?: boolean,
 	settings?: Record<string, any>,
 	todoList?: TodoItem[],
+	todoListEnabled?: boolean,
 ): Promise<string> {
 	if (!context) {
 		throw new Error("Extension context is required for generating system prompt")
@@ -98,6 +99,7 @@ ${getToolDescriptionsForMode(
 	experiments,
 	partialReadsEnabled,
 	settings,
+	todoListEnabled,
 )}
 
 ${getToolUseGuidelinesSection(codeIndexManager)}
@@ -138,6 +140,7 @@ export const SYSTEM_PROMPT = async (
 	partialReadsEnabled?: boolean,
 	settings?: Record<string, any>,
 	todoList?: TodoItem[],
+	todoListEnabled?: boolean,
 ): Promise<string> => {
 	if (!context) {
 		throw new Error("Extension context is required for generating system prompt")
@@ -205,5 +208,6 @@ ${customInstructions}`
 		partialReadsEnabled,
 		settings,
 		todoList,
+		todoListEnabled,
 	)
 }

src/core/prompts/tools/__tests__/index.spec.ts

@@ -0,0 +1,119 @@
+// npx vitest src/core/prompts/tools/__tests__/index.spec.ts
+
+import { getToolDescriptionsForMode } from "../index"
+
+describe("getToolDescriptionsForMode", () => {
+	const mockArgs = {
+		mode: "code" as const,
+		cwd: "/test",
+		supportsComputerUse: false,
+		codeIndexManager: undefined,
+		diffStrategy: undefined,
+		browserViewportSize: undefined,
+		mcpHub: undefined,
+		customModes: undefined,
+		experiments: undefined,
+		partialReadsEnabled: undefined,
+		settings: undefined,
+	}
+
+	it("should include update_todo_list tool when todoListEnabled is true", () => {
+		const tools = getToolDescriptionsForMode(
+			mockArgs.mode,
+			mockArgs.cwd,
+			mockArgs.supportsComputerUse,
+			mockArgs.codeIndexManager,
+			mockArgs.diffStrategy,
+			mockArgs.browserViewportSize,
+			mockArgs.mcpHub,
+			mockArgs.customModes,
+			mockArgs.experiments,
+			mockArgs.partialReadsEnabled,
+			mockArgs.settings,
+			true, // todoListEnabled
+		)
+
+		expect(tools).toContain("update_todo_list")
+	})
+
+	it("should exclude update_todo_list tool when todoListEnabled is false", () => {
+		const tools = getToolDescriptionsForMode(
+			mockArgs.mode,
+			mockArgs.cwd,
+			mockArgs.supportsComputerUse,
+			mockArgs.codeIndexManager,
+			mockArgs.diffStrategy,
+			mockArgs.browserViewportSize,
+			mockArgs.mcpHub,
+			mockArgs.customModes,
+			mockArgs.experiments,
+			mockArgs.partialReadsEnabled,
+			mockArgs.settings,
+			false, // todoListEnabled
+		)
+
+		expect(tools).not.toContain("update_todo_list")
+	})
+
+	it("should include update_todo_list tool when todoListEnabled is undefined (default true)", () => {
+		const tools = getToolDescriptionsForMode(
+			mockArgs.mode,
+			mockArgs.cwd,
+			mockArgs.supportsComputerUse,
+			mockArgs.codeIndexManager,
+			mockArgs.diffStrategy,
+			mockArgs.browserViewportSize,
+			mockArgs.mcpHub,
+			mockArgs.customModes,
+			mockArgs.experiments,
+			mockArgs.partialReadsEnabled,
+			mockArgs.settings,
+			undefined, // todoListEnabled
+		)
+
+		expect(tools).toContain("update_todo_list")
+	})
+
+	it("should include other tools regardless of todoListEnabled setting", () => {
+		const toolsWithTodo = getToolDescriptionsForMode(
+			mockArgs.mode,
+			mockArgs.cwd,
+			mockArgs.supportsComputerUse,
+			mockArgs.codeIndexManager,
+			mockArgs.diffStrategy,
+			mockArgs.browserViewportSize,
+			mockArgs.mcpHub,
+			mockArgs.customModes,
+			mockArgs.experiments,
+			mockArgs.partialReadsEnabled,
+			mockArgs.settings,
+			true, // todoListEnabled
+		)
+
+		const toolsWithoutTodo = getToolDescriptionsForMode(
+			mockArgs.mode,
+			mockArgs.cwd,
+			mockArgs.supportsComputerUse,
+			mockArgs.codeIndexManager,
+			mockArgs.diffStrategy,
+			mockArgs.browserViewportSize,
+			mockArgs.mcpHub,
+			mockArgs.customModes,
+			mockArgs.experiments,
+			mockArgs.partialReadsEnabled,
+			mockArgs.settings,
+			false, // todoListEnabled
+		)
+
+		// Both should have other tools like read_file, write_to_file, etc.
+		expect(toolsWithTodo.length).toBeGreaterThan(100) // Should have substantial content
+		expect(toolsWithoutTodo.length).toBeGreaterThan(100) // Should have substantial content
+
+		// Tools with todo should be longer than tools without todo
+		expect(toolsWithTodo.length).toBeGreaterThan(toolsWithoutTodo.length)
+
+		// Both should contain common tools
+		expect(toolsWithTodo).toContain("read_file")
+		expect(toolsWithoutTodo).toContain("read_file")
+	})
+})

src/core/prompts/tools/index.ts

@@ -61,6 +61,7 @@ export function getToolDescriptionsForMode(
 	experiments?: Record<string, boolean>,
 	partialReadsEnabled?: boolean,
 	settings?: Record<string, any>,
+	todoListEnabled?: boolean,
 ): string {
 	const config = getModeConfig(mode, customModes)
 	const args: ToolArgs = {
@@ -109,6 +110,11 @@ export function getToolDescriptionsForMode(
 		tools.delete("codebase_search")
 	}
 
+	// Conditionally exclude update_todo_list if feature is disabled
+	if (todoListEnabled === false) {
+		tools.delete("update_todo_list")
+	}
+
 	// Map tool descriptions for allowed tools
 	const descriptions = Array.from(tools).map((toolName) => {
 		const descriptionFn = toolDescriptionMap[toolName]

src/core/task/Task.ts

@@ -110,6 +110,7 @@ export type TaskOptions = {
 	provider: ClineProvider
 	apiConfiguration: ProviderSettings
 	enableDiff?: boolean
+	enableTodoList?: boolean
 	enableCheckpoints?: boolean
 	fuzzyMatchThreshold?: number
 	consecutiveMistakeLimit?: number
@@ -172,6 +173,7 @@ export class Task extends EventEmitter<ClineEvents> {
 	diffViewProvider: DiffViewProvider
 	diffStrategy?: DiffStrategy
 	diffEnabled: boolean = false
+	todoListEnabled: boolean = true
 	fuzzyMatchThreshold: number
 	didEditFile: boolean = false
 
@@ -213,6 +215,7 @@ export class Task extends EventEmitter<ClineEvents> {
 		provider,
 		apiConfiguration,
 		enableDiff = false,
+		enableTodoList = true,
 		enableCheckpoints = true,
 		fuzzyMatchThreshold = 1.0,
 		consecutiveMistakeLimit = 3,
@@ -253,6 +256,7 @@ export class Task extends EventEmitter<ClineEvents> {
 		this.urlContentFetcher = new UrlContentFetcher(provider.context)
 		this.browserSession = new BrowserSession(provider.context)
 		this.diffEnabled = enableDiff
+		this.todoListEnabled = enableTodoList
 		this.fuzzyMatchThreshold = fuzzyMatchThreshold
 		this.consecutiveMistakeLimit = consecutiveMistakeLimit
 		this.providerRef = new WeakRef(provider)
@@ -1649,6 +1653,8 @@ export class Task extends EventEmitter<ClineEvents> {
 				{
 					maxConcurrentFileReads,
 				},
+				this.todoList,
+				this.todoListEnabled,
 			)
 		})()
 	}

src/core/webview/ClineProvider.ts

@@ -543,6 +543,9 @@ export class ClineProvider
 			experiments,
 		} = await this.getState()
 
+		// Extract todoListEnabled from provider settings
+		const todoListEnabled = apiConfiguration.todoListEnabled ?? true
+
 		if (!ProfileValidator.isProfileAllowed(apiConfiguration, organizationAllowList)) {
 			throw new OrganizationAllowListViolationError(t("common:errors.violated_organization_allowlist"))
 		}
@@ -551,6 +554,7 @@ export class ClineProvider
 			provider: this,
 			apiConfiguration,
 			enableDiff,
+			enableTodoList: todoListEnabled,
 			enableCheckpoints,
 			fuzzyMatchThreshold,
 			task,

src/core/webview/generateSystemPrompt.ts

@@ -26,6 +26,9 @@ export const generateSystemPrompt = async (provider: ClineProvider, message: Web
 		maxConcurrentFileReads,
 	} = await provider.getState()
 
+	// Get todoListEnabled from provider settings with fallback to global setting
+	const todoListEnabled = apiConfiguration.todoListEnabled ?? true
+
 	// Check experiment to determine which diff strategy to use
 	const isMultiFileApplyDiffEnabled = experimentsModule.isEnabled(
 		experiments ?? {},
@@ -83,6 +86,8 @@ export const generateSystemPrompt = async (provider: ClineProvider, message: Web
 		{
 			maxConcurrentFileReads,
 		},
+		undefined, // todoList parameter
+		todoListEnabled,
 	)
 
 	return systemPrompt

webview-ui/src/components/settings/AdvancedSettingsSection.tsx

@@ -0,0 +1,49 @@
+import React, { useState } from "react"
+import { useAppTranslation } from "@/i18n/TranslationContext"
+import { VSCodeButton } from "@vscode/webview-ui-toolkit/react"
+import { DiffSettingsControl } from "./DiffSettingsControl"
+import { TodoListSettingsControl } from "./TodoListSettingsControl"
+
+interface AdvancedSettingsSectionProps {
+	diffEnabled?: boolean
+	fuzzyMatchThreshold?: number
+	todoListEnabled?: boolean
+	onChange: (field: "diffEnabled" | "fuzzyMatchThreshold" | "todoListEnabled", value: any) => void
+}
+
+export const AdvancedSettingsSection: React.FC<AdvancedSettingsSectionProps> = ({
+	diffEnabled,
+	fuzzyMatchThreshold,
+	todoListEnabled,
+	onChange,
+}) => {
+	const { t } = useAppTranslation()
+	const [isExpanded, setIsExpanded] = useState(false)
+
+	const toggleExpanded = () => {
+		setIsExpanded(!isExpanded)
+	}
+
+	return (
+		<div className="flex flex-col gap-3">
+			<VSCodeButton
+				appearance="secondary"
+				onClick={toggleExpanded}
+				className="flex items-center justify-between w-full text-left">
+				<span className="font-medium">{t("settings:advanced.section.label")}</span>
+				<span className="ml-2">{isExpanded ? "▼" : "▶"}</span>
+			</VSCodeButton>
+
+			{isExpanded && (
+				<div className="flex flex-col gap-4 pl-4 border-l-2 border-vscode-button-background">
+					<DiffSettingsControl
+						diffEnabled={diffEnabled}
+						fuzzyMatchThreshold={fuzzyMatchThreshold}
+						onChange={onChange}
+					/>
+					<TodoListSettingsControl todoListEnabled={todoListEnabled} onChange={onChange} />
+				</div>
+			)}
+		</div>
+	)
+}