@jsenv/plugin-commonjs 2.11.18 → 2.11.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/plugin-commonjs",
-  "version": "2.11.18",
+  "version": "2.11.20",
   "type": "module",
   "repository": {
     "type": "git",
@@ -27,16 +27,16 @@
     "@rollup/plugin-json": "6.1.0",
     "@rollup/plugin-node-resolve": "16.0.3",
     "@rollup/plugin-replace": "6.0.3",
-    "cjs-module-lexer": "2.2.0",
+    "cjs-module-lexer": "2.2.1",
     "is-valid-identifier": "2.0.2",
     "resolve": "1.22.12",
-    "rollup": "4.62.2",
+    "rollup": "4.62.4",
     "rollup-plugin-polyfill-node": "0.13.0"
   },
   "devDependencies": {
     "@jsenv/assert": "../tooling/assert",
     "@jsenv/core": "../../",
-    "playwright": "1.61.1"
+    "playwright": "1.62.1"
   },
   "publishConfig": {
     "access": "public"

package/src/cjs_to_esm_raw.js

@@ -1,5 +1,6 @@
 import { createLogger } from "@jsenv/humanize";
 import { urlToFileSystemPath } from "@jsenv/urls";
+import { isBuiltin } from "node:module";
 import { rollupPluginCommonJsNamedExports } from "./rollup_plugin_commonjs_named_exports.js";
 
 export const commonJsToJsModuleRaw = async ({
@@ -92,6 +93,16 @@ export const commonJsToJsModuleRaw = async ({
           if (specifier.startsWith("node:")) {
             return { id: specifier, external: true };
           }
+          // Keep node builtins external when targeting node. A bare builtin
+          // (e.g. `require("util")` inside ws) must NOT be resolved into a
+          // userland package of the same name (a `util` polyfill hoisted into
+          // node_modules) — Node resolves the builtin, and that polyfill lacks
+          // Node-only APIs like util.types, so bundling it breaks at runtime.
+          // When converting builtins to browser equivalents this must stay off,
+          // so the polyfill plugin can do its job.
+          if (!convertBuiltinsToBrowser && isBuiltin(specifier)) {
+            return { id: `node:${specifier}`, external: true };
+          }
           return null;
         },
       },