fix(wizard): report keyless web search providers as ready

parallel-m · web-flow · commit 9be1699074bf · 2026-06-11T23:18:49.000+09:00
Onboarding finalize now treats configured web search providers with requiresCredential: false as ready instead of warning that an API key is missing. This covers keyless providers such as Parallel Search (Free), DuckDuckGo, and Ollama while preserving credential-required warnings for providers that need keys.\n\nProof: focused wizard/search tests; oxlint on changed files; git diff --check; autoreview clean; Azure Crabbox check:changed cbx_b92ef084c21c passed; GitHub checks green.
diff --git a/src/wizard/i18n/locales/en.ts b/src/wizard/i18n/locales/en.ts
@@ -998,6 +998,7 @@ export const en = {
       webSearchGetKey: "Get your key at: {url}",
       webSearchAuthProfile: "Credential: existing {provider} auth profile.",
       webSearchKeyEnv: "API key: provided via {env} env var.",
+      webSearchKeyFree: "Web search is ready — this provider works with no API key.",
       webSearchKeyRef: "API key: configured via secret reference.",
       webSearchKeyStored: "API key: stored in config.",
       webSearchNeedsKey: "web_search will not work until a key is added.",
diff --git a/src/wizard/i18n/locales/zh-CN.ts b/src/wizard/i18n/locales/zh-CN.ts
@@ -962,6 +962,7 @@ export const zh_CN = {
       webSearchGetKey: "获取 key：{url}",
       webSearchAuthProfile: "凭据：使用已有 {provider} auth profile。",
       webSearchKeyEnv: "API key：通过 {env} 环境变量提供。",
+      webSearchKeyFree: "Web search 已就绪 —— 此 provider 无需 API key。",
       webSearchKeyRef: "API key：通过 secret reference 配置。",
       webSearchKeyStored: "API key：已存入配置。",
       webSearchNeedsKey: "添加 key 前 web_search 无法工作。",
diff --git a/src/wizard/i18n/locales/zh-TW.ts b/src/wizard/i18n/locales/zh-TW.ts
@@ -963,6 +963,7 @@ export const zh_TW = {
       webSearchGetKey: "取得 key：{url}",
       webSearchAuthProfile: "憑證：使用已有 {provider} auth profile。",
       webSearchKeyEnv: "API key：透過 {env} 環境變數提供。",
+      webSearchKeyFree: "Web search 已就緒 —— 此 provider 無需 API key。",
       webSearchKeyRef: "API key：透過 secret reference 設定。",
       webSearchKeyStored: "API key：已存入設定。",
       webSearchNeedsKey: "新增 key 前 web_search 無法運作。",
diff --git a/src/wizard/setup.finalize.test.ts b/src/wizard/setup.finalize.test.ts
@@ -188,6 +188,7 @@ function createWebSearchProviderEntry(
     | "placeholder"
     | "signupUrl"
     | "credentialPath"
+    | "requiresCredential"
   >,
 ): PluginWebSearchProviderEntry {
   return {
@@ -839,6 +840,49 @@ describe("finalizeSetupWizard", () => {
     );
   });
 
+  it("reports a keyless provider as ready without prompting for an API key", async () => {
+    listConfiguredWebSearchProviders.mockReturnValue([
+      createWebSearchProviderEntry({
+        id: "parallel-free",
+        label: "Parallel Search (Free)",
+        hint: "Free web search via Parallel's hosted Search MCP",
+        envVars: [],
+        placeholder: "",
+        signupUrl: "https://parallel.ai",
+        credentialPath: "",
+        requiresCredential: false,
+      }),
+    ]);
+
+    const prompter = createLaterPrompter();
+
+    await finalizeSetupWizard(
+      createAdvancedFinalizeArgs({
+        nextConfig: {
+          tools: { web: { search: { provider: "parallel-free", enabled: true } } },
+        },
+        prompter,
+      }),
+    );
+
+    expectNoteContains(
+      prompter,
+      "Web search is ready — this provider works with no API key.",
+      "Web search",
+    );
+    // The credential-required warning must NOT appear for a keyless provider.
+    expect(
+      vi
+        .mocked(prompter.note)
+        .mock.calls.some(
+          ([message, title]) =>
+            title === "Web search" &&
+            (message.includes("no API key was found") ||
+              message.includes("will not work until a key is added")),
+        ),
+    ).toBe(false);
+  });
+
   it("uses the setup token for health checks to avoid local env token drift", async () => {
     vi.stubEnv("OPENCLAW_GATEWAY_TOKEN", "env-token");
     const prompter = createLaterPrompter();
diff --git a/src/wizard/setup.finalize.ts b/src/wizard/setup.finalize.ts
@@ -644,6 +644,18 @@ export async function finalizeSetupWizard(
         ].join("\n"),
         t("wizard.finalize.webSearchTitle"),
       );
+    } else if (webSearchEnabled !== false && entry.requiresCredential === false) {
+      // Keyless providers (e.g. Parallel Search (Free), DuckDuckGo, Ollama) need
+      // no API key — report ready rather than the credential-required warning.
+      await prompter.note(
+        [
+          t("wizard.finalize.webSearchKeyFree"),
+          "",
+          t("wizard.finalize.webSearchProvider", { provider: label }),
+          t("wizard.finalize.webDocs"),
+        ].join("\n"),
+        t("wizard.finalize.webSearchTitle"),
+      );
     } else if (webSearchEnabled !== false && hasCredential) {
       await prompter.note(
         [
@@ -655,7 +667,7 @@ export async function finalizeSetupWizard(
         ].join("\n"),
         t("wizard.finalize.webSearchTitle"),
       );
-    } else if (!hasCredential) {
+    } else if (entry.requiresCredential !== false && !hasCredential) {
       await prompter.note(
         [
           t("wizard.finalize.webSearchNoKey", { provider: label }),
