fix(doctor): replace nonexistent auth add command with openclaw onboard (#24220)

damaozi · damaozi · commit a5745dd6308c · 2026-02-23T15:11:51.000+08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@ Docs: https://docs.openclaw.ai
 ### Fixes
 
 - Security/Skills: escape user-controlled prompt, filename, and output-path values in `openai-image-gen` HTML gallery generation to prevent stored XSS in generated `index.html` output. (#12538) Thanks @CornBrother0x.
+- CLI/Doctor: replace nonexistent `openclaw auth add` command suggestion with `openclaw onboard` in memory search health check. (#24220)
 - Security/OTEL: redact sensitive values (API keys, tokens, credential fields) from diagnostics-otel log bodies, log attributes, and error/reason span fields before OTLP export. (#12542) Thanks @brandonwise.
 - Providers/OpenRouter: remove conflicting top-level `reasoning_effort` when injecting nested `reasoning.effort`, preventing OpenRouter 400 payload-validation failures for reasoning models. (#24120) thanks @tenequm.
 - Skills/Python: add CI + pre-commit linting (`ruff`) and pytest discovery coverage for Python scripts/tests under `skills/`, including package test execution from repo root. Thanks @vincentkoc.
diff --git a/src/commands/doctor-memory-search.test.ts b/src/commands/doctor-memory-search.test.ts
@@ -126,6 +126,38 @@ describe("noteMemorySearchHealth", () => {
     });
     expect(note).not.toHaveBeenCalled();
   });
+
+  it("suggests valid CLI commands when provider key is missing (#24220)", async () => {
+    resolveMemorySearchConfig.mockReturnValue({
+      provider: "openai",
+      local: {},
+      remote: {},
+    });
+    resolveApiKeyForProvider.mockRejectedValue(new Error("no key"));
+
+    await noteMemorySearchHealth(cfg);
+
+    expect(note).toHaveBeenCalledTimes(1);
+    const message = note.mock.calls[0][0] as string;
+    expect(message).not.toContain("openclaw auth add");
+    expect(message).toContain("openclaw onboard");
+  });
+
+  it("suggests valid CLI commands in auto mode when all keys are missing (#24220)", async () => {
+    resolveMemorySearchConfig.mockReturnValue({
+      provider: "auto",
+      local: {},
+      remote: {},
+    });
+    resolveApiKeyForProvider.mockRejectedValue(new Error("no key"));
+
+    await noteMemorySearchHealth(cfg);
+
+    expect(note).toHaveBeenCalledTimes(1);
+    const message = note.mock.calls[0][0] as string;
+    expect(message).not.toContain("openclaw auth add");
+    expect(message).toContain("openclaw onboard");
+  });
 });
 
 describe("detectLegacyWorkspaceDirs", () => {
diff --git a/src/commands/doctor-memory-search.ts b/src/commands/doctor-memory-search.ts
@@ -62,7 +62,7 @@ export async function noteMemorySearchHealth(cfg: OpenClawConfig): Promise<void>
         "",
         "Fix (pick one):",
         `- Set ${envVar} in your environment`,
-        `- Add credentials: ${formatCliCommand(`openclaw auth add --provider ${resolved.provider}`)}`,
+        `- Add credentials: ${formatCliCommand("openclaw onboard")}`,
         `- To disable: ${formatCliCommand("openclaw config set agents.defaults.memorySearch.enabled false")}`,
         "",
         `Verify: ${formatCliCommand("openclaw memory status --deep")}`,
@@ -89,7 +89,7 @@ export async function noteMemorySearchHealth(cfg: OpenClawConfig): Promise<void>
       "",
       "Fix (pick one):",
       "- Set OPENAI_API_KEY, GEMINI_API_KEY, VOYAGE_API_KEY, or MISTRAL_API_KEY in your environment",
-      `- Add credentials: ${formatCliCommand("openclaw auth add --provider openai")}`,
+      `- Add credentials: ${formatCliCommand("openclaw onboard")}`,
       `- For local embeddings: configure agents.defaults.memorySearch.provider and local model path`,
       `- To disable: ${formatCliCommand("openclaw config set agents.defaults.memorySearch.enabled false")}`,
       "",
