refactor: convert QuickJSSandbox.create, SandboxManager methods to use object parameters

🤖 Generated with Codebuff
Co-Authored-By: Codebuff <noreply@codebuff.com>

Author: charleslien

backend/src/util/messages.ts

@@ -95,10 +95,11 @@ export function castAssistantMessage(message: Message): Message | null {
 // Number of terminal command outputs to keep in full form before simplifying
 const numTerminalCommandsToKeep = 5
 
-function simplifyTerminalHelper(
-  toolResult: CodebuffToolOutput<'run_terminal_command'>,
-  numKept: number,
-): { result: CodebuffToolOutput<'run_terminal_command'>; numKept: number } {
+function simplifyTerminalHelper(params: {
+  toolResult: CodebuffToolOutput<'run_terminal_command'>
+  numKept: number
+}): { result: CodebuffToolOutput<'run_terminal_command'>; numKept: number } {
+  const { toolResult, numKept } = params
   const simplified = simplifyTerminalCommandResults(toolResult)
 
   // Keep the full output for the N most recent commands
@@ -165,10 +166,10 @@ export function trimMessagesToFitTokenLimit(
         m,
       ) as CodebuffToolMessage<'run_terminal_command'>
 
-      const result = simplifyTerminalHelper(
-        terminalResultMessage.content.output,
+      const result = simplifyTerminalHelper({
+        toolResult: terminalResultMessage.content.output,
         numKept,
-      )
+      })
       terminalResultMessage.content.output = result.result
       numKept = result.numKept
 

backend/src/util/quickjs-sandbox.ts

@@ -48,17 +48,18 @@ export class QuickJSSandbox {
   /**
    * Create a new QuickJS sandbox with the specified configuration
    */
-  static async create(
-    generatorCode: string,
-    initialInput: any,
-    config: SandboxConfig = {},
+  static async create(params: {
+    generatorCode: string
+    initialInput: any
+    config?: SandboxConfig
     logger?: {
       debug: (data: any, msg?: string) => void
       info: (data: any, msg?: string) => void
       warn: (data: any, msg?: string) => void
       error: (data: any, msg?: string) => void
-    },
-  ): Promise<QuickJSSandbox> {
+    }
+  }): Promise<QuickJSSandbox> {
+    const { generatorCode, initialInput, config = {}, logger } = params
     const {
       memoryLimit = 1024 * 1024 * 20, // 20MB
       maxStackSize = 1024 * 512, // 512KB
@@ -285,12 +286,12 @@ export class SandboxManager {
     }
 
     // Create new sandbox
-    const sandbox = await QuickJSSandbox.create(
+    const sandbox = await QuickJSSandbox.create({
       generatorCode,
       initialInput,
       config,
       logger,
-    )
+    })
     this.sandboxes.set(runId, sandbox)
     return sandbox
   }