@trops/dash-core 0.1.649 → 0.1.650

package/dist/electron/index.js

@@ -21,7 +21,7 @@ var require$$11 = require('https');
 var require$$0$8 = require('@modelcontextprotocol/sdk/client/index.js');
 var require$$1$6 = require('@modelcontextprotocol/sdk/client/stdio.js');
 var require$$0$7 = require('pkce-challenge');
-var require$$13 = require('child_process');
+var require$$14 = require('child_process');
 var require$$1$5 = require('http');
 var require$$2$2 = require('algoliasearch');
 var require$$3$2 = require('node:path');
@@ -37264,6 +37264,128 @@ var mcpServerKey = {
   SEP,
 };
 
+/**
+ * mcpStartError.js — explain why an MCP server failed to start.
+ *
+ * A stdio server that exits during startup (a missing token, a bad path)
+ * surfaces from the MCP SDK as "MCP error -32000: Connection closed". The
+ * real reason is on the server's stderr, e.g. Slack's
+ *   {"level":"fatal","message":"Authentication required: …"}
+ * This turns that into "Slack couldn't start: Authentication required: ….
+ * Check its settings in Settings › Providers." — one trimmed line, with
+ * anything that looks like a secret redacted.
+ */
+
+const NEXT_STEP = "Check its settings in Settings › Providers.";
+const MAX_REASON = 300;
+
+// SDK messages that say nothing about the cause.
+const UNINFORMATIVE = /connection closed|-32000|process exited|terminated/i;
+// A line worth showing.
+const ERRORISH =
+  /error|required|fatal|denied|not found|invalid|missing|unauthori[sz]ed|forbidden|failed/i;
+// Noise around a crash: stack frames, Node banners, the npx wrapper.
+const NOISE = [
+  /^at\s/,
+  /^node:/,
+  /^Node\.js v/,
+  /^throw\s/,
+  /^\^+$/,
+  /Command failed:/,
+];
+
+const SECRETS = [
+  [/\bxox[a-z]-[A-Za-z0-9-]+/g, "[redacted]"], // Slack
+  [/\bsk-[A-Za-z0-9_-]{10,}/g, "[redacted]"], // OpenAI / Anthropic style
+  [/\bgh[pousr]_[A-Za-z0-9]{20,}/g, "[redacted]"], // GitHub
+  [/\bBearer\s+\S+/gi, "Bearer [redacted]"],
+  // Long random-looking strings (letters and digits mixed).
+  [
+    /\b(?=[A-Za-z0-9_-]*\d)(?=[A-Za-z0-9_-]*[A-Za-z])[A-Za-z0-9_-]{32,}\b/g,
+    "[redacted]",
+  ],
+];
+
+function redact(text) {
+  return SECRETS.reduce((t, [re, sub]) => t.replace(re, sub), text);
+}
+
+function jsonReason(line) {
+  if (!line.startsWith("{")) return null;
+  try {
+    const obj = JSON.parse(line);
+    const level = String(obj.level || obj.severity || "").toLowerCase();
+    const msg = obj.message || obj.msg || obj.error;
+    if (typeof msg !== "string" || !msg.trim()) return null;
+    if (level && !/fatal|error|crit|panic/.test(level)) return null;
+    return msg.trim();
+  } catch (_e) {
+    return null;
+  }
+}
+
+/** The most useful line of a failed server's stderr, or null. */
+function reasonFromStderr(stderr) {
+  const lines = String(stderr || "")
+    .split(/\r?\n/)
+    .map((l) => l.trim())
+    .filter(Boolean);
+  // A structured fatal/error log line is the most precise.
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const r = jsonReason(lines[i]);
+    if (r) return r;
+  }
+  for (let i = lines.length - 1; i >= 0; i--) {
+    const line = lines[i];
+    if (NOISE.some((re) => re.test(line))) continue;
+    if (!ERRORISH.test(line)) continue;
+    return line.replace(/^(Error|Fatal|FATAL|ERROR):\s*/, "");
+  }
+  return null;
+}
+
+/**
+ * @param {{ serverName: string, message: string, stderr?: string|null }} args
+ * @returns {string} the message to show
+ */
+function describeStartFailure$1({ serverName, message, stderr }) {
+  if (message && !UNINFORMATIVE.test(message)) return message;
+  const name = serverName || "The MCP server";
+  const raw = reasonFromStderr(stderr);
+  if (!raw)
+    return `${name} couldn't start (it closed the connection). ${NEXT_STEP}`;
+  let reason = redact(raw).replace(/[.\s]+$/, "");
+  if (reason.length > MAX_REASON)
+    reason = reason.slice(0, MAX_REASON).trimEnd() + "…";
+  return `${name} couldn't start: ${reason}. ${NEXT_STEP}`;
+}
+
+/**
+ * The last `limit` characters of a stream (a failed server's stderr), so a
+ * chatty server can't grow memory without bound.
+ */
+let StderrTail$1 = class StderrTail {
+  constructor(limit = 4096) {
+    this._limit = limit;
+    this._text = "";
+  }
+
+  push(chunk) {
+    this._text += Buffer.isBuffer(chunk)
+      ? chunk.toString("utf8")
+      : String(chunk);
+    if (this._text.length > this._limit) {
+      this._text = this._text.slice(-this._limit);
+    }
+  }
+
+  text() {
+    return this._text;
+  }
+};
+
+var mcpStartError = { describeStartFailure: describeStartFailure$1, StderrTail: StderrTail$1 };
+
 /**
  * toolSources.js — how a bot finds, starts, and uses the user's Dash MCP
  * providers (the integrations configured under Settings → Providers).
@@ -37896,6 +38018,7 @@ const os$2 = require$$2;
 const responseCache$2 = responseCache_1;
 const { gateToolCall, gateToolCallWithJit } = permissionGate;
 const { serverKey, parseServerKey } = mcpServerKey;
+const { describeStartFailure, StderrTail } = mcpStartError;
 const { connectedServersFromMap } = toolSources;
 const { applyPathScopeToCredentials } = mcpScopeResolver;
 const { readEnforceFlag: readEnforceFlag$1, readJitFlag: readJitFlag$1 } = securityFlags;
@@ -38070,7 +38193,7 @@ function getShellPath$1() {
     return _shellPath$1;
   }
 
-  const { execSync } = require$$13;
+  const { execSync } = require$$14;
   const fallbackDirs = ["/usr/local/bin", "/opt/homebrew/bin"];
 
   // Scan nvm versions, tracking both latest and best compatible version
@@ -38418,6 +38541,9 @@ const mcpController$3 = {
 
     // 3. Fresh start — wrap in a promise and track it
     const startPromise = (async () => {
+      // A stdio server's recent stderr — the real reason when it exits
+      // during startup (e.g. a missing token).
+      let stderrTail = null;
       try {
         // Stop if in stale/error state
         if (activeServers.has(key)) {
@@ -38569,7 +38695,15 @@ const mcpController$3 = {
             command: resolved.command,
             args,
             env: resolved.env,
+            stderr: "pipe",
           });
+          stderrTail = new StderrTail();
+          if (transport.stderr) {
+            transport.stderr.on("data", (chunk) => {
+              stderrTail.push(chunk);
+              process.stderr.write(chunk);
+            });
+          }
         }
 
         // Update status to connecting
@@ -38659,6 +38793,13 @@ const mcpController$3 = {
           errorMessage =
             "This MCP server is incompatible with your system Node.js version. " +
             "Install Node.js v22 (LTS) using nvm and restart the app.";
+        } else {
+          // "Connection closed" says nothing; use the server's own reason.
+          errorMessage = describeStartFailure({
+            serverName,
+            message: error.message,
+            stderr: stderrTail ? stderrTail.text() : "",
+          });
         }
 
         // Mark as error state
@@ -39196,7 +39337,7 @@ const mcpController$3 = {
    * @returns {{ success } | { error, message }}
    */
   runAuth: async (win, mcpConfig, credentials, authCommand) => {
-    const { spawn } = require$$13;
+    const { spawn } = require$$14;
 
     const env = cleanEnvForChildProcess();
 
@@ -60061,7 +60202,7 @@ var mcpDashServerController_1 = mcpDashServerController$4;
  * can use the Chat widget without a separate API key.
  */
 
-const { spawn, execSync } = require$$13;
+const { spawn, execSync } = require$$14;
 const {
   LLM_STREAM_DELTA: LLM_STREAM_DELTA$2,
   LLM_STREAM_TOOL_CALL: LLM_STREAM_TOOL_CALL$2,