indexeddbshim 17.3.0 → 17.3.1

package/dist/setGlobalVars.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"setGlobalVars.d.ts","sourceRoot":"","sources":["../src/setGlobalVars.js"],"names":[],"mappings":";;uBAkBa,GAAG;+BAKL,OAAO,UAAU,EAAE,iBAAiB,GAC1C,OAAO,CAAC,OAAO,UAAU,EAAE,YAAY,CAAC,QAClC,QAAQ;wBAoCN,CACR,IAAI,EAAE,OAAO,UAAU,EAAE,iBAAiB,KACvC,OAAO,UAAU,EAAE,WAAW;oCAIzB,CAAC,GAAG,EAAE;IACd,cAAc,EAAE,MAAM,CAAC;IACvB,iBAAiB,EAAE,MAAM,CAAA;CAC1B,KAAK,IAAI;4BAIA,CAAC,UAAU,GAAC,MAAM,CAAC,GAAG;IAC5B,SAAS,EAAE,MAAM,IAAI,CAAC;IACtB,OAAO,EAAE,CAAC,GAAG,EAAE,OAAO,KAAK,IAAI,CAAC;IAChC,WAAW,EAAE,SAAS,CAAC;IACvB,WAAW,EAAE,SAAS,CAAC;IACvB,uBAAuB,EAAE,qBAAqB,CAAC;IAC/C,0BAA0B,EAAE,CAAC,MAAM,CAAC,EAAE,MAAM,KAAK,IAAI,CAAA;CACtD;sBAIO,MAAM;4BAIN,CAAC,OAAO,UAAU,GAAC,MAAM,CAAC,GAAG;IACrC,SAAS,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,kBAAkB,EAAE,OAAO,kBAAkB,CAAC;IAC9C,SAAS,EAAE,OAAO,SAAS,CAAC;IAC5B,WAAW,EAAE,OAAO,WAAW,CAAC;IAChC,cAAc,EAAE,OAAO,cAAc,CAAC;IACtC,WAAW,EAAE,OAAO,WAAW,CAAC;IAChC,aAAa,CAAC,EAAE,aAAa,CAAA;CAC9B;mCAhG+C,mBAAmB;AAwDtE;;;;GAIG;AAEH;;;;;GAKG;AAEH;;;;;;;;;GASG;AAEH;;GAEG;AAEH;;;;;;;;;;;;;GAaG;AAEH;;;;;GAKG;AACH,qCAJW,aAAa,kBACb,OAAO,CAAC,OAAO,UAAU,EAAE,YAAY,CAAC,GACtC,aAAa,CAkSzB;2BAxY8C,iBAAiB"}
+{"version":3,"file":"setGlobalVars.d.ts","sourceRoot":"","sources":["../src/setGlobalVars.js"],"names":[],"mappings":";;uBAkBa,GAAG;+BAKL,OAAO,UAAU,EAAE,iBAAiB,GAC1C,OAAO,CAAC,OAAO,UAAU,EAAE,YAAY,CAAC,QAClC,QAAQ;wBAoCN,CACR,IAAI,EAAE,OAAO,UAAU,EAAE,iBAAiB,KACvC,OAAO,UAAU,EAAE,WAAW;oCAIzB,CAAC,GAAG,EAAE;IACd,cAAc,EAAE,MAAM,CAAC;IACvB,iBAAiB,EAAE,MAAM,CAAA;CAC1B,KAAK,IAAI;4BAIA,CAAC,UAAU,GAAC,MAAM,CAAC,GAAG;IAC5B,SAAS,EAAE,MAAM,IAAI,CAAC;IACtB,OAAO,EAAE,CAAC,GAAG,EAAE,OAAO,KAAK,IAAI,CAAC;IAChC,WAAW,EAAE,SAAS,CAAC;IACvB,WAAW,EAAE,SAAS,CAAC;IACvB,uBAAuB,EAAE,qBAAqB,CAAC;IAC/C,0BAA0B,EAAE,CAAC,MAAM,CAAC,EAAE,MAAM,KAAK,IAAI,CAAA;CACtD;sBAIO,MAAM;4BAIN,CAAC,OAAO,UAAU,GAAC,MAAM,CAAC,GAAG;IACrC,SAAS,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,CAAC;IAChC,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,UAAU,EAAE,OAAO,UAAU,CAAC;IAC9B,kBAAkB,EAAE,OAAO,kBAAkB,CAAC;IAC9C,SAAS,EAAE,OAAO,SAAS,CAAC;IAC5B,WAAW,EAAE,OAAO,WAAW,CAAC;IAChC,cAAc,EAAE,OAAO,cAAc,CAAC;IACtC,WAAW,EAAE,OAAO,WAAW,CAAC;IAChC,aAAa,CAAC,EAAE,aAAa,CAAA;CAC9B;mCAhG+C,mBAAmB;AAwDtE;;;;GAIG;AAEH;;;;;GAKG;AAEH;;;;;;;;;GASG;AAEH;;GAEG;AAEH;;;;;;;;;;;;;GAaG;AAEH;;;;;GAKG;AACH,qCAJW,aAAa,kBACb,OAAO,CAAC,OAAO,UAAU,EAAE,YAAY,CAAC,GACtC,aAAa,CAkTzB;2BAxZ8C,iBAAiB"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "indexeddbshim",
-  "version": "17.3.0",
+  "version": "17.3.1",
   "type": "module",
   "author": "Parashuram <code@nparashuram.com>",
   "contributors": [

package/src/IDBFactory.js

@@ -668,7 +668,22 @@ IDBFactory.prototype.open = function (name /* , version */) {
                                     //   open/close the transaction's active-handler window itself.
                                     req.transaction.__handlerActive = true;
                                     req.dispatchEvent(e);
-                                    req.transaction.__handlerActive = false;
+                                    // Give any same-tick microtask scheduled from within the
+                                    //   `upgradeneeded` handler (e.g. a plain
+                                    //   `Promise.resolve().then(...)`) a chance to run -- and still
+                                    //   observe the transaction as active -- before we deactivate it
+                                    //   again, per https://github.com/w3c/IndexedDB/issues/87. Only
+                                    //   the flag reset itself is deferred here -- unlike
+                                    //   `IDBTransaction.js`'s `advanceAfterDispatch`, `finished()`
+                                    //   (this transaction's own queue-advancement/completion signal)
+                                    //   still runs synchronously, at exactly its previous timing: an
+                                    //   earlier attempt at deferring `finished()` too raced against
+                                    //   unrelated test setup that assumed a freshly deleted/created
+                                    //   database's upgrade transaction had already fully completed by
+                                    //   the time this function returns.
+                                    queueMicrotask(() => {
+                                        req.transaction.__handlerActive = false;
+                                    });
 
                                     if (e.__legacyOutputDidListenersThrowError) {
                                         logError('Error', 'An error occurred in an upgradeneeded handler attached to request chain', /** @type {Error} */ (e.__legacyOutputDidListenersThrowError)); // We do nothing else with this error as per spec

package/src/setGlobalVars.js

@@ -274,8 +274,24 @@ function setGlobalVars (idb, initialConfig) {
                     Object.setPrototypeOf(shimIDBRequest, ShimEventTarget);
                     Object.setPrototypeOf(shimIDBTransaction, ShimEventTarget);
                     Object.setPrototypeOf(shimIDBVersionChangeEvent, ShimEvent);
-                    Object.setPrototypeOf(ShimDOMException, Error);
-                    Object.setPrototypeOf(ShimDOMException.prototype, Error.prototype);
+                    // `ShimDOMException` is the real native `DOMException` when one
+                    //   is available (see `DOMException.js`'s `useNativeDOMException`)
+                    //   -- which, unlike the shim classes above, is a single,
+                    //   process-wide singleton shared by reference across every
+                    //   sandbox this library gets installed into (see
+                    //   `node-idb-test.js`'s `sandboxObj`). A native `DOMException`
+                    //   already has the correct prototype chain out of the box
+                    //   (`Object.getPrototypeOf(DOMException) === Function.prototype`,
+                    //   not `Error`), so forcing it here isn't just unneeded but
+                    //   actively wrong -- and, because it's shared, permanently
+                    //   wrong for every later use of `DOMException` in the same
+                    //   process (e.g. a later WPT test file's own idlharness-style
+                    //   check that `DOMException` does *not* inherit from `Error`
+                    //   on the class side), not just this one shim install.
+                    if (typeof DOMException === 'undefined' || ShimDOMException !== DOMException) {
+                        Object.setPrototypeOf(ShimDOMException, Error);
+                        Object.setPrototypeOf(ShimDOMException.prototype, Error.prototype);
+                    }
                 }
                 if (IDB.indexedDB && !IDB.indexedDB.toString().includes('[native code]')) {
                     if (CFG.addNonIDBGlobals) {