@ateam-ai/mcp 0.3.3 → 0.3.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ateam-ai/mcp",
-  "version": "0.3.3",
+  "version": "0.3.5",
   "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

@@ -120,7 +120,7 @@ export const tools = [
     name: "ateam_build_and_run",
     core: true,
     description:
-      "Build and deploy a governed AI Team solution in one step. ⚠️ HEAVIEST OPERATION (60-180s): validates solution+skills → deploys all connectors+skills to A-Team Core (regenerates MCP servers) → health-checks → optionally runs a warm test → auto-pushes to GitHub. Use this instead of calling validate, deploy, and health separately. For small changes to an already-deployed solution, prefer ateam_patch (faster, incremental). Requires authentication.",
+      "Build and deploy a governed AI Team solution in one step. ⚠️ HEAVIEST OPERATION (60-180s): validates solution+skills → deploys all connectors+skills to A-Team Core (regenerates MCP servers) → health-checks → optionally runs a warm test → auto-pushes to GitHub. AUTO-DETECTS GitHub repo: if you omit mcp_store and a repo exists, connector code is pulled from GitHub automatically. First deploy requires mcp_store. After that, write files via ateam_github_write, then just call build_and_run without mcp_store. For small changes to an already-deployed solution, prefer ateam_patch (faster, incremental). Requires authentication.",
     inputSchema: {
       type: "object",
       properties: {
@@ -144,7 +144,7 @@ export const tools = [
         },
         github: {
           type: "boolean",
-          description: "Optional: if true, pull connector source code from the solution's GitHub repo instead of requiring mcp_store. Use this after the first deploy (which creates the repo). Cannot be used on first deploy.",
+          description: "Optional: if true, pull connector source code from the solution's GitHub repo. AUTO-DETECTED: if you omit both mcp_store and github, the system checks if a repo exists and pulls from it automatically. You rarely need to set this explicitly.",
         },
         test_message: {
           type: "string",
@@ -850,6 +850,36 @@ export const tools = [
       required: ["solution_id", "path"],
     },
   },
+  {
+    name: "ateam_github_write",
+    core: true,
+    description:
+      "Write a file to the solution's GitHub repo. Use this to create new connector files or replace existing ones — one file per call. " +
+      "This is the PRIMARY way to write connector code after first deploy. " +
+      "Write each file individually (server.js, package.json, UI assets), then call ateam_build_and_run() to deploy.",
+    inputSchema: {
+      type: "object",
+      properties: {
+        solution_id: {
+          type: "string",
+          description: "The solution ID",
+        },
+        path: {
+          type: "string",
+          description: "File path to write (e.g. 'connectors/my-mcp/server.js', 'connectors/my-mcp/package.json')",
+        },
+        content: {
+          type: "string",
+          description: "The full file content",
+        },
+        message: {
+          type: "string",
+          description: "Optional commit message (default: 'Write <path>')",
+        },
+      },
+      required: ["solution_id", "path", "content"],
+    },
+  },
   {
     name: "ateam_github_log",
     core: true,
@@ -872,14 +902,14 @@ export const tools = [
   },
 
   // ═══════════════════════════════════════════════════════════════════
-  // RELEASE MANAGEMENT — promote, rollback, version listing
+  // RELEASE MANAGEMENT — checkpoint, rollback, version listing
   // ═══════════════════════════════════════════════════════════════════
 
   {
     name: "ateam_github_promote",
     core: true,
     description:
-      "Promote a dev version to main (production). By default promotes the latest dev tag. Optionally specify a specific dev tag to promote. Creates a prod-YYYY-MM-DD-NNN tag on main.",
+      "Create a checkpoint (safe point) on the current main branch. Tags the current state with safe-YYYY-MM-DD-NNN so you can rollback to it later. Use this before risky changes or when the solution is in a known-good state.",
     inputSchema: {
       type: "object",
       properties: {
@@ -887,9 +917,9 @@ export const tools = [
           type: "string",
           description: "The solution ID",
         },
-        tag: {
+        label: {
           type: "string",
-          description: "Optional: specific dev tag to promote (e.g., 'dev-2026-03-11-005'). If omitted, promotes the latest dev tag.",
+          description: "Optional: human-readable label for this checkpoint (e.g., 'before refactor', 'v2 stable')",
         },
       },
       required: ["solution_id"],
@@ -899,7 +929,7 @@ export const tools = [
     name: "ateam_github_rollback",
     core: true,
     description:
-      "Rollback main (production) to a previous production tag. Resets main branch to the specified prod tag commit. ⚠️ DESTRUCTIVE — use with caution. Use ateam_github_list_versions to find available production tags first.",
+      "Rollback main branch to a previous checkpoint (safe-* tag). Resets main to the specified checkpoint commit. ⚠️ DESTRUCTIVE — use with caution. Use ateam_github_list_versions to find available checkpoints first.",
     inputSchema: {
       type: "object",
       properties: {
@@ -909,7 +939,7 @@ export const tools = [
         },
         tag: {
           type: "string",
-          description: "Required: production tag to rollback to (e.g., 'prod-2026-03-10-001')",
+          description: "Required: checkpoint tag to rollback to (e.g., 'safe-2026-03-11-001')",
         },
       },
       required: ["solution_id", "tag"],
@@ -919,7 +949,7 @@ export const tools = [
     name: "ateam_github_list_versions",
     core: true,
     description:
-      "List all available dev version tags for a solution. Shows tag name, date, counter, and commit SHA. Use before promoting to see what's available.",
+      "List all available checkpoints (safe-* tags) for a solution. Shows tag name, date, counter, and commit SHA. Use before rollback to see available safe points.",
     inputSchema: {
       type: "object",
       properties: {
@@ -1083,19 +1113,18 @@ const handlers = {
       steps: [
         { step: 1, action: "Learn", description: "Get the spec and study examples", tools: ["ateam_get_spec", "ateam_get_examples"] },
         { 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 the 'dev' branch 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 via ateam_github_patch (commits to dev), then redeploy with ateam_build_and_run(github:true). For skill definition changes, use ateam_patch (also pushes to dev).", tools: ["ateam_github_patch", "ateam_build_and_run", "ateam_patch"] },
+        { 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: "Test the decision pipeline or full execution, then diagnose with logs and metrics. For voice-enabled solutions, use ateam_test_voice to simulate phone conversations.", tools: ["ateam_test_pipeline", "ateam_test_skill", "ateam_test_voice", "ateam_get_execution_logs", "ateam_get_metrics"] },
-        { step: 6, action: "Promote", description: "When stable, promote the dev branch to main (production). This creates a production tag and merges dev → main.", tools: ["ateam_github_promote"] },
+        { 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"] },
       ],
     },
     branching: {
-      _important: "ALL changes go to the 'dev' branch. The 'main' branch is production — only updated via explicit promote.",
-      dev: "Staging branch. Every build_and_run, patch, and github_patch commits here automatically.",
-      main: "Production branch. Updated ONLY via ateam_github_promote(). First deploy also pushes to main as baseline.",
-      workflow: "Build → iterate on dev → test → promote to main when stable.",
-      promote: "ateam_github_promote(solution_id) — merges latest dev tag to main, creates prod tag.",
-      rollback: "ateam_github_rollback(solution_id) — reverts main to previous production tag.",
+      _important: "Single-branch model: ALL changes go directly to 'main'. Use checkpoints (safe-* tags) as safe rollback points.",
+      main: "The only branch. All deploys, patches, and github_patches commit here automatically. This IS the live running system.",
+      checkpoints: "ateam_github_promote(solution_id) — creates a safe-YYYY-MM-DD-NNN tag on current main HEAD. Use before risky changes.",
+      rollback: "ateam_github_rollback(solution_id, tag) — reverts main to a previous checkpoint tag.",
+      workflow: "Build → iterate on main → test → checkpoint when stable → continue iterating.",
     },
     first_questions: [
       { id: "goal", question: "What do you want your Team to accomplish?", type: "text" },
@@ -1104,31 +1133,31 @@ const handlers = {
       { id: "security", question: "What environment constraints?", type: "enum", options: ["sandbox", "controlled", "regulated"] },
     ],
     github_tools: {
-      _note: "Version control for solutions. ALL operations target the 'dev' branch. Promote to 'main' (production) explicitly.",
-      tools: ["ateam_github_push", "ateam_github_pull", "ateam_github_status", "ateam_github_read", "ateam_github_patch", "ateam_github_log", "ateam_github_promote", "ateam_github_rollback"],
+      _note: "Version control for solutions. Single-branch model — everything on 'main'. Use checkpoints as safe rollback points.",
+      tools: ["ateam_github_push", "ateam_github_pull", "ateam_github_status", "ateam_github_read", "ateam_github_patch", "ateam_github_log", "ateam_github_promote", "ateam_github_rollback", "ateam_github_list_versions"],
       repo_structure: {
         "solution.json": "Full solution definition",
         "skills/{skill-id}/skill.json": "Individual skill definitions",
         "connectors/{connector-id}/server.js": "Connector MCP server code",
         "connectors/{connector-id}/package.json": "Connector dependencies",
       },
-      branches: {
-        dev: "All changes land here (auto-push on build/patch). This is the working branch.",
-        main: "Production snapshot. Only updated via ateam_github_promote(). Safe rollback target.",
-      },
+      branch: "main — the only branch. All changes land here directly.",
+      checkpoints: "safe-YYYY-MM-DD-NNN tags mark safe rollback points. Create with ateam_github_promote().",
       iteration_workflow: {
-        code_changes: "ateam_github_patch (commits to dev) → ateam_build_and_run(github:true) (pulls from dev, redeploys)",
-        definition_changes: "ateam_patch (updates + redeploys + auto-pushes to dev)",
-        first_deploy: "Must include mcp_store — this creates the GitHub repo with both dev and main branches",
-        promote: "ateam_github_promote(solution_id) — when solution is stable, merge dev → main",
+        code_changes: "ateam_github_patch (one file at a time, commits to main) → ateam_build_and_run() (auto-pulls from GitHub, redeploys)",
+        definition_changes: "ateam_patch (updates + redeploys + auto-pushes to main)",
+        first_deploy: "Must include mcp_store — this creates the GitHub repo",
+        after_first_deploy: "NEVER pass mcp_store again. Write files via ateam_github_patch, then ateam_build_and_run() auto-detects the repo.",
+        checkpoint: "ateam_github_promote(solution_id) — tag current state as a safe rollback point",
       },
       when_to_use_what: {
-        ateam_github_patch: "Edit connector source code on dev branch (server.js, utils, package.json, UI assets)",
-        ateam_patch: "Edit skill definitions (intents, tools, policy) — auto-pushes to dev",
-        "ateam_build_and_run(github:true)": "Redeploy solution pulling latest connector code from dev branch",
-        "ateam_build_and_run(mcp_store)": "First deploy or when you want to pass connector code inline",
-        ateam_github_promote: "Promote dev → main when stable (creates production tag)",
-        ateam_github_rollback: "Revert main to previous production tag",
+        ateam_github_write: "Write/create connector files on main — ONE FILE PER CALL (server.js, package.json, UI assets). Use this after first deploy.",
+        ateam_github_patch: "Edit existing files with search/replace (surgical edits to large files)",
+        ateam_patch: "Edit skill definitions (intents, tools, policy) — auto-pushes to main",
+        "ateam_build_and_run()": "Redeploy — auto-pulls from GitHub if repo exists. No need to pass mcp_store or github flag.",
+        "ateam_build_and_run(mcp_store)": "FIRST DEPLOY ONLY — creates the GitHub repo. Never use mcp_store again after first deploy.",
+        ateam_github_promote: "Create a checkpoint (safe-* tag) — use before risky changes",
+        ateam_github_rollback: "Revert main to a previous checkpoint",
       },
     },
     advanced_tools: {
@@ -1173,15 +1202,17 @@ const handlers = {
         "Use ateam_github_patch + ateam_build_and_run(github:true) for connector code changes after first deploy",
         "Study the connector example (ateam_get_examples type='connector') before writing connector code",
         "Ask discovery questions if goal unclear",
-        "ALL changes go to the dev branch — remind user to ateam_github_promote() when ready for production",
-        "After every build/patch, tell the user: 'Deployed to Core ✅ | Pushed to dev branch | Promote to main: ateam_github_promote()'",
+        "ALL changes go directly to main — suggest ateam_github_promote() to create a checkpoint before risky changes",
+        "After every build/patch, tell the user: 'Deployed to Core ✅ | Pushed to main | Create checkpoint: ateam_github_promote(solution_id)'",
       ],
       never: [
         "Call validate + deploy + health separately when ateam_build_and_run does it in one step",
         "Call update + redeploy separately when ateam_patch does it in one step",
         "Dump raw spec unless requested",
         "Write connector code that starts a web server — connectors MUST use stdio transport",
-        "Assume changes are on main — they are always on dev until explicitly promoted",
+        "Mention dev branch — there is no dev branch, everything is on main",
+        "Pass large connector code via mcp_store after the first deploy — use ateam_github_write/ateam_github_patch one file at a time instead",
+        "Try to pass ALL connector files at once in a single tool call — write them individually to GitHub",
       ],
     },
   }),
@@ -1263,8 +1294,37 @@ const handlers = {
   ateam_build_and_run: async ({ solution, skills, connectors, mcp_store, github, test_message, test_skill_id }, sid) => {
     const phases = [];
 
-    // Phase 0: GitHub pull (if github:true — pull connector source from repo)
+    // Guard: reject large mcp_store — agent should use github_patch instead
+    if (mcp_store) {
+      const totalSize = Object.values(mcp_store).reduce((sum, files) => {
+        return sum + (Array.isArray(files) ? files.reduce((s, f) => s + (f.content?.length || 0), 0) : 0);
+      }, 0);
+      if (totalSize > 200_000) {
+        return {
+          ok: false,
+          phase: "pre_check",
+          error: `mcp_store is too large (${Math.round(totalSize / 1024)}KB). Max ~200KB inline.`,
+          message: "Connector code is too large to pass inline. Write files individually to GitHub, then deploy from there.",
+          _fix: [
+            "1. Write each file: ateam_github_patch(solution_id, path: 'connectors/<id>/server.js', content: '...')",
+            "2. Repeat for package.json, UI assets, etc.",
+            "3. Deploy: ateam_build_and_run(solution, skills) — will auto-pull from GitHub",
+          ],
+        };
+      }
+    }
+
+    // Phase 0: Auto-detect GitHub repo — if no mcp_store passed and repo exists, pull from GitHub automatically
     let effectiveMcpStore = mcp_store;
+    if (!mcp_store) {
+      try {
+        const ghStatus = await get(`/deploy/solutions/${solution.id}/github/status`, sid);
+        if (ghStatus?.repo_url) {
+          // Repo exists — auto-pull from GitHub
+          github = true;
+        }
+      } catch { /* no repo — first deploy, mcp_store expected */ }
+    }
     if (github && !mcp_store) {
       try {
         const pullResult = await post(
@@ -1405,7 +1465,7 @@ const handlers = {
     return {
       ok: true,
       solution_id: solution.id,
-      branch: 'dev',
+      branch: 'main',
       phases,
       deploy: {
         skills_deployed: deploy.import?.skills || [],
@@ -1417,8 +1477,8 @@ const handlers = {
       ...(test_result && { test_result }),
       ...(github_result && !github_result.error && !github_result.skipped && { github: github_result }),
       ...(validation.warnings?.length > 0 && { validation_warnings: validation.warnings }),
-      _status: '✅ Deployed to Core + pushed to dev branch.',
-      _next: 'To promote to production (main branch): ateam_github_promote(solution_id)',
+      _status: '✅ Deployed to Core + pushed to main.',
+      _next: 'Create a checkpoint before making more changes: ateam_github_promote(solution_id)',
     };
   },
 
@@ -1485,7 +1545,7 @@ const handlers = {
       }
     }
 
-    // Phase 4: Auto-push to dev branch (non-blocking)
+    // Phase 4: Auto-push to main branch (non-blocking)
     let github_result;
     try {
       github_result = await post(
@@ -1503,14 +1563,14 @@ const handlers = {
     return {
       ok: true,
       solution_id,
-      branch: 'dev',
+      branch: 'main',
       phases,
       patch: patchResult,
       redeploy: redeployResult,
       ...(test_result && { test_result }),
       ...(github_result && !github_result.error && !github_result.skipped && { github: github_result }),
-      _status: '✅ Patched + redeployed + pushed to dev branch.',
-      _next: 'To promote to production (main branch): ateam_github_promote(solution_id)',
+      _status: '✅ Patched + redeployed + pushed to main.',
+      _next: 'Create a checkpoint before making more changes: ateam_github_promote(solution_id)',
     };
   },
 
@@ -1651,13 +1711,16 @@ const handlers = {
   ateam_github_patch: async ({ solution_id, path: filePath, content, search, replace, message }, sid) =>
     post(`/deploy/solutions/${solution_id}/github/patch`, { path: filePath, content, search, replace, message }, sid),
 
+  ateam_github_write: async ({ solution_id, path: filePath, content, message }, sid) =>
+    post(`/deploy/solutions/${solution_id}/github/patch`, { path: filePath, content, message }, sid),
+
   ateam_github_log: async ({ solution_id, limit }, sid) => {
     const qs = limit ? `?limit=${limit}` : "";
     return get(`/deploy/solutions/${solution_id}/github/log${qs}`, sid);
   },
 
-  ateam_github_promote: async ({ solution_id, tag }, sid) =>
-    post(`/deploy/solutions/${solution_id}/promote`, tag ? { tag } : {}, sid),
+  ateam_github_promote: async ({ solution_id, label }, sid) =>
+    post(`/deploy/solutions/${solution_id}/promote`, label ? { label } : {}, sid),
 
   ateam_github_rollback: async ({ solution_id, tag }, sid) =>
     post(`/deploy/solutions/${solution_id}/rollback`, { tag }, sid),