fix(openai-chat): skip tool-result images without a usable URL

The tool-output parser accepts any string for image_url, including
the empty file_id shape, and a {"url":""} carrier part would fail
the whole request where the "[image]" marker degrades safely.

Author: DevMello

src/adapters/openai-chat.ts

@@ -87,7 +87,9 @@ function toolResultImageChatParts(content: string | OcxContentPart[]): unknown[]
   if (typeof content === "string") return [];
   const parts: unknown[] = [];
   for (const p of content) {
-    if (p.type !== "image") continue;
+    // Skip parts without a usable URL (the tool-output parser accepts the empty file_id shape):
+    // a {"url":""} part would fail the whole request where the "[image]" marker degrades safely.
+    if (p.type !== "image" || !p.imageUrl) continue;
     parts.push({ type: "image_url", image_url: { url: p.imageUrl, ...(p.detail ? { detail: p.detail } : {}) } });
   }
   return parts;

tests/openai-chat-tool-result-images.test.ts

@@ -85,11 +85,12 @@ test("tool-result images ride a follow-up user message; text, detail, and https
       { type: "text", text: "1 match found" },
       { type: "image", imageUrl: IMAGE_URL, detail: "high" },
       { type: "image", imageUrl: "https://example.test/shot.png" },
+      { type: "image", imageUrl: "" }, // empty file_id shape: keeps its marker, never reaches the carrier
     ]),
   ]);
   assertRoundsUnbroken(messages);
   const tool = messages.find(m => m.role === "tool")!;
-  expect(tool.content).toBe("1 match found[image][image]");
+  expect(tool.content).toBe("1 match found[image][image][image]");
   const carrier = messages.find(isImageCarrier)!;
   expect(carrier).toBeDefined();
   expect(messages.indexOf(carrier)).toBe(messages.indexOf(tool) + 1);