@koda-sl/baker-cli 0.158.2 → 0.159.0-dev.a0159ea6f

package/dist/cli.js

@@ -36,7 +36,7 @@ import {
   toModelSafeImage,
   ulid,
   validateCanvasDeep
-} from "./chunk-CG6OTNUU.js";
+} from "./chunk-K424D3CY.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -3501,7 +3501,7 @@ registerSchema({
     name: { type: "string", description: "Action name (short, action-verb, \u22646 words)", required: true },
     description: {
       type: "string",
-      description: "Context-complete description: what / why / where / done-when",
+      description: "Context-complete description in markdown: what / why / where / done-when. Use `##` headings for those sections, `-` bullets for lists of items, and `**bold**` for the figures that drive the decision. Anything longer than a couple of sentences needs headings \u2014 it is read in a narrow panel.",
       required: false
     },
     tags: {
@@ -4154,7 +4154,11 @@ registerSchema({
   args: {
     id: { type: "string", description: "Action ID or tempId (temp_*) of a task staged in this chat", required: true },
     name: { type: "string", description: "New name", required: false },
-    description: { type: "string", description: "New description", required: false },
+    description: {
+      type: "string",
+      description: "New description, in markdown \u2014 `##` headings per section, `-` bullets, `**bold**` for the decisive figures. REPLACES the whole description, so carry over anything still true.",
+      required: false
+    },
     tags: {
       type: "string",
       description: "Comma-separated tag slugs \u2014 REPLACES existing tags ('' clears)",
@@ -4176,7 +4180,7 @@ var updateCommand = defineCommand17({
     id: { type: "positional", description: "Action ID or tempId (temp_*)", required: false },
     "action-id": { type: "string", description: "Action ID or tempId (temp_*)", required: false },
     name: { type: "string", description: "New name", required: false },
-    description: { type: "string", description: "New description", required: false },
+    description: { type: "string", description: "New description (markdown)", required: false },
     tags: { type: "string", description: "Comma-separated tag slugs \u2014 REPLACES existing ('' clears)", required: false },
     priority: { type: "string", description: `Priority: ${ACTION_PRIORITIES.join("|")}|none`, required: false }
   },
@@ -29494,6 +29498,20 @@ function parseScope(raw) {
 function actingUserId() {
   return getEnv().BAKER_ACTING_USER_ID;
 }
+function customEntry(server) {
+  const failed = server.health && !server.health.ok;
+  const problem = server.health?.message ?? "This server's address is not serving tools.";
+  return {
+    name: server.name,
+    scope: server.scope,
+    enabled: server.enabled,
+    ...failed && !server.health?.transient ? { usable: false, problem } : {},
+    ...failed && server.health?.transient ? { lastCheck: "failed", problem } : {},
+    ...failed && server.health?.suggestedUrl ? { suggestedAddress: server.health.suggestedUrl } : {}
+  };
+}
+var BROKEN_SERVER_HINT = "A server marked usable=false will NOT answer: its tools are absent from this turn. Say plainly that the tool is not working and relay `problem`; never claim its tools are loading, coming next message, or temporarily unavailable. Fixing it needs a person: the user updates its address in the dashboard (Brain \u2192 Integrations \u2192 MCPs). If `suggestedAddress` is present, that is the address to use.";
+var LAST_CHECK_FAILED_HINT = "A server with lastCheck=failed was unreachable or erroring when it was last checked \u2014 that may already have passed, and its tools ARE loaded this turn. Try the tool; if it works, say nothing about the check. If it fails, report `problem` as what happened rather than guessing.";
 function parseHeaders(raw) {
   const list = raw === void 0 ? [] : Array.isArray(raw) ? raw : [raw];
   const headers = {};
@@ -29540,8 +29558,14 @@ A tool the user names that is NOT listed here is simply not connected yet.`
         ...c.accountLabel ? { account: c.accountLabel } : {},
         ...c.readOnly ? { readOnly: true } : {}
       }));
-      const custom = data.custom.map((s) => ({ name: s.name, scope: s.scope, enabled: s.enabled }));
+      const custom = data.custom.map(customEntry);
       const hints = [];
+      if (data.custom.some((s) => s.health && !s.health.ok && !s.health.transient)) {
+        hints.push(BROKEN_SERVER_HINT);
+      }
+      if (data.custom.some((s) => s.health && !s.health.ok && s.health.transient)) {
+        hints.push(LAST_CHECK_FAILED_HINT);
+      }
       if (data.managedDisabled) {
         hints.push("Managed integrations are disabled for this company \u2014 only custom MCP servers load.");
       }
@@ -29577,7 +29601,15 @@ var listCommand13 = defineCommand139({
         "/api/mcp/custom",
         user ? { actingUserId: user } : void 0
       );
-      writeJson({ ok: true, data: data.servers });
+      writeJson({
+        ok: true,
+        // Detail view: the full row (address included) plus the usability verdict.
+        // `health` itself is folded into `usable`/`problem` by `customEntry`.
+        data: data.servers.map((server) => ({ ...server, health: void 0, ...customEntry(server) })),
+        ...data.servers.some((s) => s.health && !s.health.ok) ? {
+          hints: data.servers.some((s) => s.health && !s.health.ok && !s.health.transient) ? [BROKEN_SERVER_HINT] : [LAST_CHECK_FAILED_HINT]
+        } : {}
+      });
     } catch (err) {
       fail6(err);
     }
@@ -29588,7 +29620,11 @@ registerSchema({
   description: "Register a custom MCP server. Tools appear as mcp__<name>__* on the next message. URL must be HTTPS. Scope: company (default, all chats) | org (org admin key) | user (only the current sender).",
   args: {
     name: { type: "string", description: "Server name \u2192 tools appear as mcp__<name>__*", required: true },
-    url: { type: "string", description: "HTTPS MCP endpoint", required: true },
+    url: {
+      type: "string",
+      description: "Full HTTPS MCP endpoint (commonly ends in /mcp, not just the domain)",
+      required: true
+    },
     scope: { type: "string", description: "user | user_org | company | org (default company)", required: false },
     header: { type: "string", description: 'Auth header "Key: Value" (repeatable)', required: false }
   }
@@ -29607,7 +29643,11 @@ Examples:
   },
   args: {
     name: { type: "string", description: "Server name (mcp__<name>__*)", required: true },
-    url: { type: "string", description: "HTTPS MCP endpoint", required: true },
+    url: {
+      type: "string",
+      description: "Full HTTPS MCP endpoint (commonly ends in /mcp, not just the domain)",
+      required: true
+    },
     scope: { type: "string", description: "user | user_org | company | org (default company)", required: false },
     header: { type: "string", description: 'Auth header "Key: Value" (repeatable)', required: false }
   },
@@ -29623,7 +29663,14 @@ Examples:
         ...user ? { actingUserId: user } : {},
         ...headers ? { headers } : {}
       });
-      writeJson({ ok: true, data });
+      writeJson({
+        ok: true,
+        data,
+        hints: [
+          "The address must be the full MCP endpoint (commonly ending in /mcp), not just the domain \u2014 a domain on its own usually answers sign-in and then serves no tools.",
+          "The address is checked in the background. Run `baker mcp connected` on the next message to confirm it serves tools before promising the user anything."
+        ]
+      });
     } catch (err) {
       fail6(err);
     }