fix: put 'run' subcommand first in args

Author: RhizomaticRobin

src/tools/opencode.tool.ts

@@ -78,8 +78,8 @@ function spawnOpenCodeProcess(
   const taskManager = getTaskManager();
 
   // Build command arguments
-  
   const args: string[] = [
+    CLI.SUBCOMMANDS.RUN,
     "--format", "json",
     CLI.FLAGS.MODEL, model,
   ];
@@ -88,9 +88,6 @@ function spawnOpenCodeProcess(
     args.push(CLI.FLAGS.AGENT, agent);
   }
 
-  // Add the 'run' subcommand
-  args.push(CLI.SUBCOMMANDS.RUN);
-
   // Combine task with output guidance if provided
   const fullPrompt = outputGuidance
     ? `${task}\n\nOutput guidance: ${outputGuidance}`
@@ -309,10 +306,9 @@ export function spawnContinueProcess(
   const taskManager = getTaskManager();
 
   // Build command arguments for session continuation
-  
   const args: string[] = [
+    CLI.SUBCOMMANDS.RUN,
     "--format", "json",
-    "run",
     "--session", sessionId,
   ];
 
@@ -326,7 +322,7 @@ export function spawnContinueProcess(
 
   Logger.debug(`Spawning OpenCode continuation: ${CLI.COMMANDS.OPENCODE} ${args.map(a => `"${a}"`).join(" ")}`);
 
-  // Spawn the process
+  // Spawn of process
   const proc = spawn(CLI.COMMANDS.OPENCODE, args, {
     stdio: ["ignore", "pipe", "pipe"],
     shell: true,
@@ -388,6 +384,8 @@ export function spawnContinueProcess(
       } else if (signal) {
         taskManager.failTask(taskId, `Process killed by signal ${signal}`);
       }
+      // If code is 0 but status is still working, the step_finish event should have handled completion
+      // This is a fallback for edge cases
     }
 
     activeProcesses.delete(taskId);