indexeddbshim 17.7.0 → 19.0.0

package/src/util.js

@@ -230,8 +230,32 @@ function sqlLIKEEscape (str) {
 }
 
 /**
- * @typedef {Function} AnyClass
+ * Minimal replacement for `path.join(base, name)` covering our only use case:
+ *   `name` is always a bare file name (no separators and no `.`/`..`), and
+ *   `base` is a directory path expected to be absolute and normalized (an
+ *   empty string means the current working directory). Unlike `path.join`,
+ *   this performs no `.`/`..` resolution. It reuses the separator style of
+ *   `base` (backslash only when `base` uses backslashes and no forward
+ *   slashes, i.e. a Windows path), otherwise `/`; Node's `fs`, SQLite, and
+ *   `websql-configurable` accept either separator on Windows regardless.
+ *   Avoiding `node:path` keeps browser bundles free of a path polyfill.
+ * @param {string} base
+ * @param {string} name
+ * @returns {string}
+ */
+function joinPath (base, name) {
+    if (!base) {
+        return name;
+    }
+    const sep = base.includes('\\') && !base.includes('/') ? '\\' : '/';
+    return base.replace(/[/\\]+$/u, '') + sep + name;
+}
+
+/* eslint-disable jsdoc/valid-types -- Bug */
+/**
+ * @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?)
 /**
@@ -255,7 +279,7 @@ function isObj (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isDate (obj) {
@@ -264,7 +288,7 @@ function isDate (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isBlob (obj) {
@@ -274,7 +298,7 @@ function isBlob (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isRegExp (obj) {
@@ -284,7 +308,7 @@ function isRegExp (obj) {
 
 /**
  *
- * @param {object} obj
+ * @param {AnyValue} obj
  * @returns {boolean}
  */
 function isFile (obj) {
@@ -307,7 +331,7 @@ function isBinary (obj) {
 /**
  *
  * @param {AnyValue} obj
- * @returns {boolean}
+ * @returns {obj is Iterable<unknown>}
  */
 function isIterable (obj) {
     return isObj(obj) &&
@@ -362,9 +386,7 @@ function defineReadonlyOuterInterface (obj, props) {
 
 /**
  *
- * @param {object & {
- *   [key: string]: any
- * }} obj
+ * @param {object} obj
  * @param {string[]} listeners
  * @returns {void}
  */
@@ -373,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} */ (
@@ -390,7 +412,7 @@ function defineListenerProperties (obj, listeners) {
         Object.defineProperty(obj, listener, desc);
     });
     listeners.forEach((l) => {
-        obj[l] = null;
+        Reflect.set(obj, l, null);
     });
 }
 
@@ -399,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}
  */
@@ -539,7 +561,7 @@ function enforceRange (number, type) {
 }
 
 /**
- * @typedef {any} AnyValue
+ * @typedef {unknown} AnyValue
  */
 
 /**
@@ -556,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
 }
 
 /**
@@ -637,7 +662,7 @@ function runContinuationSafely (fn) {
 export {escapeSQLiteStatement, unescapeSQLiteResponse,
     escapeDatabaseNameForSQLAndFiles, unescapeDatabaseNameForSQLAndFiles,
     escapeStoreNameForSQL, escapeIndexNameForSQL, escapeIndexNameForSQLKeyColumn,
-    sqlLIKEEscape, sqlQuote,
+    sqlLIKEEscape, sqlQuote, joinPath,
     instanceOf,
     isObj, isDate, isBlob, isRegExp, isFile, isBinary, isIterable,
     defineOuterInterface, defineReadonlyOuterInterface,