fix(doctor): keep skipped local model diagnostics

vincentkoc · vincentkoc · commit f6ab4a6535ea · 2026-06-23T23:49:14.000+08:00
diff --git a/src/commands/doctor-memory-search.test.ts b/src/commands/doctor-memory-search.test.ts
@@ -257,6 +257,27 @@ describe("noteMemorySearchHealth", () => {
     expect(note).not.toHaveBeenCalled();
   });
 
+  it("warns when local provider skipped readiness but configured local model is missing", async () => {
+    resolveMemorySearchConfig.mockReturnValue({
+      provider: "local",
+      local: { modelPath: "/definitely/missing/openclaw-memory-model.gguf" },
+      remote: {},
+    });
+
+    await noteMemorySearchHealth(cfg, {
+      gatewayMemoryProbe: {
+        checked: false,
+        ready: false,
+        error:
+          "memory embedding readiness not checked; run `openclaw memory status --deep` to probe",
+        skipped: true,
+      },
+    });
+
+    expect(note).toHaveBeenCalledTimes(1);
+    expect(firstNoteMessage()).toContain('Memory search provider is set to "local"');
+  });
+
   it("warns when local provider readiness probe is inconclusive", async () => {
     resolveMemorySearchConfig.mockReturnValue({
       provider: "local",
diff --git a/src/commands/doctor-memory-search.ts b/src/commands/doctor-memory-search.ts
@@ -472,10 +472,12 @@ export async function noteMemorySearchHealth(
     if (opts?.gatewayMemoryProbe?.checked && opts.gatewayMemoryProbe.ready) {
       return;
     }
-    if (opts?.gatewayMemoryProbe?.skipped) {
+    const hasExplicitLocalModel = hasLocalEmbeddings(resolved.local);
+    const hasUnavailableConfiguredLocalModel =
+      Boolean(normalizeOptionalString(resolved.local.modelPath)) && !hasExplicitLocalModel;
+    if (opts?.gatewayMemoryProbe?.skipped && !hasUnavailableConfiguredLocalModel) {
       return;
     }
-    const hasExplicitLocalModel = hasLocalEmbeddings(resolved.local);
     const detail = opts?.gatewayMemoryProbe?.error?.trim();
     note(
       [

Original file line number	Diff line number	Diff line change
`@@ -472,10 +472,12 @@ export async function noteMemorySearchHealth(`
`472`	`472`	`if (opts?.gatewayMemoryProbe?.checked && opts.gatewayMemoryProbe.ready) {`
`473`	`473`	`return;`
`474`	`474`	`}`
`475`		`- if (opts?.gatewayMemoryProbe?.skipped) {`
	`475`	`+ const hasExplicitLocalModel = hasLocalEmbeddings(resolved.local);`
	`476`	`+ const hasUnavailableConfiguredLocalModel =`
	`477`	`+ Boolean(normalizeOptionalString(resolved.local.modelPath)) && !hasExplicitLocalModel;`
	`478`	`+ if (opts?.gatewayMemoryProbe?.skipped && !hasUnavailableConfiguredLocalModel) {`
`476`	`479`	`return;`
`477`	`480`	`}`
`478`		`- const hasExplicitLocalModel = hasLocalEmbeddings(resolved.local);`
`479`	`481`	`const detail = opts?.gatewayMemoryProbe?.error?.trim();`
`480`	`482`	`note(`
`481`	`483`	`[`