fix: wait for child stdio close before resolving exec

Resolve on `cp.on("close", ...)` instead of `cp.on("exit", ...)`. `exit` fires before stdio streams flush, so the `appendFileSync` calls in the data handler may not have written the last lines to the log file yet — leading to flaky failures where the post-FAIL summary block reads an incomplete log (seen on PR #1854 CI run).

Author: firecow

src/job.ts

@@ -1165,7 +1165,9 @@ If you know what you're doing and would like to suppress this warning, use one o
                 cp.stdout?.pipe(split2()).on("data", (e: string) => outFunc(e, writeStreams.stdout.bind(writeStreams), (s) => chalk`{greenBright ${s}}`));
                 cp.stderr?.pipe(split2()).on("data", (e: string) => outFunc(e, writeStreams.stderr.bind(writeStreams), (s) => chalk`{redBright ${s}}`));
             }
-            void cp.on("exit", (code) => {
+            // Wait for "close" rather than "exit" so all stdout/stderr data events
+            // have flushed to the output log file before we resolve.
+            void cp.on("close", (code) => {
                 clearTimeout(this._longRunningSilentTimeout);
                 return resolve(code ?? 0);},
             );