@jsenv/test 3.7.22 → 3.7.23

package/dist/jsenv_test.js

@@ -9,7 +9,7 @@ import { cpuUsage, memoryUsage } from "node:process";
 import { stripVTControlCharacters } from "node:util";
 import crypto from "node:crypto";
 import { dirname } from "node:path";
-import { createServer } from "node:net";
+import { createConnection, createServer } from "node:net";
 import { spawn, spawnSync, fork } from "node:child_process";
 import { availableParallelism, cpus, totalmem, release, freemem } from "node:os";
 import { SOURCEMAP, generateSourcemapDataUrl } from "@jsenv/sourcemap";
@@ -4184,13 +4184,11 @@ const applyPackageResolve = (packageSpecifier, resolutionContext) => {
   if (packageSpecifier === "") {
     throw new Error("invalid module specifier");
   }
-  if (
-    conditions.includes("node") &&
-    isSpecifierForNodeBuiltin(packageSpecifier)
-  ) {
+  // "node:" prefixed specifiers always resolve to node builtins
+  if (packageSpecifier.startsWith("node:")) {
     return createResolutionResult({
       type: "node_builtin_specifier",
-      url: `node:${packageSpecifier}`,
+      url: packageSpecifier,
     });
   }
   let { packageName, packageSubpath } = parsePackageSpecifier(packageSpecifier);
@@ -4250,6 +4248,17 @@ const applyPackageResolve = (packageSpecifier, resolutionContext) => {
       packageJson,
     });
   }
+  // Bare builtin names (without "node:" prefix) are valid only if no local package found
+  // Local packages always take priority over builtins with the same name
+  if (
+    conditions.includes("node") &&
+    isSpecifierForNodeBuiltin(packageSpecifier)
+  ) {
+    return createResolutionResult({
+      type: "node_builtin_specifier",
+      url: `node:${packageSpecifier}`,
+    });
+  }
   throw createModuleNotFoundError(packageName, resolutionContext);
 };
 
@@ -7203,32 +7212,18 @@ const basicFetch = async (
 };
 
 const pingServer = async (url) => {
-  const server = createServer();
   const { hostname, port } = new URL(url);
 
-  try {
-    await new Promise((resolve, reject) => {
-      server.on("error", reject);
-      server.on("listening", () => {
-        resolve();
-      });
-      server.listen(port, hostname);
+  return new Promise((resolve) => {
+    const socket = createConnection({ hostname, port: Number(port) });
+    socket.on("connect", () => {
+      socket.destroy();
+      resolve(true);
+    });
+    socket.on("error", () => {
+      resolve(false);
     });
-  } catch (error) {
-    if (error && error.code === "EADDRINUSE") {
-      return true;
-    }
-    if (error && error.code === "EACCES") {
-      return true;
-    }
-    throw error;
-  }
-  await new Promise((resolve, reject) => {
-    server.on("error", reject);
-    server.on("close", resolve);
-    server.close();
   });
-  return false;
 };
 
 const startServerUsingCommand = async (

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/test",
-  "version": "3.7.22",
+  "version": "3.7.23",
   "type": "module",
   "repository": {
     "type": "git",
@@ -30,9 +30,9 @@
   },
   "dependencies": {
     "@c88/v8-coverage": "0.1.1",
-    "@jsenv/ast": "6.8.1",
-    "@jsenv/plugin-supervisor": "1.8.2",
-    "@jsenv/sourcemap": "1.3.17",
+    "@jsenv/ast": "6.8.2",
+    "@jsenv/plugin-supervisor": "1.8.3",
+    "@jsenv/sourcemap": "1.3.18",
     "he": "1.2.0",
     "istanbul-lib-coverage": "3.2.2",
     "istanbul-lib-instrument": "6.0.3",

package/src/helpers/ping_server.js

@@ -1,30 +1,16 @@
-import { createServer } from "node:net";
+import { createConnection } from "node:net";
 
 export const pingServer = async (url) => {
-  const server = createServer();
   const { hostname, port } = new URL(url);
 
-  try {
-    await new Promise((resolve, reject) => {
-      server.on("error", reject);
-      server.on("listening", () => {
-        resolve();
-      });
-      server.listen(port, hostname);
+  return new Promise((resolve) => {
+    const socket = createConnection({ hostname, port: Number(port) });
+    socket.on("connect", () => {
+      socket.destroy();
+      resolve(true);
+    });
+    socket.on("error", () => {
+      resolve(false);
     });
-  } catch (error) {
-    if (error && error.code === "EADDRINUSE") {
-      return true;
-    }
-    if (error && error.code === "EACCES") {
-      return true;
-    }
-    throw error;
-  }
-  await new Promise((resolve, reject) => {
-    server.on("error", reject);
-    server.on("close", resolve);
-    server.close();
   });
-  return false;
 };