reptree 0.6.0 → 0.7.0

package/README.md

@@ -33,9 +33,10 @@ const qa = company.newNamedChild("qa");
 // Create a vertex in another vertex
 const alice = qa.newChild();
 
-// Set properties
+// Set properties (supports any JSON-serializable values)
 alice.setProperty("name", "Alice");
 alice.setProperty("age", 32);
+alice.setProperty("meta", { department: "QA", skills: ["cypress", "playwright"], flags: { lead: false } });
 
 // Move the vertex inside a different vertex
 alice.moveTo(devs);
@@ -89,8 +90,7 @@ const logoFile = imagesFolder.newNamedChild("logo.png");
 logoFile.setProperties({
   type: "file",
   size: 15360,
-  dimensions: "512x512",
-  format: "png",
+  meta: { dimensions: "512x512", format: "png" },
   s3Path: "s3://my-bucket/images/logo.png",
 });
 

package/dist/index.cjs

@@ -385,7 +385,7 @@ var TreeState = class {
   }
 };
 
-// src/uuid.ts
+// src/utils/uuid.ts
 var removeDashes = (guid) => guid.replace(/-/g, "");
 function uuid() {
   return removeDashes(crypto.randomUUID());
@@ -555,7 +555,7 @@ var Vertex = class _Vertex {
     if (props && typeof props === "object" && "children" in props) {
       throw new Error("Passing children inside props is not supported at the moment");
     }
-    const normalized = _Vertex.normalizePropsForCreation(props, name);
+    const normalized = _Vertex.normalizePropsForCreation(props);
     return this.tree.newNamedVertex(this.id, name, normalized);
   }
   setProperty(key, value) {
@@ -634,21 +634,32 @@ var Vertex = class _Vertex {
    * - Filters unsupported field types with a console warning
    * - When a name param is provided to newNamedChild, ignores conflicting name in props
    */
-  static normalizePropsForCreation(props, explicitName) {
+  static normalizePropsForCreation(props) {
     if (!props) return null;
     const input = props;
     const out = {};
     const skipped = [];
+    const isJsonValue2 = (v) => {
+      if (v === null) return true;
+      const t = typeof v;
+      if (t === "string" || t === "number" || t === "boolean") return true;
+      if (Array.isArray(v)) return v.every(isJsonValue2);
+      if (t === "object") {
+        const proto = Object.getPrototypeOf(v);
+        if (proto !== Object.prototype && proto !== null) return false;
+        for (const val of Object.values(v)) {
+          if (!isJsonValue2(val)) return false;
+        }
+        return true;
+      }
+      return false;
+    };
     for (const [rawKey, rawValue] of Object.entries(input)) {
       if (rawValue === void 0) {
         continue;
       }
       if (rawKey === "children") continue;
       let key = rawKey;
-      if (rawKey === "name" && explicitName !== void 0) {
-        console.warn('newNamedChild: "name" in props is ignored because a name argument was provided');
-        continue;
-      }
       let value = rawValue;
       if (key === "_c") {
         if (value instanceof Date) {
@@ -659,23 +670,14 @@ var Vertex = class _Vertex {
           continue;
         }
       }
-      const isPrimitive = (v) => typeof v === "string" || typeof v === "number" || typeof v === "boolean";
-      if (Array.isArray(value)) {
-        if (!value.every(isPrimitive)) {
-          skipped.push(rawKey);
-          continue;
-        }
-      } else if (typeof value === "object" && value !== null) {
-        skipped.push(rawKey);
-        continue;
-      } else if (!isPrimitive(value)) {
+      if (!isJsonValue2(value)) {
         skipped.push(rawKey);
         continue;
       }
       out[key] = value;
     }
     if (skipped.length > 0) {
-      console.warn(`Some fields were skipped due to unsupported types: ${skipped.join(", ")}`);
+      throw new Error(`Unsupported property types for keys: ${skipped.join(", ")}`);
     }
     return Object.keys(out).length > 0 ? out : null;
   }
@@ -866,6 +868,61 @@ var StateVector = class _StateVector {
   }
 };
 
+// src/utils/deepEqual.ts
+function deepEqual(a, b) {
+  if (a === b) return true;
+  if (a === null || b === null) return a === b;
+  const ta = typeof a;
+  const tb = typeof b;
+  if (ta !== tb) return false;
+  if (ta !== "object") return false;
+  const aIsArr = Array.isArray(a);
+  const bIsArr = Array.isArray(b);
+  if (aIsArr || bIsArr) {
+    if (!(aIsArr && bIsArr)) return false;
+    if (a.length !== b.length) return false;
+    for (let i = 0; i < a.length; i++) {
+      if (!deepEqual(a[i], b[i])) return false;
+    }
+    return true;
+  }
+  const aProto = Object.getPrototypeOf(a);
+  const bProto = Object.getPrototypeOf(b);
+  if (aProto !== Object.prototype && aProto !== null || bProto !== Object.prototype && bProto !== null) {
+    return false;
+  }
+  const aKeys = Object.keys(a);
+  const bKeys = Object.keys(b);
+  if (aKeys.length !== bKeys.length) return false;
+  for (const key of aKeys) {
+    if (!Object.prototype.hasOwnProperty.call(b, key)) return false;
+    if (!deepEqual(a[key], b[key])) return false;
+  }
+  return true;
+}
+
+// src/utils/isJsonValue.ts
+function isJsonValue(v) {
+  if (v === void 0) return true;
+  if (v === null) return true;
+  const t = typeof v;
+  if (t === "string" || t === "number" || t === "boolean") return true;
+  if (t === "bigint" || t === "function" || t === "symbol") return false;
+  if (Array.isArray(v)) return v.every(isJsonValue);
+  if (t === "object") {
+    if (v instanceof Date) return false;
+    if (v instanceof Map || v instanceof Set || v instanceof RegExp) return false;
+    if (ArrayBuffer.isView(v)) return false;
+    const proto = Object.getPrototypeOf(v);
+    if (proto !== Object.prototype && proto !== null) return false;
+    for (const val of Object.values(v)) {
+      if (!isJsonValue(val)) return false;
+    }
+    return true;
+  }
+  return false;
+}
+
 // src/RepTree.ts
 var _RepTree = class _RepTree {
   /**
@@ -1032,6 +1089,9 @@ var _RepTree = class _RepTree {
     this.moveVertex(vertexId, _RepTree.NULL_VERTEX_ID);
   }
   setTransientVertexProperty(vertexId, key, value) {
+    if (!isJsonValue(value)) {
+      throw new Error(`Unsupported transient property value for key "${key}"`);
+    }
     this.lamportClock++;
     const op = newSetTransientVertexPropertyOp(this.lamportClock, this.peerId, vertexId, key, value);
     this.localOps.push(op);
@@ -1057,6 +1117,29 @@ var _RepTree = class _RepTree {
     vertex.clearAllTransientProperties();
   }
   setVertexProperty(vertexId, key, value) {
+    const isJsonValue2 = (v) => {
+      if (v === void 0) return true;
+      if (v === null) return true;
+      const t = typeof v;
+      if (t === "string" || t === "number" || t === "boolean") return true;
+      if (t === "bigint" || t === "function" || t === "symbol") return false;
+      if (Array.isArray(v)) return v.every(isJsonValue2);
+      if (t === "object") {
+        if (v instanceof Date) return false;
+        if (v instanceof Map || v instanceof Set || v instanceof RegExp) return false;
+        if (ArrayBuffer.isView(v)) return false;
+        const proto = Object.getPrototypeOf(v);
+        if (proto !== Object.prototype && proto !== null) return false;
+        for (const val of Object.values(v)) {
+          if (!isJsonValue2(val)) return false;
+        }
+        return true;
+      }
+      return false;
+    };
+    if (!isJsonValue2(value)) {
+      throw new Error(`Unsupported property value for key "${key}"`);
+    }
     this.lamportClock++;
     const op = newSetVertexPropertyOp(this.lamportClock, this.peerId, vertexId, key, value);
     this.localOps.push(op);
@@ -1218,12 +1301,8 @@ var _RepTree = class _RepTree {
       }
       for (const propA of propertiesA) {
         const propB = propertiesB.find((p) => p.key === propA.key);
-        if (!propB) {
-          return false;
-        }
-        if (propA.value !== propB.value) {
-          return false;
-        }
+        if (!propB) return false;
+        if (!deepEqual(propA.value, propB.value)) return false;
       }
     }
     for (const childId of childrenA) {
@@ -1353,7 +1432,6 @@ var _RepTree = class _RepTree {
       }
     }
   }
-  // Non-LWW modify-property flow removed
   applyOperation(op) {
     if (isMoveVertexOp(op)) {
       this.applyMove(op);