@abraca/dabra 1.0.18 → 1.0.20

package/dist/index.d.ts

@@ -699,7 +699,7 @@ declare class AbracadabraProvider extends AbracadabraBaseProvider {
    */
   loadChild(childId: string): Promise<AbracadabraProvider>;
   private _doLoadChild;
-  private unloadChild;
+  unloadChild(childId: string): void;
   /** Return all currently-loaded child providers. */
   get children(): Map<string, AbracadabraProvider>;
   /**

package/package.json

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

package/src/AbracadabraProvider.ts

@@ -497,7 +497,7 @@ export class AbracadabraProvider extends AbracadabraBaseProvider {
 		return childProvider;
 	}
 
-	private unloadChild(childId: string) {
+	unloadChild(childId: string) {
 		const provider = this.childProviders.get(childId);
 		if (provider) {
 			provider.destroy();

package/src/AbracadabraWS.ts

@@ -220,9 +220,8 @@ export class AbracadabraWS extends EventEmitter {
   attach(provider: AbracadabraBaseProvider) {
     const existing = this.configuration.providerMap.get(provider.configuration.name);
     if (existing && existing !== provider) {
-      console.warn(
-        `[AbracadabraWS] attach: overwriting provider for "${provider.configuration.name}". ` +
-        `This may indicate a duplicate loadChild for the same document.`
+      console.debug(
+        `[AbracadabraWS] attach: replacing provider for "${provider.configuration.name}".`
       );
     }
     this.configuration.providerMap.set(provider.configuration.name, provider);

package/src/BackgroundSyncManager.ts

@@ -391,7 +391,28 @@ export class BackgroundSyncManager extends EventEmitter {
 	}
 
 	private async _syncNonE2EDoc(docId: string): Promise<DocSyncState> {
-		// loadChild() returns cached provider if already open
+		// Check if the provider already exists (user is viewing it) before loading.
+		const alreadyCached = this.rootProvider.children.has(docId);
+
+		// Another provider (e.g. a nested renderer) may have already loaded
+		// this doc through a different parent.  The shared WS providerMap
+		// tracks all attached providers; if one exists, the doc is already
+		// syncing — skip to avoid creating a duplicate.
+		if (!alreadyCached) {
+			const wsProviderMap = (this.rootProvider as any).configuration
+				?.websocketProvider?.configuration?.providerMap as
+				| Map<string, unknown>
+				| undefined;
+			if (wsProviderMap?.has(docId)) {
+				return {
+					docId,
+					status: "synced",
+					lastSynced: Date.now(),
+					isE2E: false,
+				};
+			}
+		}
+
 		const childProvider = await this.rootProvider.loadChild(docId);
 
 		// Wait for ready (offline snapshot loaded) then synced (server sync done)
@@ -403,6 +424,13 @@ export class BackgroundSyncManager extends EventEmitter {
 			this._prefetchDocFiles(docId, childProvider.document).catch(() => null);
 		}
 
+		// Release provider if it was created solely for background sync.
+		// This closes its IDB database, freeing the file descriptor.
+		// Providers that were already cached (user is viewing them) are kept alive.
+		if (!alreadyCached) {
+			this.rootProvider.unloadChild(docId);
+		}
+
 		return { docId, status: "synced", lastSynced: Date.now(), isE2E: false };
 	}
 

package/src/OfflineStore.ts

@@ -277,11 +277,15 @@ export class OfflineStore {
 
 	destroy() {
 		// Set the destroyed flag first so getDb() returns null for any new operations.
-		// Do NOT call db.close() here — in-flight IDB transactions hold their own
-		// reference to the database and closing it prematurely causes InvalidStateError.
-		// The DB will be closed and garbage-collected once all transactions complete.
 		this._destroyed = true;
+		const db = this.db;
 		this.db = null;
 		this.dbPromise = null;
+		// Close the IDB connection to free the file descriptor.  Deferred to the
+		// next microtask so any in-flight transaction callbacks settle first.
+		// IDB spec: close() waits for running transactions to finish before closing.
+		if (db) {
+			Promise.resolve().then(() => db.close());
+		}
 	}
 }