reptree 0.5.0 → 0.7.0

package/dist/index.d.ts

@@ -10,8 +10,8 @@ declare class VertexState {
     private transientProperties;
     children: string[];
     constructor(id: string, parentId: TreeVertexId | null);
-    setProperty(key: string, value: any): void;
-    setTransientProperty(key: string, value: any): void;
+    setProperty(key: string, value: VertexPropertyType): void;
+    setTransientProperty(key: string, value: VertexPropertyType): void;
     getProperty(key: string, includingTransient?: boolean): VertexPropertyType | undefined;
     getAllProperties(includingTransient?: boolean): ReadonlyArray<TreeVertexProperty>;
     removeProperty(key: string): void;
@@ -21,11 +21,12 @@ declare class VertexState {
 }
 
 type TreeVertexId = string;
-/**
- * Serializable CRDT data for operations
- */
-/** Property type for state */
-type VertexPropertyType = string | number | boolean | string[] | number[] | boolean[] | undefined;
+type JsonPrimitive = string | number | boolean | null;
+type JsonValue = JsonPrimitive | JsonValue[] | {
+    [key: string]: JsonValue;
+};
+/** Property type for state (undefined means removal) */
+type VertexPropertyType = JsonValue | undefined;
 type TreeVertexProperty = {
     readonly key: string;
     readonly value: VertexPropertyType;
@@ -95,16 +96,8 @@ type SchemaLike<T> = {
     parse?: (input: unknown) => T;
     shape?: Record<string, FieldSchemaLike>;
 };
-type AliasRule = {
-    publicKey: string;
-    internalKey: string;
-    toPublic?: (value: unknown) => unknown;
-    toInternal?: (value: unknown) => unknown;
-};
-declare const defaultAliases: AliasRule[];
 type BindOptions<T> = {
     schema?: SchemaLike<T>;
-    aliases?: AliasRule[];
     includeInternalKeys?: boolean;
 };
 /**
@@ -205,7 +198,6 @@ declare class Vertex {
     bind<T extends Record<string, unknown>>(schemaOrOptions?: SchemaLike<T> | BindOptions<T>): BindedVertex<T>;
     /**
      * 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
      */
@@ -353,8 +345,8 @@ declare class TreeState {
     getChildrenIds(vertexId: TreeVertexId): string[];
     getChildren(vertexId: TreeVertexId): VertexState[];
     moveVertex(vertexId: TreeVertexId, newParentId: TreeVertexId | null): VertexState;
-    setProperty(vertexId: string, key: string, value: any): void;
-    setTransientProperty(vertexId: string, key: string, value: any): void;
+    setProperty(vertexId: string, key: string, value: VertexPropertyType): void;
+    setTransientProperty(vertexId: string, key: string, value: VertexPropertyType): void;
     addChangeCallback(vertexId: TreeVertexId, listener: (events: VertexChangeEvent[]) => void): void;
     removeChangeCallback(vertexId: TreeVertexId, listener: (events: VertexChangeEvent[]) => void): void;
     addGlobalChangeCallback(listener: (events: VertexChangeEvent[]) => void): void;
@@ -422,4 +414,4 @@ declare class StateVector {
 
 declare function uuid(): string;
 
-export { type AliasRule, type BindOptions, type BindedVertex, type MoveVertex, type OpId, type OpIdRange, RepTree, type SchemaLike, type SetVertexProperty, StateVector, TreeState, type TreeVertexId, type TreeVertexProperty, Vertex, type VertexChangeEvent, type VertexChildrenChangeEvent, type VertexMoveEvent, type VertexOperation, type VertexPropertyChangeEvent, type VertexPropertyType, VertexState, bindVertex, defaultAliases, isAnyPropertyOp, isMoveVertexOp, newMoveVertexOp, newSetTransientVertexPropertyOp, newSetVertexPropertyOp, uuid };
+export { type BindOptions, type BindedVertex, type JsonPrimitive, type JsonValue, type MoveVertex, type OpId, type OpIdRange, RepTree, type SchemaLike, type SetVertexProperty, StateVector, TreeState, type TreeVertexId, type TreeVertexProperty, Vertex, type VertexChangeEvent, type VertexChildrenChangeEvent, type VertexMoveEvent, type VertexOperation, type VertexPropertyChangeEvent, type VertexPropertyType, VertexState, bindVertex, isAnyPropertyOp, isMoveVertexOp, newMoveVertexOp, newSetTransientVertexPropertyOp, newSetVertexPropertyOp, uuid };

package/dist/index.js

@@ -317,7 +317,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 ";
@@ -330,8 +330,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);
       }
@@ -348,37 +348,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 },
@@ -415,19 +395,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);
@@ -457,10 +432,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") {
@@ -474,19 +447,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;
     }
   });
@@ -503,10 +471,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");
@@ -550,7 +518,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) {
@@ -626,30 +594,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) {
@@ -660,23 +633,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;
   }
@@ -867,6 +831,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 {
   /**
@@ -1016,7 +1035,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");
@@ -1033,6 +1052,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);
@@ -1058,6 +1080,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);
@@ -1090,7 +1135,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));
       }
     }
@@ -1219,12 +1264,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) {
@@ -1354,7 +1395,6 @@ var _RepTree = class _RepTree {
       }
     }
   }
-  // Non-LWW modify-property flow removed
   applyOperation(op) {
     if (isMoveVertexOp(op)) {
       this.applyMove(op);
@@ -1477,7 +1517,6 @@ export {
   Vertex,
   VertexState,
   bindVertex,
-  defaultAliases,
   isAnyPropertyOp,
   isMoveVertexOp,
   newMoveVertexOp,