@kontextmind/kxm 0.7.91 → 0.7.92

package/.claude-plugin/marketplace.json

@@ -11,7 +11,7 @@
       "name": "kxm",
       "source": "./plugins/kxm",
       "description": "Durable workflows, peer agents, and kxm tui",
-      "version": "0.7.91",
+      "version": "0.7.92",
       "category": "development",
       "tags": ["kxm", "multi-agent", "workflows", "mcp"]
     }

package/CHANGELOG.md

@@ -185,6 +185,14 @@ All notable user-facing changes are documented here. The project follows [Semant
   none exist. The block is placed at the end of a file that has no markers yet, rather
   than before a `## Do not` heading that only this repository uses.
 
+- **`kxm memory sync` refuses malformed memory markers instead of eating text.** Sync
+  used to replace from the first `<!-- kxm:memory:start -->` to the first
+  `<!-- kxm:memory:end -->` wherever they appeared, so an orphan start marker got a new
+  block appended and the following sync deleted everything between the orphan and that
+  block; an end before its start duplicated text; a second block went stale. A file
+  must now hold exactly one start marker followed by one end marker, or neither.
+  Otherwise sync exits non-zero naming the file and the problem, and writes no file.
+
 - **Release version surfaces cover workspace packages:** a merged PR no longer
   breaks the release pipeline. `scripts/kxm-bump-version.mjs` now writes the
   version into every package manifest under `packages/`, using the same package

package/docs/configuration.md

@@ -303,7 +303,7 @@ The current hub command groups are `agent`, `session`, `workflow`, `gate`, `hub`
 | `kxm improve` | Bucket `.kxm/logs/telemetry.jsonl` events into a proposed-only improvement report; does not read the workflow journal |
 | `kxm context get \| recall \| state \| episode \| promote \| explain \| wiki-compile \| wiki-lint` | Context operating system: role-aware packets, metadata search, temporal state, episodes, evidence-backed lineage (`explain`), wiki compile/lint |
 | `kxm skills` | Governed skill candidate lifecycle |
-| `kxm memory sync` | Regenerate the read-only project-memory block — authored facts from `.kxm/memory/` only — between `<!-- kxm:memory:start -->` and `<!-- kxm:memory:end -->` in whichever of `AGENTS.md`, `CLAUDE.md` and `GEMINI.md` already exist in the current directory. Text outside the markers is never rewritten; a file without markers gets the block appended. Missing files are reported as `missing` and **not created** — which harness a project uses is its own choice — and with none present sync exits 1 without writing. `--json` reports `updated`, `unchanged` and `missing` |
+| `kxm memory sync` | Regenerate the read-only project-memory block — authored facts from `.kxm/memory/` only — between `<!-- kxm:memory:start -->` and `<!-- kxm:memory:end -->` in whichever of `AGENTS.md`, `CLAUDE.md` and `GEMINI.md` already exist in the current directory. Text outside the markers is never rewritten; a file without markers gets the block appended. Each file must hold exactly one start marker followed by exactly one end marker, or neither: an orphan marker, an end before its start, or a second block makes sync exit 1 naming the file and the problem, and **no file is written**, the well-formed ones included. Missing files are reported as `missing` and **not created** — which harness a project uses is its own choice — and with none present sync exits 1 without writing. `--json` reports `updated`, `unchanged` and `missing` |
 
 Improvement telemetry is classified as `project` whenever a project or workflow identity is present, and as `cli` for unscoped operator behavior. Set `KXM_IMPROVE_TARGET=cli` or `KXM_IMPROVE_TARGET=project` only when an operator needs to override that generic classification; this changes report bucketing, not workflow state.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kontextmind/kxm",
-  "version": "0.7.91",
+  "version": "0.7.92",
   "description": "KXM local-first multi-agent orchestration and operator dashboard",
   "type": "module",
   "author": "KontextMind",

package/plugins/kxm/.claude-plugin/plugin.json

@@ -2,7 +2,7 @@
   "$schema": "https://json.schemastore.org/claude-code-plugin-manifest.json",
   "name": "kxm",
   "displayName": "KXM",
-  "version": "0.7.91",
+  "version": "0.7.92",
   "description": "Headless multi-agent orchestration, durable workflows, and a live operator dashboard for Pi and Claude Code",
   "author": {
     "name": "KontextMind",

package/plugins/kxm/dist/cli.js

@@ -33618,26 +33618,27 @@ function formatHarnessMemoryBlock(records) {
   return contentLines.join("\n");
 }
 var HARNESS_INSTRUCTION_FILES = ["AGENTS.md", "CLAUDE.md", "GEMINI.md"];
-function updateHarnessDocument(filePath, block, dryRun = false) {
-  const original = readFileSync24(filePath, "utf8");
-  let updated;
-  if (original.includes(MEMORY_MARKER_START) && original.includes(MEMORY_MARKER_END)) {
-    const startIdx = original.indexOf(MEMORY_MARKER_START);
-    const endIdx = original.indexOf(MEMORY_MARKER_END) + MEMORY_MARKER_END.length;
-    updated = original.slice(0, startIdx) + block + original.slice(endIdx);
-  } else {
+function memoryMarkerProblem(text) {
+  const starts = text.split(MEMORY_MARKER_START).length - 1;
+  const ends = text.split(MEMORY_MARKER_END).length - 1;
+  if (starts > 1 || ends > 1) return `has ${starts} ${MEMORY_MARKER_START} and ${ends} ${MEMORY_MARKER_END} markers, not one block`;
+  if (starts > ends) return `has ${MEMORY_MARKER_START} with no ${MEMORY_MARKER_END}`;
+  if (ends > starts) return `has ${MEMORY_MARKER_END} with no ${MEMORY_MARKER_START}`;
+  if (text.indexOf(MEMORY_MARKER_END) < text.indexOf(MEMORY_MARKER_START)) return `has ${MEMORY_MARKER_END} before ${MEMORY_MARKER_START}`;
+  return void 0;
+}
+function withMemoryBlock(original, block) {
+  const startIdx = original.indexOf(MEMORY_MARKER_START);
+  if (startIdx === -1) {
     const head = original.trimEnd();
-    updated = head ? `${head}
+    return head ? `${head}
 
 ${block}
 ` : `${block}
 `;
   }
-  if (updated !== original) {
-    if (!dryRun) writeFileSync18(filePath, updated, "utf8");
-    return true;
-  }
-  return false;
+  const endIdx = original.indexOf(MEMORY_MARKER_END) + MEMORY_MARKER_END.length;
+  return original.slice(0, startIdx) + block + original.slice(endIdx);
 }
 function syncHarnessMemory(repoRoot, options = {}) {
   const root = resolve22(repoRoot);
@@ -33648,11 +33649,27 @@ function syncHarnessMemory(repoRoot, options = {}) {
       `none of ${HARNESS_INSTRUCTION_FILES.join(", ")} exists in ${root}; sync updates the instruction files a project already has and does not create them`
     );
   }
+  const documents = present.map((name) => {
+    const original = readFileSync24(join28(root, name), "utf8");
+    return { name, original, problem: memoryMarkerProblem(original) };
+  });
+  const malformed = documents.filter((doc) => doc.problem !== void 0);
+  if (malformed.length > 0) {
+    throw new Error(
+      `${malformed.map((doc) => `${doc.name} ${doc.problem}`).join("; ")}; wrote no file. Keep exactly one ${MEMORY_MARKER_START} followed by one ${MEMORY_MARKER_END} in each file, or delete both so sync appends a fresh block`
+    );
+  }
   const block = formatHarnessMemoryBlock(loadAuthoredMemory(root));
   const updated = [];
   const unchanged = [];
-  for (const name of present) {
-    (updateHarnessDocument(join28(root, name), block, options.dryRun) ? updated : unchanged).push(name);
+  for (const { name, original } of documents) {
+    const next = withMemoryBlock(original, block);
+    if (next === original) {
+      unchanged.push(name);
+    } else {
+      if (!options.dryRun) writeFileSync18(join28(root, name), next, "utf8");
+      updated.push(name);
+    }
   }
   return { updated, unchanged, missing };
 }

package/plugins/kxm/dist/mcp-server.js

@@ -17182,7 +17182,7 @@ async function deliverInboxNotification(messageId, delivered, notify) {
 }
 
 // plugins/kxm/src/mcp-server.ts
-var VERSION = "0.7.91";
+var VERSION = "0.7.92";
 var inbox = /* @__PURE__ */ new Map();
 var notifiedInbox = /* @__PURE__ */ new Set();
 var meshClient;

package/plugins/kxm/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-plugin",
-  "version": "0.7.91",
+  "version": "0.7.92",
   "private": true,
   "type": "module",
   "engines": {

package/plugins/kxm/skills/kxm-context-memory/SKILL.md

@@ -57,6 +57,12 @@ never creates an instruction file. If the project has none of the three, sync
 refuses; create the file your harness reads yourself rather than expecting KXM
 to author it.
 
+Each file must hold exactly one start marker followed by exactly one end
+marker, or neither. An orphan marker, an end before its start, or a second
+block makes sync refuse, name the file and the problem, and write no file at
+all. Fix the markers by hand — keep one pair, or delete both so sync appends a
+fresh block — rather than letting sync guess which text is yours.
+
 Do not invent search, save, delete, list, or clear subcommands under memory.
 Use context retrieval for discovery and reviewed authored changes for
 corrections, respecting provenance and historical records.

package/plugins/kxm/src/mcp-server.ts

@@ -8,7 +8,7 @@ import { AGENT_COMMANDS_MAP, enforceToolPolicy, getMcpTools, reconcileInbox } fr
 import { deliverInboxNotification } from "./inbox.ts";
 import type { HubEvent, MessageRecord } from "./protocol.ts";
 
-const VERSION = "0.7.91";
+const VERSION = "0.7.92";
 const inbox = new Map<string, MessageRecord>();
 const notifiedInbox = new Set<string>();
 let meshClient: HubClient | undefined;

package/plugins/kxm/src/memory.ts

@@ -323,26 +323,30 @@ export function formatHarnessMemoryBlock(records: MemoryRecord[]): string {
  * carry KXM's words instead of the project's. */
 const HARNESS_INSTRUCTION_FILES = ["AGENTS.md", "CLAUDE.md", "GEMINI.md"] as const;
 
-/** Replace the marker-delimited block, or append one when the file has none. Nothing
- * outside the markers is rewritten. Returns whether the block changed the file; `dryRun`
- * answers without writing. */
-export function updateHarnessDocument(filePath: string, block: string, dryRun = false): boolean {
-  const original = readFileSync(filePath, "utf8");
-  let updated: string;
-  if (original.includes(MEMORY_MARKER_START) && original.includes(MEMORY_MARKER_END)) {
-    const startIdx = original.indexOf(MEMORY_MARKER_START);
-    const endIdx = original.indexOf(MEMORY_MARKER_END) + MEMORY_MARKER_END.length;
-    updated = original.slice(0, startIdx) + block + original.slice(endIdx);
-  } else {
-    const head = original.trimEnd();
-    updated = head ? `${head}\n\n${block}\n` : `${block}\n`;
-  }
+/** What is wrong with the memory markers in `text`, or undefined when it has exactly one
+ * start marker followed by exactly one end marker, or neither. Any other shape has no single
+ * block to replace: an orphan start would make the next sync delete the project's text up to
+ * a later end, an end before its start would duplicate text, and a second block goes stale. */
+function memoryMarkerProblem(text: string): string | undefined {
+  const starts = text.split(MEMORY_MARKER_START).length - 1;
+  const ends = text.split(MEMORY_MARKER_END).length - 1;
+  if (starts > 1 || ends > 1) return `has ${starts} ${MEMORY_MARKER_START} and ${ends} ${MEMORY_MARKER_END} markers, not one block`;
+  if (starts > ends) return `has ${MEMORY_MARKER_START} with no ${MEMORY_MARKER_END}`;
+  if (ends > starts) return `has ${MEMORY_MARKER_END} with no ${MEMORY_MARKER_START}`;
+  if (text.indexOf(MEMORY_MARKER_END) < text.indexOf(MEMORY_MARKER_START)) return `has ${MEMORY_MARKER_END} before ${MEMORY_MARKER_START}`;
+  return undefined;
+}
 
-  if (updated !== original) {
-    if (!dryRun) writeFileSync(filePath, updated, "utf8");
-    return true;
+/** `original` with its memory block replaced, or with one appended when it has no markers.
+ * Nothing outside the markers is rewritten. Markers must already pass `memoryMarkerProblem`. */
+function withMemoryBlock(original: string, block: string): string {
+  const startIdx = original.indexOf(MEMORY_MARKER_START);
+  if (startIdx === -1) {
+    const head = original.trimEnd();
+    return head ? `${head}\n\n${block}\n` : `${block}\n`;
   }
-  return false;
+  const endIdx = original.indexOf(MEMORY_MARKER_END) + MEMORY_MARKER_END.length;
+  return original.slice(0, startIdx) + block + original.slice(endIdx);
 }
 
 export function syncHarnessMemory(
@@ -358,11 +362,30 @@ export function syncHarnessMemory(
     );
   }
 
+  // Check every file before writing any: one malformed file leaves all of them untouched.
+  const documents = present.map((name) => {
+    const original = readFileSync(join(root, name), "utf8");
+    return { name, original, problem: memoryMarkerProblem(original) };
+  });
+  const malformed = documents.filter((doc) => doc.problem !== undefined);
+  if (malformed.length > 0) {
+    throw new Error(
+      `${malformed.map((doc) => `${doc.name} ${doc.problem}`).join("; ")}; wrote no file. `
+        + `Keep exactly one ${MEMORY_MARKER_START} followed by one ${MEMORY_MARKER_END} in each file, or delete both so sync appends a fresh block`,
+    );
+  }
+
   const block = formatHarnessMemoryBlock(loadAuthoredMemory(root));
   const updated: string[] = [];
   const unchanged: string[] = [];
-  for (const name of present) {
-    (updateHarnessDocument(join(root, name), block, options.dryRun) ? updated : unchanged).push(name);
+  for (const { name, original } of documents) {
+    const next = withMemoryBlock(original, block);
+    if (next === original) {
+      unchanged.push(name);
+    } else {
+      if (!options.dryRun) writeFileSync(join(root, name), next, "utf8");
+      updated.push(name);
+    }
   }
   return { updated, unchanged, missing };
 }