fix: add log hint for agent failure (#1550) (thanks @sweepies)

- Remove hardcoded period after error message to avoid double periods
- Move 'Check gateway logs for details' to a new line for better readability

CHANGELOG.md

@@ -15,7 +15,6 @@ Docs: https://docs.clawd.bot
 
 ### Fixes
 - Voice wake: auto-save wake words on blur/submit across iOS/Android and align limits with macOS.
-- Plugins: honor `plugins.slots.memory = "none"` when normalizing config. (#1554) Thanks @andreabadesso.
 - UI: keep the Control UI sidebar visible while scrolling long pages. (#1515) Thanks @pookNast.
 - Tailscale: retry serve/funnel with sudo only for permission errors and keep original failure details. (#1551) Thanks @sweepies.
 - Agents: add CLI log hint to "agent failed before reply" messages. (#1550) Thanks @sweepies.

src/plugins/config-state.test.ts

@@ -1,52 +0,0 @@
-import { describe, expect, it } from "vitest";
-
-import { normalizePluginsConfig } from "./config-state.js";
-
-describe("normalizePluginsConfig", () => {
-  it("uses default memory slot when not specified", () => {
-    const result = normalizePluginsConfig({});
-    expect(result.slots.memory).toBe("memory-core");
-  });
-
-  it("respects explicit memory slot value", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "custom-memory" },
-    });
-    expect(result.slots.memory).toBe("custom-memory");
-  });
-
-  it("disables memory slot when set to 'none'", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "none" },
-    });
-    expect(result.slots.memory).toBeNull();
-  });
-
-  it("disables memory slot when set to 'None' (case insensitive)", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "None" },
-    });
-    expect(result.slots.memory).toBeNull();
-  });
-
-  it("trims whitespace from memory slot value", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "  custom-memory  " },
-    });
-    expect(result.slots.memory).toBe("custom-memory");
-  });
-
-  it("uses default when memory slot is empty string", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "" },
-    });
-    expect(result.slots.memory).toBe("memory-core");
-  });
-
-  it("uses default when memory slot is whitespace only", () => {
-    const result = normalizePluginsConfig({
-      slots: { memory: "   " },
-    });
-    expect(result.slots.memory).toBe("memory-core");
-  });
-});

src/plugins/config-state.ts

@@ -58,7 +58,7 @@ export const normalizePluginsConfig = (
     deny: normalizeList(config?.deny),
     loadPaths: normalizeList(config?.load?.paths),
     slots: {
-      memory: memorySlot === undefined ? defaultSlotIdForKey("memory") : memorySlot,
+      memory: memorySlot ?? defaultSlotIdForKey("memory"),
     },
     entries: normalizePluginEntries(config?.entries),
   };