indexeddbshim 17.1.0 → 17.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "indexeddbshim",
-  "version": "17.1.0",
+  "version": "17.2.1",
   "type": "module",
   "author": "Parashuram <code@nparashuram.com>",
   "contributors": [
@@ -21,8 +21,8 @@
       "web-platform-tests/**",
       "coverage",
       "ignore",
-      ".eslintrc.js",
-      ".mocharc.cjs",
+      "eslint.config.js",
+      ".mocharc.js",
       "tests-mocha/**",
       "tests-polyfill/**",
       "test-support/**"
@@ -161,8 +161,8 @@
     "canvas": "^3.2.3",
     "eventtargeter": "0.9.0",
     "sync-promise-expanded": "^2.0.0",
-    "typeson": "10.0.0",
-    "typeson-registry": "13.0.0",
+    "typeson": "10.2.0",
+    "typeson-registry": "13.1.0",
     "websql-configurable": "^3.0.3"
   },
   "devDependencies": {
@@ -176,11 +176,11 @@
     "@rollup/plugin-node-resolve": "^16.0.3",
     "@rollup/plugin-terser": "^1.0.0",
     "@types/better-sqlite3": "^9.6.0",
-    "@types/node": "^26.1.2",
+    "@types/node": "^26.2.0",
     "@types/websql": "^0.0.30",
     "@unicode/unicode-17.0.0": "^1.6.17",
     "babel-plugin-add-module-exports": "1.0.4",
-    "bson": "^7.3.1",
+    "bson": "^7.3.2",
     "builtin-modules": "^5.3.0",
     "c8": "^12.0.0",
     "chai": "6.2.2",
@@ -191,9 +191,9 @@
     "coveradge": "^0.8.2",
     "cyclonejs": "1.1.4",
     "delay-cli": "^3.0.0",
-    "eslint": "^10.8.0",
-    "eslint-config-ash-nazg": "^42.1.1",
-    "globals": "^17.9.0",
+    "eslint": "^10.8.1",
+    "eslint-config-ash-nazg": "^42.5.0",
+    "globals": "^17.11.0",
     "is-date-object": "1.1.0",
     "isomorphic-fetch": "^3.0.0",
     "jsdom": "^30.0.1",
@@ -210,10 +210,9 @@
     "rollup-plugin-node-polyfills": "^0.2.1",
     "sinon": "^22.1.0",
     "source-map-support": "0.5.21",
-    "sourcemap-transformer": "git+https://github.com/brettz9/sourcemap-transformer.git",
     "typescript": "^6.0.3",
     "w3c-blob": "0.0.1",
-    "ws": "^8.21.2"
+    "ws": "^8.21.3"
   },
   "allowScripts": {
     "es5-ext@0.10.64": true,

package/src/DOMException.js

@@ -368,22 +368,22 @@ try {
 const createDOMException = useNativeDOMException
     // eslint-disable-next-line @stylistic/operator-linebreak -- Need JSDoc
     ? /**
-     * @param {string} name
-     * @param {string} message
-     * @param {ErrorLike} [error]
-     * @returns {DOMException}
-     */
+       * @param {string} name
+       * @param {string} message
+       * @param {ErrorLike} [error]
+       * @returns {DOMException}
+       */
     function (name, message, error) {
         logError(name, message, error);
         return createNativeDOMException(name, message);
     }
     // eslint-disable-next-line @stylistic/operator-linebreak -- Need JSDoc
     : /**
-    * @param {string} name
-    * @param {string} message
-    * @param {ErrorLike} [error]
-    * @returns {Error}
-    */
+       * @param {string} name
+       * @param {string} message
+       * @param {ErrorLike} [error]
+       * @returns {Error}
+       */
     function (name, message, error) {
         logError(name, message, error);
         return createNonNativeDOMException(name, message);

package/src/IDBCursor.js

@@ -482,19 +482,19 @@ IDBCursor.prototype.__findMultiEntry = function (key, primaryKey, tx, success, e
  */
 
 /**
-* @callback SuccessArg
-* @param {StructuredCloneValue} value
-* @param {import('./IDBRequest.js').IDBRequestFull} req
-* @returns {void}
-*/
+ * @callback SuccessArg
+ * @param {StructuredCloneValue} value
+ * @param {import('./IDBRequest.js').IDBRequestFull} req
+ * @returns {void}
+ */
 
 /**
-* @callback SuccessCallback
-* @param {IndexedDBKey} key
-* @param {StructuredCloneValue} value
-* @param {IndexedDBKey} primaryKey
-* @returns {void}
-*/
+ * @callback SuccessCallback
+ * @param {IndexedDBKey} key
+ * @param {StructuredCloneValue} value
+ * @param {IndexedDBKey} primaryKey
+ * @returns {void}
+ */
 
 /**
  * Creates an "onsuccess" callback.
@@ -522,11 +522,11 @@ IDBCursor.prototype.__onsuccess = function (success) {
 
 /**
  * @typedef {{
-*   matchingKey: string,
-*   key: string,
-*   [k: string]: string
-* }} RowItemNonNull
-*/
+ *   matchingKey: string,
+ *   key: string,
+ *   [k: string]: string
+ * }} RowItemNonNull
+ */
 
 /**
  *

package/src/IDBFactory.js

@@ -388,7 +388,7 @@ function IDBFactory () {
  *   __connections: {
  *     [key: string]: import('./IDBDatabase.js').IDBDatabaseFull[]
  *   }
-* }} IDBFactoryFull
+ * }} IDBFactoryFull
  */
 
 const IDBFactoryAlias = IDBFactory;
@@ -856,9 +856,17 @@ IDBFactory.prototype.deleteDatabase = function (name) {
      * @returns {boolean}
      */
     function dbError (tx, err) {
-        if (calledDBError || err === true) {
+        if (calledDBError) {
             return false;
         }
+        // Must be set synchronously (not inside the `sysdbFinishedCbDelete` callback below) to
+        //  guard against reentrancy: `sysdbFinishedCbDelete` -> `rollback`/`commit` -> websql-configurable's
+        //  `done()` -> `currentTask.errorCallback(er)`, which is this very `dbError`, calling back into
+        //  us before the first invocation has finished. Without this, a second rollback/commit gets
+        //  issued on the shared `sysdb` connection after it has already moved on to the next queued
+        //  transaction, corrupting that connection's task queue (see issue with unrelated `open()` calls
+        //  failing/crashing after a `deleteDatabase` file-removal error).
+        calledDBError = true;
         const er = webSQLErrback(/** @type {SQLError} */ (err || tx));
         sysdbFinishedCbDelete(true, function () {
             req.__done = true;
@@ -867,7 +875,6 @@ IDBFactory.prototype.deleteDatabase = function (name) {
             // Re: why bubbling here (and how cancelable is only really relevant for `window.onerror`) see: https://github.com/w3c/IndexedDB/issues/86
             const e = createEvent('error', er, {bubbles: true, cancelable: true});
             req.dispatchEvent(e);
-            calledDBError = true;
         });
         return false;
     }
@@ -970,14 +977,14 @@ IDBFactory.prototype.cmp = function (key1, key2) {
 };
 
 /**
-* May return outdated information if a database has since been deleted.
-* @see https://github.com/w3c/IndexedDB/pull/240/files
-* @this {IDBFactoryFull}
-* @returns {Promise<{
-*   name: string,
-*   version: Integer
-* }[]>}
-*/
+ * May return outdated information if a database has since been deleted.
+ * @see https://github.com/w3c/IndexedDB/pull/240/files
+ * @this {IDBFactoryFull}
+ * @returns {Promise<{
+ *   name: string,
+ *   version: Integer
+ * }[]>}
+ */
 IDBFactory.prototype.databases = function () {
     const me = this;
     let calledDbCreateError = false;
@@ -1022,17 +1029,17 @@ IDBFactory.prototype.databases = function () {
 };
 
 /**
-* @todo forceClose: Test
-* This is provided to facilitate unit-testing of the
-*  closing of a database connection with a forced flag:
-* <https://w3c.github.io/IndexedDB/#steps-for-closing-a-database-connection>
-* @param {string} dbName
-* @param {Integer} connIdx
-* @param {string} msg
-* @throws {TypeError}
-* @this {IDBFactoryFull}
-* @returns {void}
-*/
+ * @todo forceClose: Test
+ * This is provided to facilitate unit-testing of the
+ *  closing of a database connection with a forced flag:
+ * <https://w3c.github.io/IndexedDB/#steps-for-closing-a-database-connection>
+ * @param {string} dbName
+ * @param {Integer} connIdx
+ * @param {string} msg
+ * @throws {TypeError}
+ * @this {IDBFactoryFull}
+ * @returns {void}
+ */
 IDBFactory.prototype.__forceClose = function (dbName, connIdx, msg) {
     const me = this;
     /**

package/src/IDBKeyRange.js

@@ -6,11 +6,11 @@ const readonlyProperties = /** @type {const} */ (['lower', 'upper', 'lowerOpen',
 
 /**
  * @typedef {globalThis.IDBKeyRange & {
-*   __lowerCached: string|null|false,
-*   __upperCached: string|null|false,
-*   __lowerOpen: boolean,
-* }} IDBKeyRangeFull
-*/
+ *   __lowerCached: string|null|false,
+ *   __upperCached: string|null|false,
+ *   __lowerOpen: boolean,
+ * }} IDBKeyRangeFull
+ */
 
 /**
  * The IndexedDB KeyRange object.

package/src/IDBTransaction.js

@@ -776,10 +776,10 @@ IDBTransaction.__assertActive = function (tx) {
 };
 
 /**
-* Used by our `EventTarget.prototype` library to implement bubbling/capturing.
+ * Used by our `EventTarget.prototype` library to implement bubbling/capturing.
  * @this {IDBTransactionFull}
-* @returns {import('./IDBDatabase.js').IDBDatabaseFull}
-*/
+ * @returns {import('./IDBDatabase.js').IDBDatabaseFull}
+ */
 IDBTransaction.prototype.__getParent = function () {
     return this.db;
 };

package/src/Key.js

@@ -32,14 +32,14 @@ import CFG from './CFG.js';
  */
 
 /**
-* @typedef {object} KeyValueObject
-* @property {KeyType|"NaN"|"null"|"undefined"|"boolean"|"object"|"symbol"|
-*   "function"|"bigint"} type If not `KeyType`, indicates invalid value
-* @property {Value} [value]
-* @property {boolean} [invalid]
-* @property {string} [message]
-* @todo Specify acceptable `value` more precisely
-*/
+ * @typedef {object} KeyValueObject
+ * @property {KeyType|"NaN"|"null"|"undefined"|"boolean"|"object"|"symbol"|
+ *   "function"|"bigint"} type If not `KeyType`, indicates invalid value
+ * @property {Value} [value]
+ * @property {boolean} [invalid]
+ * @property {string} [message]
+ * @todo Specify acceptable `value` more precisely
+ */
 
 /**
  * @typedef {number|string|Date|ArrayBuffer} ValueTypePrimitive
@@ -474,10 +474,10 @@ function convertValueToKey (input, seen) {
 }
 
 /**
-* Currently not in use.
-* @param {Value} input
-* @returns {KeyValueObject}
-*/
+ * Currently not in use.
+ * @param {Value} input
+ * @returns {KeyValueObject}
+ */
 function convertValueToMultiEntryKey (input) {
     return convertValueToKeyValueDecoded(input, null, true, true);
 }
@@ -516,17 +516,17 @@ function getCopyBytesHeldByBufferSource (O) {
 }
 
 /**
-* Shortcut utility to avoid returning full keys from `convertValueToKey`
-*   and subsequent need to process in calling code unless `fullKeys` is
-*   set; may throw.
-* @param {Value} input
-* @param {Value[]|null} [seen]
-* @param {boolean} [multiEntry]
-* @param {boolean} [fullKeys]
-* @throws {TypeError} See `getCopyBytesHeldByBufferSource`
-* @todo Document other allowable `input`
-* @returns {KeyValueObject}
-*/
+ * Shortcut utility to avoid returning full keys from `convertValueToKey`
+ *   and subsequent need to process in calling code unless `fullKeys` is
+ *   set; may throw.
+ * @param {Value} input
+ * @param {Value[]|null} [seen]
+ * @param {boolean} [multiEntry]
+ * @param {boolean} [fullKeys]
+ * @throws {TypeError} See `getCopyBytesHeldByBufferSource`
+ * @todo Document other allowable `input`
+ * @returns {KeyValueObject}
+ */
 function convertValueToKeyValueDecoded (input, seen, multiEntry, fullKeys) {
     seen ||= [];
     if (seen.includes(input)) {
@@ -623,12 +623,12 @@ function convertValueToMultiEntryKeyDecoded (key, fullKeys) {
 }
 
 /**
-* An internal utility.
-* @param {Value} input
-* @param {Value[]|null|undefined} [seen]
-* @throws {DOMException} `DataError`
-* @returns {KeyValueObject}
-*/
+ * An internal utility.
+ * @param {Value} input
+ * @param {Value[]|null|undefined} [seen]
+ * @throws {DOMException} `DataError`
+ * @returns {KeyValueObject}
+ */
 function convertValueToKeyRethrowingAndIfInvalid (input, seen) {
     const key = convertValueToKey(input, seen);
     if (key.invalid) {
@@ -649,26 +649,26 @@ function extractKeyFromValueUsingKeyPath (value, keyPath, multiEntry) {
     return extractKeyValueDecodedFromValueUsingKeyPath(value, keyPath, multiEntry, true);
 }
 /**
-* Not currently in use.
-* @param {Value} value
-* @param {KeyPath} keyPath
-* @param {boolean} multiEntry
-* @returns {KeyPathEvaluateValue}
-*/
+ * Not currently in use.
+ * @param {Value} value
+ * @param {KeyPath} keyPath
+ * @param {boolean} multiEntry
+ * @returns {KeyPathEvaluateValue}
+ */
 function evaluateKeyPathOnValue (value, keyPath, multiEntry) {
     return evaluateKeyPathOnValueToDecodedValue(value, keyPath, multiEntry, true);
 }
 
 /**
-* May throw, return `{failure: true}` (e.g., non-object on keyPath resolution)
-*    or `{invalid: true}` (e.g., `NaN`).
-* @param {Value} value
-* @param {KeyPath} keyPath
-* @param {boolean} [multiEntry]
-* @param {boolean} [fullKeys]
-* @returns {KeyValueObject|KeyPathEvaluateValue}
-* @todo Document other possible return?
-*/
+ * May throw, return `{failure: true}` (e.g., non-object on keyPath resolution)
+ *    or `{invalid: true}` (e.g., `NaN`).
+ * @param {Value} value
+ * @param {KeyPath} keyPath
+ * @param {boolean} [multiEntry]
+ * @param {boolean} [fullKeys]
+ * @returns {KeyValueObject|KeyPathEvaluateValue}
+ * @todo Document other possible return?
+ */
 function extractKeyValueDecodedFromValueUsingKeyPath (value, keyPath, multiEntry, fullKeys) {
     const r = evaluateKeyPathOnValueToDecodedValue(value, keyPath, multiEntry, fullKeys);
     if (r.failure) {
@@ -899,11 +899,11 @@ function findMultiEntryMatches (keyEntry, range) {
 }
 
 /**
-* Not currently in use but keeping for spec parity.
-* @param {Key} key
-* @throws {Error} Upon a "bad key"
-* @returns {ValueType}
-*/
+ * Not currently in use but keeping for spec parity.
+ * @param {Key} key
+ * @throws {Error} Upon a "bad key"
+ * @returns {ValueType}
+ */
 function convertKeyToValue (key) {
     const {type, value} = key;
     switch (type) {
@@ -985,10 +985,10 @@ const MAX_ALLOWED_CURRENT_NUMBER = 9007199254740992; // 2 ^ 53 (Also equal to `N
  */
 
 /**
-* @callback SQLFailureCallback
-* @param {DOMException|Error} exception
-* @returns {void}
-*/
+ * @callback SQLFailureCallback
+ * @param {DOMException|Error} exception
+ * @returns {void}
+ */
 
 /**
  *