@schmock/core 2.1.1 → 2.1.3

package/src/builder.ts

@@ -1,5 +1,4 @@
 import type { Server } from "node:http";
-import { createServer } from "node:http";
 import { normalizePath, toHttpMethod } from "./constants.js";
 import {
   errorMessage,
@@ -321,6 +320,17 @@ export class CallableMockInstance {
       );
     }
 
+    // Lazy-load node:http so browser bundles never pull it in. See issue #395.
+    return import("node:http").then(({ createServer }) =>
+      this.#startHttpServer(createServer, port, hostname),
+    );
+  }
+
+  #startHttpServer(
+    createServer: typeof import("node:http").createServer,
+    port: number,
+    hostname: string,
+  ): Promise<Schmock.ServerInfo> {
     const httpServer = createServer((req, res) => {
       const handleRequest = async () => {
         const url = new URL(req.url ?? "/", `http://${req.headers.host}`);

package/src/dist-shape.test.ts

@@ -0,0 +1,35 @@
+import { existsSync, readFileSync } from "node:fs";
+import { resolve } from "node:path";
+import { describe, expect, it } from "vitest";
+
+/**
+ * Regression coverage for issue #395 — a top-level `import { createServer }
+ * from "node:http"` in `dist/builder.js` was preserved by browser bundlers
+ * (esbuild / Angular / Vite) even when `.listen()` was never called, causing
+ * `node:http` resolution failures in browsers.
+ *
+ * Inspect published artifacts: top-level `node:http` imports must not exist.
+ * `node:http` may only appear inside dynamic imports (`await import(...)`).
+ */
+const distRoot = resolve(__dirname, "..", "dist");
+
+// Static ESM imports from node: modules — `from "node:..."`. Dynamic imports
+// (`await import("node:...")`) have no `from` clause and are not matched.
+function staticNodeImports(src: string): string[] {
+  return src.match(/from\s*["']node:[^"']+["']/g) ?? [];
+}
+
+describe.skipIf(!existsSync(distRoot))(
+  "dist shape — browser compatibility (#395)",
+  () => {
+    it("dist/builder.js has no static node:* imports", () => {
+      const src = readFileSync(resolve(distRoot, "builder.js"), "utf8");
+      expect(staticNodeImports(src)).toEqual([]);
+    });
+
+    it("dist/index.js has no static node:* imports", () => {
+      const src = readFileSync(resolve(distRoot, "index.js"), "utf8");
+      expect(staticNodeImports(src)).toEqual([]);
+    });
+  },
+);