@ateam-ai/mcp 0.3.26 → 0.3.28

package/package.json

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

@@ -156,20 +156,28 @@ ${AGENT_DOC_SENTINEL}
  * Merge the newly-rendered header with an existing CLAUDE.md, preserving
  * anything below the sentinel (solution-specific notes).
  *
+ * Output is deterministic so consecutive merges on the same inputs produce
+ * byte-identical output (required for the idempotent-write path in
+ * ateam_write_agent_doc).
+ *
  * @param {string} freshHeader   - Output of renderAgentDocHeader
  * @param {string|null} existing - Current CLAUDE.md contents (or null if new file)
  * @returns {string} merged markdown
  */
 export function mergeAgentDoc(freshHeader, existing) {
-  if (!existing) return freshHeader;
-  const idx = existing.indexOf(AGENT_DOC_SENTINEL);
-  if (idx === -1) {
-    // Existing file has no sentinel — treat the whole file as manual notes,
-    // prepend the auto-generated header, separate by sentinel.
-    return freshHeader + "\n" + existing.trimStart();
-  }
-  const tail = existing.slice(idx + AGENT_DOC_SENTINEL.length);
-  return freshHeader + tail;
+  // Normalize freshHeader: strip trailing whitespace, end with exactly "\n".
+  // This prevents newline accumulation when the output is read back and merged again.
+  const headerNormalized = freshHeader.replace(/\s+$/, "") + "\n";
+
+  const extractNotes = (src) => {
+    if (!src) return "";
+    const idx = src.indexOf(AGENT_DOC_SENTINEL);
+    const afterSentinel = idx === -1 ? src : src.slice(idx + AGENT_DOC_SENTINEL.length);
+    return afterSentinel.trim();
+  };
+
+  const notes = extractNotes(existing);
+  return notes ? `${headerNormalized}\n${notes}\n` : headerNormalized;
 }
 
 // ──────────────────────────────────────────────────────────────────────────

package/src/tools.js

@@ -1717,22 +1717,21 @@ const handlers = {
     }
 
     // Auto-seed / refresh the agent onboarding doc. Non-fatal — any failure
-    // here must not break a successful deploy. The doc renders from the
-    // deployed definition, so only attempt after the repo is present.
+    // here is swallowed so it can't break a successful deploy. The tool is
+    // idempotent: if the rendered doc is byte-identical to what's in the
+    // repo, it returns unchanged:true and writes no commit.
     let agent_doc_result = null;
-    if (github_result && !github_result.error) {
-      try {
-        agent_doc_result = await handlers.ateam_write_agent_doc({ solution_id: solutionId }, sid);
-        phases.push({
-          phase: "agent_doc",
-          status: "done",
-          created: agent_doc_result?.created || false,
-          preserved_notes: agent_doc_result?.preserved_notes || false,
-        });
-      } catch (err) {
-        agent_doc_result = { error: err.message };
-        phases.push({ phase: "agent_doc", status: "error", error: err.message });
-      }
+    try {
+      agent_doc_result = await handlers.ateam_write_agent_doc({ solution_id: solutionId }, sid);
+      phases.push({
+        phase: "agent_doc",
+        status: agent_doc_result?.unchanged ? "unchanged" : "done",
+        created: agent_doc_result?.created || false,
+        preserved_notes: agent_doc_result?.preserved_notes || false,
+      });
+    } catch (err) {
+      agent_doc_result = { error: err.message };
+      phases.push({ phase: "agent_doc", status: "skipped", reason: err.message });
     }
 
     return {
@@ -2120,17 +2119,29 @@ const handlers = {
 
     const freshHeader = renderAgentDocHeader({ solution, skills, connectors });
     let existing = null;
-    if (!overwrite) {
-      try {
-        const r = await get(
-          `/deploy/solutions/${solution_id}/github/read?path=${encodeURIComponent("CLAUDE.md")}`,
-          sid,
-        );
-        existing = r?.content || null;
-      } catch { /* file doesn't exist yet — treat as fresh create */ }
-    }
+    try {
+      const r = await get(
+        `/deploy/solutions/${solution_id}/github/read?path=${encodeURIComponent("CLAUDE.md")}`,
+        sid,
+      );
+      existing = r?.content || null;
+    } catch { /* file doesn't exist yet — treat as fresh create */ }
     const merged = overwrite ? freshHeader : mergeAgentDoc(freshHeader, existing);
 
+    // Idempotent write: if the merged content is byte-identical to what's
+    // already committed, skip the patch entirely. Prevents a noise commit on
+    // every build_and_run when nothing meaningful changed.
+    if (existing && existing === merged) {
+      return {
+        ok: true,
+        solution_id,
+        unchanged: true,
+        created: false,
+        preserved_notes: existing.includes(AGENT_DOC_SENTINEL),
+        bytes: merged.length,
+      };
+    }
+
     const res = await post(
       `/deploy/solutions/${solution_id}/github/patch`,
       {
@@ -2143,6 +2154,7 @@ const handlers = {
     return {
       ok: Boolean(res?.ok ?? true),
       solution_id,
+      unchanged: false,
       created: !existing,
       preserved_notes: Boolean(existing && existing.includes(AGENT_DOC_SENTINEL)),
       bytes: merged.length,