fix: llm-task tool schema compatibility with llama.cpp + doctor SecretRef crash

jmzlx · jmzlx · commit 0909f2bbbc1d · 2026-03-04T20:41:48.000-08:00
Two fixes: 1. llm-task plugin: Type.Unknown() produces JSON Schema without a 'type' field, which llama.cpp rejects during schema-to-grammar conversion. Replace with Type.Unsafe<unknown>({type: 'object', ...}) to emit a concrete type. Fixes #35443. 2. openclaw doctor: resolved?.remote?.apiKey?.trim() crashes when apiKey is a SecretRef object (not a string). Guard with typeof check. Fixes #35444.
diff --git a/extensions/llm-task/src/llm-task-tool.ts b/extensions/llm-task/src/llm-task-tool.ts
@@ -74,9 +74,17 @@ export function createLlmTaskTool(api: OpenClawPluginApi) {
       "Run a generic JSON-only LLM task and return schema-validated JSON. Designed for orchestration from Lobster workflows via openclaw.invoke.",
     parameters: Type.Object({
       prompt: Type.String({ description: "Task instruction for the LLM." }),
-      input: Type.Optional(Type.Unknown({ description: "Optional input payload for the task." })),
+      input: Type.Optional(
+        Type.Unsafe<unknown>({
+          type: "object",
+          description: "Optional input payload for the task.",
+        }),
+      ),
       schema: Type.Optional(
-        Type.Unknown({ description: "Optional JSON Schema to validate the returned JSON." }),
+        Type.Unsafe<unknown>({
+          type: "object",
+          description: "Optional JSON Schema to validate the returned JSON.",
+        }),
       ),
       provider: Type.Optional(
         Type.String({ description: "Provider override (e.g. openai-codex, anthropic)." }),
diff --git a/src/commands/doctor-memory-search.ts b/src/commands/doctor-memory-search.ts
@@ -26,7 +26,10 @@ export async function noteMemorySearchHealth(
   const agentId = resolveDefaultAgentId(cfg);
   const agentDir = resolveAgentDir(cfg, agentId);
   const resolved = resolveMemorySearchConfig(cfg, agentId);
-  const hasRemoteApiKey = Boolean(resolved?.remote?.apiKey?.trim());
+  const rawApiKey = resolved?.remote?.apiKey;
+  const hasRemoteApiKey = Boolean(
+    typeof rawApiKey === "string" ? rawApiKey.trim() : rawApiKey,
+  );
 
   if (!resolved) {
     note("Memory search is explicitly disabled (enabled: false).", "Memory search");
