@rubytech/create-maxy 1.0.665 → 1.0.666

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-maxy",
-  "version": "1.0.665",
+  "version": "1.0.666",
   "description": "Install Maxy — AI for Productive People",
   "bin": {
     "create-maxy": "./dist/index.js"

package/payload/platform/neo4j/schema.cypher

@@ -308,9 +308,19 @@ OPTIONS {
 // Linked via (Message)-[:NEXT]->(Message) to the next message in
 // insertion order (Task 621). The chain is linear — one outgoing
 // NEXT per message — so a Conversation with N messages has N-1
-// NEXT edges. Concurrent writes against the same Conversation
-// CAN fork the chain under Neo4j's READ_COMMITTED default; tracked
-// under Task 624.
+// NEXT edges.
+//
+// Chain-fork invariant (Task 624): the write path in
+// platform/ui/app/lib/neo4j-store.ts#persistMessage holds a
+// per-conversationId async mutex around the Cypher that observes
+// the tail and CREATEs the new [:NEXT] edge. Without this, two
+// concurrent writes under READ_COMMITTED both see the same tail
+// and each CREATE an outgoing NEXT → two edges from one node.
+// Scope: single node-process only; multi-replica coordination is
+// a follow-up (see .docs/neo4j.md). Any new write path that creates
+// Message nodes on the chain (bulk import, migration, rehydrate)
+// MUST go through persistMessage or hold an equivalent lock.
+//
 // Vector-indexed for semantic search over conversation history.
 // ----------------------------------------------------------
 

package/payload/server/server.js

@@ -5281,6 +5281,7 @@ async function persistToolCall(record) {
   }
 }
 var SUMMARY_MAX_LEN = 200;
+var persistMessageLocks = /* @__PURE__ */ new Map();
 async function persistMessage(conversationId, role, content, accountId, tokens, createdAt, sender) {
   if (!content) return null;
   const messageId = randomUUID();
@@ -5291,6 +5292,15 @@ async function persistMessage(conversationId, role, content, accountId, tokens,
   } catch (err) {
     console.error(`[persist] Embedding failed, storing without: ${err instanceof Error ? err.message : String(err)}`);
   }
+  const prev = persistMessageLocks.get(conversationId);
+  const waited = prev !== void 0;
+  let release;
+  const mine = new Promise((resolve31) => {
+    release = resolve31;
+  });
+  const chained = (prev ?? Promise.resolve()).then(() => mine);
+  persistMessageLocks.set(conversationId, chained);
+  await prev;
   const session = getSession();
   try {
     const result = await session.run(
@@ -5353,7 +5363,7 @@ async function persistMessage(conversationId, role, content, accountId, tokens,
     const summarySetThisWrite = record.get("summarySetThisWrite") === true;
     const chainLenRaw = record.get("chainLen");
     const chainLen = typeof chainLenRaw === "bigint" ? Number(chainLenRaw) : typeof chainLenRaw?.toNumber === "function" ? chainLenRaw.toNumber() : Number(chainLenRaw ?? 0);
-    console.error(`[neo4j-store] append-message conversationId=${conversationId.slice(0, 8)}\u2026 messageId=${messageId.slice(0, 8)}\u2026 prev=${prevMessageId ? prevMessageId.slice(0, 8) + "\u2026" : "null"} chainLen=${chainLen}`);
+    console.error(`[neo4j-store] append-message conversationId=${conversationId.slice(0, 8)}\u2026 messageId=${messageId.slice(0, 8)}\u2026 prev=${prevMessageId ? prevMessageId.slice(0, 8) + "\u2026" : "null"} chainLen=${chainLen} waited=${waited}`);
     if (summarySetThisWrite) {
       console.error(`[neo4j-store] conversation-summary-set conversationId=${conversationId.slice(0, 8)}\u2026 len=${summary.length}`);
     }
@@ -5363,6 +5373,10 @@ async function persistMessage(conversationId, role, content, accountId, tokens,
     console.error(`[persist] Neo4j write failed: ${err instanceof Error ? err.message : String(err)}`);
     return null;
   } finally {
+    release();
+    if (persistMessageLocks.get(conversationId) === chained) {
+      persistMessageLocks.delete(conversationId);
+    }
     await session.close();
   }
 }