donobu 5.61.1 → 5.61.3

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

@@ -1510,7 +1510,45 @@ async function attemptAutoHealRun(params) {
 function countUnexpectedTests(report) {
     const stats = report.stats;
     const unexpected = stats?.unexpected;
-    return typeof unexpected === 'number' ? unexpected : undefined;
+    if (typeof unexpected === 'number') {
+        return unexpected;
+    }
+    const suites = (report.suites ?? []);
+    if (suites.length === 0) {
+        return undefined;
+    }
+    let failures = 0;
+    for (const suite of suites) {
+        for (const spec of (0, reportWalk_1.collectSpecs)(suite)) {
+            for (const test of spec.tests ?? []) {
+                const status = (0, reportWalk_1.statusOf)(test);
+                if (status === 'failed' ||
+                    status === 'timedOut' ||
+                    status === 'interrupted' ||
+                    status === 'unknown') {
+                    failures += 1;
+                }
+            }
+        }
+    }
+    return failures;
+}
+/**
+ * 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
@@ -2038,6 +2076,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 +2307,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

@@ -1510,7 +1510,45 @@ async function attemptAutoHealRun(params) {
 function countUnexpectedTests(report) {
     const stats = report.stats;
     const unexpected = stats?.unexpected;
-    return typeof unexpected === 'number' ? unexpected : undefined;
+    if (typeof unexpected === 'number') {
+        return unexpected;
+    }
+    const suites = (report.suites ?? []);
+    if (suites.length === 0) {
+        return undefined;
+    }
+    let failures = 0;
+    for (const suite of suites) {
+        for (const spec of (0, reportWalk_1.collectSpecs)(suite)) {
+            for (const test of spec.tests ?? []) {
+                const status = (0, reportWalk_1.statusOf)(test);
+                if (status === 'failed' ||
+                    status === 'timedOut' ||
+                    status === 'interrupted' ||
+                    status === 'unknown') {
+                    failures += 1;
+                }
+            }
+        }
+    }
+    return failures;
+}
+/**
+ * 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
@@ -2038,6 +2076,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 +2307,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.3",
   "description": "Create browser automations with an LLM agent and replay them as Playwright scripts.",
   "main": "dist/main.js",
   "module": "dist/esm/main.js",