fix(tools): refine execute command background handling

Adjusts the logic for handling the `runInBackground` parameter in the `executeCommand` tool.

Move the backgrounding logic into the process started callback instead of the online callback in case the process we're launching doesn't produce any output.

Author: hassoncs

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

@@ -422,15 +422,20 @@ describe("Command Execution Timeout Integration", () => {
 
 			mockTerminal = {
 				runCommand: vitest.fn().mockImplementation((command, callbacks) => {
+					// Simulate shell execution starting (this is where runInBackground logic triggers)
+					setTimeout(() => {
+						callbacks.onShellExecutionStarted(12345, mockProcess)
+					}, 5)
+
 					// Simulate command producing output
 					setTimeout(() => {
 						callbacks.onLine("Starting development server...\n", mockProcess)
 					}, 10)
 
 					// Simulate command completion
 					setTimeout(() => {
-						callbacks.onCompleted("Development server started successfully")
-						callbacks.onShellExecutionComplete({ exitCode: 0 })
+						callbacks.onCompleted("Development server started successfully", mockProcess)
+						callbacks.onShellExecutionComplete({ exitCode: 0 }, mockProcess)
 					}, 50)
 
 					return Promise.resolve()

src/core/tools/executeCommandTool.ts

@@ -178,8 +178,8 @@ export async function executeCommand(
 	}
 
 	let message: { text?: string; images?: string[] } | undefined
-	let runInBackground = runInBackgroundRequested
 	let completed = false
+	let userChoseBackgroundMode = false
 	let result: string = ""
 	let exitDetails: ExitCodeDetails | undefined
 	let shellIntegrationError: string | undefined
@@ -199,16 +199,13 @@ export async function executeCommand(
 			const status: CommandExecutionStatus = { executionId, status: "output", output: compressedOutput }
 			provider?.postMessageToWebview({ type: "commandExecutionStatus", text: JSON.stringify(status) })
 
-			// If runInBackgroundRequested, automatically continue the process
-			if (runInBackgroundRequested && !completed) {
-				completed = true
-				process.continue()
+			if (userChoseBackgroundMode) {
 				return
 			}
 
 			try {
 				const { response, text, images } = await task.ask("command_output", "")
-				runInBackground = true
+				userChoseBackgroundMode = true
 
 				if (response === "messageResponse") {
 					message = { text, images }
@@ -226,10 +223,16 @@ export async function executeCommand(
 			task.say("command_output", result)
 			completed = true
 		},
-		onShellExecutionStarted: (pid: number | undefined) => {
+		onShellExecutionStarted: (pid: number | undefined, process: RooTerminalProcess) => {
 			console.log(`[executeCommand] onShellExecutionStarted: ${pid}`)
 			const status: CommandExecutionStatus = { executionId, status: "started", pid, command }
 			provider?.postMessageToWebview({ type: "commandExecutionStatus", text: JSON.stringify(status) })
+
+			if (runInBackgroundRequested) {
+				userChoseBackgroundMode = true
+				process.continue()
+				return
+			}
 		},
 		onShellExecutionComplete: (details: ExitCodeDetails) => {
 			const status: CommandExecutionStatus = { executionId, status: "exited", exitCode: details.exitCode }