@byline/admin 3.19.0 → 3.20.1

package/dist/fields/array/array-field.js

@@ -24,6 +24,13 @@ const ArrayField = ({ field, defaultValue, path, disableSorting = false, collect
         getFieldValue,
         path
     ]);
+    const patchItemId = (item, index)=>{
+        if (item && 'object' == typeof item) {
+            if ('_id' in item) return String(item._id);
+            if ('id' in item) return String(item.id);
+        }
+        return String(index);
+    };
     const handleDragEnd = ({ moveFromIndex, moveToIndex })=>{
         setItems((prev)=>moveItem(prev, moveFromIndex, moveToIndex));
         const currentArray = getFieldValue(path) ?? defaultValue;
@@ -32,11 +39,10 @@ const ArrayField = ({ field, defaultValue, path, disableSorting = false, collect
             const clampedTo = Math.max(0, Math.min(moveToIndex, currentArray.length - 1));
             if (clampedFrom === clampedTo) return;
             const item = currentArray[clampedFrom];
-            const itemId = item && 'object' == typeof item && 'id' in item ? String(item.id) : String(clampedFrom);
             appendPatch({
                 kind: 'array.move',
                 path: path,
-                itemId,
+                itemId: patchItemId(item, clampedFrom),
                 toIndex: clampedTo
             });
         }
@@ -45,7 +51,9 @@ const ArrayField = ({ field, defaultValue, path, disableSorting = false, collect
         const childFields = field.fields ?? [];
         if (0 === childFields.length) return;
         const newId = crypto.randomUUID();
-        const newItem = {};
+        const newItem = {
+            _id: newId
+        };
         for (const childField of childFields)if ('group' === childField.type && childField.fields && childField.fields.length > 0) {
             const groupObj = {};
             for (const innerField of childField.fields)groupObj[innerField.name] = await defaultScalarForField(innerField, getFieldValues);
@@ -80,12 +88,11 @@ const ArrayField = ({ field, defaultValue, path, disableSorting = false, collect
         const currentArray = getFieldValue(path) ?? defaultValue;
         if (!Array.isArray(currentArray) || index < 0 || index >= currentArray.length) return;
         const item = currentArray[index];
-        const itemId = item && 'object' == typeof item && 'id' in item ? String(item.id) : String(index);
         setItems((prev)=>prev.filter((_, i)=>i !== index));
         appendPatch({
             kind: 'array.remove',
             path: path,
-            itemId
+            itemId: patchItemId(item, index)
         });
         const newArrayValue = [
             ...currentArray

package/package.json

@@ -2,7 +2,7 @@
   "name": "@byline/admin",
   "private": false,
   "license": "MPL-2.0",
-  "version": "3.19.0",
+  "version": "3.20.1",
   "engines": {
     "node": ">=20.9.0"
   },
@@ -155,10 +155,10 @@
     "react-diff-viewer-continued": "^4.2.2",
     "uuid": "^14.0.1",
     "zod": "^4.4.3",
-    "@byline/auth": "3.19.0",
-    "@byline/ui": "3.19.0",
-    "@byline/core": "3.19.0",
-    "@byline/i18n": "3.19.0"
+    "@byline/auth": "3.20.1",
+    "@byline/i18n": "3.20.1",
+    "@byline/ui": "3.20.1",
+    "@byline/core": "3.20.1"
   },
   "peerDependencies": {
     "react": "^19.0.0",

package/src/fields/array/array-field.tsx

@@ -80,6 +80,23 @@ export const ArrayField = ({
     }
   }, [defaultValue, getFieldValue, path])
 
+  /**
+   * Stable patch identity for an array item. Persisted items carry `_id`
+   * (the array-item identity from `store_meta`) — that is what the server's
+   * patch engine matches on (`applyArrayPatch`: `item._id === patch.itemId`),
+   * so it MUST be preferred here. `id` is accepted as a legacy/seed-data
+   * alias. Items added this session have neither (the storage layer assigns
+   * `_id` at write time), so fall back to the item's current index — the
+   * patch engine resolves a pure-integer itemId as an index fallback.
+   */
+  const patchItemId = (item: unknown, index: number): string => {
+    if (item && typeof item === 'object') {
+      if ('_id' in item) return String((item as { _id: string })._id)
+      if ('id' in item) return String((item as { id: string }).id)
+    }
+    return String(index)
+  }
+
   const handleDragEnd = ({
     moveFromIndex,
     moveToIndex,
@@ -96,15 +113,11 @@ export const ArrayField = ({
       if (clampedFrom === clampedTo) return
 
       const item = currentArray[clampedFrom]
-      const itemId =
-        item && typeof item === 'object' && 'id' in item
-          ? String((item as { id: string }).id)
-          : String(clampedFrom)
 
       appendPatch({
         kind: 'array.move',
         path: path,
-        itemId,
+        itemId: patchItemId(item, clampedFrom),
         toIndex: clampedTo,
       })
     }
@@ -116,8 +129,12 @@ export const ArrayField = ({
 
     const newId = crypto.randomUUID()
 
-    // Build a new item with default values for ALL child fields
-    const newItem: Record<string, any> = {}
+    // Build a new item with default values for ALL child fields.
+    // Assign the stable `_id` client-side — exactly like BlocksField's
+    // handleAddItem — so the item is patch-addressable (move / remove by
+    // `_id`) within the same editing session, before any server write has
+    // assigned identity. The storage layer persists `_id` via store_meta.
+    const newItem: Record<string, any> = { _id: newId }
     for (const childField of childFields) {
       if (childField.type === 'group' && childField.fields && childField.fields.length > 0) {
         // Group child — build a nested object with defaults for each inner field
@@ -158,17 +175,13 @@ export const ArrayField = ({
     if (!Array.isArray(currentArray) || index < 0 || index >= currentArray.length) return
 
     const item = currentArray[index]
-    const itemId =
-      item && typeof item === 'object' && 'id' in item
-        ? String((item as { id: string }).id)
-        : String(index)
 
     setItems((prev) => prev.filter((_, i) => i !== index))
 
     appendPatch({
       kind: 'array.remove',
       path: path,
-      itemId,
+      itemId: patchItemId(item, index),
     })
 
     const newArrayValue = [...currentArray]