fix(doctor): keep shell completion lint informational

giodl73-repo · giodl73-repo · commit 7b8ecb1f9c7e · 2026-05-25T08:11:28.000-07:00
diff --git a/src/commands/doctor-completion.test.ts b/src/commands/doctor-completion.test.ts
@@ -23,7 +23,7 @@ describe("shell completion health mapping", () => {
     expect(shellCompletionStatusToHealthFindings(current)).toEqual([
       expect.objectContaining({
         checkId: "core/doctor/shell-completion",
-        severity: "warning",
+        severity: "info",
         path: "shellCompletion.zsh",
       }),
     ]);
@@ -48,6 +48,7 @@ describe("shell completion health mapping", () => {
 
     expect(shellCompletionStatusToHealthFindings(current)).toEqual([
       expect.objectContaining({
+        severity: "info",
         message: expect.stringContaining("cache is missing"),
         fixHint: expect.stringContaining("openclaw doctor --fix"),
       }),
diff --git a/src/commands/doctor-completion.ts b/src/commands/doctor-completion.ts
@@ -95,7 +95,7 @@ export function shellCompletionStatusToHealthFindings(
     return [
       {
         checkId,
-        severity: "warning",
+        severity: "info",
         message: `Your ${status.shell} profile uses slow dynamic completion (source <(...)).`,
         path,
         fixHint: "Run `openclaw doctor --fix` to upgrade to cached completion.",
@@ -106,7 +106,7 @@ export function shellCompletionStatusToHealthFindings(
     return [
       {
         checkId,
-        severity: "warning",
+        severity: "info",
         message: `Shell completion is configured in your ${status.shell} profile but the cache is missing.`,
         path,
         fixHint: `Run \`openclaw completion --write-state\` or \`openclaw doctor --fix\` to regenerate ${status.cachePath}.`,
diff --git a/src/flows/doctor-lint-flow.test.ts b/src/flows/doctor-lint-flow.test.ts
@@ -92,4 +92,10 @@ describe("exitCodeFromFindings", () => {
     expect(exitCodeFromFindings(findings, "warning")).toBe(1);
     expect(exitCodeFromFindings(findings, "error")).toBe(0);
   });
+
+  it("does not fail default lint for informational findings", () => {
+    const findings = [{ checkId: "a", severity: "info" as const, message: "info" }];
+
+    expect(exitCodeFromFindings(findings)).toBe(0);
+  });
 });
