remove node:module polyfills

Author: anonrig

.changeset/cruel-terms-smell.md

@@ -0,0 +1,6 @@
+---
+"@cloudflare/unenv-preset": patch
+"wrangler": patch
+---
+
+Remove node:module polyfills

packages/unenv-preset/src/preset.ts

@@ -30,6 +30,7 @@ const nativeModules = [
 	"path/posix",
 	"path/win32",
 	"querystring",
+	"module",
 	"stream",
 	"stream/consumers",
 	"stream/promises",
@@ -43,7 +44,7 @@ const nativeModules = [
 ];
 
 // Modules implemented via a mix of workerd APIs and polyfills.
-const hybridModules = ["console", "crypto", "module", "process", "tls", "util"];
+const hybridModules = ["console", "crypto", "process", "tls", "util"];
 
 /**
  * Creates the Cloudflare preset for the given compatibility date and compatibility flags

packages/unenv-preset/src/runtime/node/module.ts

@@ -115,6 +115,7 @@ export const WorkerdTests: Record<string, () => void> = {
 			"path/posix",
 			"path/win32",
 			"querystring",
+			"module",
 			"stream",
 			"stream/consumers",
 			"stream/promises",
@@ -329,4 +330,45 @@ export const WorkerdTests: Record<string, () => void> = {
 		});
 		assert.deepStrictEqual(result, { test: "require" });
 	},
+
+	async testModule() {
+		const module = await import("node:module");
+		const exportNames = [
+			"createRequire",
+			"enableCompileCache",
+			"findSourceMap",
+			"getCompileCacheDir",
+			"getSourceMapsSupport",
+			"isBuiltin",
+			"register",
+			"runMain",
+			"setSourceMapsSupport",
+			"stripTypeScriptTypes",
+			"syncBuiltinESMExports",
+			"wrap",
+			"flushCompileCache",
+			"findPackageJSON",
+			"_debug",
+			"_findPath",
+			"_initPaths",
+			"_load",
+			"_preloadModules",
+			"_resolveFilename",
+			"_resolveLookupPaths",
+			"_nodeModulePaths",
+			"Module",
+			"SourceMap",
+		];
+
+		for (const name of exportNames) {
+			assert.strictEqual(typeof module[name], "function");
+		}
+
+		assert.ok(Array.isArray(module.globalPaths));
+		assert.ok(Array.isArray(module.builtinModules));
+		assert.strictEqual(typeof module.constants, "object");
+		assert.strictEqual(typeof module._cache, "object");
+		assert.strictEqual(typeof module._extensions, "object");
+		assert.strictEqual(typeof module._pathCache, "object");
+	},
 };