@ateam-ai/mcp 0.4.86 → 0.4.88

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ateam-ai/mcp",
-  "version": "0.4.86",
+  "version": "0.4.88",
   "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"
+    "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"
   },
   "keywords": [
     "mcp",

package/src/tools.js

@@ -633,9 +633,9 @@ export const tools = [
       properties: {
         type: {
           type: "string",
-          enum: ["skill", "connector", "connector-ui", "solution", "script-cache-skill", "ui-plugin-native", "index"],
+          enum: ["skill", "connector", "connector-ui", "solution", "script-cache-skill", "ui-plugin-native", "ui-plugin-iframe", "device-tools", "index"],
           description:
-            "Example type: 'skill' = Order Support Agent, 'connector' = stdio MCP connector, 'connector-ui' = UI-capable connector, 'solution' = full 3-skill e-commerce solution, 'script-cache-skill' = fat-tool skill with script_cache opt-in (reference implementation of script-level JIT shortcuts — study this before building any browser-automation skill), 'ui-plugin-native' = complete working React Native (mobile) UI plugin (rn-src/index.tsx + esbuild build:rn → rn-bundle, @adas/plugin-sdk, es2015), 'index' = list all available examples",
+            "Example type: 'skill' = Order Support Agent, 'connector' = stdio MCP connector, 'connector-ui' = UI-capable connector, 'solution' = full 3-skill e-commerce solution, 'script-cache-skill' = fat-tool skill with script_cache opt-in (reference implementation of script-level JIT shortcuts — study this before building any browser-automation skill), 'ui-plugin-native' = complete working React Native (mobile) UI plugin (rn-src/index.tsx + esbuild build:rn → rn-bundle, @adas/plugin-sdk, es2015), 'ui-plugin-iframe' = complete working web (iframe) UI plugin with the postMessage protocol, 'device-tools' = a solution's OWN tools that execute ON THE PHONE (runtime:\"device\") — BOTH halves and why they must agree: the plugin-bundle implementation, the connector manifest that declares it (Core cannot introspect a phone, so the manifest is the entire contract), and the skill wiring without which the skill gets none of them. Read this before designing anything that needs a LIVE device reading rather than the last synced one, 'index' = list all available examples",
         },
       },
       required: ["type"],
@@ -2522,6 +2522,8 @@ const EXAMPLE_PATHS = {
   solution: "/spec/examples/solution",
   "script-cache-skill": "/spec/examples/script-cache-skill",
   "ui-plugin-native": "/spec/examples/ui-plugin-native",
+  "ui-plugin-iframe": "/spec/examples/ui-plugin-iframe",
+  "device-tools": "/spec/examples/device-tools",
 };
 
 // Tools that are tenant-aware — require EXPLICIT ateam_auth (env vars alone not enough).
@@ -3559,7 +3561,18 @@ const handlers = {
 
   ateam_get_workflows: async (_args, sid) => get("/spec/workflows", sid),
 
-  ateam_get_examples: async ({ type }, sid) => get(EXAMPLE_PATHS[type], sid),
+  ateam_get_examples: async ({ type }, sid) => {
+    // An unknown type used to reach get(undefined) and fetch the API root, so a
+    // typo answered with something that looked like a valid response. Say what
+    // exists instead — the caller cannot see this map.
+    const path = EXAMPLE_PATHS[type];
+    if (!path) {
+      throw new Error(
+        `Unknown example type "${type}". Available: ${Object.keys(EXAMPLE_PATHS).join(", ")}.`
+      );
+    }
+    return get(path, sid);
+  },
 
   // Design-time capability advisor. Proxies to the Builder's /spec/advisor
   // (LLM over the curated capability catalog). Public endpoint (auth-exempt),
@@ -3957,12 +3970,28 @@ const handlers = {
       health,
       ...(widget_health && { widget_health }),
       ...(test_result && { test_result }),
-      ...(github_result && !github_result.error && !github_result.skipped && { github: github_result }),
+      // The GitHub outcome is reported WHATEVER it was. Filtering out the error
+      // case left a failed push visible only inside `phases` — which nothing
+      // reads — while _status went on claiming the push succeeded.
+      ...(github_result && { github: github_result }),
       ...(agent_doc_result && !agent_doc_result.error && { agent_doc: agent_doc_result }),
       ...(validation.warnings?.length > 0 && { validation_warnings: validation.warnings }),
-      _status: widget_health && !widget_health.ok
-        ? `✅ Deployed to Core + pushed to main. ⚠️ ${widget_health.issues?.length || 0} widget(s) not rendering — see widget_health.`
-        : '✅ Deployed to Core + pushed to main.',
+      // _status must describe WHAT HAPPENED. It previously asserted
+      // "pushed to main" unconditionally — when the push failed, when it was
+      // skipped (GitHub disabled, or push_to_github not opted in), and when no
+      // push was attempted at all. Worse, its only variable was widget_health,
+      // an unrelated flag: whether the code reached GitHub could not change the
+      // sentence claiming the code reached GitHub.
+      _status: [
+        '✅ Deployed to Core',
+        github_result?.error ? `⚠️ GitHub push FAILED: ${github_result.error}`
+          : github_result?.skipped ? `⚠️ GitHub push skipped${github_result.reason ? ` (${github_result.reason})` : ''} — Core and GitHub now differ`
+          : github_result ? '+ pushed to main'
+          : '⚠️ no GitHub push attempted — Core and GitHub may differ',
+        ...(widget_health && !widget_health.ok
+          ? [`⚠️ ${widget_health.issues?.length || 0} widget(s) not rendering — see widget_health.`]
+          : []),
+      ].join(' '),
       _next: 'Create a checkpoint before making more changes: ateam_github_promote(solution_id)',
     };
   },