feat: Support Node.js v20 (#71)

package.json

@@ -32,7 +32,7 @@
 		"prepack": "pnpm build && clean-pkg-json"
 	},
 	"dependencies": {
-		"@esbuild-kit/core-utils": "^3.2.3",
+		"@esbuild-kit/core-utils": "^3.3.0",
 		"get-tsconfig": "^4.7.0"
 	},
 	"devDependencies": {

src/loaders.ts

@@ -1,6 +1,9 @@
+import type { MessagePort } from 'node:worker_threads';
 import path from 'path';
 import { pathToFileURL, fileURLToPath } from 'url';
-import type { ResolveFnOutput, ResolveHookContext, LoadHook } from 'module';
+import type {
+	ResolveFnOutput, ResolveHookContext, LoadHook, GlobalPreloadHook,
+} from 'module';
 import {
 	transform,
 	transformDynamicImport,
@@ -31,6 +34,20 @@ type resolve = (
 	recursiveCall?: boolean,
 ) => MaybePromise<ResolveFnOutput>;
 
+/**
+ * Technically globalPreload is deprecated so it should be in loaders-deprecated
+ * but it shares a closure with the new load hook
+ */
+let mainThreadPort: MessagePort | undefined;
+export const globalPreload: GlobalPreloadHook = ({ port }) => {
+	mainThreadPort = port;
+	return `
+	const require = getBuiltin('module').createRequire(getBuiltin('process').cwd() + '/<preload>');
+	require('@esbuild-kit/core-utils').installSourceMapSupport(port);
+	port.unref(); // Allows process to exit without waiting for port to close
+	`;
+};
+
 const extensions = ['.js', '.json', '.ts', '.tsx', '.jsx'] as const;
 
 async function tryExtensions(
@@ -223,6 +240,7 @@ export const load: LoadHook = async function (
 	const code = loaded.source.toString();
 
 	if (
+		// Support named imports in JSON modules
 		loaded.format === 'json'
 		|| tsExtensionsPattern.test(url)
 	) {
@@ -236,7 +254,7 @@ export const load: LoadHook = async function (
 
 		return {
 			format: 'module',
-			source: applySourceMap(transformed, url),
+			source: applySourceMap(transformed, url, mainThreadPort),
 		};
 	}
 
@@ -246,6 +264,7 @@ export const load: LoadHook = async function (
 			loaded.source = applySourceMap(
 				dynamicImportTransformed,
 				url,
+				mainThreadPort,
 			);
 		}
 	}

tests/index.ts

@@ -11,6 +11,7 @@ const nodeVersions = [
 				'16',
 				'17',
 				'18',
+				'20',
 			]
 			: []
 	),

tests/specs/typescript/tsconfig.ts

@@ -1,3 +1,5 @@
+import path from 'path';
+import fs from 'fs/promises';
 import { testSuite, expect } from 'manten';
 import { createFixture } from 'fs-fixture';
 import type { NodeApis } from '../../utils/node-with-loader.js';
@@ -37,6 +39,12 @@ export default testSuite(async ({ describe }, node: NodeApis) => {
 					},
 				});
 
+				await fs.symlink(
+					path.resolve('node_modules'),
+					path.join(fixture.path, 'node_modules'),
+					'dir',
+				);
+
 				onTestFinish(async () => await fixture.rm());
 
 				// Strict mode is not tested because ESM is strict by default
@@ -71,6 +79,12 @@ export default testSuite(async ({ describe }, node: NodeApis) => {
 					'src/jsx.jsx': checkJsx,
 				});
 
+				await fs.symlink(
+					path.resolve('node_modules'),
+					path.join(fixture.path, 'node_modules'),
+					'dir',
+				);
+
 				onTestFinish(async () => await fixture.rm());
 
 				const jsxJs = await node.load('./src/jsx.jsx', {