vite errors out because the manager uses version 3

Author: abdo643-HULK

packages/installations/package.json

@@ -50,10 +50,10 @@
   },
   "devDependencies": {
     "@firebase/app": "0.7.8",
-    "rollup": "2.57.0",
     "@rollup/plugin-commonjs": "21.0.0",
     "@rollup/plugin-json": "4.1.0",
     "@rollup/plugin-node-resolve": "13.0.5",
+    "rollup": "2.57.0",
     "rollup-plugin-typescript2": "0.30.0",
     "rollup-plugin-uglify": "6.0.4",
     "typescript": "4.2.2"
@@ -62,9 +62,9 @@
     "@firebase/app": "0.x"
   },
   "dependencies": {
-    "@firebase/util": "1.4.2",
     "@firebase/component": "0.5.9",
-    "idb": "3.0.2",
+    "@firebase/util": "1.4.2",
+    "idb": "6.1.5",
     "tslib": "^2.1.0"
   }
-}
+}

packages/installations/src/helpers/idb-manager.ts

@@ -15,28 +15,31 @@
  * limitations under the License.
  */
 
-import { DB, openDb } from 'idb';
+import { openDB } from 'idb';
 import { AppConfig } from '../interfaces/installation-impl';
 import { InstallationEntry } from '../interfaces/installation-entry';
 import { getKey } from '../util/get-key';
 import { fidChanged } from './fid-changed';
+import type { IDBPDatabase } from 'idb';
 
 const DATABASE_NAME = 'firebase-installations-database';
 const DATABASE_VERSION = 1;
 const OBJECT_STORE_NAME = 'firebase-installations-store';
 
-let dbPromise: Promise<DB> | null = null;
-function getDbPromise(): Promise<DB> {
+let dbPromise: Promise<IDBPDatabase> | null = null;
+function getDbPromise(): Promise<IDBPDatabase> {
   if (!dbPromise) {
-    dbPromise = openDb(DATABASE_NAME, DATABASE_VERSION, upgradeDB => {
-      // We don't use 'break' in this switch statement, the fall-through
-      // behavior is what we want, because if there are multiple versions between
-      // the old version and the current version, we want ALL the migrations
-      // that correspond to those versions to run, not only the last one.
-      // eslint-disable-next-line default-case
-      switch (upgradeDB.oldVersion) {
-        case 0:
-          upgradeDB.createObjectStore(OBJECT_STORE_NAME);
+    dbPromise = openDB(DATABASE_NAME, DATABASE_VERSION, {
+      upgrade: (upgradeDB, oldVersion) => {
+        // We don't use 'break' in this switch statement, the fall-through
+        // behavior is what we want, because if there are multiple versions between
+        // the old version and the current version, we want ALL the migrations
+        // that correspond to those versions to run, not only the last one.
+        // eslint-disable-next-line default-case
+        switch (oldVersion) {
+          case 0:
+            upgradeDB.createObjectStore(OBJECT_STORE_NAME);
+        }
       }
     });
   }
@@ -66,7 +69,7 @@ export async function set<ValueType extends InstallationEntry>(
   const objectStore = tx.objectStore(OBJECT_STORE_NAME);
   const oldValue = await objectStore.get(key);
   await objectStore.put(value, key);
-  await tx.complete;
+  await tx.done;
 
   if (!oldValue || oldValue.fid !== value.fid) {
     fidChanged(appConfig, value.fid);
@@ -81,7 +84,7 @@ export async function remove(appConfig: AppConfig): Promise<void> {
   const db = await getDbPromise();
   const tx = db.transaction(OBJECT_STORE_NAME, 'readwrite');
   await tx.objectStore(OBJECT_STORE_NAME).delete(key);
-  await tx.complete;
+  await tx.done;
 }
 
 /**
@@ -106,7 +109,7 @@ export async function update<ValueType extends InstallationEntry | undefined>(
   } else {
     await store.put(newValue, key);
   }
-  await tx.complete;
+  await tx.done;
 
   if (newValue && (!oldValue || oldValue.fid !== newValue.fid)) {
     fidChanged(appConfig, newValue.fid);
@@ -119,5 +122,5 @@ export async function clear(): Promise<void> {
   const db = await getDbPromise();
   const tx = db.transaction(OBJECT_STORE_NAME, 'readwrite');
   await tx.objectStore(OBJECT_STORE_NAME).clear();
-  await tx.complete;
+  await tx.done;
 }