reptree 0.5.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

@@ -26,7 +26,6 @@ __export(index_exports, {
   Vertex: () => Vertex,
   VertexState: () => VertexState,
   bindVertex: () => bindVertex,
-  defaultAliases: () => defaultAliases,
   isAnyPropertyOp: () => isAnyPropertyOp,
   isMoveVertexOp: () => isMoveVertexOp,
   newMoveVertexOp: () => newMoveVertexOp,
@@ -355,7 +354,7 @@ var TreeState = class {
       const vertex = this.getVertex(vertexId);
       if (vertex) {
         for (const prop of vertex.getAllProperties()) {
-          if (prop.key === "_n") {
+          if (prop.key === "name") {
             vertexName = prop.value;
           }
           const propPrefix = indent + (isLast ? "    " : "\u2502   ") + "\u2022 ";
@@ -368,8 +367,8 @@ var TreeState = class {
     const sortedChildren = [...children].sort((a, b) => {
       const vertexA = this.getVertex(a);
       const vertexB = this.getVertex(b);
-      const nameA = vertexA?.getProperty("_n");
-      const nameB = vertexB?.getProperty("_n");
+      const nameA = vertexA?.getProperty("name");
+      const nameB = vertexB?.getProperty("name");
       if (nameA && nameB) {
         return nameA.localeCompare(nameB);
       }
@@ -386,37 +385,17 @@ var TreeState = class {
   }
 };
 
-// src/uuid.ts
+// src/utils/uuid.ts
 var removeDashes = (guid) => guid.replace(/-/g, "");
 function uuid() {
   return removeDashes(crypto.randomUUID());
 }
 
 // src/reactive.ts
-var defaultAliases = [
-  { publicKey: "name", internalKey: "_n" },
-  {
-    publicKey: "createdAt",
-    internalKey: "_c",
-    toPublic: (v) => typeof v === "string" ? new Date(v) : v,
-    toInternal: (v) => v instanceof Date ? v.toISOString() : v
-  }
-];
-function buildAliasMaps(aliases) {
-  const publicToInternal = /* @__PURE__ */ new Map();
-  const internalToPublic = /* @__PURE__ */ new Map();
-  for (const rule of aliases) {
-    publicToInternal.set(rule.publicKey, rule);
-    internalToPublic.set(rule.internalKey, rule);
-  }
-  return { publicToInternal, internalToPublic };
-}
 function bindVertex(tree, id, schemaOrOptions) {
-  const isOptions = typeof schemaOrOptions === "object" && schemaOrOptions !== null && (Object.prototype.hasOwnProperty.call(schemaOrOptions, "aliases") || Object.prototype.hasOwnProperty.call(schemaOrOptions, "includeInternalKeys") || Object.prototype.hasOwnProperty.call(schemaOrOptions, "schema"));
+  const isOptions = typeof schemaOrOptions === "object" && schemaOrOptions !== null && (Object.prototype.hasOwnProperty.call(schemaOrOptions, "includeInternalKeys") || Object.prototype.hasOwnProperty.call(schemaOrOptions, "schema"));
   const options = isOptions ? schemaOrOptions : { schema: schemaOrOptions };
   const schema = options.schema;
-  const aliases = options.aliases ?? defaultAliases;
-  const { publicToInternal } = buildAliasMaps(aliases);
   const obj = {};
   Object.defineProperties(obj, {
     $vertex: { get: () => tree.getVertex(id), enumerable: false, configurable: true },
@@ -453,19 +432,14 @@ function bindVertex(tree, id, schemaOrOptions) {
         const transientProxy = new Proxy({}, {
           set(_, prop, value) {
             if (typeof prop === "string") {
-              const rule = publicToInternal.get(prop);
-              const internalKey = rule?.internalKey ?? prop;
-              const internalValue = rule?.toInternal ? rule.toInternal(value) : value;
-              tree.setTransientVertexProperty(id, internalKey, internalValue);
+              tree.setTransientVertexProperty(id, prop, value);
             }
             return true;
           },
           get(_, prop) {
             if (typeof prop !== "string") return void 0;
-            const rule = publicToInternal.get(prop);
-            const internalKey = rule?.internalKey ?? prop;
-            const rawValue = tree.getVertexProperty(id, internalKey, true);
-            return rule?.toPublic ? rule.toPublic(rawValue) : rawValue;
+            const rawValue = tree.getVertexProperty(id, prop, true);
+            return rawValue;
           }
         });
         fn(transientProxy);
@@ -495,10 +469,8 @@ function bindVertex(tree, id, schemaOrOptions) {
       if (prop in target) {
         return Reflect.get(target, prop, receiver);
       }
-      const rule = publicToInternal.get(prop);
-      const internalKey = rule?.internalKey ?? prop;
-      const rawValue = tree.getVertexProperty(id, internalKey, true);
-      return rule?.toPublic ? rule.toPublic(rawValue) : rawValue;
+      const rawValue = tree.getVertexProperty(id, prop, true);
+      return rawValue;
     },
     set(target, prop, value) {
       if (typeof prop !== "string") {
@@ -512,19 +484,14 @@ function bindVertex(tree, id, schemaOrOptions) {
           value = res.data;
         }
       }
-      const rule = publicToInternal.get(prop);
-      const internalKey = rule?.internalKey ?? prop;
-      const internalValue = rule?.toInternal ? rule.toInternal(value) : value;
-      tree.setVertexProperty(id, internalKey, internalValue);
+      tree.setVertexProperty(id, prop, value);
       return true;
     },
     deleteProperty(_target, prop) {
       if (typeof prop !== "string") {
         return true;
       }
-      const rule = publicToInternal.get(prop);
-      const internalKey = rule?.internalKey ?? prop;
-      tree.setVertexProperty(id, internalKey, void 0);
+      tree.setVertexProperty(id, prop, void 0);
       return true;
     }
   });
@@ -541,10 +508,10 @@ var Vertex = class _Vertex {
     return this.state.id;
   }
   get name() {
-    return this.getProperty("_n");
+    return this.getProperty("name");
   }
   set name(name) {
-    this.tree.setVertexProperty(this.id, "_n", name);
+    this.tree.setVertexProperty(this.id, "name", name);
   }
   get createdAt() {
     const createdAt = this.getProperty("_c");
@@ -588,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) {
@@ -664,30 +631,35 @@ var Vertex = class _Vertex {
   }
   /**
    * Normalizes an input props object for vertex creation:
-   * - Aliases name -> _n, createdAt -> _c (Date -> ISO string)
    * - 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") {
-        if (explicitName !== void 0) {
-          console.warn('newNamedChild: "name" in props is ignored because a name argument was provided');
-          continue;
-        }
-        key = "_n";
-      } else if (rawKey === "createdAt") {
-        key = "_c";
-      }
       let value = rawValue;
       if (key === "_c") {
         if (value instanceof Date) {
@@ -698,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;
   }
@@ -905,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 {
   /**
@@ -1054,7 +1072,7 @@ var _RepTree = class _RepTree {
     if (typedProps) {
       this.setVertexProperties(vertexId, typedProps);
     }
-    this.setVertexProperty(vertexId, "_n", name);
+    this.setVertexProperty(vertexId, "name", name);
     const vertex = this.state.getVertex(vertexId);
     if (!vertex) {
       throw new Error("Failed to create named vertex");
@@ -1071,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);
@@ -1096,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);
@@ -1128,7 +1172,7 @@ var _RepTree = class _RepTree {
     const targetName = path[0];
     const children = this.getChildren(vertex.id);
     for (const child of children) {
-      if (child.getProperty("_n") === targetName) {
+      if (child.getProperty("name") === targetName) {
         return this.getVertexByPathArray(child, path.slice(1));
       }
     }
@@ -1257,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) {
@@ -1392,7 +1432,6 @@ var _RepTree = class _RepTree {
       }
     }
   }
-  // Non-LWW modify-property flow removed
   applyOperation(op) {
     if (isMoveVertexOp(op)) {
       this.applyMove(op);
@@ -1516,7 +1555,6 @@ var RepTree = _RepTree;
   Vertex,
   VertexState,
   bindVertex,
-  defaultAliases,
   isAnyPropertyOp,
   isMoveVertexOp,
   newMoveVertexOp,