fix: CodeRabbit auto-fixes for PR #772

tests/codegen-deno/deno_scripting.harness.mjs

@@ -27,6 +27,7 @@ const fs = {
 };
 
 const fsStub = {
+  /** Returns the in-memory `fs` entries for `path` as Dirent-like objects, or throws ENOENT. */
   readdirSync: (path) => {
     const entries = fs[path];
     if (!entries) {
@@ -40,17 +41,24 @@ const fsStub = {
       isDirectory: () => e.isDirectory,
     }));
   },
+  /** No-op stand-in; this corpus never asserts on directory creation. */
   mkdirSync: () => {},
+  /** No-op stand-in; this corpus never asserts on file writes. */
   writeFileSync: () => {},
+  /** No-op stand-in; this corpus never asserts on file removal. */
   rmSync: () => {},
+  /** Returns an empty buffer; this corpus never asserts on file contents. */
   readFileSync: () => new Uint8Array(),
+  /** Always throws ENOENT; this corpus never exercises a successful stat. */
   statSync: (path) => {
     const err = new Error(`stub: no such path ${path}`);
     err.code = "ENOENT";
     throw err;
   },
 };
 
+// Intercepts the Bun-ESM prelude's lazy `node:fs` resolution so `fsStub`
+// backs the emitted FS calls; delegates any other builtin to the real host.
 const realGetBuiltinModule = process.getBuiltinModule?.bind(process);
 process.getBuiltinModule = (name) => {
   if (name === "node:fs") return fsStub;
@@ -65,6 +73,7 @@ process.getBuiltinModule = (name) => {
 const realArgv = process.argv;
 process.argv = ["node", "deno_scripting.harness.mjs", "alpha", "beta", "gamma"];
 let lastExit = null;
+/** Captures the exit code instead of terminating the harness process. */
 process.exit = (code) => { lastExit = code; };
 
 // Capture stderr writes from consoleError.

tests/codegen-deno/deno_scripting_part2.harness.mjs

@@ -18,6 +18,7 @@ const files = {
 };
 
 const fsStub = {
+  /** Returns size/isFile/isDirectory for a known path, or throws ENOENT. */
   statSync: (path) => {
     const e = files[path];
     if (!e) {
@@ -31,6 +32,7 @@ const fsStub = {
       isDirectory: () => e.kind === "dir",
     };
   },
+  /** Returns the stubbed bytes for a known file path, or throws ENOENT. */
   readFileSync: (path) => {
     const e = files[path];
     if (!e || e.kind !== "file") {
@@ -42,6 +44,8 @@ const fsStub = {
   },
 };
 
+// Intercepts the Bun-ESM prelude's lazy `node:fs` resolution so `fsStub`
+// backs the emitted FS calls; delegates any other builtin to the real host.
 const realGetBuiltinModule = process.getBuiltinModule?.bind(process);
 process.getBuiltinModule = (name) => {
   if (name === "node:fs") return fsStub;

tools/run_codegen_bun_tests.sh

@@ -24,6 +24,8 @@ if ! command -v bun >/dev/null 2>&1; then
 fi
 
 failures=()
+# Records a check failure and echoes it as a `::error::` annotation, without
+# aborting the script, so later checks still run (fail-late).
 fail() {
   failures+=("$1")
   echo "::error::$1" >&2