fix(agent): keep gpt shell arguments through dispatch

Author: TheGreatAxios

src/agent/tool-aliases.test.ts

@@ -191,6 +191,43 @@ describe("hidden alias dispatch", () => {
     expect(seen.timeout).toBe(5000);
   });
 
+  test("gpt shell keeps run_shell cwd, timeout, background, and stop", async () => {
+    let seen: Record<string, unknown> = {};
+    const runner = createDynamicToolRunner([
+      {
+        kind: "string",
+        definition: posixDef("run_shell"),
+        handler: async (args) => {
+          seen = args;
+          return "ok";
+        },
+      },
+    ]);
+    runner.setCallGate(() => true);
+    const result = await runner.run(
+      {
+        id: "1",
+        name: "shell",
+        arguments: {
+          command: "npm test",
+          cwd: "/tmp/proj",
+          timeout: 5000,
+          background: true,
+          stop: "sh_1",
+        },
+      },
+      new AbortController().signal,
+    );
+    expect(result.content).toBe("ok");
+    expect(seen).toEqual({
+      command: "npm test",
+      cwd: "/tmp/proj",
+      timeout: 5000,
+      background: true,
+      stop: "sh_1",
+    });
+  });
+
   test("update_plan hidden-dispatches onto manage_tasks; apply_patch does not", async () => {
     const runner = createDynamicToolRunner([
       {

src/agent/tool-aliases.ts

@@ -428,18 +428,16 @@ function incomingAlias(requested: string): string {
 }
 
 /** Coerce hidden Codex-shaped arguments onto the engine tool. Callers pass
- * the already-resolved engine. */
+ * the already-resolved engine. gpt advertises `shell` with run_shell's own
+ * arguments (`cwd`, `timeout`, `background`, `stop`); those pass through.
+ * Only Codex argv, `workdir`, and `timeout_ms` are rewritten. */
 export function prepareDispatchedToolCall(
   call: ToolCall,
   engine: string,
 ): ToolCall {
   const incoming = incomingAlias(call.name);
   let args = call.arguments;
-  if (
-    incoming === "shell" ||
-    incoming.toLowerCase() === "shell" ||
-    (engine === "run_shell" && looksLikeCodexShellArgs(args))
-  ) {
+  if (engine === "run_shell" && looksLikeCodexShellArgs(args)) {
     args = coerceShellArgs(args);
   }
   if (