fix(tavily): reject blank extract URLs

VectorPeak · chatgpt-codex-connector[bot] · VectorPeak · commit cfe24e7fb687 · 2026-07-19T21:07:32.000+08:00
Co-authored-by: chatgpt-codex-connector[bot] &lt;199175422+chatgpt-codex-connector[bot]@users.noreply.github.com&gt;
diff --git a/extensions/tavily/src/tavily-extract-tool.ts b/extensions/tavily/src/tavily-extract-tool.ts
@@ -50,7 +50,7 @@ export function createTavilyExtractTool(api: OpenClawPluginApi, ctx?: TavilyTool
     parameters: TavilyExtractToolSchema,
     execute: async (_toolCallId: string, rawParams: Record<string, unknown>) => {
       const urls = Array.isArray(rawParams.urls)
-        ? (rawParams.urls as string[]).filter(Boolean)
+        ? rawParams.urls.map((url) => (typeof url === "string" ? url.trim() : "")).filter(Boolean)
         : [];
       if (urls.length === 0) {
         throw new Error("tavily_extract requires at least one URL.");
diff --git a/extensions/tavily/src/tavily-tools.test.ts b/extensions/tavily/src/tavily-tools.test.ts
@@ -351,6 +351,28 @@ describe("tavily tools", () => {
     expect(runTavilyExtract).not.toHaveBeenCalled();
   });
 
+  it("rejects blank extract URLs before Tavily calls and trims valid URLs", async () => {
+    const tool = createTavilyExtractTool(fakeApi());
+
+    await expect(
+      tool.execute("extract-call", {
+        urls: ["   "],
+      }),
+    ).rejects.toThrow("tavily_extract requires at least one URL.");
+
+    expect(runTavilyExtract).not.toHaveBeenCalled();
+
+    await tool.execute("extract-call", {
+      urls: [" https://example.com/article "],
+    });
+
+    const extractParams = requireFirstMockArg(
+      runTavilyExtract,
+      "Tavily extract params",
+    ) as TavilyExtractParams;
+    expect(extractParams.urls).toEqual(["https://example.com/article"]);
+  });
+
   it("rejects fractional and out-of-range integer options before Tavily calls", async () => {
     const searchTool = createTavilySearchTool(fakeApi());
     await expect(
