@helping-ai-workflow/md2doc 3.5.0 → 3.6.0

package/lib/editor/wave-store.js

@@ -909,9 +909,44 @@ function diffMembers(ctx, path, was, now, skipKey, edits) {
     const r = diffValue(ctx, path.concat([key]), was[key], now[key], edits);
     if (r.ok !== true) return r;
   }
+  // v3.6.0 Task 13 fix round 1 (Critical 1): every new key on THIS object
+  // goes into ONE edit, not one `insert` per key.
+  //
+  // `toPatch()` (this file, `store.toPatch`) always diffs the WHOLE session
+  // against the document as it stood when the editor opened — not
+  // incrementally, gesture by gesture — so two new sibling fields set at any
+  // two points in the same session (the Lane panel's period then phase; two
+  // of the six ruler fields) both show up as "new" here in the SAME call.
+  // The old code pushed one independent `{op: 'insert', path: […key]}` per
+  // key, and `wave-codec.js`'s resolver turns EVERY one of those into a
+  // patch anchored at the object's same last EXISTING member — because
+  // neither edit can see the other, both compute the identical insertion
+  // point, and `patchSource`'s "two edits land at the same place" guard
+  // (which exists for exactly this ambiguity) refused the batch. Every
+  // session that set two new fields on one object failed, always, not
+  // intermittently.
+  //
+  // Folding them into one `insert-members` edit — resolved by
+  // `wave-codec.js`'s `resolveInsertMembers` — gives the object ONE new
+  // anchor for however many new keys it picked up this session. Key order
+  // inside it is `Object.keys(now)` order (insertion order for a plain
+  // object, so it matches the order the fields were actually set in); it
+  // does not have to be THE order, because `toPatch()`'s own read-back
+  // check just below in this file verifies the combined edit reproduces the
+  // exact document regardless of key order (`sameValue` does not compare
+  // key order either).
+  const newKeys = [];
   for (const key of Object.keys(now)) {
     if (key === skipKey) continue;
-    if (!has(was, key)) edits.push({ op: 'insert', path: path.concat([key]), value: now[key] });
+    if (!has(was, key)) newKeys.push(key);
+  }
+  if (newKeys.length === 1) {
+    edits.push({ op: 'insert', path: path.concat([newKeys[0]]), value: now[newKeys[0]] });
+  } else if (newKeys.length > 1) {
+    edits.push({
+      op: 'insert-members', path: path,
+      entries: newKeys.map(function (key) { return [key, now[key]]; }),
+    });
   }
   return OK;
 }