@ateam-ai/mcp 0.4.43 → 0.4.45

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ateam-ai/mcp",
-  "version": "0.4.43",
+  "version": "0.4.45",
   "mcpName": "io.github.ariekogan/ateam-mcp",
   "description": "A-Team MCP Server — build, validate, and deploy multi-agent solutions from any AI environment",
   "type": "module",

package/src/api.js

@@ -461,7 +461,21 @@ function formatError(method, path, status, body, baseUrl) {
   }
 
   const hint = hints[status] || "";
-  const detail = typeof body === "string" && body.length > 0 && body.length < 2000 ? body : "";
+  // A JSON error body used to be dropped entirely — only a string body became
+  // `detail`. So an endpoint that answers 4xx WITH the diagnosis (ui.surfaceProbe
+  // returns 422 + the failures that explain why the surface is broken) reached
+  // the caller as a bare "returned 422", throwing away the very thing it was
+  // asked to produce. Serialize objects too, capped the same way.
+  let detail = "";
+  if (typeof body === "string") {
+    if (body.length > 0 && body.length < 2000) detail = body;
+  } else if (body && typeof body === "object") {
+    try {
+      const s = JSON.stringify(body);
+      if (s.length < 2000) detail = s;
+      else detail = s.slice(0, 2000) + "… (truncated)";
+    } catch { /* non-serializable — leave detail empty */ }
+  }
 
   // Always show the FULL URL actually hit — ateam-mcp is a PUBLIC MCP with a
   // configurable base (prod default, dev/self-host overrides), so a bare

package/src/tools.js

@@ -1965,6 +1965,37 @@ export const tools = [
       required: ["solution_id", "plugin_id"],
     },
   },
+  {
+    name: "ateam_connector_logs",
+    core: true,
+    description:
+      "Read what a connector process actually PRINTED to stderr. This is the only place a connector's " +
+      "internal failure is visible: a tool that catches its own error still returns ok:true, and the widget " +
+      "then renders an empty state that looks like real data.\n\n" +
+      "Real case (2026-08-11): a dashboard connector's ledger.getData got 401 Authentication required from " +
+      "Core, swallowed it, returned an empty ledger, and displayed 0.00 everywhere — while the upload said " +
+      "ok, the tool said ok:true, and the surface probe said surface_ok. The word 'Authentication' appeared " +
+      "ONLY here.\n\n" +
+      "USE IT whenever a tool succeeds but the data is empty, wrong, or zero — that combination is the " +
+      "signature of a swallowed error, and 'the call returned ok' is not evidence it worked. Pass the " +
+      "returned `cursor` back as `since` to read only what is new since your last look, so you can bracket " +
+      "an action and see exactly what it printed. Only stdio (solution) connectors stream stderr through " +
+      "Core; a platform/HTTP connector answers ok:false with a reason.",
+    inputSchema: {
+      type: "object",
+      properties: {
+        solution_id: { type: "string", description: "The solution ID" },
+        connector_id: { type: "string", description: "The connector ID (e.g. 'accounting-dashboard-mcp')" },
+        since: {
+          type: "number",
+          description: "Cursor from a previous call — returns only lines printed after it. Omit for the whole retained tail.",
+        },
+        limit: { type: "number", description: "Max lines (default 100, max 300)" },
+        errors_only: { type: "boolean", description: "Keep only lines that read as errors (401/failed/exception/refused/…)" },
+      },
+      required: ["solution_id", "connector_id"],
+    },
+  },
   {
     name: "ateam_redeploy",
     core: true,
@@ -2078,6 +2109,7 @@ const TENANT_TOOLS = new Set([
   "ateam_list_solutions",
   "ateam_get_solution",
   "ateam_get_execution_logs",
+  "ateam_connector_logs",
   "ateam_conversation",
   "ateam_test_skill",
   "ateam_test_notification",
@@ -2619,7 +2651,7 @@ const handlers = {
         { step: 2, action: "Build & Run", description: "Define your solution + skills + connector code, then validate, deploy, and health-check in one call. Include mcp_store with connector source code on the first deploy.", tools: ["ateam_build_and_run"] },
         { step: 3, action: "Version", description: "Every deploy auto-pushes to main on GitHub. The repo (tenant--solution-id) is the source of truth for connector code.", tools: ["ateam_github_status", "ateam_github_log"] },
         { step: 4, action: "Iterate", description: "Edit connector code ONE FILE AT A TIME via ateam_github_patch, then redeploy with ateam_build_and_run (auto-pulls from GitHub). NEVER re-pass all connector code inline after first deploy. For skill definitions, use ateam_patch.", tools: ["ateam_github_patch", "ateam_build_and_run", "ateam_patch"] },
-        { step: 5, action: "Test & Debug", description: "Chat with the solution via ateam_conversation (auto-routes; multi-turn via actor_id). It is ASYNC — see conversation_flow below: kick off → get chain_id → poll ateam_chain_status until chain_done → read the reply. Use ateam_test_pipeline for intent debugging, ateam_test_voice for voice. For a UI plugin, ateam_verify_surface PROVES it renders with data (required evidence for a user-visible fix). Diagnose with logs and metrics.", tools: ["ateam_conversation", "ateam_chain_status", "ateam_get_chain", "ateam_test_pipeline", "ateam_test_skill", "ateam_test_voice", "ateam_verify_surface", "ateam_get_execution_logs", "ateam_get_metrics"] },
+        { step: 5, action: "Test & Debug", description: "Chat with the solution via ateam_conversation (auto-routes; multi-turn via actor_id). It is ASYNC — see conversation_flow below: kick off → get chain_id → poll ateam_chain_status until chain_done → read the reply. Use ateam_test_pipeline for intent debugging, ateam_test_voice for voice. For a UI plugin, ateam_verify_surface PROVES it renders with data (required evidence for a user-visible fix). Diagnose with logs and metrics. ⚠️ A tool answering ok:true with EMPTY/zero data is not proof it worked — that is the signature of a connector swallowing its own error. Read ateam_connector_logs before you believe a green result.", tools: ["ateam_conversation", "ateam_chain_status", "ateam_get_chain", "ateam_test_pipeline", "ateam_test_skill", "ateam_test_voice", "ateam_verify_surface", "ateam_connector_logs", "ateam_get_execution_logs", "ateam_get_metrics"] },
         { step: 6, action: "Checkpoint", description: "When solution is in a good state, create a checkpoint (safe point). You can rollback to any checkpoint if something breaks.", tools: ["ateam_github_promote", "ateam_github_list_versions"] },
       ],
     },
@@ -2681,7 +2713,7 @@ const handlers = {
     },
     advanced_tools: {
       _note: "These tools are available but hidden from the default tool list. Call them by name when you need fine-grained control.",
-      debugging: ["ateam_get_execution_logs", "ateam_get_metrics", "ateam_diff", "ateam_get_connector_source"],
+      debugging: ["ateam_get_execution_logs", "ateam_connector_logs", "ateam_get_metrics", "ateam_diff", "ateam_get_connector_source"],
       manual_lifecycle: ["ateam_validate_skill", "ateam_validate_solution", "ateam_deploy_solution", "ateam_deploy_skill", "ateam_deploy_connector", "ateam_update", "ateam_redeploy"],
       async_testing: ["ateam_test_status", "ateam_test_abort"],
       other: ["ateam_upload_connector_files", "ateam_solution_chat"],
@@ -3982,6 +4014,20 @@ const handlers = {
     };
   },
 
+  ateam_connector_logs: async ({ solution_id, connector_id, since, limit, errors_only }, sid) => {
+    if (!solution_id) throw new Error("solution_id required");
+    if (!connector_id) throw new Error("connector_id required");
+    const qs = new URLSearchParams();
+    if (since) qs.set("since", String(since));
+    if (limit) qs.set("limit", String(limit));
+    if (errors_only === true) qs.set("errors_only", "true");
+    const qsStr = qs.toString() ? `?${qs}` : "";
+    return get(
+      `/deploy/solutions/${solution_id}/connectors/${encodeURIComponent(connector_id)}/logs${qsStr}`,
+      sid
+    );
+  },
+
   ateam_verify_surface: async ({ solution_id, plugin_id, expect, actor_id }, sid) => {
     if (!solution_id) return { ok: false, error: "solution_id required" };
     if (!plugin_id) return { ok: false, error: "plugin_id required" };
@@ -3990,10 +4036,26 @@ const handlers = {
     if (actor_id) body.actor_id = actor_id;
     // Forwards to the skill-validator, which runs Core's ui.surfaceProbe via /mcp.
     // 60s: navigate + settle + warm-retry inside Core, plus the hop.
-    return await post(
-      `/deploy/solutions/${solution_id}/plugins/${encodeURIComponent(plugin_id)}/verify-surface`,
-      body, sid, { timeoutMs: 60000 }
-    );
+    try {
+      return await post(
+        `/deploy/solutions/${solution_id}/plugins/${encodeURIComponent(plugin_id)}/verify-surface`,
+        body, sid, { timeoutMs: 60000 }
+      );
+    } catch (err) {
+      // A FAILING surface is this tool's whole job, not a transport error. The
+      // route maps a negative verdict to 422 (503 for inconclusive), which the
+      // generic error path turned into "returned 422" — hiding the failures that
+      // say WHY the screen is broken, and leaving the caller as blind as before
+      // it ran the probe. Hand the verdict back as a result instead of throwing.
+      const m = /returned (4\d\d|503)\b[\s\S]*?— (\{[\s\S]*)$/.exec(err?.message || "");
+      if (m) {
+        try {
+          const parsed = JSON.parse(m[2].replace(/\n?Hint: [\s\S]*$/, "").trim());
+          if (parsed && (parsed.verdict || Array.isArray(parsed.failures))) return parsed;
+        } catch { /* not the probe's body — fall through */ }
+      }
+      throw err;
+    }
   },
 
   ateam_test_skill: async ({ solution_id, skill_id, message, wait, wait_for, chain_timeout_ms, actor_id }, sid) => {