indexeddbshim 14.0.0 → 15.0.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "indexeddbshim",
-  "version": "14.0.0",
+  "version": "15.0.1",
   "type": "module",
   "author": "Parashuram <code@nparashuram.com>",
   "contributors": [
@@ -28,78 +28,6 @@
       "test-support/**"
     ]
   },
-  "scripts": {
-    "tsc": "tsc",
-    "tsc-prod": "tsc -p tsconfig-prod.json",
-    "build": "npm run rollup && npm run tsc-prod",
-    "prepublishOnly": "npm run license-badges",
-    "license-badge": "license-badger --corrections --uncategorizedLicenseTemplate \"\\${license} (\\${name} (\\${version}))\" --filteredTypes=nonempty --textTemplate \"License types\n(project, deps, and bundled devDeps)\" --packageJson --production badges/licenses-badge.svg",
-    "license-badge-dev": "license-badger --corrections --filteredTypes=nonempty --textTemplate \"License types\n(all devDeps)\" --allDevelopment badges/licenses-badge-dev.svg",
-    "license-badges": "run-p license-badge license-badge-dev",
-    "start": "static -p 9999 -H '{\"Access-Control-Allow-Origin\": \"http://web-platform.test:8000\", \"Access-Control-Allow-Methods\": \"GET\", \"Access-Control-Allow-Headers\": \"Content-Type\"}'",
-    "watch": "chokidar \"src/*\" \"node_modules/eventtarget/EventTarget.js\" \"node_modules/websql/lib/websql/WebSQLTransaction.js\" \"node_modules/websql/lib/websql/WebSQLDatabase.js\" -c \"npm run rollup\"",
-    "dev": "run-p start watch",
-    "clean": "run-p clean-sysDB clean-mocha clean-fake clean-mock clean-w3c clean-w3cOld",
-    "clean-sysDB": "rimraf __sysdb__*",
-    "clean-polyfill": "run-p clean-fake clean-mock clean-w3cOld clean-sysDB",
-    "clean-mocha": "rimraf D_indexeddbshim_test_database_* D_test.sqlite",
-    "clean-fake": "rimraf D_test0.*",
-    "clean-mock": "rimraf D_test_database*",
-    "clean-w3c": "rimraf D_^D^B2* D_db* D_test* D_about* D_^I^D^B* D_blank* D_database_name* D_idbtransaction* D_x D_x.sqlite D_y D_y.sqlite D_webworker* D_http*",
-    "clean-w3cOld": "rimraf D_db.sqlite* D_test* D_database_name* D_idbtransaction*",
-    "copy": "copyup node_modules/core-js-bundle/minified.js node_modules/source-map-support/browser-source-map-support.js node_modules/mocha/mocha.css node_modules/mocha/mocha.js node_modules/chai/chai.js node_modules/sinon/pkg/sinon-no-sourcemaps.js test-support",
-    "lint": "eslint --ext=js,md,html .",
-    "eslint": "npm run lint",
-    "sqlite-rebuild": "cd node_modules/sqlite3 && npm i",
-    "w3c-build-only": "node test-support/node-buildjs",
-    "w3c-build": "run-s clean-w3c w3c-build-only",
-    "fake": "run-s clean-fake coverage:fake clean-fake",
-    "mock": "run-s clean-mock coverage:mock clean-mock",
-    "w3c-old": "run-s clean-w3cOld coverage:w3cOld clean-w3cOld",
-    "tests-polyfill": "run-s clean-polyfill coverage:fake coverage:mock coverage:w3cOld",
-    "mocha": "run-s clean-mocha coverage:mocha clean-mocha",
-    "rollup": "rollup -c",
-    "coverage:fake": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+fake --require source-map-support/register tests-polyfill/fakeIndexedDB/test-node.js",
-    "coverage:mock": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+mock --require source-map-support/register tests-polyfill/indexedDBmock/test-node.js",
-    "coverage:w3cOld": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+w3c --require source-map-support/register tests-polyfill/w3c/test-node.js",
-    "coverage:w3c": "NODE_OPTIONS=--max_old_space_size=16384 c8 --silent --no-clean npm run w3c-test-only",
-    "test:w3c": "NODE_OPTIONS=--max_old_space_size=16384 NODE_OPTIONS=--trace-warnings node test-support/node-idb-test.js",
-    "w3c-test-only": "node --trace-warnings test-support/node-idb-test.js",
-    "coverage:mocha": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+mocha --require source-map-support/register tests-mocha/test-node.js",
-    "coverage-badge": "coveradge badges/coverage-badge",
-    "coverage:report-only": "c8 report",
-    "coverage:report": "run-s coverage:report-only",
-    "remove-cache": "rimraf node_modules/.cache",
-    "remove-cov": "rimraf .nyc_output ",
-    "remove-results": "rimraf test-support/results/file*.json",
-    "coverage:polyfill": "run-s coverage:fake coverage:mock coverage:w3cOld",
-    "tests-badge": "mbg --fileGlob \"./test-support/results/file*.json\" --badge_output badges/tests-badge.svg",
-    "coverage": "run-s -c clean remove-cache remove-cov remove-results coverage:fake coverage:mock coverage:mocha coverage:w3c clean tests-badge coverage-badge coverage:report",
-    "open-coverage-only": "open http://127.0.0.1:9999/coverage/lcov-report/",
-    "open-coverage": "run-p start open-coverage-only",
-    "test": "run-s lint coverage",
-    "submodules-init": "git submodule update --init --recursive",
-    "submodules-fetch": "git submodule foreach --recursive git fetch",
-    "submodules-merge": "git submodule foreach git merge origin master",
-    "submodules": "run-s submodules-init submodules-fetch submodules-merge",
-    "wpt": "run-p start web-platform-tests",
-    "web-platform-tests": "cd web-platform-tests && ./wpt serve",
-    "w3c-find-unused": "node test-support/find-unused.js",
-    "w3c": "run-s w3c-build w3c-test",
-    "w3c-add-wrap": "node test-support/wrap-w3c-tests",
-    "w3c-remove-wrap": "node test-support/wrap-w3c-tests remove",
-    "w3c-wrap": "node test-support/wrap-w3c-tests",
-    "test-and-start:w3c": "run-p -r start web-platform-tests delayed-w3c-test",
-    "delay": "delay 9",
-    "delayed-w3c-test": "run-s delay test:w3c",
-    "w3c-test": "run-s clean-w3c test-and-start:w3c",
-    "w3c-good-only": "node test-support/node-idb-test.js good",
-    "w3c-bad-only": "node test-support/node-idb-test.js bad",
-    "w3c-good": "run-s w3c-build w3c-good-only clean-w3c",
-    "w3c-bad": "run-s w3c-build w3c-bad-only clean-w3c",
-    "w3c-good-test": "run-s w3c-good-only clean-w3c",
-    "w3c-bad-test": "run-s w3c-bad-only clean-w3c"
-  },
   "repository": {
     "type": "git",
     "url": "https://github.com/indexeddbshim/IndexedDBShim"
@@ -155,20 +83,20 @@
     "last 3 Samsung versions"
   ],
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=18.14.0"
   },
   "dependencies": {
     "canvas": "^2.11.2",
     "eventtargeter": "0.9.0",
     "sync-promise-expanded": "^1.0.0",
     "typeson": "9.0.3",
-    "typeson-registry": "10.0.0",
+    "typeson-registry": "11.0.0",
     "websql-configurable": "^3.0.3"
   },
   "devDependencies": {
     "@brettz9/eslint-plugin": "^1.0.4",
-    "@babel/core": "^7.24.7",
-    "@babel/preset-env": "^7.24.7",
+    "@babel/core": "^7.24.9",
+    "@babel/preset-env": "^7.24.8",
     "@brettz9/node-static": "^0.1.1",
     "@es-joy/json-file-reporter": "^1.4.0",
     "@rollup/plugin-babel": "^6.0.4",
@@ -201,7 +129,7 @@
     "eslint-plugin-markdown": "^3.0.1",
     "eslint-plugin-n": "^16.6.2",
     "eslint-plugin-import": "^2.29.1",
-    "eslint-plugin-jsdoc": "^48.5.2",
+    "eslint-plugin-jsdoc": "^48.7.0",
     "eslint-plugin-no-unsanitized": "^4.0.2",
     "eslint-plugin-no-use-extend-native": "^0.5.0",
     "eslint-plugin-sonarjs": "^0.23.0",
@@ -220,15 +148,86 @@
     "npm-run-all": "^4.1.5",
     "open-cli": "^8.0.0",
     "rimraf": "^5.0.8",
-    "rollup": "^4.18.0",
+    "rollup": "^4.18.1",
     "rollup-plugin-filesize": "^10.0.0",
     "rollup-plugin-node-globals": "^1.4.0",
     "rollup-plugin-node-polyfills": "^0.2.1",
     "sinon": "^18.0.0",
     "source-map-support": "0.5.21",
     "sourcemap-transformer": "git+https://github.com/brettz9/sourcemap-transformer.git",
-    "typescript": "^5.5.3",
+    "typescript": "^5.5.4",
     "w3c-blob": "0.0.1",
     "ws": "^8.18.0"
+  },
+  "scripts": {
+    "tsc": "tsc",
+    "tsc-prod": "tsc -p tsconfig-prod.json",
+    "build": "npm run rollup && npm run tsc-prod",
+    "license-badge": "license-badger --corrections --uncategorizedLicenseTemplate \"\\${license} (\\${name} (\\${version}))\" --filteredTypes=nonempty --textTemplate \"License types\n(project, deps, and bundled devDeps)\" --packageJson --production badges/licenses-badge.svg",
+    "license-badge-dev": "license-badger --corrections --filteredTypes=nonempty --textTemplate \"License types\n(all devDeps)\" --allDevelopment badges/licenses-badge-dev.svg",
+    "license-badges": "run-p license-badge license-badge-dev",
+    "start": "static -p 9999 -H '{\"Access-Control-Allow-Origin\": \"http://web-platform.test:8000\", \"Access-Control-Allow-Methods\": \"GET\", \"Access-Control-Allow-Headers\": \"Content-Type\"}'",
+    "watch": "chokidar \"src/*\" \"node_modules/eventtarget/EventTarget.js\" \"node_modules/websql/lib/websql/WebSQLTransaction.js\" \"node_modules/websql/lib/websql/WebSQLDatabase.js\" -c \"npm run rollup\"",
+    "dev": "run-p start watch",
+    "clean": "run-p clean-sysDB clean-mocha clean-fake clean-mock clean-w3c clean-w3cOld",
+    "clean-sysDB": "rimraf __sysdb__*",
+    "clean-polyfill": "run-p clean-fake clean-mock clean-w3cOld clean-sysDB",
+    "clean-mocha": "rimraf D_indexeddbshim_test_database_* D_test.sqlite",
+    "clean-fake": "rimraf D_test0.*",
+    "clean-mock": "rimraf D_test_database*",
+    "clean-w3c": "rimraf D_^D^B2* D_db* D_test* D_about* D_^I^D^B* D_blank* D_database_name* D_idbtransaction* D_x D_x.sqlite D_y D_y.sqlite D_webworker* D_http*",
+    "clean-w3cOld": "rimraf D_db.sqlite* D_test* D_database_name* D_idbtransaction*",
+    "copy": "copyup node_modules/core-js-bundle/minified.js node_modules/source-map-support/browser-source-map-support.js node_modules/mocha/mocha.css node_modules/mocha/mocha.js node_modules/chai/chai.js node_modules/sinon/pkg/sinon-no-sourcemaps.js test-support",
+    "lint": "eslint --ext=js,md,html .",
+    "eslint": "npm run lint",
+    "sqlite-rebuild": "cd node_modules/sqlite3 && npm i",
+    "w3c-build-only": "node test-support/node-buildjs",
+    "w3c-build": "run-s clean-w3c w3c-build-only",
+    "fake": "run-s clean-fake coverage:fake clean-fake",
+    "mock": "run-s clean-mock coverage:mock clean-mock",
+    "w3c-old": "run-s clean-w3cOld coverage:w3cOld clean-w3cOld",
+    "tests-polyfill": "run-s clean-polyfill coverage:fake coverage:mock coverage:w3cOld",
+    "mocha": "run-s clean-mocha coverage:mocha clean-mocha",
+    "rollup": "rollup -c",
+    "coverage:fake": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+fake --require source-map-support/register tests-polyfill/fakeIndexedDB/test-node.js",
+    "coverage:mock": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+mock --require source-map-support/register tests-polyfill/indexedDBmock/test-node.js",
+    "coverage:w3cOld": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+w3c --require source-map-support/register tests-polyfill/w3c/test-node.js",
+    "coverage:w3c": "NODE_OPTIONS=--max_old_space_size=16384 c8 --silent --no-clean npm run w3c-test-only",
+    "test:w3c": "NODE_OPTIONS=--max_old_space_size=16384 NODE_OPTIONS=--trace-warnings node test-support/node-idb-test.js",
+    "w3c-test-only": "node --trace-warnings test-support/node-idb-test.js",
+    "coverage:mocha": "c8 --silent --no-clean mocha --reporterOptions configFile=mocha-multi-reporters.json,cmrOutput=@es-joy/json-file-reporter+output+mocha --require source-map-support/register tests-mocha/test-node.js",
+    "coverage-badge": "coveradge badges/coverage-badge",
+    "coverage:report-only": "c8 report",
+    "coverage:report": "run-s coverage:report-only",
+    "remove-cache": "rimraf node_modules/.cache",
+    "remove-cov": "rimraf .nyc_output ",
+    "remove-results": "rimraf test-support/results/file*.json",
+    "coverage:polyfill": "run-s coverage:fake coverage:mock coverage:w3cOld",
+    "tests-badge": "mbg --fileGlob \"./test-support/results/file*.json\" --badge_output badges/tests-badge.svg",
+    "coverage": "run-s -c clean remove-cache remove-cov remove-results coverage:fake coverage:mock coverage:mocha coverage:w3c clean tests-badge coverage-badge coverage:report",
+    "open-coverage-only": "open http://127.0.0.1:9999/coverage/lcov-report/",
+    "open-coverage": "run-p start open-coverage-only",
+    "test": "run-s lint coverage",
+    "submodules-init": "git submodule update --init --recursive",
+    "submodules-fetch": "git submodule foreach --recursive git fetch",
+    "submodules-merge": "git submodule foreach git merge origin master",
+    "submodules": "run-s submodules-init submodules-fetch submodules-merge",
+    "wpt": "run-p start web-platform-tests",
+    "web-platform-tests": "cd web-platform-tests && ./wpt serve",
+    "w3c-find-unused": "node test-support/find-unused.js",
+    "w3c": "run-s w3c-build w3c-test",
+    "w3c-add-wrap": "node test-support/wrap-w3c-tests",
+    "w3c-remove-wrap": "node test-support/wrap-w3c-tests remove",
+    "w3c-wrap": "node test-support/wrap-w3c-tests",
+    "test-and-start:w3c": "run-p -r start web-platform-tests delayed-w3c-test",
+    "delay": "delay 9",
+    "delayed-w3c-test": "run-s delay test:w3c",
+    "w3c-test": "run-s clean-w3c test-and-start:w3c",
+    "w3c-good-only": "node test-support/node-idb-test.js good",
+    "w3c-bad-only": "node test-support/node-idb-test.js bad",
+    "w3c-good": "run-s w3c-build w3c-good-only clean-w3c",
+    "w3c-bad": "run-s w3c-build w3c-bad-only clean-w3c",
+    "w3c-good-test": "run-s w3c-good-only clean-w3c",
+    "w3c-bad-test": "run-s w3c-bad-only clean-w3c"
   }
-}
+}

package/src/IDBDatabase.js

@@ -29,6 +29,8 @@ const readonlyProperties = ['name', 'version', 'objectStoreNames'];
 function IDBDatabase () {
     this.__versionTransaction = null;
     this.__objectStores = null;
+    /** @type {import('./IDBTransaction.js').IDBTransactionFull[]} */
+    this.__transactions = [];
     throw new TypeError('Illegal constructor');
 }
 const IDBDatabaseAlias = IDBDatabase;
@@ -92,6 +94,7 @@ IDBDatabase.__createInstance = function (db, name, oldVersion, version, storePro
             legacyOutputDidListenersThrowFlag: true // Event hook for IndexedB
         });
 
+        /** @type {import('./IDBTransaction.js').IDBTransactionFull[]} */
         this.__transactions = [];
 
         /** @type {{[key: string]: IDBObjectStore}} */
@@ -222,6 +225,7 @@ IDBDatabase.prototype.close = function () {
     if (this.__unblocking) {
         this.__unblocking.check();
     }
+    this.__transactions = [];
 };
 
 /**
@@ -325,6 +329,7 @@ IDBDatabase.prototype.__forceClose = function (msg) {
             'The connection was force-closed: ' + (msg || '')
         ));
     });
+    me.__transactions = [];
 };
 
 util.defineOuterInterface(IDBDatabase.prototype, listeners);

package/src/IDBFactory.js

@@ -635,6 +635,11 @@ IDBFactory.prototype.open = function (name /* , version */) {
 
                             // eslint-disable-next-line camelcase -- Clear API
                             req.transaction.on__complete = function () {
+                                const pos = connection.__transactions.indexOf(req.transaction);
+                                if (pos > -1) {
+                                    connection.__transactions.splice(pos, 1);
+                                }
+
                                 if (/** @type {import('./IDBDatabase.js').IDBDatabaseFull} */ (
                                     req.__result
                                 ).__closePending) {
@@ -1028,9 +1033,10 @@ IDBFactory.prototype.__forceClose = function (dbName, connIdx, msg) {
         conn.__forceClose(msg);
     }
     if (util.isNullish(dbName)) {
-        (Object.values(me.__connections)).forEach((conn) => {
-            // @ts-expect-error It's ok
-            forceClose(conn);
+        (Object.values(me.__connections)).forEach((connections) => {
+            connections.forEach((connection) => {
+                forceClose(connection);
+            });
         });
     } else if (!me.__connections[dbName]) {
         console.log('No database connections with that name to force close');
@@ -1041,7 +1047,7 @@ IDBFactory.prototype.__forceClose = function (dbName, connIdx, msg) {
     } else if (!Number.isInteger(connIdx) || connIdx < 0 || connIdx > me.__connections[dbName].length - 1) {
         throw new TypeError(
             'If providing an argument, __forceClose must be called with a ' +
-            'numeric index to indicate a specific connection to lose'
+            'numeric index to indicate a specific connection to close'
         );
     } else {
         forceClose(me.__connections[dbName][connIdx]);

package/src/Key.js

@@ -91,6 +91,14 @@ const signValues = ['negativeInfinity', 'bigNegative', 'smallNegative', 'smallPo
  * @typedef {any} AnyValue
  */
 
+/**
+ * @type {{
+ *   [key: string]: {
+ *     encode: (param: any, inArray?: boolean) => string,
+ *     decode: (param: string, inArray?: boolean) => any
+ *   }
+ * }}
+ */
 const types = {
     invalid: {
         /**
@@ -923,7 +931,6 @@ function encode (key, inArray) {
         return null;
     }
     // array, date, number, string, binary (should already have detected "invalid")
-    // @ts-expect-error Argument may be ignored
     return types[getKeyType(key)].encode(key, inArray);
 }