@dabble/patches 0.8.12 → 0.8.13

package/dist/client/Patches.d.ts

@@ -123,12 +123,6 @@ declare class Patches {
      * Should be called when shutting down the client.
      */
     close(): Promise<void>;
-    /**
-     * Submits ops for a document through the serialized change queue.
-     * Used by PatchesBranchClient to merge branch changes without racing
-     * against concurrent user edits on the same document.
-     */
-    submitDocChange(docId: string, ops: JSONPatchOp[], metadata?: Record<string, any>): Promise<void>;
     /**
      * Internal handler for doc changes. Called when doc.onChange emits ops.
      * Serializes calls per docId to prevent concurrent handleDocChange from

package/dist/client/Patches.js

@@ -197,17 +197,6 @@ class Patches {
     this.onServerCommit.clear();
     this.onError.clear();
   }
-  /**
-   * Submits ops for a document through the serialized change queue.
-   * Used by PatchesBranchClient to merge branch changes without racing
-   * against concurrent user edits on the same document.
-   */
-  submitDocChange(docId, ops, metadata = {}) {
-    const managed = this.docs.get(docId);
-    const algorithm = this.getDocAlgorithm(docId) ?? this.algorithms[this.defaultAlgorithm];
-    if (!algorithm) throw new Error(`No algorithm found for document ${docId}`);
-    return this._handleDocChange(docId, ops, managed?.doc, algorithm, metadata);
-  }
   /**
    * Internal handler for doc changes. Called when doc.onChange emits ops.
    * Serializes calls per docId to prevent concurrent handleDocChange from

package/dist/client/PatchesBranchClient.d.ts

@@ -26,8 +26,8 @@ interface PatchesBranchClientOptions {
  * (offline-first, local store handles caching/pending/tombstones). The API shape
  * determines merge behavior:
  *
- * - `BranchAPI` has `mergeBranch` — server performs the merge
- * - `BranchClientStore` has `updateBranch` — client merges locally, updates `lastMergedRev`
+ * - `BranchAPI` — server performs the merge via `mergeBranch`
+ * - `BranchClientStore` — merge is not supported; call the server merge endpoint directly
  */
 declare class PatchesBranchClient {
     private readonly api;
@@ -81,16 +81,15 @@ declare class PatchesBranchClient {
     /**
      * Merge a branch's changes back into this document.
      *
-     * Online (BranchAPI with `mergeBranch`): server performs the merge.
-     * Offline (BranchClientStore with `updateBranch`): client reads branch changes,
-     * re-stamps them with `batchId: branchId`, submits via algorithm.handleDocChange
-     * on the source doc, then updates `lastMergedRev` locally.
+     * Requires a `BranchAPI` (online mode) — the server performs the merge.
+     * Throws if the API is a `BranchClientStore` (offline-first mode) because
+     * client stores don't maintain full change history needed for correct merging.
+     * Offline-first consumers should call the server merge endpoint directly.
      */
     mergeBranch(branchId: string): Promise<void>;
     /** Clear state */
     clear(): void;
     private _createBranchOffline;
-    private _mergeBranchLocally;
 }
 
 export { PatchesBranchClient, type PatchesBranchClientOptions };

package/dist/client/PatchesBranchClient.js

@@ -2,6 +2,7 @@ import "../chunk-IZ2YBCUP.js";
 import { store } from "easy-signal";
 import { breakChanges } from "../algorithms/ot/shared/changeBatching.js";
 import { createChange } from "../data/change.js";
+const OFFLINE_MERGE_ERROR = "Branch merging requires a server connection. Use a BranchAPI or call the server merge endpoint directly.";
 class PatchesBranchClient {
   constructor(id, api, patches, options) {
     this.api = api;
@@ -87,18 +88,17 @@ class PatchesBranchClient {
   /**
    * Merge a branch's changes back into this document.
    *
-   * Online (BranchAPI with `mergeBranch`): server performs the merge.
-   * Offline (BranchClientStore with `updateBranch`): client reads branch changes,
-   * re-stamps them with `batchId: branchId`, submits via algorithm.handleDocChange
-   * on the source doc, then updates `lastMergedRev` locally.
+   * Requires a `BranchAPI` (online mode) — the server performs the merge.
+   * Throws if the API is a `BranchClientStore` (offline-first mode) because
+   * client stores don't maintain full change history needed for correct merging.
+   * Offline-first consumers should call the server merge endpoint directly.
    */
   async mergeBranch(branchId) {
-    if (!this.isOffline) {
-      await this.api.mergeBranch(branchId);
-      await this.listBranches();
-      return;
+    if (this.isOffline) {
+      throw new Error(OFFLINE_MERGE_ERROR);
     }
-    await this._mergeBranchLocally(branchId);
+    await this.api.mergeBranch(branchId);
+    await this.listBranches();
   }
   /** Clear state */
   clear() {
@@ -148,29 +148,6 @@ class PatchesBranchClient {
     this.patches.onChange.emit(branchDocId);
     return branchDocId;
   }
-  async _mergeBranchLocally(branchId) {
-    const offlineApi = this.api;
-    const branch = this.branches.state.find((b) => b.id === branchId);
-    if (!branch) throw new Error(`Branch ${branchId} not found`);
-    const sourceDocId = branch.docId;
-    const algorithmName = this.options?.algorithm ?? this.patches.defaultAlgorithm;
-    const algorithm = this.patches.algorithms[algorithmName];
-    if (!algorithm?.listChanges) {
-      throw new Error("Offline merge requires an algorithm with listChanges support");
-    }
-    const startAfter = branch.lastMergedRev ?? (branch.contentStartRev ?? 2) - 1;
-    const branchChanges = await algorithm.listChanges(branchId, { startAfter });
-    if (branchChanges.length === 0) return;
-    const lastBranchRev = branchChanges[branchChanges.length - 1].rev;
-    for (const change of branchChanges) {
-      await this.patches.submitDocChange(sourceDocId, change.ops, { batchId: branchId });
-    }
-    await offlineApi.updateBranch(branchId, { lastMergedRev: lastBranchRev });
-    this.patches.onChange.emit(sourceDocId);
-    this.branches.state = this.branches.state.map(
-      (b) => b.id === branchId ? { ...b, lastMergedRev: lastBranchRev } : b
-    );
-  }
 }
 export {
   PatchesBranchClient

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dabble/patches",
-  "version": "0.8.12",
+  "version": "0.8.13",
   "description": "Immutable JSON Patch implementation based on RFC 6902 supporting operational transformation and last-writer-wins",
   "author": "Jacob Wright <jacwright@gmail.com>",
   "bugs": {