@elytracms/studio 0.0.99 → 0.0.100

package/dist/lib/operations/cms-persistence.js

@@ -1,4 +1,4 @@
-import { applyDocumentPatches, documentKey, documentValueBuckets, expandValuePath, fieldPath, formatValuePath, isValuePathPrefix, readValueAt, selfTypeDocument, semanticValueEqual, touchedPaths, touchedPathsOfEntry, valuePathsOverlap, } from '@elytracms/core/cms-core';
+import { applyDocumentPatches, documentKey, expandValuePath, fieldPositionPatches, formatValuePath, isValuePathPrefix, readValueAt, selfTypeDocument, semanticValueEqual, touchedPaths, touchedPathsOfEntry, valuePathsOverlap, } from '@elytracms/core/cms-core';
 import { createPersistQueue } from './persist-queue.js';
 /** EC-686 — a decomposed alias is re-sent at most this often (each time a colleague wrote the document in between). */
 const ALIAS_ATTEMPTS = 3;
@@ -231,22 +231,16 @@ function withAlias(document, alias) {
 /**
  * EC-686 item 1 — what a whole-document snapshot changed against the view it was made from,
  * position by position: a `set` (or an `unset`, the value gone) of every field position —
- * `values[f]`, `localized[l][f]` — whose value differs by THE equality
- * (`documentValueBuckets` + `semanticValueEqual`, the document diff's own reading). Every
- * value change of a document is one of these; its slot `alias` is the one key that is not.
+ * `values[f]`, `localized[l][f]` — whose value differs by THE equality. Every value change of
+ * a document is one of these; its slot `alias` is the one key that is not.
+ *
+ * EC-703 (Codex round 6 F33): THE own-property per-position diff (`fieldPositionPatches`,
+ * cms-core — the one the publish strip reads too). This copy read `bucket.after[key]`, so a
+ * removed field named after an `Object.prototype` member became `set constructor = Object`
+ * and every later `documentOf` threw (measured).
  */
 function positionPatches(view, snapshot) {
-    const patches = [];
-    for (const bucket of documentValueBuckets(view, snapshot)) {
-        for (const key of new Set([...Object.keys(bucket.before), ...Object.keys(bucket.after)])) {
-            if (semanticValueEqual(bucket.before[key], bucket.after[key]))
-                continue;
-            const path = fieldPath(key, bucket.locale);
-            const value = bucket.after[key];
-            patches.push(value === undefined ? { op: 'unset', path } : { op: 'set', path, value });
-        }
-    }
-    return patches;
+    return fieldPositionPatches(view, snapshot);
 }
 export function createCmsPersister(options) {
     const queue = createPersistQueue((error) => {