@dabble/patches 0.5.19 → 0.5.20

package/dist/json-patch/ops/add.js

@@ -3,7 +3,7 @@ import { deepEqual } from "../utils/deepEqual.js";
 import { getOpData } from "../utils/getOpData.js";
 import {
   isArrayPath,
-  isEmptyObject,
+  isEmptyContainer,
   log,
   updateArrayIndexes,
   updateRemovedOps,
@@ -47,8 +47,8 @@ const add = {
     log("Transforming", otherOps, 'against "add"', thisOp);
     if (isArrayPath(thisOp.path, state)) {
       return updateArrayIndexes(state, thisOp.path, otherOps, 1);
-    } else if (isEmptyObject(thisOp.value)) {
-      return updateSoftWrites(thisOp.path, otherOps);
+    } else if (isEmptyContainer(thisOp.value)) {
+      return updateSoftWrites(thisOp.path, otherOps, thisOp.value);
     } else {
       return updateRemovedOps(state, thisOp.path, otherOps);
     }

package/dist/json-patch/utils/index.d.ts

@@ -7,7 +7,7 @@ export { isAdd, mapAndFilterOps, transformRemove, updateRemovedOps } from './ops
 export { getArrayIndex, getArrayPrefixAndIndex, getIndexAndEnd, getPrefix, getPrefixAndProp, getProp, getPropAfter, isArrayPath } from './paths.js';
 export { EMPTY, EMPTY_ARRAY, getValue, pluck, pluckWithShallowCopy } from './pluck.js';
 export { shallowCopy } from './shallowCopy.js';
-export { isEmptyObject, updateSoftWrites } from './softWrites.js';
+export { isEmptyContainer, updateSoftWrites } from './softWrites.js';
 export { toArrayIndex } from './toArrayIndex.js';
 export { toKeys } from './toKeys.js';
 export { updateArrayIndexes } from './updateArrayIndexes.js';

package/dist/json-patch/utils/softWrites.d.ts

@@ -1,10 +1,11 @@
 import { JSONPatchOp } from '../types.js';
 
-declare function isEmptyObject(value: any): boolean;
+declare function isEmptyContainer(value: any): boolean;
 /**
- * If other objects were added to this same path, assume they are maps/hashes/lookups and don't overwrite, allow
- * subsequent ops to merge onto the first map created. `soft` will also do this for any value that already exists.
+ * If other empty objects or arrays were added to this same path, assume they are maps/hashes/lookups/collections
+ * and don't overwrite, allow subsequent ops to merge onto the first container created. `soft` will also do this
+ * for any value that already exists.
  */
-declare function updateSoftWrites(overPath: string, ops: JSONPatchOp[]): JSONPatchOp[];
+declare function updateSoftWrites(overPath: string, ops: JSONPatchOp[], originalValue: any): JSONPatchOp[];
 
-export { isEmptyObject, updateSoftWrites };
+export { isEmptyContainer, updateSoftWrites };

package/dist/json-patch/utils/softWrites.js

@@ -1,19 +1,21 @@
 import "../../chunk-IZ2YBCUP.js";
 import { log } from "./log.js";
 import { mapAndFilterOps } from "./ops.js";
-function isEmptyObject(value) {
-  return Boolean(value && typeof value === "object" && Object.keys(value).length === 0);
+function isEmptyContainer(value) {
+  if (!value || typeof value !== "object") return false;
+  return Array.isArray(value) ? value.length === 0 : Object.keys(value).length === 0;
 }
-function updateSoftWrites(overPath, ops) {
+function updateSoftWrites(overPath, ops, originalValue) {
+  const originalIsArray = Array.isArray(originalValue);
   return mapAndFilterOps(ops, (op) => {
-    if (op.op === "add" && op.path === overPath && isEmptyObject(op.value)) {
-      log("Removing empty object", op);
+    if (op.op === "add" && op.path === overPath && isEmptyContainer(op.value) && Array.isArray(op.value) === originalIsArray) {
+      log("Removing empty container", op);
       return null;
     }
     return op;
   });
 }
 export {
-  isEmptyObject,
+  isEmptyContainer,
   updateSoftWrites
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dabble/patches",
-  "version": "0.5.19",
+  "version": "0.5.20",
   "description": "Immutable JSON Patch implementation based on RFC 6902 supporting operational transformation and last-writer-wins",
   "author": "Jacob Wright <jacwright@gmail.com>",
   "bugs": {