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/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 }
@@ -230,6 +227,12 @@ export async function executeCommand(
 			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 }
@@ -330,7 +333,7 @@ export async function executeCommand(
 				images,
 			),
 		]
-	} else if (completed || exitDetails) {
+	} else if (exitDetails) {
 		let exitStatus: string = ""
 
 		if (exitDetails !== undefined) {