Node 20 support

.github/workflows/npm-publish.yml

@@ -22,7 +22,7 @@ jobs:
           java-version: '11'
       - uses: actions/setup-node@v3
         with:
-          node-version: '19'
+          node-version: '20'
           registry-url: https://registry.npmjs.org/
       - run: npm ci
       - run: npm publish --provenance --access public

.github/workflows/pull-request.yml

@@ -28,6 +28,8 @@ jobs:
       - run: npm run lint
       - run: npm test
       - run: deno test --import-map=import_map.json test
+        env:
+          DENO_NO_PACKAGE_JSON=1
 
   run-tests-windows:
     runs-on: windows-latest
@@ -47,3 +49,5 @@ jobs:
       - run: npm run lint
       - run: npm test
       - run: deno test --import-map=import_map.json test
+        env:
+          DENO_NO_PACKAGE_JSON=1

.mocharc.json

@@ -1,4 +1,4 @@
 {
 	"extension": ["ts"],
-	"loader": "./loader.mjs"
+	"node-option": ["no-warnings=ExperimentalWarning", "loader=./loader.mjs"]
 }

loader.mjs

@@ -14,10 +14,10 @@
  */
 
 import module from 'node:module';
+import { join } from 'node:path';
+import { fileURLToPath } from 'node:url';
 import { resolve as resolveTs } from 'ts-node/esm';
 import importMap from './import_map.json' assert { type: 'json' };
-import { fileURLToPath } from 'node:url';
-import { join } from 'node:path';
 
 export * from 'ts-node/esm';
 
@@ -47,10 +47,16 @@ module._resolveFilename = (...args) => {
 export const resolve = (specifier, context, defaultResolver) => {
 	if (typeof specifier === 'string') {
 		Object.entries(importMap.imports).forEach(([k, v]) => {
-			if (specifier.startsWith(absoluteBaseUrl + k))
-				specifier = specifier.replace(k, v);
+			if (specifier.startsWith(k)) {
+				specifier = specifier.replace(k, absoluteBaseUrl + v);
+			}
 		});
 	}
 
-	return resolveTs(specifier, context, defaultResolver);
+	return resolveTs(specifier, context, defaultResolver).then((v) => {
+		if (v.url.endsWith('.ts')) {
+			v.format = 'module';
+		}
+		return v;
+	});
 };

package.json

@@ -132,7 +132,7 @@
 		"test:unit": "nyc mocha src/**/*.spec.ts",
 		"test:e2e": "mocha test/**/*.spec.ts",
 		"test": "npm run test:unit && npm run build && npm run test:e2e",
-		"prepack": "npm run build && rimraf -g -- 'dist/**/*.tsbuildinfo'",
+		"prepack": "npm run build",
 		"prepublishOnly": "npm test && npm run lint",
 		"preversion": "npm run lint",
 		"version": "npm run lint && git add -A src",

src/test/lib/buildTimeSettings.ts

@@ -0,0 +1,23 @@
+/* Copyright © 2023 Exact Realty Limited.
+ *
+ * Permission to use, copy, modify, and distribute this software for any
+ * purpose with or without fee is hereby granted, provided that the above
+ * copyright notice and this permission notice appear in all copies.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES WITH
+ * REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
+ * AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT,
+ * INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
+ * LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR
+ * OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
+ * PERFORMANCE OF THIS SOFTWARE.
+ */
+
+if (typeof __buildtimeSettings__ !== 'object')
+	Reflect.set(globalThis, '__buildtimeSettings__', {});
+
+Reflect.set(__buildtimeSettings__, 'bidirectionalMessaging', true);
+Reflect.set(__buildtimeSettings__, 'buildType', 'debug');
+Reflect.set(__buildtimeSettings__, 'defaultAllowedGlobalProps', []);
+Reflect.set(__buildtimeSettings__, 'hardenGlobals', true);
+Reflect.set(__buildtimeSettings__, 'sandboxInitDeadlineInMs', 100);

src/test/tsconfig.json

@@ -0,0 +1,10 @@
+{
+	"extends": "../../tsconfig.json",
+	"compilerOptions": {
+		"rootDir": ".",
+		"composite": true,
+		"lib": ["es2022"],
+		"outDir": "../../build/test"
+	},
+	"include": ["**/*"]
+}

src/trusted/lib/setupSandboxListeners.spec.ts

@@ -13,10 +13,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-if (typeof __buildtimeSettings__ !== 'object')
-	Reflect.set(globalThis, '__buildtimeSettings__', {});
-
-Reflect.set(__buildtimeSettings__, 'sandboxInitDeadlineInMs', 100);
+import '~/test/lib/buildTimeSettings.js';
 
 import setupSandboxListeners from './setupSandboxListeners.js';
 

src/trusted/tsconfig.json

@@ -3,8 +3,9 @@
 	"compilerOptions": {
 		"rootDir": ".",
 		"outDir": "../../dist/trusted",
+		"tsBuildInfoFile": "../../build/trusted.tsbuildinfo",
 		"composite": true
 	},
-	"references": [{ "path": "../types" }, { "path": "../untrusted" }],
+	"references": [{ "path": "../types" }, { "path": "../untrusted" }, { "path": "../test" }],
 	"include": ["**/*", "**/*.json"]
 }

src/types/tsconfig.json

@@ -3,6 +3,7 @@
 	"compilerOptions": {
 		"rootDir": ".",
 		"outDir": "../../dist/types",
+		"tsBuildInfoFile": "../../build/types.tsbuildinfo",
 		"composite": true
 	},
 	"include": ["**/*", "**/*.json"]

src/untrusted/lib/createContext.spec.ts

@@ -13,11 +13,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-if (typeof __buildtimeSettings__ !== 'object')
-	Reflect.set(globalThis, '__buildtimeSettings__', {});
-
-Reflect.set(__buildtimeSettings__, 'bidirectionalMessaging', true);
-Reflect.set(__buildtimeSettings__, 'defaultAllowedGlobalProps', []);
+import '~/test/lib/buildTimeSettings.js';
 
 import assert from 'node:assert/strict';
 import createContext, { setupExternalMethods } from './createContext.js';

src/untrusted/lib/hardenGlobals.spec.ts

@@ -13,10 +13,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-if (typeof __buildtimeSettings__ !== 'object')
-	Reflect.set(globalThis, '__buildtimeSettings__', {});
-
-Reflect.set(__buildtimeSettings__, 'hardenGlobals', true);
+import '~/test/lib/buildTimeSettings.js';
 
 import assert from 'node:assert/strict';
 import hardenGlobals, { disableURLStaticMethods } from './hardenGlobals.js';

src/untrusted/lib/performTaskFactory.spec.ts

@@ -13,10 +13,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-if (typeof __buildtimeSettings__ !== 'object')
-	Reflect.set(globalThis, '__buildtimeSettings__', {});
-
-Reflect.set(__buildtimeSettings__, 'buildType', 'debug');
+import '~/test/lib/buildTimeSettings.js';
 
 import assert from 'node:assert/strict';
 import performTaskFactory from './performTaskFactory.js';

src/untrusted/lib/requestHandler.spec.ts

@@ -13,10 +13,7 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
-if (typeof __buildtimeSettings__ !== 'object')
-	Reflect.set(globalThis, '__buildtimeSettings__', {});
-
-Reflect.set(__buildtimeSettings__, 'buildType', 'debug');
+import '~/test/lib/buildTimeSettings.js';
 
 import assert from 'node:assert/strict';
 import { reconstructErrorInformation } from './errorModem.js';

src/untrusted/tsconfig.json

@@ -3,8 +3,9 @@
 	"compilerOptions": {
 		"rootDir": ".",
 		"outDir": "../../dist/untrusted",
+		"tsBuildInfoFile": "../../build/untrusted.tsbuildinfo",
 		"composite": true
 	},
-	"references": [{ "path": "../types" }],
+	"references": [{ "path": "../types" }, { "path": "../test" }],
 	"include": ["**/*", "**/*.json"]
 }

test/e2e/bare/bareSandbox.spec.ts

@@ -15,7 +15,7 @@
 
 import runNodejsTests from '@test/lib/runNodejsTests.js';
 
-import * as bare from '@dist/exports/bare';
+import * as bare from '@dist/exports/bare.js';
 
 // TODO: Import from '@dist/exports/bare'
 import { hardenGlobals, freezePrototypes } from '@dist/index.js';
@@ -28,13 +28,10 @@ if (process.version) {
 		.slice(1)
 		.split('.', 2)
 		.map((n) => parseInt(n));
-	if (
-		(major === 18 && minor >= 18) ||
-		(major === 20 && minor >= 6) ||
-		major > 20
-	) {
+	if ((major === 20 && minor >= 6) || major > 20) {
 		freezePrototypes();
 	}
 }
 
-runNodejsTests('Bare', bare.default);
+// Due to Node's CJS-from-ESM implementation, the import is bare.default.default
+runNodejsTests('Bare', (bare.default as unknown as typeof bare).default);

test/e2e/nodejs/nodejsSandbox.spec.ts

@@ -15,6 +15,6 @@
 
 import runNodejsTests from '@test/lib/runNodejsTests.js';
 
-import { nodejsSandbox as m } from '@dist/index';
+import { nodejsSandbox as m } from '@dist/index.js';
 
 runNodejsTests('Node.js', m);

test/lib/runBrowserTest.ts

@@ -13,9 +13,12 @@
  * PERFORMANCE OF THIS SOFTWARE.
  */
 
+import { fileURLToPath } from 'node:url';
 import getCodeHelper from './getCodeHelper.js';
 import { enabledBrowsers, webdriverTestSuites } from './webdriverTestSuites.js';
 
+const __dirname = fileURLToPath(new URL('.', import.meta.url));
+
 const runBrowserTest = (m: string, commonBundle?: boolean) => {
 	const code = commonBundle
 		? getCodeHelper(__dirname, '../../dist/index.mjs', m)

test/lib/runNodejsTests.ts

@@ -16,7 +16,7 @@
 import '~untrusted/lib/nodejsLoadWebcrypto.js'; // MUST BE AT THE TOP
 
 import assertRejectsWithFactory from '@test/lib/assertRejectsWithFactory.js';
-import baseTests from '@test/lib/baseTests.json';
+import baseTests from '@test/lib/baseTests.json' assert { type: 'json' };
 import wrapper from '@test/lib/wrapper.js';
 import assert from 'node:assert/strict';
 import type { ISandbox } from '~/types/index.js';

test/lib/webdriverTestSuites.ts

@@ -15,11 +15,11 @@
 
 import assert from 'node:assert/strict';
 import webdriver from 'selenium-webdriver';
-import { Options as ChromeOptions } from 'selenium-webdriver/chrome';
-import { Options as EdgeOptions } from 'selenium-webdriver/edge';
-import { Options as FirefoxOptions } from 'selenium-webdriver/firefox';
-import { Options as SafariOptions } from 'selenium-webdriver/safari';
-import baseTests from './baseTests.json';
+import { Options as ChromeOptions } from 'selenium-webdriver/chrome.js';
+import { Options as EdgeOptions } from 'selenium-webdriver/edge.js';
+import { Options as FirefoxOptions } from 'selenium-webdriver/firefox.js';
+import { Options as SafariOptions } from 'selenium-webdriver/safari.js';
+import baseTests from './baseTests.json' assert { type: 'json' };
 
 export const enabledBrowsers = () => {
 	const webdriverBrowsers = new Set(

tsconfig.json

@@ -2,7 +2,7 @@
 	"compilerOptions": {
 		"target": "es6",
 		"composite": true,
-		"module": "commonjs",
+		"module": "esnext",
 		"moduleResolution": "node",
 		"outDir": "dist",
 		"rootDir": "src",
@@ -35,7 +35,10 @@
 		"checkJs": false
 	},
 	"ts-node": {
-		"experimentalResolver": true
+		"experimentalResolver": true,
+		"compilerOptions": {
+			"module": "esnext"
+		}
 	},
 	"references": [
 		{ "path": "src/trusted" },