fix: readline named exports, process.env in ESM run mode, ESM exec mode detection, grace period. Remaining: V8 run mode event loop

Author: NathanFlurry

packages/nodejs/src/builtin-modules.ts

@@ -138,6 +138,10 @@ const KNOWN_BUILTIN_MODULES = new Set([
  * `import { readFile } from 'fs'` works inside the isolate.
  */
 export const BUILTIN_NAMED_EXPORTS: Record<string, string[]> = {
+	readline: [
+		"createInterface",
+		"promises",
+	],
 	fs: [
 		"promises",
 		"readFileSync",

packages/nodejs/src/execution-driver.ts

@@ -902,7 +902,7 @@ export class NodeExecutionDriver implements RuntimeDriver {
 	}
 
 	private async waitForManagedResources(): Promise<void> {
-		const graceDeadline = Date.now() + 100;
+		const graceDeadline = Date.now() + 5000;
 
 		// Give async bridge callbacks a moment to register their host-side handles.
 		while (!this.disposed && !this.hasManagedResources() && Date.now() < graceDeadline) {
@@ -1330,7 +1330,7 @@ export class NodeExecutionDriver implements RuntimeDriver {
 				},
 			});
 
-			if (options.mode === "exec" && !result.error) {
+			if (!result.error) {
 				await this.waitForManagedResources();
 			}
 

packages/nodejs/src/kernel-runtime.ts

@@ -905,14 +905,24 @@ class NodeRuntimeDriver implements RuntimeDriver {
               // where the CJS transform succeeds, return transformed CJS code instead.
               if (this._isOverlayEsmEntry(hostPath)) {
                 const transformed = transformSourceForRequireSync(content, scriptPath);
-                // Check if the transform actually converted ESM to CJS (not just stripped shebang).
-                // The require-setup marker indicates a successful ESM→CJS conversion.
                 const REQUIRE_ESM_MARKER = "/*__secure_exec_require_esm__*/";
                 if (transformed.startsWith(REQUIRE_ESM_MARKER)) {
                   console.error(`[_resolveEntry] ESM→CJS transform OK: ${scriptPath} (${content.length}→${transformed.length})`);
                   return { code: transformed, filePath: scriptPath };
                 }
-                console.error(`[_resolveEntry] ESM→CJS transform skipped: ${scriptPath} (${content.length} bytes, no marker), using V8 ESM`);
+                // CJS transform failed (e.g., top-level await). Wrap in a CJS
+                // async launcher that imports the module and then waits for active
+                // handles (event loop pump). This runs in exec mode with full
+                // event loop support via _waitForActiveHandles.
+                console.error(`[_resolveEntry] ESM async launcher: ${scriptPath} (${content.length} bytes)`);
+                const launcher = [
+                  `/*__secure_exec_require_esm__*/`,
+                  `(async function() {`,
+                  `  try { await __dynamicImport(${JSON.stringify(scriptPath)}, ${JSON.stringify(scriptPath)}); }`,
+                  `  catch(e) { process.stderr.write(String(e.stack || e.message) + "\\n"); process.exit(1); }`,
+                  `})();`,
+                ].join("\n");
+                return { code: launcher, filePath: scriptPath };
               }
               return { code: content, filePath: scriptPath };
             } catch {

packages/nodejs/src/module-source.ts

@@ -265,7 +265,8 @@ export function transformSourceForRequireSync(
 
 	try {
 		return transformSync(normalizedSource, getRequireTransformOptions(filePath, syntax)).code;
-	} catch {
+	} catch (e) {
+		console.error(`[transformSourceForRequireSync] FAILED for ${filePath}: ${(e as Error).message?.slice(0, 200)}`);
 		return normalizedSource;
 	}
 }