refactor: use same builtinModules from runtime

concat in old impl was causing duplicate entries!

src/presets/node.ts

@@ -1,4 +1,4 @@
-import { NodeBuiltinModules } from "./utils";
+import { builtinModules } from "../runtime/node/module";
 import type { Preset } from "../../lib/index.d.mts";
 import { version } from "../../package.json";
 
@@ -17,5 +17,5 @@ export default {
 
   polyfill: [],
 
-  external: [...NodeBuiltinModules],
+  external: [...builtinModules],
 } as Preset;

src/presets/nodeless.ts

@@ -1,4 +1,5 @@
-import { NodeBuiltinModules, mapArrToVal } from "./utils";
+import { mapArrToVal } from "./utils";
+import { builtinModules } from "../runtime/node/module";
 import type { Preset } from "../../lib/index.d.mts";
 import { version } from "../../package.json";
 
@@ -10,7 +11,7 @@ const nodeless: Preset & { alias: Map<string, string> } = {
 
   alias: {
     // Generic mock for built-ins
-    ...mapArrToVal("unenv/mock/proxy-cjs", NodeBuiltinModules),
+    ...mapArrToVal("unenv/mock/proxy-cjs", builtinModules),
 
     // Built-ins implemented by unenv
     "buffer/index.js": "buffer",
@@ -102,7 +103,7 @@ const nodeless: Preset & { alias: Map<string, string> } = {
 };
 
 // Add node: aliases
-for (const m of NodeBuiltinModules) {
+for (const m of builtinModules) {
   nodeless.alias[`node:${m}`] = nodeless.alias[m];
 }
 

src/presets/utils.ts

@@ -1,21 +1,3 @@
-import { Module } from "node:module";
-
-// eslint-disable-next-line unicorn/prefer-spread
-export const NodeBuiltinModules = ([] as string[]).concat(
-  Module.builtinModules,
-  [
-    "assert/strict",
-    "fs/promises",
-    "path/posix",
-    "path/win32",
-    "stream/consumers",
-    "stream/promises",
-    "stream/web",
-    "timers/promises",
-    "util/types",
-  ],
-);
-
-export function mapArrToVal(val: any, arr: any[]) {
+export function mapArrToVal(val: any, arr: readonly any[]) {
   return Object.fromEntries(arr.map((c) => [c, val]));
 }