@ateam-ai/mcp 0.4.89 → 0.4.91

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ateam-ai/mcp",
-  "version": "0.4.89",
+  "version": "0.4.91",
   "mcpName": "io.github.ariekogan/ateam-mcp",
   "description": "A-Team MCP Server — build, validate, and deploy multi-agent solutions from any AI environment",
   "type": "module",
@@ -13,7 +13,7 @@
     "start:http": "node src/index.js --http",
     "dev": "node --watch src/index.js",
     "dev:http": "node --watch src/index.js --http",
-    "test": "node test/session-isolation.test.mjs && node test/widget-protocol.test.mjs && node test/actor-binding.test.mjs && node test/spec-topics.test.mjs && node test/example-types.test.mjs && node --test test/deploy-status-truth.test.mjs && node --test test/connector-source-provenance.test.mjs"
+    "test": "node test/session-isolation.test.mjs && node test/widget-protocol.test.mjs && node test/actor-binding.test.mjs && node test/spec-topics.test.mjs && node test/example-types.test.mjs && node --test test/deploy-status-truth.test.mjs && node --test test/connector-source-provenance.test.mjs && node --test test/bootstrap-base-url.test.mjs"
   },
   "keywords": [
     "mcp",

package/src/tools.js

@@ -3176,11 +3176,25 @@ function chainTreeOf(resp) {
 // is the difference between proving behaviour and matching a string that a
 // rename would quietly satisfy.
 export const handlers = {
-  ateam_bootstrap: async () => ({
+  // (_args, sid) — the SESSION ID IS LOAD-BEARING HERE.
+  //
+  // getBaseUrl(sessionId) resolves per-session first (api.js:326-339): a caller
+  // that passed `url` to ateam_auth is talking to THAT api, and the session
+  // holds it. Bootstrap called it with NO argument, so it skipped the
+  // per-session and bearer branches every time and reported the process
+  // DEFAULT — https://api.ateam-ai.com.
+  //
+  // The result was a tool that says PROD while the session is authenticated to
+  // DEV, in the one field whose whole job is to tell you which environment you
+  // are on. Observed live: a session working entirely against
+  // dev-api.ateam-ai.com was told base_url: https://api.ateam-ai.com, and had
+  // to learn from deploy errors which environment it was actually on. The
+  // dangerous direction is the mirror image — believing you are on dev.
+  ateam_bootstrap: async (_args, sid) => ({
     runtime: {
       ateam_mcp_version: MCP_VERSION,
-      base_url: getBaseUrl(),
-      _note: "The version of the ateam-mcp process actually serving this call, and the API it talks to. If a fix looks missing, check this FIRST — a local MCP process keeps running the code it loaded at session start, so a pushed/published fix is not live until the process restarts.",
+      base_url: getBaseUrl(sid),
+      _note: "The version of the ateam-mcp process actually serving this call, and the API THIS SESSION talks to (per-session, as set by ateam_auth's `url`). If a fix looks missing, check this FIRST — a local MCP process keeps running the code it loaded at session start, so a pushed/published fix is not live until the process restarts.",
     },
     platform_positioning: {
       name: "A-Team",
@@ -3926,10 +3940,29 @@ export const handlers = {
     }
 
     // Phase 2.5: Restart connectors that have source code (upload triggers stop+start)
+    //
+    // A runtime:"device" connector is SKIPPED. It has authored source — the RN
+    // bundle (rn-src/, package.json, the esbuild config) — so it looks like an
+    // ordinary connector to a loop that only asks "does it have files?". But
+    // there is no server process to restart: the bundle ships to the phone.
+    // Uploading it here 409s on the merge and then health marked the connector
+    // "error", failing a deploy whose connector was working as designed.
+    //
+    // Classified from the DECLARED connectors[], which the caller authored —
+    // not by asking Core. Same ownership rule the Builder now follows.
+    const deviceConnectorIds = new Set(
+      (connectors || [])
+        .filter((c) => c && typeof c === "object" && c.runtime === "device" && c.id)
+        .map((c) => c.id),
+    );
     if (effectiveMcpStore && Object.keys(effectiveMcpStore).length > 0) {
       const connectorResults = [];
       for (const [connId, files] of Object.entries(effectiveMcpStore)) {
         if (!Array.isArray(files) || files.length === 0) continue;
+        if (deviceConnectorIds.has(connId)) {
+          connectorResults.push({ id: connId, ok: true, tools: 0, skipped: "device_runtime" });
+          continue;
+        }
         try {
           const uploadResult = await post(
             `/deploy/solutions/${solutionId}/connectors/${connId}/upload`,