tool results are strings in handleSteps. Make terminal params optional

Author: jahooma

common/src/tools/params/tool/run-terminal-command.ts

@@ -17,6 +17,7 @@ export const runTerminalCommandParams = {
       process_type: z
         .enum(['SYNC', 'BACKGROUND'])
         .default('SYNC')
+        .optional()
         .describe(
           `Either SYNC (waits, returns output) or BACKGROUND (runs in background). Default SYNC`,
         ),
@@ -29,6 +30,7 @@ export const runTerminalCommandParams = {
       timeout_seconds: z
         .number()
         .default(30)
+        .optional()
         .describe(
           `Set to -1 for no timeout. Does not apply for BACKGROUND commands. Default 30`,
         ),

common/src/util/types/agent-config.d.ts

@@ -148,7 +148,7 @@ export interface AgentConfig {
   ) => Generator<
     ToolCall | 'STEP' | 'STEP_ALL',
     void,
-    { agentState: AgentState; toolResult: ToolResult | undefined }
+    { agentState: AgentState; toolResult: string | undefined }
   >
 }
 

common/src/util/types/tools.d.ts

@@ -157,11 +157,11 @@ export interface RunTerminalCommandParams {
   /** CLI command valid for user's OS. */
   command: string
   /** Either SYNC (waits, returns output) or BACKGROUND (runs in background). Default SYNC */
-  process_type: 'SYNC' | 'BACKGROUND'
+  process_type?: 'SYNC' | 'BACKGROUND'
   /** The working directory to run the command in. Default is the project root. */
   cwd?: string
   /** Set to -1 for no timeout. Does not apply for BACKGROUND commands. Default 30 */
-  timeout_seconds: number
+  timeout_seconds?: number
 }
 
 /**