Fix initial start sample project load (#179)

* fix initialize db on first start issue

* fix sample project loading

Author: hristoiankov

cider-app/package.json

@@ -6,7 +6,7 @@
     "email": "contact@oatear.com",
     "url": "https://oatear.com"
   },
-  "version": "0.6.13",
+  "version": "0.6.14",
   "repository": "https://github.com/oatear/cider",
   "main": "electron-app/dist/main.js",
   "scripts": {

cider-app/src/app/data-services/indexed-db/db.ts

@@ -4,7 +4,7 @@ import { Asset } from "../types/asset.type";
 import { CardTemplate } from "../types/card-template.type";
 import { FieldType } from "../types/field-type.type";
 import { Deck } from "../types/deck.type";
-import { exportDB, importDB } from "dexie-export-import";
+import { exportDB, importDB, importInto } from "dexie-export-import";
 import FileUtils from "src/app/shared/utils/file-utils";
 import { ExportProgress } from "dexie-export-import/dist/export";
 import { ImportProgress } from "dexie-export-import/dist/import";
@@ -39,6 +39,7 @@ export class AppDB extends Dexie {
     private httpClient;
     private changeSubject: Subject<any>;
     private loadSubject: Subject<null>;
+    private isPopulating: boolean = false;
 
     constructor(httpClient: HttpClient,
         electronService: ElectronService,
@@ -151,9 +152,10 @@ export class AppDB extends Dexie {
             .then(count => {
                 if (!electronService.isElectron() && count > 0) {
                     console.log('db already populated');
+                    return Promise.resolve();
                 } else {
                     console.log('populate from file');
-                    this.populateFromFile().then(() => {
+                    return this.populateFromFile().then(() => {
                         // cause asset urls to update
                         this.loadSubject.next(null);
                     });
@@ -176,7 +178,9 @@ export class AppDB extends Dexie {
 
         // Add hooks for granular dirty tracking
         const trackChange = (tableName: string, type: string, key: any) => {
-            this.changeSubject.next({ tableName, type, key });
+            if (!this.isPopulating) {
+                this.changeSubject.next({ tableName, type, key });
+            }
         };
 
         // We need to wait for tables to be initialized or just access them via this.table()
@@ -211,15 +215,39 @@ export class AppDB extends Dexie {
 
     }
 
-    async populateFromFile() {
-        const blob = await firstValueFrom(this.httpClient.get(AppDB.SAMPLE_DB_FILE, { responseType: 'blob' }));
-        const file = new File([blob], 'database.json', { type: 'application/json', lastModified: Date.now() });
-        return importDB(file, {
-            //noTransaction: true
-        }).then((db) => {
-            this.initializeData();
-            return db;
-        });
+    async populateFromFile(emitChange: boolean = true) {
+        // This is a destructive operation, but it's only called when the DB is effectively empty
+        // or we fully intend to reset to the sample state.
+        this.isPopulating = true;
+
+        try {
+            // 1. Fetch the sample file
+            const blob = await firstValueFrom(this.httpClient.get(AppDB.SAMPLE_DB_FILE, { responseType: 'blob' }));
+            const file = new File([blob], 'database.json', { type: 'application/json', lastModified: Date.now() });
+
+            // 2. Import into the CURRENT existing database logic
+            // We do NOT close or delete the DB. We just clear existing tables and import data.
+            // options:
+            // - clearTablesBeforeImport: true -> effectively a reset
+            // - acceptVersionDiff: true -> ignores that the file might be v2 and the DB is v10
+            // - overwriteValues: true -> ensures import wins
+            await importInto(this, file, {
+                clearTablesBeforeImport: true,
+                acceptVersionDiff: true,
+                overwriteValues: true
+            });
+
+            // 3. Notify that the database has been populated (refreshes UI/Sidebar)
+            // We use a generic event so listeners know something changed.
+            // We also wait a tick to ensure listeners are ready if this is part of initial load.
+            if (emitChange) {
+                this.changeSubject.next({ tableName: 'all', type: 'reset', key: null });
+            }
+        } finally {
+            this.isPopulating = false;
+        }
+
+        return this;
     }
 
     /**
@@ -264,11 +292,13 @@ export class AppDB extends Dexie {
     public async resetDatabase(keepEmpty?: boolean) {
         this.close();
         await this.delete();
+        await this.open();
         if (!keepEmpty) {
-            const tempDb = await this.populateFromFile();
-            tempDb.close();
+            await this.populateFromFile(false);
+            // Don't close here! populateFromFile leaves the DB open (via importInto), 
+            // and we want it to stay open for initializeData.
         }
-        await this.open().then(() => this.initializeData());
+        await this.initializeData();
         this.changeSubject.next(null);
         return true;
     }