fix(collection): deundefined/null value when deleting array

src/collection/chain.ts

@@ -225,13 +225,14 @@ export class Chain<
     const mutations = flatten<PatchOperation>(
       this.hashTable.nodes
         .map((node) => {
+          const { data } = node;
           const updated = isCallable<
             (arg0: DataType) => DataType | Partial<DataType>
           >(updateFnOrObject)
-            ? updateFnOrObject(node.data)
+            ? updateFnOrObject(data)
             : updateFnOrObject;
 
-          return patchFn(node.data as unknown as object, updated, [
+          return patchFn(data as unknown as object, updated, [
             node.hash,
             node.$id,
           ]);

src/collection/collection.test.ts

@@ -402,6 +402,23 @@ describe("set", () => {
     ]);
     expect(1).toBe(1);
   });
+
+  it("removes an item from an array using set", () => {
+    const $ = new Collection([
+      { color: "red", contents: ["apple", "orange", "banana"] },
+    ]);
+
+    const $patch = $.get({ color: "red" })
+      .set({ contents: ["orange", "banana"] })
+      .commit();
+
+    console.log($patch);
+
+    expect($.get({ color: "red" }).data).toEqual({
+      color: "red",
+      contents: ["orange", "banana"],
+    });
+  });
 });
 
 describe("replace", () => {

src/collection/collection.ts

@@ -72,7 +72,8 @@ export interface Chainable<
   readonly nodes: HashTableItem<Index, Data>[];
   readonly count: number;
   readonly exists: boolean;
-  readonly or: any;
+  // @todo fix
+  readonly or: any; // eslint-disable-line
   readonly chain: Chain<
     Data,
     IndexKeys,

src/utils/collections/unset.test.ts

@@ -4,4 +4,6 @@ test("unset", () => {
   expect(unset({ a: { b: { c: "hello", d: {} } } }, "a.b.c")).toEqual({
     a: { b: { d: {} } },
   });
+
+  expect(unset(["banana", "apple", "pear"], "0")).toEqual(["apple", "pear"]);
 });

src/utils/collections/unset.ts

@@ -1,14 +1,17 @@
-import { isNumeric } from "../types";
+import { isArray, isNumeric } from "../types";
 
 export function unset<T>(obj: T, path: string) {
   const parts = path.split(".");
 
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   parts.reduce((o: any, key: string, index: number) => {
     const isLast = index === parts.length - 1;
-
     if (isLast) {
-      delete o[key];
+      if (isArray(o)) {
+        o.splice(Number(key), 1);
+      } else {
+        delete o[key];
+      }
     } else if (!o[key]) {
       o[key] = isNumeric(key) ? [] : {};
     }