@ateam-ai/mcp 0.3.20 → 0.3.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ateam-ai/mcp",
-  "version": "0.3.20",
+  "version": "0.3.22",
   "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/tools.js

@@ -108,9 +108,9 @@ export const tools = [
       properties: {
         type: {
           type: "string",
-          enum: ["skill", "connector", "connector-ui", "solution", "index"],
+          enum: ["skill", "connector", "connector-ui", "solution", "script-cache-skill", "index"],
           description:
-            "Example type: 'skill' = Order Support Agent, 'connector' = stdio MCP connector, 'connector-ui' = UI-capable connector, 'solution' = full 3-skill e-commerce solution, '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), 'index' = list all available examples",
         },
       },
       required: ["type"],
@@ -303,7 +303,9 @@ export const tools = [
       "- Update problem: updates: { \"problem.statement\": \"...\", \"problem.goals\": [\"goal1\"] }\n" +
       "- Add a tool: updates: { \"tools_push\": [{ name: \"conn.tool\", description: \"...\", inputs: [...], output: {...} }] }\n" +
       "- Change intent: updates: { \"intents.supported_update\": [{ id: \"i1\", description: \"new desc\" }] }\n" +
-      "- Force redeploy: updates: { \"_force_redeploy\": true }\n\n" +
+      "- Force redeploy: updates: { \"_force_redeploy\": true }\n" +
+      "- CREATE a new skill: target='skill', skill_id='my-new-skill', updates: { \"problem.statement\": \"...\", \"role.persona\": \"...\" }\n" +
+      "  If the skill doesn't exist yet, a default scaffold is created and the updates are applied on top. The skill is automatically added to the solution topology.\n\n" +
       "Use target='skill' + skill_id for skill fields. Use target='solution' for solution-level fields (linked_skills, platform_connectors, ui_plugins).",
     inputSchema: {
       type: "object",
@@ -1108,6 +1110,7 @@ const EXAMPLE_PATHS = {
   connector: "/spec/examples/connector",
   "connector-ui": "/spec/examples/connector-ui",
   solution: "/spec/examples/solution",
+  "script-cache-skill": "/spec/examples/script-cache-skill",
 };
 
 // Tools that are tenant-aware — require EXPLICIT ateam_auth (env vars alone not enough).
@@ -1658,11 +1661,12 @@ const handlers = {
 
   ateam_patch: async ({ solution_id, target, skill_id, updates, test_message }, sid) => {
     const phases = [];
+    let isNewSkill = false;
 
     // GitHub-first patch: read from GitHub → apply patch → write back → redeploy
     // This ensures GitHub stays the single source of truth.
 
-    // Phase 1: Read current state from GitHub
+    // Phase 1: Read current state from GitHub (or create scaffold if new skill)
     let current;
     const filePath = target === "skill" && skill_id
       ? `skills/${skill_id}/skill.json`
@@ -1671,7 +1675,38 @@ const handlers = {
       const readResult = await get(`/deploy/solutions/${solution_id}/github/read?path=${encodeURIComponent(filePath)}`, sid);
       current = JSON.parse(readResult.content);
     } catch (err) {
-      return { ok: false, phase: "read", error: `Failed to read ${filePath} from GitHub: ${err.message}` };
+      // If it's a skill that doesn't exist yet, create a default scaffold.
+      // This lets agents use ateam_patch to both CREATE and UPDATE skills —
+      // no separate "create" step needed.
+      if (target === "skill" && skill_id) {
+        console.log(`[ateam_patch] Skill "${skill_id}" not found on GitHub — creating new skill scaffold`);
+        isNewSkill = true;
+        current = {
+          id: skill_id,
+          name: skill_id.replace(/-/g, " ").replace(/\b\w/g, c => c.toUpperCase()),
+          description: "",
+          version: "0.1.0",
+          phase: "PROBLEM_DISCOVERY",
+          connectors: [],
+          problem: { statement: "", context: "", goals: [] },
+          scenarios: [],
+          role: { name: "", persona: "", goals: [], limitations: [], communication_style: { tone: "professional", verbosity: "concise" } },
+          intents: { supported: [], thresholds: { accept: 0.8, clarify: 0.5, reject: 0.5 }, out_of_domain: { action: "redirect", message: "" } },
+          tools: [],
+          policy: { guardrails: { never: [], always: [] }, approvals: [], workflows: [], escalation: { enabled: false, conditions: [], target: "" } },
+          engine: { rv2: { max_iterations: 10, iteration_timeout_ms: 120000, allow_parallel_tools: false, on_max_iterations: "ask_user" }, hlr: { enabled: true, critic: { enabled: true, check_interval: 3, strictness: "medium" }, reflection: { enabled: true, depth: "shallow" }, replanning: { enabled: true, max_replans: 3 } }, autonomy: { level: "supervised" }, finalization_gate: { enabled: true, max_retries: 2 } },
+          access_policy: { rules: [{ tools: ["*"], effect: "allow" }] },
+          grant_mappings: [],
+          channels: [],
+          conversation: [],
+          triggers: [],
+          meta_tools: [],
+          glossary: {},
+        };
+        phases.push({ phase: "read", status: "created_scaffold", skill_id });
+      } else {
+        return { ok: false, phase: "read", error: `Failed to read ${filePath} from GitHub: ${err.message}` };
+      }
     }
 
     // Phase 2: Apply patch in memory
@@ -1729,23 +1764,52 @@ const handlers = {
       return { ok: false, phase: "github_write", error: `Patch applied but failed to write to GitHub: ${err.message}`, phases };
     }
 
-    // Phase 4: Redeploy from GitHub
+    // Phase 3b: If new skill, add it to solution.json topology (skills[], linked_skills)
+    if (isNewSkill && skill_id) {
+      try {
+        const solRead = await get(`/deploy/solutions/${solution_id}/github/read?path=solution.json`, sid);
+        const sol = JSON.parse(solRead.content);
+        const skillEntry = { id: skill_id, name: patched.name || skill_id, role: "worker", description: patched.description || "", connectors: patched.connectors || [] };
+        // Add to skills[] if not already present
+        if (!sol.skills) sol.skills = [];
+        if (!sol.skills.find(s => s.id === skill_id)) {
+          sol.skills.push(skillEntry);
+        }
+        // Add to linked_skills if not already present
+        if (!sol.linked_skills) sol.linked_skills = [];
+        if (!sol.linked_skills.includes(skill_id)) {
+          sol.linked_skills.push(skill_id);
+        }
+        await post(`/deploy/solutions/${solution_id}/github/patch`, {
+          path: "solution.json",
+          content: JSON.stringify(sol, null, 2),
+          message: `Add skill "${skill_id}" to solution topology`,
+        }, sid, { timeoutMs: 30_000 });
+        phases.push({ phase: "solution_topology", status: "done", added: skill_id });
+      } catch (err) {
+        // Non-fatal: skill.json was written, topology can be fixed manually
+        phases.push({ phase: "solution_topology", status: "warning", error: err.message });
+        console.warn(`[ateam_patch] Failed to add ${skill_id} to solution topology: ${err.message}`);
+      }
+    }
+
+    // Phase 4: Redeploy from GitHub (extended timeout — deploys can take 60-120s)
+    // IMPORTANT: if redeploy times out or fails, we return ok:true with a warning
+    // because the patch IS saved to GitHub — it's not lost. The agent can retry
+    // the redeploy with ateam_redeploy(solution_id, skill_id).
     let redeployResult;
     try {
       if (target === "skill" && skill_id) {
-        redeployResult = await post(`/deploy/solutions/${solution_id}/skills/${skill_id}/redeploy`, {}, sid);
+        redeployResult = await post(`/deploy/solutions/${solution_id}/skills/${skill_id}/redeploy`, {}, sid, { timeoutMs: 180_000 });
       } else {
-        redeployResult = await post(`/deploy/solutions/${solution_id}/redeploy`, {}, sid);
+        redeployResult = await post(`/deploy/solutions/${solution_id}/redeploy`, {}, sid, { timeoutMs: 180_000 });
       }
       phases.push({ phase: "redeploy", status: "done" });
     } catch (err) {
-      return {
-        ok: false,
-        phase: "redeploy",
-        phases,
-        error: err.message,
-        message: "Patch saved to GitHub but redeploy failed. Try ateam_redeploy manually.",
-      };
+      // Partial success: patch is saved to GitHub, only redeploy failed.
+      // Return ok:true so the agent doesn't think the patch was lost.
+      phases.push({ phase: "redeploy", status: "timeout_or_error", error: err.message });
+      console.warn(`[ateam_patch] Redeploy failed after successful patch: ${err.message}`);
     }
 
     // Phase 3: Optional re-test
@@ -1766,16 +1830,19 @@ const handlers = {
       }
     }
 
-    // Phase 4: Auto-push to main branch (non-blocking)
+    const redeployOk = phases.some(p => p.phase === "redeploy" && p.status === "done");
     return {
       ok: true,
       solution_id,
       branch: 'main',
       phases,
       patched: patched,
-      redeploy: redeployResult,
+      ...(isNewSkill && { created_skill: skill_id }),
+      ...(redeployResult && { redeploy: redeployResult }),
       ...(test_result && { test_result }),
-      _status: '✅ Patched on GitHub + redeployed.',
+      _status: redeployOk
+        ? '✅ Patched on GitHub + redeployed.'
+        : '⚠️ Patched on GitHub ✅ but redeploy timed out. Run: ateam_redeploy(solution_id' + (skill_id ? `, skill_id: "${skill_id}"` : '') + ')',
       _next: 'Create a checkpoint before making more changes: ateam_github_promote(solution_id)',
     };
   },