Format codex proxy files with prettier

Author: TheGreatAxios

src/agent/codex-apply-patch.test.ts

@@ -157,11 +157,7 @@ describe("extractAffectedPaths", () => {
 *** Delete File: obsolete.txt
 *** End Patch
 `);
-    expect(extractAffectedPaths(patch)).toEqual([
-      "hello.txt",
-      "src/app.py",
-      "obsolete.txt",
-    ]);
+    expect(extractAffectedPaths(patch)).toEqual(["hello.txt", "src/app.py", "obsolete.txt"]);
   });
 
   test("move path extraction includes source and destination", () => {

src/agent/codex-apply-patch.ts

@@ -115,8 +115,7 @@ export function parseCodexApplyPatch(input: string): ParsedPatch {
         );
       }
       // Codex-rs: each '+' line contributes text + "\n".
-      const content =
-        contentLines.length === 0 ? "" : contentLines.map((l) => `${l}\n`).join("");
+      const content = contentLines.length === 0 ? "" : contentLines.map((l) => `${l}\n`).join("");
       ops.push({ type: "add", path, content });
       continue;
     }
@@ -147,7 +146,11 @@ export function parseCodexApplyPatch(input: string): ParsedPatch {
           `malformed Update File '${path}': expected hunk ('@@') or next file op, got: ${body[i]}`,
         );
       }
-      ops.push(moveTo === undefined ? { type: "update", path, hunks } : { type: "update", path, moveTo, hunks });
+      ops.push(
+        moveTo === undefined
+          ? { type: "update", path, hunks }
+          : { type: "update", path, moveTo, hunks },
+      );
       continue;
     }
 
@@ -198,9 +201,7 @@ export function applyUpdateHunks(original: string, hunks: PatchHunk[]): string {
     if (hunk.header !== undefined && hunk.header.length > 0) {
       const idx = findLineFrom(lines, hunk.header, cursor);
       if (idx === -1) {
-        throw new CodexApplyPatchError(
-          `failed to find hunk context header '${hunk.header}'`,
-        );
+        throw new CodexApplyPatchError(`failed to find hunk context header '${hunk.header}'`);
       }
       cursor = idx + 1;
     }
@@ -245,10 +246,7 @@ export function contentFromAddOp(op: PatchAddOp): string {
   return op.content;
 }
 
-function parseHunk(
-  body: string[],
-  start: number,
-): { hunk: PatchHunk; next: number } {
+function parseHunk(body: string[], start: number): { hunk: PatchHunk; next: number } {
   const headerLine = body[start]!;
   let header: string | undefined;
   if (headerLine === "@@") {
@@ -268,9 +266,8 @@ function parseHunk(
     if (raw === END_OF_FILE) {
       i += 1;
       return {
-        hunk: header === undefined
-          ? { lines, endOfFile: true }
-          : { header, lines, endOfFile: true },
+        hunk:
+          header === undefined ? { lines, endOfFile: true } : { header, lines, endOfFile: true },
         next: i,
       };
     }
@@ -305,11 +302,7 @@ function parseHunk(
 }
 
 function isFileOpHeader(line: string): boolean {
-  return (
-    line.startsWith(ADD_FILE) ||
-    line.startsWith(DELETE_FILE) ||
-    line.startsWith(UPDATE_FILE)
-  );
+  return line.startsWith(ADD_FILE) || line.startsWith(DELETE_FILE) || line.startsWith(UPDATE_FILE);
 }
 
 function isHunkStart(line: string): boolean {
@@ -366,9 +359,7 @@ function findSequence(
   if (pattern.length > lines.length) return -1;
 
   const searchStart =
-    endOfFile && lines.length >= pattern.length
-      ? lines.length - pattern.length
-      : from;
+    endOfFile && lines.length >= pattern.length ? lines.length - pattern.length : from;
 
   const tryFrom = (start: number, eq: (a: string, b: string) => boolean): number => {
     for (let i = start; i <= lines.length - pattern.length; i++) {
@@ -385,8 +376,7 @@ function findSequence(
   };
 
   // When eof, try the eof-aligned window first, then fall through from `from`.
-  const starts =
-    endOfFile && searchStart !== from ? [searchStart, from] : [searchStart];
+  const starts = endOfFile && searchStart !== from ? [searchStart, from] : [searchStart];
 
   for (const start of starts) {
     const exact = tryFrom(start, (a, b) => a === b);

src/agent/codex-tool-mount.test.ts

@@ -132,11 +132,7 @@ describe("Codex tool proxy mount", () => {
       runTool: async () => ({ content: "ok" }),
       runManageTasks: async () => ({ content: "ok" }),
     });
-    expect(proxies.map((t) => t.definition.name)).toEqual([
-      "apply_patch",
-      "shell",
-      "update_plan",
-    ]);
+    expect(proxies.map((t) => t.definition.name)).toEqual(["apply_patch", "shell", "update_plan"]);
 
     const allow = new Set<string>(BUILD_TOOLS);
     const kept = proxies.filter((t) => allow.has(t.definition.name));
@@ -160,11 +156,7 @@ describe("Codex tool proxy mount", () => {
       allowDelete: allowDeleteFromCapabilities(docsCapabilities),
       allowShell: allowShellFromCapabilities(docsCapabilities),
     });
-    expect(proxies.map((t) => t.definition.name)).toEqual([
-      "apply_patch",
-      "shell",
-      "update_plan",
-    ]);
+    expect(proxies.map((t) => t.definition.name)).toEqual(["apply_patch", "shell", "update_plan"]);
 
     const docsAllow = new Set<string>(DOCS_TOOLS);
     const docsKept = proxies.filter((t) => docsAllow.has(t.definition.name));

src/agent/codex-tool-proxies.test.ts

@@ -120,7 +120,11 @@ describe("createCodexToolProxies", () => {
 
   test("add forwards write_file with Codex trailing newline", async () => {
     const { calls, files, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -143,7 +147,11 @@ describe("createCodexToolProxies", () => {
 
   test("delete forwards delete_file", async () => {
     const { calls, files, runTool } = makeRecorder({ "obsolete.txt": "gone" });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -159,7 +167,12 @@ describe("createCodexToolProxies", () => {
 
   test("allowDelete false refuses Delete without calling delete_file", async () => {
     const { calls, files, runTool } = makeRecorder({ "obsolete.txt": "gone" });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, allowDelete: false, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      allowDelete: false,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -179,7 +192,12 @@ describe("createCodexToolProxies", () => {
 print("Hi")
 `;
     const { calls, files, runTool } = makeRecorder({ "src/app.py": original });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, allowDelete: false, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      allowDelete: false,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -203,7 +221,12 @@ print("Hi")
 print("Hi")
 `;
     const { calls, files, runTool } = makeRecorder({ "src/app.py": original });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, allowDelete: false, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      allowDelete: false,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -227,7 +250,11 @@ print("Hi")
 print("bye")
 `;
     const { calls, files, runTool } = makeRecorder({ "src/app.py": original });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -253,7 +280,11 @@ print("bye")
 print("Hi")
 `;
     const { calls, files, runTool } = makeRecorder({ "src/app.py": original });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -284,7 +315,11 @@ print("Hello, world!")
       "src/app.py": "old\n",
       "obsolete.txt": "x",
     });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -312,7 +347,11 @@ print("Hello, world!")
 
   test("parse failure surfaces as tool error (isError)", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Add File: a.txt
@@ -342,7 +381,11 @@ print("Hello, world!")
 
   test("runTool isError aborts the patch with isError", async () => {
     const { runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeApplyPatch(
       tools,
       `*** Begin Patch
@@ -361,29 +404,45 @@ print("Hello, world!")
 describe("shell proxy", () => {
   test("string command forwards to run_shell", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeTool(tools, "shell", { command: "ls -la" });
     expect(result.isError).toBeFalsy();
     expect(calls).toEqual([{ name: "run_shell", args: { command: "ls -la" } }]);
   });
 
   test("bash -lc argv triple unwraps to the script", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     await invokeTool(tools, "shell", { command: ["bash", "-lc", "echo 'hi there'"] });
     expect(calls).toEqual([{ name: "run_shell", args: { command: "echo 'hi there'" } }]);
   });
 
   test("other argv arrays are shell-quoted and joined", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     await invokeTool(tools, "shell", { command: ["echo", "hello world"] });
     expect(calls).toEqual([{ name: "run_shell", args: { command: "echo 'hello world'" } }]);
   });
 
   test("workdir and timeout_ms translate to cwd and timeout", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     await invokeTool(tools, "shell", {
       command: "pwd",
       workdir: "/tmp/work",
@@ -396,7 +455,11 @@ describe("shell proxy", () => {
 
   test("missing command surfaces as tool error", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeTool(tools, "shell", {});
     expect(result.isError).toBe(true);
     expect(result.content).toMatch(/command/);
@@ -405,7 +468,12 @@ describe("shell proxy", () => {
 
   test("allowShell false refuses without calling run_shell", async () => {
     const { calls, runTool } = makeRecorder();
-    const tools = createCodexToolProxies({ isCodex: true, runTool, allowShell: false, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      allowShell: false,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeTool(tools, "shell", { command: "ls" });
     expect(result.isError).toBe(true);
     expect(result.content).toMatch(/not allowed/);
@@ -414,7 +482,11 @@ describe("shell proxy", () => {
 
   test("run_shell isError propagates as tool error", async () => {
     const runTool: CodexRunTool = async () => ({ content: "boom", isError: true });
-    const tools = createCodexToolProxies({ isCodex: true, runTool, runManageTasks: unusedManageTasks });
+    const tools = createCodexToolProxies({
+      isCodex: true,
+      runTool,
+      runManageTasks: unusedManageTasks,
+    });
     const result = await invokeTool(tools, "shell", { command: "ls" });
     expect(result.isError).toBe(true);
     expect(result.content).toMatch(/boom/);
@@ -497,19 +569,11 @@ describe("update_plan proxy", () => {
 
 describe("allowDeleteFromCapabilities", () => {
   test("docs allowlist (no delete_file) → false; build → true", () => {
-    expect(
-      allowDeleteFromCapabilities({ mode: "allow", tools: DOCS_TOOLS }),
-    ).toBe(false);
-    expect(
-      allowDeleteFromCapabilities({ mode: "allow", tools: BUILD_TOOLS }),
-    ).toBe(true);
+    expect(allowDeleteFromCapabilities({ mode: "allow", tools: DOCS_TOOLS })).toBe(false);
+    expect(allowDeleteFromCapabilities({ mode: "allow", tools: BUILD_TOOLS })).toBe(true);
     expect(allowDeleteFromCapabilities(undefined)).toBe(true);
-    expect(
-      allowDeleteFromCapabilities({ mode: "exclude", tools: ["run_shell"] }),
-    ).toBe(true);
-    expect(
-      allowDeleteFromCapabilities({ mode: "exclude", tools: ["delete_file"] }),
-    ).toBe(false);
+    expect(allowDeleteFromCapabilities({ mode: "exclude", tools: ["run_shell"] })).toBe(true);
+    expect(allowDeleteFromCapabilities({ mode: "exclude", tools: ["delete_file"] })).toBe(false);
   });
 });
 
@@ -518,11 +582,7 @@ describe("allowShellFromCapabilities", () => {
     expect(allowShellFromCapabilities({ mode: "allow", tools: DOCS_TOOLS })).toBe(false);
     expect(allowShellFromCapabilities({ mode: "allow", tools: BUILD_TOOLS })).toBe(true);
     expect(allowShellFromCapabilities(undefined)).toBe(true);
-    expect(
-      allowShellFromCapabilities({ mode: "exclude", tools: ["delete_file"] }),
-    ).toBe(true);
-    expect(
-      allowShellFromCapabilities({ mode: "exclude", tools: ["run_shell"] }),
-    ).toBe(false);
+    expect(allowShellFromCapabilities({ mode: "exclude", tools: ["delete_file"] })).toBe(true);
+    expect(allowShellFromCapabilities({ mode: "exclude", tools: ["run_shell"] })).toBe(false);
   });
 });