feat(module-plugin-polyfill): support polyfill Node.js builtin module…

…s starting with node: (#6315)

.changeset/modern-feet-bake.md

@@ -0,0 +1,6 @@
+---
+'@modern-js/plugin-module-node-polyfill': patch
+---
+
+feat(module-plugin-polyfill): support polyfill Node.js builtin modules starting with node:
+feat(module-plugin-polyfill): 支持对 node: 开头的 Node.js builtin modules 进行 polyfill

packages/module/plugin-module-node-polyfill/src/index.ts

@@ -5,7 +5,7 @@ import type {
   ICompiler,
   ModuleTools,
 } from '@modern-js/module-tools';
-import { addResolveFallback, excludeObjectKeys } from './utils';
+import { addNodePrefix, addResolveFallback, excludeObjectKeys } from './utils';
 
 export interface NodePolyfillPluginOptions {
   // like https://github.com/Richienb/node-polyfill-webpack-plugin#excludealiases
@@ -14,7 +14,7 @@ export interface NodePolyfillPluginOptions {
   overrides?: Partial<Record<keyof typeof modules, string>>;
 }
 
-export const modules = {
+let modules = {
   assert: require.resolve('assert/'),
   buffer: require.resolve('buffer/'),
   child_process: null,
@@ -60,6 +60,8 @@ export const modules = {
 export const getNodePolyfillHook = (
   polyfillOption: NodePolyfillPluginOptions = {},
 ) => {
+  const nodeModules = addNodePrefix(modules);
+  modules = Object.assign(modules, nodeModules);
   const polyfillModules = {
     ...excludeObjectKeys(
       addResolveFallback(modules, polyfillOption.overrides),

packages/module/plugin-module-node-polyfill/src/utils.ts

@@ -42,3 +42,16 @@ export function addResolveFallback(
 
   return newObject;
 }
+
+export function addNodePrefix(
+  modules: Record<string, string | null>,
+): Record<string, string | null> {
+  return Object.fromEntries(
+    Object.entries(modules).map(([key, value]) => {
+      if (!key.startsWith('_')) {
+        return [`node:${key}`, value];
+      }
+      return [key, value];
+    }),
+  );
+}

tests/integration/module/plugins/node-polyfill/node-polyfill.test.ts

@@ -15,6 +15,8 @@ describe('plugin-node-polyfill', () => {
       path.resolve(__dirname, 'dist/index.js'),
       'utf8',
     );
-    expect(content).toContain('init_globals');
+    expect(content).toContain('init_globals()');
+    expect(content).toContain('__toESM(require_browser2())');
+    expect(content).toContain('__toESM(require_path_browserify())');
   });
 });

tests/integration/module/plugins/node-polyfill/src/index.js

@@ -1 +1,4 @@
+import os from 'node:os';
+import path from 'path';
+
 export const value = Buffer.from('value');