Merge pull request #666 from streamich/json-patch-state-2

`JsonPatchState` improvements

Author: streamich

src/json-crdt/json-patch/JsonPatch.ts

@@ -15,7 +15,9 @@ export class JsonPatch<N extends JsonNode = JsonNode<any>> {
 
   public apply(ops: Operation[]): this {
     const length = ops.length;
-    for (let i = 0; i < length; i++) this.applyOp(ops[i]);
+    this.model.api.transaction(() => {
+      for (let i = 0; i < length; i++) this.applyOp(ops[i]);
+    });
     return this;
   }
 

src/json-crdt/json-patch/JsonPatchStore.ts

@@ -9,23 +9,23 @@ import type {JsonNode, JsonNodeView} from '../nodes';
 export class JsonPatchStore<N extends JsonNode = JsonNode<any>> implements SyncStore<Readonly<JsonNodeView<N>>> {
   public readonly node: N;
   public readonly api: JsonNodeApi<N>;
-  protected _patcher: JsonPatch<N>;
-  protected _pfx: string;
+  public readonly patcher: JsonPatch<N>;
+  public readonly pfx: string;
 
   constructor(
     protected readonly model: Model<N>,
-    protected readonly path: Path = [],
+    public readonly path: Path = [],
   ) {
-    this._pfx = path.length ? path.join() : '';
+    this.pfx = path.length ? path.join() : '';
     const api = model.api;
     this.node = api.find(path) as N;
     this.api = api.wrap(this.node) as unknown as JsonNodeApi<N>;
-    this._patcher = new JsonPatch(model, path);
+    this.patcher = new JsonPatch(model, path);
   }
 
   public readonly update = (change: Operation | Operation[]): void => {
     const ops = Array.isArray(change) ? change : [change];
-    this._patcher.apply(ops);
+    this.patcher.apply(ops);
   };
 
   // ---------------------------------------------------------------- SyncStore