@abraca/dabra 0.4.0 → 0.6.0

package/dist/index.d.ts

@@ -933,6 +933,7 @@ declare class OfflineStore {
    *                    per-server, preventing cross-server data contamination.
    */
   constructor(docId: string, serverOrigin?: string);
+  private dbPromise;
   private getDb;
   persistUpdate(update: Uint8Array): Promise<void>;
   getPendingUpdates(): Promise<Uint8Array[]>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abraca/dabra",
-  "version": "0.4.0",
+  "version": "0.6.0",
   "description": "abracadabra provider",
   "keywords": [
     "abracadabra",

package/src/AbracadabraProvider.ts

@@ -188,12 +188,16 @@ export class AbracadabraProvider extends HocuspocusProvider {
 	private async _initFromOfflineStore(): Promise<void> {
 		if (!this.offlineStore) return;
 
-		const snapshot = await this.offlineStore.getDocSnapshot().catch(() => null);
+		// Fetch snapshot and pending updates in parallel — each call opens the
+		// IDB database, so running them concurrently roughly halves the latency.
+		const [snapshot, pending] = await Promise.all([
+			this.offlineStore.getDocSnapshot().catch(() => null),
+			this.offlineStore.getPendingUpdates().catch(() => []),
+		]);
+
 		if (snapshot) {
 			Y.applyUpdate(this.document, snapshot, this.offlineStore);
 		}
-
-		const pending = await this.offlineStore.getPendingUpdates().catch(() => []);
 		for (const update of pending) {
 			Y.applyUpdate(this.document, update, this.offlineStore);
 		}
@@ -417,29 +421,12 @@ export class AbracadabraProvider extends HocuspocusProvider {
 	private async _doLoadChild(childId: string): Promise<AbracadabraProvider> {
 		const childDoc = new Y.Doc({ guid: childId });
 
-		if (this.isConnected) {
-			// Online: notify the server that this child belongs to the parent and
-			// wait for server confirmation before opening the child WebSocket.
-			// Without waiting, the child's SyncStep1 may race against subdoc
-			// creation and get a NotFound error on first sync.
-			await new Promise<void>((resolve) => {
-				const onRegistered = ({ childId: cid }: onSubdocRegisteredParameters) => {
-					if (cid === childId) {
-						this.off("subdocRegistered", onRegistered);
-						resolve();
-					}
-				};
-				this.on("subdocRegistered", onRegistered);
-				this.registerSubdoc(childDoc);
-				// Fallback: don't block forever if the server is slow or the
-				// doc was already registered in a previous session.
-				setTimeout(resolve, 3000);
-			});
-		} else {
-			// Offline: queue the registration for replay on reconnect and proceed
-			// immediately so the child provider can populate from its local store.
-			this.registerSubdoc(childDoc);
-		}
+		// Fire-and-forget: tell the server this child belongs to the parent.
+		// The child provider's own WebSocket handshake (TCP + TLS + auth) is
+		// slow enough that the server will have processed the registration
+		// before the child's first SyncStep1 arrives.  In the rare race the
+		// child's built-in reconnect logic retries automatically.
+		this.registerSubdoc(childDoc);
 
 		// Each child gets its own WebSocket connection.  Omitting
 		// websocketProvider lets HocuspocusProvider create one automatically

package/src/OfflineStore.ts

@@ -77,12 +77,18 @@ export class OfflineStore {
 		this.storeKey = serverOrigin ? `${serverOrigin}/${docId}` : docId;
 	}
 
-	private async getDb(): Promise<IDBDatabase | null> {
-		if (!idbAvailable()) return null;
-		if (!this.db) {
-			this.db = await openDb(this.storeKey).catch(() => null);
+	private dbPromise: Promise<IDBDatabase | null> | null = null;
+
+	private getDb(): Promise<IDBDatabase | null> {
+		if (!idbAvailable()) return Promise.resolve(null);
+		if (!this.dbPromise) {
+			// Cache the promise so concurrent callers share a single open operation.
+			this.dbPromise = openDb(this.storeKey).catch(() => null).then(db => {
+				this.db = db;
+				return db;
+			});
 		}
-		return this.db;
+		return this.dbPromise;
 	}
 
 	// ── Pending (unsynced) updates ────────────────────────────────────────────