fix: ensure resume args precede image args

Author: cryptonerdcn

sdk/typescript/src/exec.ts

@@ -115,16 +115,16 @@ export class CodexExec {
       commandArgs.push("--config", `approval_policy="${args.approvalPolicy}"`);
     }
 
+    if (args.threadId) {
+      commandArgs.push("resume", args.threadId);
+    }
+
     if (args.images?.length) {
       for (const image of args.images) {
         commandArgs.push("--image", image);
       }
     }
 
-    if (args.threadId) {
-      commandArgs.push("resume", args.threadId);
-    }
-
     const env: Record<string, string> = {};
     if (this.envOverride) {
       Object.assign(env, this.envOverride);

sdk/typescript/tests/exec.test.ts

@@ -67,4 +67,30 @@ describe("CodexExec", () => {
       expect(result.error.message).toMatch(/Codex Exec exited/);
     }
   });
+
+  it("places resume args before image args", async () => {
+    const { CodexExec } = await import("../src/exec");
+    spawnMock.mockClear();
+    const child = new FakeChildProcess();
+    spawnMock.mockReturnValue(child as unknown as child_process.ChildProcess);
+
+    setImmediate(() => {
+      child.stdout.end();
+      child.stderr.end();
+      child.emit("exit", 0, null);
+    });
+
+    const exec = new CodexExec("codex");
+    for await (const _ of exec.run({ input: "hi", images: ["img.png"], threadId: "thread-id" })) {
+      // no-op
+    }
+
+    const commandArgs = spawnMock.mock.calls[0]?.[1] as string[] | undefined;
+    expect(commandArgs).toBeDefined();
+    const resumeIndex = commandArgs!.indexOf("resume");
+    const imageIndex = commandArgs!.indexOf("--image");
+    expect(resumeIndex).toBeGreaterThan(-1);
+    expect(imageIndex).toBeGreaterThan(-1);
+    expect(resumeIndex).toBeLessThan(imageIndex);
+  });
 });