feat: add setting to disable LLM command suggestions

- Added new configuration setting 'roo.disableLLMCommandSuggestions'
- Made command suggestions conditional based on the setting
- Updated system prompt to pass the setting value
- Added comprehensive tests for the new functionality
- Added localization support for the new setting

Fixes #5491

Author: hannesrudolph

src/core/prompts/tools/__tests__/execute-command.spec.ts

@@ -0,0 +1,72 @@
+import { describe, it, expect } from "vitest"
+import { getExecuteCommandDescription } from "../execute-command"
+import { ToolArgs } from "../types"
+
+describe("getExecuteCommandDescription", () => {
+	const baseArgs: ToolArgs = {
+		cwd: "/test/path",
+		supportsComputerUse: false,
+	}
+
+	it("should include suggestions section when disableLlmCommandSuggestions is false", () => {
+		const args: ToolArgs = {
+			...baseArgs,
+			settings: {
+				disableLlmCommandSuggestions: false,
+			},
+		}
+
+		const description = getExecuteCommandDescription(args)
+
+		// Check that the description includes the suggestions parameter
+		expect(description).toContain("<suggestions>")
+		expect(description).toContain("- suggestions: (optional) Command patterns for the user to allow/deny")
+		expect(description).toContain("Suggestion Guidelines")
+	})
+
+	it("should include suggestions section when disableLlmCommandSuggestions is not set", () => {
+		const args: ToolArgs = {
+			...baseArgs,
+			settings: {},
+		}
+
+		const description = getExecuteCommandDescription(args)
+
+		// Check that the description includes the suggestions parameter
+		expect(description).toContain("<suggestions>")
+		expect(description).toContain("- suggestions: (optional) Command patterns for the user to allow/deny")
+		expect(description).toContain("Suggestion Guidelines")
+	})
+
+	it("should exclude suggestions section when disableLlmCommandSuggestions is true", () => {
+		const args: ToolArgs = {
+			...baseArgs,
+			settings: {
+				disableLlmCommandSuggestions: true,
+			},
+		}
+
+		const description = getExecuteCommandDescription(args)
+
+		// Check that the description does NOT include the suggestions parameter
+		expect(description).not.toContain("<suggestions>")
+		expect(description).not.toContain("- suggestions: (optional) Command patterns for the user to allow/deny")
+		expect(description).not.toContain("Suggestion Guidelines")
+	})
+
+	it("should include basic command and cwd parameters regardless of settings", () => {
+		const args: ToolArgs = {
+			...baseArgs,
+			settings: {
+				disableLlmCommandSuggestions: true,
+			},
+		}
+
+		const description = getExecuteCommandDescription(args)
+
+		// Check that basic parameters are always included
+		expect(description).toContain("- command: (required)")
+		expect(description).toContain("- cwd: (optional)")
+		expect(description).toContain("execute_command")
+	})
+})

src/core/prompts/tools/execute-command.ts

@@ -1,12 +1,42 @@
 import { ToolArgs } from "./types"
 
 export function getExecuteCommandDescription(args: ToolArgs): string | undefined {
-	return `## execute_command
+	const disableLlmSuggestions = args.settings?.disableLlmCommandSuggestions ?? false
+
+	const baseDescription = `## execute_command
 Description: Request to execute a CLI command on the system. Use this when you need to perform system operations or run specific commands to accomplish any step in the user's task. You must tailor your command to the user's system and provide a clear explanation of what the command does. For command chaining, use the appropriate chaining syntax for the user's shell. Prefer to execute complex CLI commands over creating executable scripts, as they are more flexible and easier to run. Prefer relative commands and paths that avoid location sensitivity for terminal consistency, e.g: \`touch ./testdata/example.file\`, \`dir ./examples/model1/data/yaml\`, or \`go test ./cmd/front --config ./cmd/front/config.yml\`. If directed by the user, you may open a terminal in a different directory by using the \`cwd\` parameter.
 
 Parameters:
 - command: (required) The CLI command to execute. This should be valid for the current operating system. Ensure the command is properly formatted and does not contain any harmful instructions.
-- cwd: (optional) The working directory to execute the command in (default: ${args.cwd})
+- cwd: (optional) The working directory to execute the command in (default: ${args.cwd})`
+
+	if (disableLlmSuggestions) {
+		return (
+			baseDescription +
+			`
+
+Usage:
+<execute_command>
+<command>Your command here</command>
+<cwd>Working directory path (optional)</cwd>
+</execute_command>
+
+Example: Requesting to execute npm run dev
+<execute_command>
+<command>npm run dev</command>
+</execute_command>
+
+Example: Requesting to execute ls in a specific directory
+<execute_command>
+<command>ls -la</command>
+<cwd>/home/user/projects</cwd>
+</execute_command>`
+		)
+	}
+
+	return (
+		baseDescription +
+		`
 - suggestions: (optional) Command patterns for the user to allow/deny for future auto-approval. Include 1-2 relevant patterns when executing common development commands. Use <suggest> tags.
 
 **Suggestion Guidelines:**
@@ -41,4 +71,5 @@ Example: Requesting to execute ls in a specific directory
 <suggest>ls</suggest>
 </suggestions>
 </execute_command>`
+	)
 }

src/core/task/Task.ts

@@ -1648,6 +1648,9 @@ export class Task extends EventEmitter<ClineEvents> {
 				maxReadFileLine !== -1,
 				{
 					maxConcurrentFileReads,
+					disableLlmCommandSuggestions: vscode.workspace
+						.getConfiguration(Package.name)
+						.get<boolean>("disableLlmCommandSuggestions", false),
 				},
 			)
 		})()