🧪 test: harden bun:sqlite bundle regression test

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>

Author: warengonzaga

tests/runtime.test.ts

@@ -6,22 +6,30 @@ import { describe, expect, test } from "bun:test";
 import { mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { fileURLToPath } from "node:url";
 
 describe("runtime bundling", () => {
 	test("does not emit a static bun:sqlite import in Node bundles", async () => {
 		const workDir = mkdtempSync(join(tmpdir(), "config-engine-runtime-"));
 		const entryPath = join(workDir, "entry.ts");
 		const outDir = join(workDir, "dist");
-		// Use a file:// URL so the import path is portable across platforms (including Windows).
-		const runtimeUrl = new URL("../src/runtime.ts", import.meta.url).href;
+		// Resolve through a file:// URL, then back to a filesystem path: the bundler
+		// resolves plain paths, and JSON.stringify escapes Windows separators safely.
+		const runtimePath = fileURLToPath(new URL("../src/runtime.ts", import.meta.url));
 
 		try {
 			mkdirSync(outDir, { recursive: true });
-			// Import openDatabase (not just isBun) so the bundler cannot tree-shake
-			// the SQLite adapter code — this ensures the bun:sqlite guard is actually tested.
+			// The bundle is emitted as ESM, so mark the output directory as a module
+			// package, otherwise Node parses the .js output as CommonJS and fails.
+			writeFileSync(join(outDir, "package.json"), `${JSON.stringify({ type: "module" })}\n`);
+			// Actually reference openDatabase (not just import it) so the bundler cannot
+			// tree-shake the SQLite adapter code, so the bun:sqlite guard is really
+			// exercised by the bundle.
 			writeFileSync(
 				entryPath,
-				`import { isBun, openDatabase } from ${JSON.stringify(runtimeUrl)};\nconsole.log(isBun());\n`,
+				`import { isBun, openDatabase } from ${JSON.stringify(runtimePath)};\n` +
+					"console.log(isBun());\n" +
+					"console.log(typeof openDatabase);\n",
 			);
 
 			const result = await Bun.build({
@@ -37,15 +45,20 @@ describe("runtime bundling", () => {
 
 			const outputPath = join(outDir, "entry.js");
 			const bundled = readFileSync(outputPath, "utf8");
-			expect(bundled).not.toMatch(/(?:from|import)\s+["']bun:sqlite["']/);
+			// Allow for minified output: `from"bun:sqlite"`, `import"bun:sqlite"`,
+			// and `require("bun:sqlite")` must all be absent.
+			expect(bundled).not.toMatch(/(?:from|import|require\s*\()\s*["']bun:sqlite["']/);
 
 			const run = Bun.spawnSync(["node", outputPath], {
 				stdout: "pipe",
 				stderr: "pipe",
 			});
 
 			expect(run.exitCode).toBe(0);
-			expect(new TextDecoder().decode(run.stdout).trim()).toBe("false");
+			expect(new TextDecoder().decode(run.stdout).trim().split(/\r?\n/)).toEqual([
+				"false",
+				"function",
+			]);
 		} finally {
 			rmSync(workDir, { recursive: true, force: true });
 		}