feat(implemented-node-modules): don't drop node: prefix

and add `node:test` module to the list

Author: SunsetTechuila

src/assets/implemented-node-modules.json

@@ -1,53 +1,104 @@
 {
   "assert": true,
   "assert/strict": true,
+  "node:assert": true,
+  "node:assert/strict": true,
   "async_hooks": true,
+  "node:async_hooks": true,
   "async_hooks/async_context": true,
   "buffer": true,
+  "node:buffer": true,
   "child_process": true,
+  "node:child_process": true,
   "constants": true,
+  "node:constants": true,
   "cluster": ">= 1.1.25",
+  "node:cluster": ">= 1.1.25",
   "console": true,
+  "node:console": true,
   "crypto": true,
+  "node:crypto": true,
   "dgram": ">= 1.1.6",
+  "node:dgram": ">= 1.1.6",
   "diagnostics_channel": true,
+  "node:diagnostics_channel": true,
   "dns": true,
   "dns/promises": true,
+  "node:dns": true,
+  "node:dns/promises": true,
   "domain": true,
+  "node:domain": true,
   "events": true,
+  "node:events": true,
   "fs": true,
   "fs/promises": true,
+  "node:fs": true,
+  "node:fs/promises": true,
   "http": true,
+  "node:http": true,
   "http2": ">= 1.0.13",
+  "node:http2": ">= 1.0.13",
   "https": true,
+  "node:https": true,
   "module": true,
+  "node:module": true,
   "net": true,
+  "node:net": true,
   "os": true,
+  "node:os": true,
   "path": true,
   "path/posix": true,
   "path/win32": true,
+  "node:path": true,
+  "node:path/posix": true,
+  "node:path/win32": true,
   "perf_hooks": true,
+  "node:perf_hooks": true,
   "process": true,
+  "node:process": true,
   "punycode": true,
+  "node:punycode": true,
   "querystring": true,
+  "node:querystring": true,
   "readline": true,
   "readline/promises": true,
+  "node:readline": true,
+  "node:readline/promises": true,
   "stream": true,
   "stream/consumers": true,
   "stream/promises": true,
   "stream/web": true,
+  "node:stream": true,
+  "node:stream/consumers": true,
+  "node:stream/promises": true,
+  "node:stream/web": true,
   "string_decoder": true,
+  "node:string_decoder": true,
   "sys": true,
+  "node:sys": true,
   "timers": true,
   "timers/promises": true,
+  "node:timers": true,
+  "node:timers/promises": true,
   "tls": true,
+  "node:tls": true,
   "tty": true,
+  "node:tty": true,
   "url": true,
+  "node:url": true,
   "util": true,
   "util/types": true,
+  "node:util": true,
+  "node:util/types": true,
   "v8": true,
+  "node:v8": true,
   "vm": true,
+  "node:vm": true,
   "wasi": true,
+  "node:wasi": true,
   "worker_threads": true,
-  "zlib": true
+  "node:worker_threads": true,
+  "zlib": true,
+  "node:zlib": true,
+  "node:test": ">=1.2.6"
 }

src/bun.ts

@@ -21,11 +21,7 @@ export function isBunModule(moduleName: string, bunVersion?: BunVersion): boolea
 }
 
 export function isBunImplementedNodeModule(moduleName: string, bunVersion?: BunVersion): boolean {
-  return checkModule(
-    moduleName.replace(/^node:/, ""),
-    implementedNodeModules,
-    bunVersion ?? currentBunVersion,
-  );
+  return checkModule(moduleName, implementedNodeModules, bunVersion ?? currentBunVersion);
 }
 
 export function isBunBuiltin(moduleName: string, bunVersion?: BunVersion): boolean {

src/generic.ts

@@ -9,11 +9,7 @@ export function isBunModule(moduleName: string, bunVersion?: BunVersion): boolea
 }
 
 export function isBunImplementedNodeModule(moduleName: string, bunVersion?: BunVersion): boolean {
-  return checkModule(
-    moduleName.replace(/^node:/, ""),
-    implementedNodeModules,
-    bunVersion ?? "latest",
-  );
+  return checkModule(moduleName, implementedNodeModules, bunVersion ?? "latest");
 }
 
 export function isBunBuiltin(moduleName: string, bunVersion?: BunVersion): boolean {

test/getModules.test.ts

@@ -16,17 +16,17 @@ describe("Module lists checking", () => {
   const nodeModulesList = Object.keys(bundledNodeModules);
   const builtinModulesList = [...bunModulesList, ...nodeModulesList];
 
-  test("getBunModules() equals keys of bun-modules.json", () => {
-    expect(getBunModules()).toContainAllValues(bunModulesList);
+  test("getBunModules('latest') equals keys of bun-modules.json", () => {
+    expect(getBunModules("latest")).toContainAllValues(bunModulesList);
   });
   test("getBunModules() returns only bun modules", () => {
     getBunModules().forEach((moduleName) => {
       expect(isBunModule(moduleName)).toBe(true);
     });
   });
 
-  test("getBunImplementedNodeModules() equals keys of implemented-node-modules.json", () => {
-    expect(getBunImplementedNodeModules()).toContainAllValues(nodeModulesList);
+  test("getBunImplementedNodeModules('latest') equals keys of implemented-node-modules.json", () => {
+    expect(getBunImplementedNodeModules("latest")).toContainAllValues(nodeModulesList);
   });
   test("getBunImplementedNodeModules() returns only node modules", () => {
     getBunImplementedNodeModules().forEach((moduleName) => {
@@ -37,8 +37,8 @@ describe("Module lists checking", () => {
     expect(getBunImplementedNodeModules("1.0.0")).not.toContain("http2");
   });
 
-  test("getBunBuiltinModules() equals keys of implemented-node-modules.json && bun-modules.json", () => {
-    expect(getBunBuiltinModules()).toContainAllValues(builtinModulesList);
+  test("getBunBuiltinModules('latest') equals keys of implemented-node-modules.json && bun-modules.json", () => {
+    expect(getBunBuiltinModules("latest")).toContainAllValues(builtinModulesList);
   });
   test("getBunBuiltinModules() returns only built-in modules", () => {
     getBunBuiltinModules().forEach((moduleName) => {

test/isBunImplementedNodeModule.test.ts

@@ -8,12 +8,14 @@ describe("Implemented Node modules checking", () => {
     expect(isBunImplementedNodeModule("node:fs")).toBe(true);
     expect(isBunImplementedNodeModule("http2", "1.0.13")).toBe(true);
     expect(isBunImplementedNodeModule("node:http2", "1.0.13")).toBe(true);
+    expect(isBunImplementedNodeModule("node:test", "1.2.6")).toBe(true);
   });
 
   test("Return false for non-node/not implemented modules", () => {
     expect(isBunImplementedNodeModule("bun")).toBe(false);
     expect(isBunImplementedNodeModule("node:bun")).toBe(false);
     expect(isBunImplementedNodeModule("node:http2", "1.0.0")).toBe(false);
     expect(isBunImplementedNodeModule("http2", "1.0.0")).toBe(false);
+    expect(isBunImplementedNodeModule("test", "1.2.6")).toBe(false);
   });
 });