refactor: remove unused source map logic (#10)

Author: privatenumber

package.json

@@ -32,7 +32,6 @@
 		},
 		"./esm": "./dist/esm/index.mjs",
 		"./cli": "./dist/cli.mjs",
-		"./source-map": "./dist/source-map.cjs",
 		"./suppress-warnings": "./dist/suppress-warnings.cjs",
 		"./preflight": "./dist/preflight.cjs",
 		"./repl": "./dist/repl.mjs"

src/esm/api/register.ts

@@ -1,8 +1,7 @@
 import module from 'node:module';
-import { installSourceMapSupport } from '../../source-map.js';
 
 export const register = () => {
-	installSourceMapSupport();
+	process.setSourceMapsEnabled(true);
 
 	module.register(
 		'./index.mjs',

src/esm/hook/index.ts

@@ -11,8 +11,8 @@ export const initialize: InitializeHook = async (data) => {
  * but it shares a closure with the new load hook
  */
 export const globalPreload: GlobalPreloadHook = () => `
-const require = getBuiltin('module').createRequire("${import.meta.url}");
-require('../source-map.cjs').installSourceMapSupport();
+const require = getBuiltin('module').createRequire('${import.meta.url}');
+process.setSourceMapsEnabled(true);
 `;
 
 export { load } from './load.js';

src/esm/hook/load.ts

@@ -3,7 +3,7 @@ import type { LoadHook } from 'node:module';
 import type { TransformOptions } from 'esbuild';
 import { transform } from '../../utils/transform/index.js';
 import { transformDynamicImport } from '../../utils/transform/transform-dynamic-import.js';
-import { installSourceMapSupport } from '../../source-map.js';
+import { inlineSourceMap } from '../../source-map.js';
 import { isFeatureSupported, importAttributes } from '../../utils/node-features.js';
 import { parent } from '../../utils/ipc/client.js';
 import {
@@ -12,7 +12,7 @@ import {
 	isJsonPattern,
 } from './utils.js';
 
-const applySourceMap = installSourceMapSupport();
+process.setSourceMapsEnabled(true);
 
 const contextAttributesProperty = (
 	isFeatureSupported(importAttributes)
@@ -69,14 +69,14 @@ export const load: LoadHook = async (
 
 		return {
 			format: 'module',
-			source: applySourceMap(transformed),
+			source: inlineSourceMap(transformed),
 		};
 	}
 
 	if (loaded.format === 'module') {
 		const dynamicImportTransformed = transformDynamicImport(filePath, code);
 		if (dynamicImportTransformed) {
-			loaded.source = applySourceMap(dynamicImportTransformed);
+			loaded.source = inlineSourceMap(dynamicImportTransformed);
 		}
 	}
 

src/source-map.ts

@@ -12,28 +12,3 @@ export const inlineSourceMap = (
 	+ inlineSourceMapPrefix
 	+ Buffer.from(JSON.stringify(map), 'utf8').toString('base64')
 );
-
-const noSourceMap = ({ code }: Transformed) => code;
-
-export const installSourceMapSupport = () => {
-	/**
-	 * Check if native source maps are supported by seeing if the API is available
-	 * https://nodejs.org/dist/latest-v18.x/docs/api/process.html#processsetsourcemapsenabledval
-	 *
-	 * Previously, we also checked Error.prepareStackTrace to opt-out of source maps
-	 * as per this recommendation:
-	 * https://nodejs.org/dist/latest-v18.x/docs/api/cli.html#:~:text=Overriding%20Error.prepareStackTrace%20prevents%20%2D%2Denable%2Dsource%2Dmaps%20from%20modifying%20the%20stack%20trace.
-	 *
-	 * But it's been removed because:
-	 * 1. It's set by default from Node v21.6.0 and v20.12.0
-	 * 2. It may have been possible for a custom prepareStackTrace to parse the source maps
-	 */
-	const hasNativeSourceMapSupport = 'setSourceMapsEnabled' in process;
-
-	if (hasNativeSourceMapSupport) {
-		process.setSourceMapsEnabled(true);
-		return inlineSourceMap;
-	}
-
-	return noSourceMap;
-};