fix: update command timeout to use seconds configuration

- Read commandExecutionTimeout from VSCode config in seconds
- Convert to milliseconds internally for timeout logic
- Display timeout in seconds in error messages
- Update integration tests to expect seconds format

Author: mrubens

src/core/tools/__tests__/executeCommandTimeout.integration.spec.ts

@@ -64,11 +64,11 @@ describe("Command Execution Timeout Integration", () => {
 	})
 
 	it("should pass timeout configuration to executeCommand", async () => {
-		const customTimeout = 15000
+		const customTimeoutMs = 15000 // 15 seconds in milliseconds
 		const options: ExecuteCommandOptions = {
 			executionId: "test-execution",
 			command: "echo test",
-			commandExecutionTimeout: customTimeout,
+			commandExecutionTimeout: customTimeoutMs,
 		}
 
 		// Mock a quick-completing process
@@ -82,11 +82,11 @@ describe("Command Execution Timeout Integration", () => {
 	})
 
 	it("should handle timeout scenario", async () => {
-		const shortTimeout = 100 // Very short timeout
+		const shortTimeoutMs = 100 // Very short timeout in milliseconds
 		const options: ExecuteCommandOptions = {
 			executionId: "test-execution",
 			command: "sleep 10",
-			commandExecutionTimeout: shortTimeout,
+			commandExecutionTimeout: shortTimeoutMs,
 		}
 
 		// Create a process that never resolves but has an abort method
@@ -105,15 +105,15 @@ describe("Command Execution Timeout Integration", () => {
 		// Should return timeout error
 		expect(result[0]).toBe(false) // Not rejected by user
 		expect(result[1]).toContain("terminated after exceeding")
-		expect(result[1]).toContain(`${shortTimeout}ms`)
+		expect(result[1]).toContain("0.1s") // Should show seconds in error message
 	}, 10000) // Increase test timeout to 10 seconds
 
 	it("should abort process on timeout", async () => {
-		const shortTimeout = 50
+		const shortTimeoutMs = 50 // Short timeout in milliseconds
 		const options: ExecuteCommandOptions = {
 			executionId: "test-execution",
 			command: "sleep 10",
-			commandExecutionTimeout: shortTimeout,
+			commandExecutionTimeout: shortTimeoutMs,
 		}
 
 		// Create a process that can be aborted

src/core/tools/executeCommandTool.ts

@@ -64,11 +64,14 @@ export async function executeCommandTool(
 			const clineProviderState = await clineProvider?.getState()
 			const { terminalOutputLineLimit = 500, terminalShellIntegrationDisabled = false } = clineProviderState ?? {}
 
-			// Get command execution timeout from VSCode configuration
-			const commandExecutionTimeout = vscode.workspace
+			// Get command execution timeout from VSCode configuration (in seconds)
+			const commandExecutionTimeoutSeconds = vscode.workspace
 				.getConfiguration(Package.name)
 				.get<number>("commandExecutionTimeout", 0)
 
+			// Convert seconds to milliseconds for internal use
+			const commandExecutionTimeout = commandExecutionTimeoutSeconds * 1000
+
 			const options: ExecuteCommandOptions = {
 				executionId,
 				command,
@@ -135,6 +138,8 @@ export async function executeCommand(
 		commandExecutionTimeout = 0,
 	}: ExecuteCommandOptions,
 ): Promise<[boolean, ToolResponse]> {
+	// Convert milliseconds back to seconds for display purposes
+	const commandExecutionTimeoutSeconds = commandExecutionTimeout / 1000
 	let workingDir: string
 
 	if (!customCwd) {
@@ -249,7 +254,7 @@ export async function executeCommand(
 
 				return [
 					false,
-					`The command was terminated after exceeding a user-configured ${commandExecutionTimeout}ms timeout. Do not try to re-run the command.`,
+					`The command was terminated after exceeding a user-configured ${commandExecutionTimeoutSeconds}s timeout. Do not try to re-run the command.`,
 				]
 			}
 			throw error