pi-mcp-client 0.3.0 → 0.3.2

package/dist/index.js

@@ -30,6 +30,8 @@ function diagnostic(code, context) {
     cancelled: "Start the operation again when ready.",
     connection_failed: `Check the URL or executable, working directory, network, and TLS setup; then run /mcp reconnect ${target}.`,
     protocol_error: "Check the server's MCP compatibility and protocol setting. Only stdio and Streamable HTTP are supported.",
+    server_unknown: "Choose a configured MCP server from the capability directory or run /mcp to list servers. Omit server in mcp_tools to search all enabled servers.",
+    server_disabled: `Run /mcp enable ${target}, or omit server in mcp_tools to search all enabled servers.`,
     tool_changed: "Check server filters and use mcp_tools with activate and the exact identifier to activate the current tool definition. Reload Pi if the connection configuration changed.",
     tool_error: "Review the server's tool result and inputs. Verify the outcome before retrying.",
     oauth_failed: `Check OAuth support and browser access to the local callback, then run /mcp auth ${target}.`,
@@ -137,6 +139,8 @@ var init_diagnostics = __esm({
       cancelled: "The operation was cancelled.",
       connection_failed: "The server connection failed.",
       protocol_error: "The server response or protocol is not supported.",
+      server_unknown: "The MCP server is not configured.",
+      server_disabled: "The MCP server is disabled.",
       tool_changed: "The tool is unavailable or its configuration or schema changed.",
       tool_error: "The tool reported an error.",
       oauth_failed: "OAuth authentication did not complete.",
@@ -3024,7 +3028,14 @@ var McpRuntime = class {
     return waitFor(state.listing, signal);
   }
   async discover(server, signal) {
-    if (typeof server === "string") this.definition(server);
+    if (typeof server === "string") {
+      const config = Object.hasOwn(this.config, server) ? this.config[server] : void 0;
+      if (!config || config.disabled)
+        throw new DiagnosticError(diagnostic(
+          config ? "server_disabled" : "server_unknown",
+          { server, operation: "search" }
+        ));
+    }
     const names = Array.isArray(server) ? [...new Set(server)] : server ? [server] : Object.keys(this.config).filter((name) => !this.config[name].disabled).sort();
     const result = {
       tools: [],
@@ -3481,27 +3492,31 @@ function renderResult(result, options, theme, isError) {
       state: isError ? "failed" : options.isPartial ? "running" : "done"
     }
   ];
+  const bodyInRow = details?.rows.length === 1 && details.rows[0].state === "failed" && details.rows[0].label === text && !/[\r\n]/.test(text);
   let formattedOutput;
   return {
     render(width) {
       if (width <= 0) return [];
       const lines = rows.flatMap((row) => {
         const status = states[row.state] ?? states.failed;
-        const value = theme.fg(status.color, status.glyph) + " " + theme.fg("accent", line(row.label)) + (row.inlineDescription ? theme.fg("dim", ` ${line(row.inlineDescription)}`) : "");
+        const value = theme.fg(status.color, status.glyph) + " " + theme.fg(status === states.failed ? "error" : "accent", line(row.label)) + (row.inlineDescription ? theme.fg(status === states.failed ? "error" : "dim", ` ${line(row.inlineDescription)}`) : "");
         const rendered = options.expanded && (!details?.searchNotes || row.state === "failed") ? new Text(value, 0, 0).render(width).map((x) => truncateToWidth2(x, width)) : [truncateToWidth2(value, width)];
         if (options.expanded && row.description)
           rendered.push(truncateToWidth2(
-            theme.fg("dim", `  ${line(row.description)}`),
+            theme.fg(status === states.failed ? "error" : "dim", `  ${line(row.description)}`),
             width
           ));
         return rendered;
       });
       for (const note of details?.searchNotes ?? [])
         lines.push(...(options.expanded ? new Text(theme.fg("warning", plain(note)), 0, 0).render(width) : [theme.fg("warning", line(note))]).map((row) => truncateToWidth2(row, width)));
-      if (options.expanded && !options.isPartial && text && !details?.searchNotes)
+      if (options.expanded && !options.isPartial && text && !details?.searchNotes && !bodyInRow)
         lines.push(
           ...new Text(
-            formattedOutput ??= validation ? formatValidation(body, theme) : formatOutput(body, details?.displayBlocks, theme),
+            formattedOutput ??= (() => {
+              const output = validation ? formatValidation(body, theme) : formatOutput(body, details?.displayBlocks, theme);
+              return isError || details?.failed ? theme.fg("error", output) : output;
+            })(),
             0,
             0
           ).render(width).map((x) => truncateToWidth2(x, width))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-mcp-client",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "MCP tools for Pi, discovered on demand and called natively through the official SDK.",
   "type": "module",
   "license": "MIT",