fix: remove all CJS transpilation, keep only env fix and V8 event loop pump. Clean state for native ESM debugging.

Author: NathanFlurry

packages/nodejs/src/bridge-handlers.ts

@@ -3858,6 +3858,11 @@ export function buildChildProcessBridgeHandlers(deps: ChildProcessBridgeDeps): B
 
 	// Serialize a child process event and push it into the V8 isolate
 	const dispatchEvent = (sessionId: number, type: "stdout" | "stderr" | "exit", data?: Uint8Array | number) => {
+		if (type === "stdout" || type === "stderr") {
+			console.error(`[child-${type}] sessionId=${sessionId} bytes=${(data as Uint8Array)?.length ?? 0} preview=${Buffer.from(data as Uint8Array).toString('utf8').slice(0,80)}`);
+		} else {
+			console.error(`[child-exit] sessionId=${sessionId} code=${data}`);
+		}
 		try {
 			let eventType: "child_stdout" | "child_stderr" | "child_exit";
 			let payload: ChildProcessStreamPayload;
@@ -3930,7 +3935,9 @@ export function buildChildProcessBridgeHandlers(deps: ChildProcessBridgeDeps): B
 
 	handlers[K.childProcessStdinWrite] = (sessionId: unknown, data: unknown) => {
 		const d = data instanceof Uint8Array ? data : Buffer.from(String(data), "base64");
-		sessions.get(Number(sessionId))?.writeStdin(d);
+		const proc = sessions.get(Number(sessionId));
+		console.error(`[stdin-write] sessionId=${sessionId} hasProc=${!!proc} bytes=${d.length} data=${Buffer.from(d).toString('utf8').slice(0,80)}`);
+		proc?.writeStdin(d);
 	};
 
 	handlers[K.childProcessStdinClose] = (sessionId: unknown) => {

packages/nodejs/src/execution-driver.ts

@@ -902,7 +902,7 @@ export class NodeExecutionDriver implements RuntimeDriver {
 	}
 
 	private async waitForManagedResources(): Promise<void> {
-		const graceDeadline = Date.now() + 5000;
+		const graceDeadline = Date.now() + 100;
 
 		// Give async bridge callbacks a moment to register their host-side handles.
 		while (!this.disposed && !this.hasManagedResources() && Date.now() < graceDeadline) {
@@ -1045,15 +1045,8 @@ export class NodeExecutionDriver implements RuntimeDriver {
 		// use CJS exec mode with a dynamic import() wrapper instead of V8's
 		// native ESM "run" mode. The native ESM resolver makes a synchronous
 		// IPC call per import, which is prohibitively slow for packages with
-		// large dependency trees (e.g., PI has hundreds of @sinclair/typebox
-		// sub-modules). CJS mode uses the in-process require-setup transform
-		// which is orders of magnitude faster.
-		// If the code was already CJS-transformed by _resolveEntry (has the
-		// require-esm marker), use exec mode so require() and __dirname work.
-		const REQUIRE_ESM_MARKER = "/*__secure_exec_require_esm__*/";
-		const alreadyTransformed = options.code.startsWith(REQUIRE_ESM_MARKER);
-		const sessionMode = alreadyTransformed ? "exec" : (options.mode === "run" || entryIsEsm ? "run" : "exec");
-		const userCode = entryIsEsm && !alreadyTransformed
+		const sessionMode = options.mode === "run" || entryIsEsm ? "run" : "exec";
+		const userCode = entryIsEsm
 			? options.code
 			: (() => {
 					const transformed = transformSourceForRequireSync(
@@ -1289,7 +1282,7 @@ export class NodeExecutionDriver implements RuntimeDriver {
 			this._currentSession = session;
 
 			// Execute in V8 session
-			console.error(`[exec] mode=${sessionMode} alreadyTransformed=${alreadyTransformed} entryIsEsm=${entryIsEsm} filePath=${options.filePath} codeLen=${userCode.length}`);
+			console.error(`[exec] mode=${sessionMode} entryIsEsm=${entryIsEsm} filePath=${options.filePath} codeLen=${userCode.length}`);
 			const result = await session.execute({
 				bridgeCode,
 				postRestoreScript,
@@ -1330,7 +1323,7 @@ export class NodeExecutionDriver implements RuntimeDriver {
 				},
 			});
 
-			if (!result.error) {
+			if (options.mode === "exec" && !result.error) {
 				await this.waitForManagedResources();
 			}
 

packages/nodejs/src/kernel-runtime.ts

@@ -900,21 +900,6 @@ class NodeRuntimeDriver implements RuntimeDriver {
           if (hostPath) {
             try {
               const content = readFileSync(hostPath, 'utf-8');
-              // Check if this is an ESM module. V8's native ESM resolver is too slow
-              // for large dep trees due to per-module IPC. For overlay ESM scripts
-              // where the CJS transform succeeds, return transformed CJS code instead.
-              if (this._isOverlayEsmEntry(hostPath)) {
-                const transformed = transformSourceForRequireSync(content, scriptPath);
-                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 };
-                }
-                // CJS transform failed (e.g., top-level await). Fall through to
-                // V8 native ESM "run" mode. The V8 runtime pumps the event loop
-                // after module evaluation so timers and callbacks fire.
-                console.error(`[_resolveEntry] ESM→CJS failed, using V8 ESM run mode: ${scriptPath}`);
-              }
               return { code: content, filePath: scriptPath };
             } catch {
               // Fall through to the error below
@@ -928,25 +913,4 @@ class NodeRuntimeDriver implements RuntimeDriver {
     // No script or -e flag — read from stdin (not supported yet)
     throw new Error('node: missing script argument (stdin mode not supported)');
   }
-
-  /**
-   * Check if a host filesystem path points to an ESM module by reading
-   * the nearest package.json for "type": "module". Used to decide whether
-   * to wrap overlay entry scripts in a CJS require() call.
-   */
-  private _isOverlayEsmEntry(hostPath: string): boolean {
-    let dir = dirname(hostPath);
-    for (let i = 0; i < 10; i++) {
-      const pkgJsonPath = join(dir, 'package.json');
-      try {
-        const pkg = JSON.parse(readFileSync(pkgJsonPath, 'utf-8'));
-        return pkg.type === 'module';
-      } catch {
-        const parent = dirname(dir);
-        if (parent === dir) break;
-        dir = parent;
-      }
-    }
-    return false;
-  }
 }