@abraca/dabra 1.0.18 → 1.0.19

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.19",
   "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/BackgroundSyncManager.ts

@@ -391,7 +391,9 @@ 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);
+
 		const childProvider = await this.rootProvider.loadChild(docId);
 
 		// Wait for ready (offline snapshot loaded) then synced (server sync done)
@@ -403,6 +405,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());
+		}
 	}
 }