indexeddbshim 17.7.0 → 19.0.0

package/src/IDBCursor.js

@@ -23,16 +23,16 @@ const cursorDirections = ['next', 'prev', 'nextunique', 'prevunique'];
 
 /**
  * @typedef {IDBCursor & {
- *   primaryKey: import('./Key.js').Key,
- *   key:  import('./Key.js').Key,
+ *   primaryKey: import('./Key.js').Key|null|undefined,
+ *   key:  import('./Key.js').Key|null|undefined,
  *   direction: string,
  *   source: import('./IDBObjectStore.js').IDBObjectStoreFull|
  *     import('./IDBIndex.js').IDBIndexFull,
  *   __request: import('./IDBRequest.js').IDBRequestFull,
  *   __advanceCount: Integer|undefined,
  *   __indexSource: boolean,
- *   __key: import('./Key.js').Key,
- *   __primaryKey: import('./Key.js').Key,
+ *   __key: import('./Key.js').Key|null|undefined,
+ *   __primaryKey: import('./Key.js').Key|null|undefined,
  *   __value: import('./Key.js').Value,
  *   __store: import('./IDBObjectStore.js').IDBObjectStoreFull,
  *   __range: import('./IDBKeyRange.js').IDBKeyRangeFull|undefined,
@@ -40,11 +40,14 @@ const cursorDirections = ['next', 'prev', 'nextunique', 'prevunique'];
  *   __valueColumnName: string,
  *   __keyOnly: boolean,
  *   __valueDecoder: {
- *     decode: (str: string) => any,
+ *     decode: (str: string) => import('./Key.js').Value,
  *   },
  *   __count: boolean,
  *   __prefetchedIndex: Integer,
- *   __prefetchedData: null|SQLResultSetRowList|{
+ *   __prefetchedData: null|{
+ *     length: number;
+ *     item(index: number): unknown;
+ *   }|{
  *     data: RowItemNonNull[],
  *     length: Integer,
  *     item: (index: Integer) => RowItemNonNull
@@ -58,7 +61,14 @@ const cursorDirections = ['next', 'prev', 'nextunique', 'prevunique'];
  *   __multiEntryExhausted: boolean,
  *   __invalidateCache: () => void,
  *   __gotValue: boolean,
- *   __find: (...args: any[]) => void,
+ *   __find: (
+ *     key: import('./Key.js').Key|undefined,
+ *     primaryKey: import('./Key.js').Key|undefined,
+ *     tx: WebSQLTransaction,
+ *     success: KeySuccess,
+ *     error: FindError,
+ *     recordsToLoad?: Integer
+ *   ) => void,
  *   __findBasic: (
  *     key: import('./Key.js').Key|undefined,
  *     primaryKey: import('./Key.js').Key|undefined,
@@ -79,17 +89,17 @@ const cursorDirections = ['next', 'prev', 'nextunique', 'prevunique'];
  *   __decode: (
  *     rowItem: RowItemNonNull,
  *     callback: (
- *       key: import('./Key.js').Key,
+ *       key: import('./Key.js').Key|undefined,
  *       val: import('./Key.js').Value,
- *       primaryKey: import('./Key.js').Key,
+ *       primaryKey: import('./Key.js').Key|undefined,
  *       encKey?: string
  *     ) => void
  *   ) => void,
  *   __sourceOrEffectiveObjStoreDeleted: () => void,
  *   __continue: (key?: import('./Key.js').Key, advanceContinue?: boolean) => void,
  *   __continueFinish: (
- *     key: import('./Key.js').Key,
- *     primaryKey: import('./Key.js').Key,
+ *     key: import('./Key.js').Key|undefined,
+ *     primaryKey: import('./Key.js').Key|undefined,
  *     advanceState: boolean
  *   ) => void
  * }} IDBCursorFull
@@ -122,7 +132,7 @@ const IDBCursorAlias = IDBCursor;
  *   import('./IDBIndex.js').IDBIndexFull} source
  * @param {string} keyColumnName
  * @param {string} valueColumnName
- * @param {boolean} count
+ * @param {boolean} [count]
  * @this {IDBCursorFull}
  * @returns {void}
  */
@@ -161,7 +171,7 @@ IDBCursor.__super = function IDBCursor (query, direction, store, source, keyColu
     this.__valueColumnName = valueColumnName;
     this.__keyOnly = valueColumnName === 'key';
     this.__valueDecoder = this.__keyOnly ? Key : Sca;
-    this.__count = count;
+    this.__count = Boolean(count);
     this.__prefetchedIndex = -1;
     this.__continuationKey = undefined;
     this.__continuationPrimaryKey = undefined;
@@ -183,25 +193,38 @@ IDBCursor.__super = function IDBCursor (query, direction, store, source, keyColu
 
 /**
  *
- * @param {...any} args
+ * @param {IDBKeyRange} query
+ * @param {string} direction
+ * @param {import('./IDBObjectStore.js').IDBObjectStoreFull} store
+ * @param {import('./IDBObjectStore.js').IDBObjectStoreFull|
+ *   import('./IDBIndex.js').IDBIndexFull} source
+ * @param {string} keyColumnName
+ * @param {string} valueColumnName
+ * @param {boolean} [count]
  * @returns {IDBCursorFull}
  */
-IDBCursor.__createInstance = function (...args) {
+IDBCursor.__createInstance = function (query, direction, store, source, keyColumnName, valueColumnName, count) {
     const IDBCursor = IDBCursorAlias.__super;
     IDBCursor.prototype = IDBCursorAlias.prototype;
 
     // @ts-expect-error It's ok
-    return new IDBCursor(...args);
+    return new IDBCursor(
+        query, direction, store, source, keyColumnName, valueColumnName, count
+    );
 };
 
 /**
  *
- * @param {...any} args
+ * @param {import('./Key.js').Key|undefined} key
+ * @param {import('./Key.js').Key|undefined} primaryKey
+ * @param {WebSQLTransaction} tx
+ * @param {KeySuccess} success
+ * @param {FindError} error
+ * @param {Integer} [recordsToLoad]
  * @this {IDBCursorFull}
  * @returns {void}
  */
-IDBCursor.prototype.__find = function (...args /* key, tx, success, error, recordsToLoad */) {
-    const [key, primaryKey, tx, success, error, recordsToLoad] = args;
+IDBCursor.prototype.__find = function (key, primaryKey, tx, success, error, recordsToLoad) {
     if (this.__multiEntryIndex) {
         this.__findMultiEntry(key, primaryKey, tx, success, error, recordsToLoad);
     } else {
@@ -211,9 +234,9 @@ IDBCursor.prototype.__find = function (...args /* key, tx, success, error, recor
 
 /**
  * @typedef {(
- *   k: import('./Key.js').Key,
+ *   k: import('./Key.js').Key|undefined,
  *   val: import('./Key.js').Value,
- *   primKey: import('./Key.js').Key
+ *   primKey: import('./Key.js').Key|undefined
  * ) => void} KeySuccess
  */
 
@@ -319,7 +342,7 @@ IDBCursor.prototype.__findBasic = function (key, primaryKey, tx, success, error,
         } else if (data.rows.length > 1) {
             me.__prefetchedIndex = 0;
             me.__prefetchedData = data.rows;
-            if (CFG.DEBUG) { console.log('Preloaded ' + me.__prefetchedData.length + ' records for cursor'); }
+            if (CFG.DEBUG) { console.log('Preloaded ' + data.rows.length + ' records for cursor'); }
             me.__decode(/** @type {RowItemNonNull} */ (data.rows.item(0)), success);
         } else if (data.rows.length === 1) {
             me.__decode(/** @type {RowItemNonNull} */ (data.rows.item(0)), success);
@@ -449,7 +472,9 @@ IDBCursor.prototype.__findMultiEntry = function (key, primaryKey, tx, success, e
                 for (let i = 0; i < data.rows.length; i++) {
                     const rowItem = /** @type {RowItemNonNull} */ (data.rows.item(i));
                     const rowKey = Key.decode(rowItem[me.__keyColumnName], true);
-                    const matches = Key.findMultiEntryMatches(rowKey, me.__range);
+                    const matches = Key.findMultiEntryMatches(
+                        /** @type {import('./Key.js').Key} */ (rowKey), me.__range
+                    );
                     ct += matches.length;
                 }
                 success(undefined, ct, undefined);
@@ -470,7 +495,9 @@ IDBCursor.prototype.__findMultiEntry = function (key, primaryKey, tx, success, e
             for (let i = 0; i < data.rows.length; i++) {
                 const rowItem = /** @type {RowItemNonNull} */ (data.rows.item(i));
                 const rowKey = Key.decode(rowItem[me.__keyColumnName], true);
-                const matches = Key.findMultiEntryMatches(rowKey, me.__range);
+                const matches = Key.findMultiEntryMatches(
+                    /** @type {import('./Key.js').Key} */ (rowKey), me.__range
+                );
 
                 for (const matchingKey of matches) {
                     /**
@@ -542,11 +569,11 @@ IDBCursor.prototype.__findMultiEntry = function (key, primaryKey, tx, success, e
 };
 
 /**
- * @typedef {any} StructuredCloneValue
+ * @typedef {import('./Key.js').Value} StructuredCloneValue
  */
 
 /**
- * @typedef {any} IndexedDBKey
+ * @typedef {import('./Key.js').Key} IndexedDBKey
  */
 
 /**
@@ -558,9 +585,9 @@ IDBCursor.prototype.__findMultiEntry = function (key, primaryKey, tx, success, e
 
 /**
  * @callback SuccessCallback
- * @param {IndexedDBKey} key
+ * @param {IndexedDBKey|undefined} key
  * @param {StructuredCloneValue} value
- * @param {IndexedDBKey} primaryKey
+ * @param {IndexedDBKey|undefined} primaryKey
  * @returns {void}
  */
 
@@ -600,9 +627,9 @@ IDBCursor.prototype.__onsuccess = function (success) {
  *
  * @param {RowItemNonNull} rowItem
  * @param {(
- *   key: import('./Key.js').Key,
+ *   key: import('./Key.js').Key|undefined,
  *   val: import('./Key.js').Value,
- *   primaryKey: import('./Key.js').Key,
+ *   primaryKey: import('./Key.js').Key|undefined,
  *   encKey?: string
  * ) => void} callback
  * @this {IDBCursorFull}
@@ -691,8 +718,8 @@ IDBCursor.prototype.__continue = function (key, advanceContinue) {
 
 /**
  *
- * @param {import('./Key.js').Key} key
- * @param {import('./Key.js').Key} primaryKey
+ * @param {import('./Key.js').Key|undefined} key
+ * @param {import('./Key.js').Key|undefined} primaryKey
  * @param {boolean} advanceState
  * @this {IDBCursorFull}
  * @returns {void}
@@ -707,9 +734,9 @@ IDBCursor.prototype.__continueFinish = function (key, primaryKey, advanceState)
         me.__store.transaction
     ).__pushToQueue(me.__request, function cursorContinue (tx, args, success, error, executeNextRequest) {
         /**
-         * @param {import('./Key.js').Key} k
+         * @param {import('./Key.js').Key|undefined} k
          * @param {import('./Key.js').Value} val
-         * @param {import('./Key.js').Key} primKey
+         * @param {import('./Key.js').Key|undefined} primKey
          * @returns {void}
          */
         function triggerSuccess (k, val, primKey) {
@@ -752,7 +779,7 @@ IDBCursor.prototype.__continueFinish = function (key, primaryKey, advanceState)
             // We have pre-loaded data for the cursor
             me.__prefetchedIndex++;
             if (me.__prefetchedIndex < me.__prefetchedData.length) {
-                me.__decode(me.__prefetchedData.item(me.__prefetchedIndex), function (k, val, primKey, encKey) {
+                me.__decode(/** @type {RowItemNonNull} */ (me.__prefetchedData.item(me.__prefetchedIndex)), function (k, val, primKey, encKey) {
                     /**
                      * @returns {void}
                      */
@@ -901,12 +928,18 @@ IDBCursor.prototype.advance = function (count) {
 };
 
 /**
- * @typedef {any} AnyValue
+ * The `{query, count, direction}` options shape shared by
+ *   `getAll`/`getAllKeys`/`getAllRecords`.
+ * @typedef {{
+ *   query?: import('./Key.js').Value,
+ *   count?: Integer,
+ *   direction?: string
+ * }} GetAllOptions
  */
 
 /**
  *
- * @param {AnyValue} valueToUpdate
+ * @param {import('./Key.js').Value} valueToUpdate
  * @this {IDBCursorFull}
  * @returns {IDBRequest}
  */
@@ -1045,16 +1078,22 @@ Object.defineProperty(IDBCursorWithValue.prototype, 'constructor', {
 const IDBCursorWithValueAlias = IDBCursorWithValue;
 /**
  *
- * @param {...any} args
+ * @param {IDBKeyRange} query
+ * @param {string} direction
+ * @param {import('./IDBObjectStore.js').IDBObjectStoreFull} store
+ * @param {import('./IDBObjectStore.js').IDBObjectStoreFull|
+ *   import('./IDBIndex.js').IDBIndexFull} source
+ * @param {string} keyColumnName
+ * @param {string} valueColumnName
+ * @param {boolean} [count]
  * @returns {IDBCursorWithValueFull}
  */
-IDBCursorWithValue.__createInstance = function (...args) {
+IDBCursorWithValue.__createInstance = function (query, direction, store, source, keyColumnName, valueColumnName, count) {
     /**
      * @class
      * @this {IDBCursorWithValueFull}
      */
     function IDBCursorWithValue () {
-        const [query, direction, store, source, keyColumnName, valueColumnName, count] = args;
         IDBCursor.__super.call(this, query, direction, store, source, keyColumnName, valueColumnName, count);
         // @ts-expect-error It's ok
         this[Symbol.toStringTag] = 'IDBCursorWithValue';
@@ -1078,12 +1117,16 @@ Object.defineProperty(IDBCursorWithValue, 'prototype', {
 /**
  * Validates `direction` and fills in defaults for the `{query, count,
  *   direction}` shape shared by `getAll`/`getAllKeys`/`getAllRecords`.
- * @param {AnyValue} options
+ * @param {unknown} options
  * @throws {TypeError}
- * @returns {{query: AnyValue, count: Integer|undefined, direction: string}}
+ * @returns {{
+ *   query: import('./Key.js').Value|undefined,
+ *   count: Integer|undefined,
+ *   direction: string
+ * }}
  */
 function normalizeGetAllOptions (options) {
-    const {query, count, direction = 'next'} = /** @type {AnyValue} */ (options ?? {});
+    const {query, count, direction = 'next'} = /** @type {GetAllOptions} */ (options ?? {});
     if (!cursorDirections.includes(direction)) {
         throw new TypeError('`' + direction + '` is not a valid direction');
     }
@@ -1104,7 +1147,11 @@ function normalizeGetAllOptions (options) {
  *   first argument wins regardless of how many arguments were actually passed.
  * @param {IArguments} args
  * @throws {TypeError}
- * @returns {{query: AnyValue, count: Integer|undefined, direction: string}}
+ * @returns {{
+ *   query: import('./Key.js').Value|undefined,
+ *   count: Integer|undefined,
+ *   direction: string
+ * }}
  */
 function parseGetAllArgs (args) {
     const arg0 = args[0];
@@ -1124,7 +1171,11 @@ function parseGetAllArgs (args) {
  *   form to disambiguate against.
  * @param {IArguments} args
  * @throws {TypeError}
- * @returns {{query: AnyValue, count: Integer|undefined, direction: string}}
+ * @returns {{
+ *   query: import('./Key.js').Value|undefined,
+ *   count: Integer|undefined,
+ *   direction: string
+ * }}
  */
 function parseGetAllRecordsArgs (args) {
     return normalizeGetAllOptions(args[0]);
@@ -1231,7 +1282,7 @@ function collectAll (source, query, count, direction, mode) {
     Object.defineProperty(state, 'direction', {writable: false, value: direction});
 
     return tx.__addToTransactionQueue(function collectAllOp (sqlTx, args, success, error) {
-        /** @type {AnyValue[]} */
+        /** @type {unknown[]} */
         const results = [];
         const recordsToLoad = count || CFG.cursorPreloadPackSize || 100;
 
@@ -1272,7 +1323,7 @@ function collectAll (source, query, count, direction, mode) {
                 state.__prefetchedIndex++;
                 if (state.__prefetchedIndex < state.__prefetchedData.length) {
                     IDBCursor.prototype.__decode.call(
-                        state, state.__prefetchedData.item(state.__prefetchedIndex),
+                        state, /** @type {RowItemNonNull} */ (state.__prefetchedData.item(state.__prefetchedIndex)),
                         /**
                          * @param {import('./Key.js').Key} k
                          * @param {import('./Key.js').Value} val

package/src/IDBFactory.js

@@ -1,6 +1,4 @@
 /* eslint-disable sonarjs/no-invariant-returns -- Convenient here */
-// eslint-disable-next-line no-restricted-imports -- Can be polyfilled
-import path from 'node:path';
 import SyncPromise from 'sync-promise-expanded';
 
 import {createEvent} from './Event.js';
@@ -367,7 +365,7 @@ function cleanupDatabaseResources (__openDatabase, name, escapedDatabaseName, da
     }
     if (fs && CFG.deleteDatabaseFiles !== false) {
         closeCachedWebSQLConnections(name, () => {
-            fs.unlink(path.join(CFG.databaseBasePath || '', escapedDatabaseName), (err) => {
+            fs.unlink(util.joinPath(CFG.databaseBasePath || '', escapedDatabaseName), (err) => {
                 if (err && err.code !== 'ENOENT') { // Ignore if file is already deleted
                     const removalError = /** @type {Error & {code?: number}} */ (
                         new Error('Error removing database file: ' + escapedDatabaseName + ' ' + err)
@@ -383,7 +381,7 @@ function cleanupDatabaseResources (__openDatabase, name, escapedDatabaseName, da
     }
 
     const sqliteDB = __openDatabase(
-        path.join(CFG.databaseBasePath || '', escapedDatabaseName),
+        util.joinPath(CFG.databaseBasePath || '', escapedDatabaseName),
         '1',
         name,
         CFG.DEFAULT_DB_SIZE
@@ -451,7 +449,7 @@ function createSysDB (__openDatabase, success, failure) {
         sysdb = __openDatabase(
             typeof CFG.memoryDatabase === 'string'
                 ? CFG.memoryDatabase
-                : path.join(
+                : util.joinPath(
                     (typeof CFG.sysDatabaseBasePath === 'string'
                         ? CFG.sysDatabaseBasePath
                         : (CFG.databaseBasePath || '')),
@@ -647,7 +645,7 @@ IDBFactory.prototype.open = function (name /* , version */) {
                     let sysdbFinishedCb = function (systx, err, cb) {
                         if (err) {
                             /**
-                             * @param {any} [errorToShow]
+                             * @param {unknown} [errorToShow]
                              * @returns {void}
                              */
                             const manualRevert = function (errorToShow) {
@@ -681,7 +679,10 @@ IDBFactory.prototype.open = function (name /* , version */) {
                                     },
                                     function () {
                                         isRevertingSysdb = false;
-                                        cb(errorToShow || reportError); // eslint-disable-line promise/no-callback-in-promise -- Convenient
+                                        // eslint-disable-next-line promise/no-callback-in-promise -- Convenient
+                                        cb(/** @type {(Error & {code?: number})|undefined} */ (
+                                            errorToShow || reportError
+                                        ));
                                     }
                                 );
                             };
@@ -720,14 +721,14 @@ IDBFactory.prototype.open = function (name /* , version */) {
                                 new IDBVersionChangeEvent('upgradeneeded', {oldVersion, newVersion: version})
                             );
                             req.__result = connection;
-                            connection.__upgradeTransaction = req.__transaction = req.__result.__versionTransaction = IDBTransaction.__createInstance(req.__result, req.__result.objectStoreNames, 'versionchange');
+                            connection.__upgradeTransaction = req.__transaction = connection.__versionTransaction = IDBTransaction.__createInstance(connection, connection.objectStoreNames, 'versionchange');
                             req.__done = true;
 
                             req.transaction.__addNonRequestToTransactionQueue(
                                 /**
                                  * @param {import('websql-configurable/lib/websql/WebSQLTransaction.js').default} tx
-                                 * @param {ObjectArray} args
-                                 * @param {(result?: any, req?: import('./IDBRequest.js').IDBRequestFull) => void} finished
+                                 * @param {unknown[]} args
+                                 * @param {(result?: unknown, req?: import('./IDBRequest.js').IDBRequestFull) => void} finished
                                  * @returns {void}
                                  */
                                 function onupgradeneeded (tx, args, finished /* , error */) {
@@ -963,7 +964,7 @@ IDBFactory.prototype.open = function (name /* , version */) {
             db = websqlDBCache[name][version];
         } else {
             db = /** @type {DatabaseFull} */ (me.__openDatabase(
-                useMemoryDatabase ? CFG.memoryDatabase : path.join(CFG.databaseBasePath || '', escapedDatabaseName),
+                useMemoryDatabase ? CFG.memoryDatabase : util.joinPath(CFG.databaseBasePath || '', escapedDatabaseName),
                 '1',
                 name,
                 CFG.DEFAULT_DB_SIZE

package/src/IDBIndex.js

@@ -68,7 +68,7 @@ const IDBIndexAlias = IDBIndex;
  *   __keyPath: import('./Key.js').KeyPath,
  *   __recreated?: boolean,
  *   __fetchIndexData: (
- *     range: any,
+ *     range: Query,
  *     opType: "value"|"key"|"count",
  *     nullDisallowed: boolean,
  *     count?: number
@@ -497,7 +497,7 @@ IDBIndex.__updateIndexList = function (store, tx, success, failure) {
 };
 
 /**
- * @typedef {any|IDBKeyRange} Query
+ * @typedef {import('./Key.js').Value|IDBKeyRange} Query
  */
 
 /**
@@ -766,7 +766,7 @@ IDBIndex.prototype.__renameIndex = function (store, oldName, newName, colInfoToP
                             })
                         );
                         SyncPromise.all(indexCreations).then(finish).catch(
-                            /** @type {(reason: any) => PromiseLike<never>} */
+                            /** @type {(reason: unknown) => PromiseLike<never>} */
                             (error)
                         ).catch((err) => {
                             console.log('Index rename error');
@@ -779,14 +779,10 @@ IDBIndex.prototype.__renameIndex = function (store, oldName, newName, colInfoToP
     });
 };
 
-/**
- * @typedef {any} AnyValue
- */
-
 /* eslint-disable unicorn/no-top-level-side-effects -- Would be good */
 Object.defineProperty(IDBIndex, Symbol.hasInstance, {
     /**
-     * @param {AnyValue} obj
+     * @param {unknown} obj
      * @returns {boolean}
      */
     value: (obj) => util.isObj(obj) &&
@@ -819,7 +815,9 @@ Object.defineProperty(IDBIndex, 'prototype', {
  * @param {string[]} sqlValues
  * @param {WebSQLTransaction} tx
  * @param {null|undefined} args
- * @param {(result: number|undefined|[]|AnyValue|AnyValue[]) => void} success
+ * @param {(
+ *   result: number|undefined|[]|import('./Key.js').Value|import('./Key.js').Value[]
+ * ) => void} success
  * @param {(tx: WebSQLTransaction, err: (Error & {code?: number})) => void} error
  * @returns {void}
  */
@@ -859,7 +857,7 @@ function executeFetchIndexData (
                  * @param {{
                  *   value: string
                  * }} record
-                 * @returns {AnyValue}
+                 * @returns {import('./Key.js').Value}
                  */
                 (record) => { // when opType is value
                     return Sca.decode(util.unescapeSQLiteResponse(record.value));
@@ -874,9 +872,9 @@ function executeFetchIndexData (
                 );
                 let record;
                 if (hasKey && (
-                    (multiChecks && range.some(
+                    (multiChecks && /** @type {import('./Key.js').ValueType[]} */ (range).some(
                         /**
-                         * @param {string} check
+                         * @param {import('./Key.js').ValueType} check
                          * @returns {boolean}
                          */
                         (check) => rowKey.includes(check)

package/src/IDBKeyRange.js

@@ -27,8 +27,8 @@ function IDBKeyRange () {
 const IDBKeyRangeAlias = IDBKeyRange;
 
 /**
- * @param {import('./Key.js').Key|null} lower
- * @param {import('./Key.js').Key|null} upper
+ * @param {import('./Key.js').Value} lower
+ * @param {import('./Key.js').Value} upper
  * @param {boolean} lowerOpen
  * @param {boolean} upperOpen
  * @returns {import('./IDBKeyRange.js').IDBKeyRangeFull}
@@ -238,10 +238,11 @@ function setSQLForKeyRange (
 function convertValueToKeyRange (value, nullDisallowed) {
     if (util.instanceOf(value, IDBKeyRange)) {
         // We still need to validate IDBKeyRange-like objects (the above check is based on loose duck-typing)
-        if (value.toString() !== '[object IDBKeyRange]') {
-            return IDBKeyRange.__createInstance(value.lower, value.upper, value.lowerOpen, value.upperOpen);
+        const range = /** @type {IDBKeyRangeFull} */ (value);
+        if (range.toString() !== '[object IDBKeyRange]') {
+            return IDBKeyRange.__createInstance(range.lower, range.upper, range.lowerOpen, range.upperOpen);
         }
-        return value;
+        return range;
     }
     if (util.isNullish(value)) {
         if (nullDisallowed) {