donobu 5.61.1 → 5.61.2

package/dist/cli/donobu-cli.d.ts

@@ -18,12 +18,20 @@ declare function ensureReporterValueHasJson(value: string): {
  * caller to fall back to the rerun's own exit code rather than assume success.
  */
 declare function countUnexpectedTests(report: DonobuReport): number | undefined;
+/**
+ * Reconcile a run's process status with its recorded report. The child process
+ * is normally authoritative, but wrappers can surface exit 0 even when the
+ * report records unexpected failures. Escalate only zero exits so infra and
+ * crash codes keep their original signal.
+ */
+declare function reconcileExitCodeWithReport(exitCode: number, report: DonobuReport | null): number;
 /** @internal Exposed for unit tests only — not part of the public API. */
 export declare const _forTesting: {
     countUnexpectedTests: typeof countUnexpectedTests;
     ensureReporterValueHasJson: typeof ensureReporterValueHasJson;
     hasReporterArg: typeof hasReporterArg;
     injectJsonReporterIntoArgs: typeof injectJsonReporterIntoArgs;
+    reconcileExitCodeWithReport: typeof reconcileExitCodeWithReport;
 };
 export {};
 //# sourceMappingURL=donobu-cli.d.ts.map

package/dist/cli/donobu-cli.js

@@ -1512,6 +1512,23 @@ function countUnexpectedTests(report) {
     const unexpected = stats?.unexpected;
     return typeof unexpected === 'number' ? unexpected : undefined;
 }
+/**
+ * Reconcile a run's process status with its recorded report. The child process
+ * is normally authoritative, but wrappers can surface exit 0 even when the
+ * report records unexpected failures. Escalate only zero exits so infra and
+ * crash codes keep their original signal.
+ */
+function reconcileExitCodeWithReport(exitCode, report) {
+    if (exitCode !== 0 || !report) {
+        return exitCode;
+    }
+    const unexpected = countUnexpectedTests(report);
+    if (unexpected !== undefined && unexpected > 0) {
+        Logger_1.appLogger.warn(`Run exited 0 but the report records ${unexpected} failed test(s); exiting non-zero.`);
+        return 1;
+    }
+    return exitCode;
+}
 /**
  * Find a test entry in a report by title + project name. File paths are
  * deliberately not part of the lookup: treatment plans carry absolute paths
@@ -2038,6 +2055,7 @@ async function runTestCommand(cliArgs) {
                     // guarantee the reporter was deferring for.
                     await postDeferredSlackFromInitialRun(playwrightOutputDir);
                 }
+                return reconcileExitCodeWithReport(autoHealOutcome.exitCode, enrichedInitialReport ?? initialDonobuReport);
             }
             return autoHealOutcome.exitCode;
         }
@@ -2268,5 +2286,6 @@ exports._forTesting = {
     ensureReporterValueHasJson,
     hasReporterArg,
     injectJsonReporterIntoArgs,
+    reconcileExitCodeWithReport,
 };
 //# sourceMappingURL=donobu-cli.js.map

package/dist/esm/cli/donobu-cli.d.ts

@@ -18,12 +18,20 @@ declare function ensureReporterValueHasJson(value: string): {
  * caller to fall back to the rerun's own exit code rather than assume success.
  */
 declare function countUnexpectedTests(report: DonobuReport): number | undefined;
+/**
+ * Reconcile a run's process status with its recorded report. The child process
+ * is normally authoritative, but wrappers can surface exit 0 even when the
+ * report records unexpected failures. Escalate only zero exits so infra and
+ * crash codes keep their original signal.
+ */
+declare function reconcileExitCodeWithReport(exitCode: number, report: DonobuReport | null): number;
 /** @internal Exposed for unit tests only — not part of the public API. */
 export declare const _forTesting: {
     countUnexpectedTests: typeof countUnexpectedTests;
     ensureReporterValueHasJson: typeof ensureReporterValueHasJson;
     hasReporterArg: typeof hasReporterArg;
     injectJsonReporterIntoArgs: typeof injectJsonReporterIntoArgs;
+    reconcileExitCodeWithReport: typeof reconcileExitCodeWithReport;
 };
 export {};
 //# sourceMappingURL=donobu-cli.d.ts.map

package/dist/esm/cli/donobu-cli.js

@@ -1512,6 +1512,23 @@ function countUnexpectedTests(report) {
     const unexpected = stats?.unexpected;
     return typeof unexpected === 'number' ? unexpected : undefined;
 }
+/**
+ * Reconcile a run's process status with its recorded report. The child process
+ * is normally authoritative, but wrappers can surface exit 0 even when the
+ * report records unexpected failures. Escalate only zero exits so infra and
+ * crash codes keep their original signal.
+ */
+function reconcileExitCodeWithReport(exitCode, report) {
+    if (exitCode !== 0 || !report) {
+        return exitCode;
+    }
+    const unexpected = countUnexpectedTests(report);
+    if (unexpected !== undefined && unexpected > 0) {
+        Logger_1.appLogger.warn(`Run exited 0 but the report records ${unexpected} failed test(s); exiting non-zero.`);
+        return 1;
+    }
+    return exitCode;
+}
 /**
  * Find a test entry in a report by title + project name. File paths are
  * deliberately not part of the lookup: treatment plans carry absolute paths
@@ -2038,6 +2055,7 @@ async function runTestCommand(cliArgs) {
                     // guarantee the reporter was deferring for.
                     await postDeferredSlackFromInitialRun(playwrightOutputDir);
                 }
+                return reconcileExitCodeWithReport(autoHealOutcome.exitCode, enrichedInitialReport ?? initialDonobuReport);
             }
             return autoHealOutcome.exitCode;
         }
@@ -2268,5 +2286,6 @@ exports._forTesting = {
     ensureReporterValueHasJson,
     hasReporterArg,
     injectJsonReporterIntoArgs,
+    reconcileExitCodeWithReport,
 };
 //# sourceMappingURL=donobu-cli.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "donobu",
-  "version": "5.61.1",
+  "version": "5.61.2",
   "description": "Create browser automations with an LLM agent and replay them as Playwright scripts.",
   "main": "dist/main.js",
   "module": "dist/esm/main.js",