fix: ensure source mapped locations correct on Windows

Use `@cspotcode/source-map-support` instead of `source-map-support`.
This has some additional fixes for resolving paths on Windows, and is
used by `ts-node`.

packages/miniflare/package.json

@@ -41,12 +41,12 @@
 		"miniflare": "bootstrap.js"
 	},
 	"dependencies": {
+		"@cspotcode/source-map-support": "0.8.1",
 		"acorn": "^8.8.0",
 		"acorn-walk": "^8.2.0",
 		"capnp-ts": "^0.7.0",
 		"exit-hook": "^2.2.1",
 		"glob-to-regexp": "^0.4.1",
-		"source-map-support": "0.5.21",
 		"stoppable": "^1.1.0",
 		"undici": "^5.22.1",
 		"workerd": "1.20231030.0",

packages/miniflare/src/plugins/core/errors/index.ts

@@ -1,7 +1,7 @@
 import fs from "fs";
 import path from "path";
 import { fileURLToPath } from "url";
-import type { UrlAndMap } from "source-map-support";
+import type { UrlAndMap } from "@cspotcode/source-map-support";
 import { z } from "zod";
 import { Request, Response } from "../../../http";
 import { Log } from "../../../shared";
@@ -49,7 +49,7 @@ interface SourceFile {
 }
 
 // Try to read a file from the file-system, returning undefined if not found
-function maybeGetDiskFile(filePath: string): SourceFile | undefined {
+function maybeGetDiskFile(filePath: string): Required<SourceFile> | undefined {
 	try {
 		const contents = fs.readFileSync(filePath, "utf8");
 		return { path: filePath, contents };

packages/miniflare/src/plugins/core/errors/sourcemap.ts

@@ -1,5 +1,5 @@
 import assert from "assert";
-import type { Options } from "source-map-support";
+import type { Options } from "@cspotcode/source-map-support";
 import { parseStack } from "./callsite";
 
 const sourceMapInstallBaseOptions: Options = {
@@ -8,6 +8,7 @@ const sourceMapInstallBaseOptions: Options = {
 	handleUncaughtExceptions: false,
 	// Don't hook Node `require` function
 	hookRequire: false,
+	redirectConflictingLibrary: false,
 
 	// Make sure we're using fresh copies of files (i.e. between `setOptions()`)
 	emptyCacheBetweenOperations: true,
@@ -36,10 +37,10 @@ export function getSourceMapper(): SourceMapper {
 	// always updated, load a fresh copy, by resetting then restoring the
 	// `require` cache.
 
-	const originalSupport = require.cache["source-map-support"];
-	delete require.cache["source-map-support"];
-	const support: typeof import("source-map-support") = require("source-map-support");
-	require.cache["source-map-support"] = originalSupport;
+	const originalSupport = require.cache["@cspotcode/source-map-support"];
+	delete require.cache["@cspotcode/source-map-support"];
+	const support: typeof import("@cspotcode/source-map-support") = require("@cspotcode/source-map-support");
+	require.cache["@cspotcode/source-map-support"] = originalSupport;
 
 	const originalPrepareStackTrace = Error.prepareStackTrace;
 	support.install(sourceMapInstallBaseOptions);
@@ -50,10 +51,13 @@ export function getSourceMapper(): SourceMapper {
 	sourceMapper = (retrieveSourceMap, error) => {
 		support.install({
 			...sourceMapInstallBaseOptions,
-			retrieveFile(file: string): string | null {
+			retrieveFile(_file: string): string {
 				// `retrieveFile` should only be called by the default implementation of
-				// `retrieveSourceMap`, so we shouldn't need to implement it
-				assert.fail(`Unexpected retrieveFile(${JSON.stringify(file)}) call`);
+				// `retrieveSourceMap`, which will only be called if `retrieveSourceMap`
+				// failed. In that case, return an empty string so the default
+				// implementation fails too (can't find `// sourceMappingURL` comment
+				// in empty string).
+				return "";
 			},
 			retrieveSourceMap,
 		});

packages/wrangler/package.json

@@ -102,6 +102,7 @@
 	},
 	"dependencies": {
 		"@cloudflare/kv-asset-handler": "^0.2.0",
+		"@cspotcode/source-map-support": "0.8.1",
 		"@esbuild-plugins/node-globals-polyfill": "^0.2.3",
 		"@esbuild-plugins/node-modules-polyfill": "^0.2.2",
 		"blake3-wasm": "^2.1.5",
@@ -113,7 +114,6 @@
 		"resolve.exports": "^2.0.2",
 		"selfsigned": "^2.0.1",
 		"source-map": "0.6.1",
-		"source-map-support": "0.5.21",
 		"xxhash-wasm": "^1.0.1"
 	},
 	"devDependencies": {

packages/wrangler/scripts/deps.ts

@@ -12,6 +12,8 @@ export const EXTERNAL_DEPENDENCIES = [
 	// todo - bundle miniflare too
 	"selfsigned",
 	"source-map",
+	// CommonJS module using `module.require()` which isn't provided by `esbuild`
+	"@cspotcode/source-map-support",
 	"@esbuild-plugins/node-globals-polyfill",
 	"@esbuild-plugins/node-modules-polyfill",
 	"chokidar",

packages/wrangler/src/sourcemap.ts

@@ -15,11 +15,11 @@ function getSourceMappingPrepareStackTrace(): NonNullable<
 	// affecting external user's use of this package, and so
 	// `Error.prepareStackTrace` is always updated, load a fresh copy, by
 	// resetting then restoring the `require` cache.
-	const originalSupport = require.cache["source-map-support"];
-	delete require.cache["source-map-support"];
+	const originalSupport = require.cache["@cspotcode/source-map-support"];
+	delete require.cache["@cspotcode/source-map-support"];
 	// eslint-disable-next-line @typescript-eslint/consistent-type-imports,@typescript-eslint/no-var-requires
-	const support: typeof import("source-map-support") = require("source-map-support");
-	require.cache["source-map-support"] = originalSupport;
+	const support: typeof import("@cspotcode/source-map-support") = require("@cspotcode/source-map-support");
+	require.cache["@cspotcode/source-map-support"] = originalSupport;
 
 	const originalPrepareStackTrace = Error.prepareStackTrace;
 	support.install({
@@ -28,6 +28,7 @@ function getSourceMappingPrepareStackTrace(): NonNullable<
 		handleUncaughtExceptions: false,
 		// Don't hook Node `require` function
 		hookRequire: false,
+		redirectConflictingLibrary: false,
 		// Make sure we're using fresh copies of files each time we source map
 		emptyCacheBetweenOperations: true,
 	});