indexeddbshim 18.0.0 → 19.0.0

package/src/setGlobalVars.js

@@ -15,15 +15,11 @@ import shimIDBDatabase from './IDBDatabase.js';
 import CFG from './CFG.js';
 import {isNullish} from './util.js';
 
-/**
- * @typedef {any} AnyValue
- */
-
 /**
  * @callback SetConfig
  * @param {import('./CFG.js').KeyofConfigValues|
  *   Partial<import('./CFG.js').ConfigValues>} prop
- * @param {AnyValue} [val]
+ * @param {import('./CFG.js').ConfigValue} [val]
  * @throws {Error}
  * @returns {void}
  */
@@ -114,18 +110,9 @@ function setGlobalVars (idb, initialConfig) {
     const IDB = /** @type {ShimmedObject & {[key: string]: unknown}} */ (
         /** @type {unknown} */ (idb || globalThis || {})
     );
-    /**
-     * @typedef {any} AnyClass
-     */
-    /**
-     * @typedef {any} AnyValue
-     */
-    /**
-     * @typedef {Function} AnyFunction
-     */
     /**
      * @param {string} name
-     * @param {AnyClass} value
+     * @param {unknown} value
      * @param {PropertyDescriptor & {
      *   shimNS?: object
      * }|undefined} [propDesc]
@@ -157,10 +144,10 @@ function setGlobalVars (idb, initialConfig) {
                 } else {
                     const o = {
                         /**
-                         * @returns {AnyValue}
+                         * @returns {unknown}
                          */
                         get [name] () {
-                            return /** @type {AnyFunction} */ (
+                            return /** @type {() => unknown} */ (
                                 /** @type {PropertyDescriptor} */ (
                                     propDesc
                                 ).get
@@ -239,7 +226,7 @@ function setGlobalVars (idb, initialConfig) {
                         return shimIndexedDB;
                     }
                 });
-                /** @type {[string, any][]} */
+                /** @type {[string, unknown][]} */
                 ([
                     ['IDBFactory', shimIDBFactory],
                     ['IDBDatabase', shimIDBDatabase],
@@ -339,7 +326,9 @@ function setGlobalVars (idb, initialConfig) {
             '__useShim', '__debug', '__setConfig',
             '__getConfig', '__setUnicodeIdentifiers'
         ]).forEach((prop) => {
-            /** @type {ShimIndexedDB} */ (IDB.shimIndexedDB)[prop] = /** @type {() => any} */ function () {
+            /** @type {{[key: string]: () => void}} */ (
+                /** @type {unknown} */ (IDB.shimIndexedDB)
+            )[prop] = function () {
                 console.warn('This browser does not have WebSQL to shim.');
             };
         });

package/src/util.js

@@ -251,9 +251,11 @@ function joinPath (base, name) {
     return base.replace(/[/\\]+$/u, '') + sep + name;
 }
 
+/* eslint-disable jsdoc/valid-types -- Bug */
 /**
- * @typedef {Function} AnyClass
+ * @typedef {{[Symbol.hasInstance]: (value: unknown) => boolean}} AnyClass
  */
+/* eslint-enable jsdoc/valid-types -- Bug */
 
 // Babel doesn't seem to provide a means of using the `instanceof` operator with Symbol.hasInstance (yet?)
 /**
@@ -277,7 +279,7 @@ function isObj (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isDate (obj) {
@@ -286,7 +288,7 @@ function isDate (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isBlob (obj) {
@@ -296,7 +298,7 @@ function isBlob (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isRegExp (obj) {
@@ -306,7 +308,7 @@ function isRegExp (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isFile (obj) {
@@ -329,7 +331,7 @@ function isBinary (obj) {
 /**
  *
  * @param {AnyValue} obj
- * @returns {boolean}
+ * @returns {obj is Iterable<unknown>}
  */
 function isIterable (obj) {
     return isObj(obj) &&
@@ -384,9 +386,7 @@ function defineReadonlyOuterInterface (obj, props) {
 
 /**
  *
- * @param {object & {
- *   [key: string]: any
- * }} obj
+ * @param {object} obj
  * @param {string[]} listeners
  * @returns {void}
  */
@@ -395,13 +395,13 @@ function defineListenerProperties (obj, listeners) {
     listeners.forEach((listener) => {
         const o = {
             get [listener] () {
-                return obj['__' + listener];
+                return Reflect.get(obj, '__' + listener);
             },
             /**
              * @param {AnyValue} val
              */
             set [listener] (val) {
-                obj['__' + listener] = val;
+                Reflect.set(obj, '__' + listener, val);
             }
         };
         const desc = /** @type {PropertyDescriptor} */ (
@@ -412,7 +412,7 @@ function defineListenerProperties (obj, listeners) {
         Object.defineProperty(obj, listener, desc);
     });
     listeners.forEach((l) => {
-        obj[l] = null;
+        Reflect.set(obj, l, null);
     });
 }
 
@@ -421,7 +421,7 @@ function defineListenerProperties (obj, listeners) {
  * @param {object} obj
  * @param {string|string[]} props
  * @param {null|{
- *   [key: string]: any
+ *   [key: string]: unknown
  * }} getter
  * @returns {void}
  */
@@ -561,7 +561,7 @@ function enforceRange (number, type) {
 }
 
 /**
- * @typedef {any} AnyValue
+ * @typedef {unknown} AnyValue
  */
 
 /**
@@ -578,8 +578,11 @@ function convertToDOMString (v, treatNullAs) {
  * @returns {string}
  */
 function ToString (o) { // Todo: See `es-abstract/es7`
-    // `String()` will not throw with Symbols
-    return '' + o; // eslint-disable-line no-implicit-coercion -- Need to throw with symbols
+    // `String()` will not throw with Symbols, but the unchecked `+ ''`
+    //   coercion deliberately does (Web IDL requires converting a symbol
+    //   to `DOMString` to throw a `TypeError`); the assertion only tells
+    //   the compiler to allow the coercion it cannot model.
+    return '' + /** @type {string} */ (o); // eslint-disable-line no-implicit-coercion -- Need to throw with symbols
 }
 
 /**