@ateam-ai/mcp 0.3.27 → 0.3.29

package/package.json

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

@@ -299,7 +299,8 @@ export const tools = [
       "5. Array update: { \"tools_update\": [{ name: \"existing_tool\", description: \"updated\" }] }\n" +
       "6. Replace whole section: { \"role\": { persona: \"...\", goals: [...] } }\n\n" +
       "EXAMPLES:\n" +
-      "- Change persona: updates: { \"role.persona\": \"You are a friendly assistant\" }\n" +
+      "- Change persona (full replace): updates: { \"role.persona\": \"You are a friendly assistant\" }\n" +
+      "- Append to persona (don't replace): updates: { \"persona_append\": \"\\n\\nALWAYS respond in 2 sentences.\" }\n" +
       "- Add a guardrail: updates: { \"policy.guardrails.never_push\": [\"Never share passwords\"] }\n" +
       "- Update problem: updates: { \"problem.statement\": \"...\", \"problem.goals\": [\"goal1\"] }\n" +
       "- Add a tool: updates: { \"tools_push\": [{ name: \"conn.tool\", description: \"...\", inputs: [...], output: {...} }] }\n" +
@@ -1813,7 +1814,17 @@ const handlers = {
     let patched = { ...current };
     try {
       for (const [key, value] of Object.entries(updates || {})) {
-        if (key.endsWith("_push") && Array.isArray(value)) {
+        if (key === "persona_append" && typeof value === "string") {
+          // persona_append: shorthand for appending text to role.persona without
+          // rewriting the whole string. Historically agents set this expecting
+          // it to work; the planner only reads role.persona, so this shorthand
+          // now merges into the correct field. A trailing separator is inserted
+          // if the existing persona doesn't already end with whitespace.
+          if (!patched.role || typeof patched.role !== "object") patched.role = {};
+          const existing = typeof patched.role.persona === "string" ? patched.role.persona : "";
+          const sep = (!existing || /\s$/.test(existing)) ? "" : "\n\n";
+          patched.role.persona = existing + sep + value;
+        } else if (key.endsWith("_push") && Array.isArray(value)) {
           // Array push: tools_push, intents_push, etc.
           const field = key.replace(/_push$/, "");
           patched[field] = [...(patched[field] || []), ...value];