🤖 test: restore script execution transformation tests & lint fixes

Change-Id: I995e0b8b97da838b451a2b78a299c21a33af8e0f
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Author: ThomasK33

src/browser/utils/messages/modelMessageTransform.ts

@@ -216,7 +216,7 @@ export function transformScriptMessagesForLLM(messages: MuxMessage[]): MuxMessag
 
     const scriptMeta = msg.metadata.muxMetadata;
     const result = scriptMeta.result;
-    
+
     let llmContent = `Script '${scriptMeta.scriptName}' executed (exit code ${result.exitCode}).`;
 
     // Include Stdout/Stderr if present
@@ -227,12 +227,12 @@ export function transformScriptMessagesForLLM(messages: MuxMessage[]): MuxMessag
     }
 
     // Include MUX_OUTPUT if present
-    if (result.outputFile && result.outputFile.trim()) {
+    if (result.outputFile?.trim()) {
       llmContent += `\nMUX_OUTPUT:\n${result.outputFile.trim()}`;
     }
 
     // Include MUX_PROMPT if present
-    if (result.promptFile && result.promptFile.trim()) {
+    if (result.promptFile?.trim()) {
       llmContent += `\nMUX_PROMPT:\n${result.promptFile.trim()}`;
     }
 

src/browser/utils/messages/transformScriptMessagesForLLM.test.ts

@@ -1,9 +1,17 @@
 import { describe, it, expect } from "@jest/globals";
 import { transformScriptMessagesForLLM } from "./modelMessageTransform";
 import type { MuxMessage } from "@/common/types/message";
+import type { BashToolResult } from "@/common/types/tools";
 
 describe("transformScriptMessagesForLLM", () => {
   it("should include stdout/stderr in script execution messages", () => {
+    const scriptResult: BashToolResult = {
+      success: true,
+      output: "some stdout output",
+      exitCode: 0,
+      wall_duration_ms: 100,
+    };
+
     const messages: MuxMessage[] = [
       {
         id: "script-1",
@@ -18,12 +26,7 @@ describe("transformScriptMessagesForLLM", () => {
             command: "/script test",
             scriptName: "test.sh",
             args: [],
-            result: {
-              success: true,
-              output: "some stdout output",
-              exitCode: 0,
-              wall_duration_ms: 100,
-            } as any,
+            result: scriptResult,
           },
         },
       },
@@ -34,13 +37,22 @@ describe("transformScriptMessagesForLLM", () => {
     const textPart = result[0].parts[0];
     expect(textPart.type).toBe("text");
     if (textPart.type === "text") {
-       expect(textPart.text).toContain("Script 'test.sh' executed");
-       expect(textPart.text).toContain("Stdout/Stderr:");
-       expect(textPart.text).toContain("some stdout output");
+      expect(textPart.text).toContain("Script 'test.sh' executed");
+      expect(textPart.text).toContain("Stdout/Stderr:");
+      expect(textPart.text).toContain("some stdout output");
     }
   });
 
   it("should include all fields (stdout, output, prompt) if present", () => {
+    const scriptResult: BashToolResult = {
+      success: true,
+      output: "stdout stuff",
+      exitCode: 0,
+      wall_duration_ms: 100,
+      outputFile: "User toast",
+      promptFile: "Model prompt",
+    };
+
     const messages: MuxMessage[] = [
       {
         id: "script-all",
@@ -55,14 +67,7 @@ describe("transformScriptMessagesForLLM", () => {
             command: "/script all",
             scriptName: "all.sh",
             args: [],
-            result: {
-              success: true,
-              output: "stdout stuff",
-              exitCode: 0,
-              wall_duration_ms: 100,
-              outputFile: "User toast",
-              promptFile: "Model prompt",
-            } as any,
+            result: scriptResult,
           },
         },
       },
@@ -73,9 +78,9 @@ describe("transformScriptMessagesForLLM", () => {
     const textPart = result[0].parts[0];
     expect(textPart.type).toBe("text");
     if (textPart.type === "text") {
-       expect(textPart.text).toContain("Stdout/Stderr:\nstdout stuff");
-       expect(textPart.text).toContain("MUX_OUTPUT:\nUser toast");
-       expect(textPart.text).toContain("MUX_PROMPT:\nModel prompt");
+      expect(textPart.text).toContain("Stdout/Stderr:\nstdout stuff");
+      expect(textPart.text).toContain("MUX_OUTPUT:\nUser toast");
+      expect(textPart.text).toContain("MUX_PROMPT:\nModel prompt");
     }
   });
 });