Skip to content

Commit 374a148

Browse files
committed
fix(devx): let the self-test's failure branch reach the verdict handshake
The handshake guards a `return` that skips the verdict block, not the verdict's direction; leaving the flag false on a red printed "never finished" over a run that finished and failed. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012zGPuVVX3deAx9LdjK8jCk
1 parent de225e8 commit 374a148

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

scripts/check-cli-examples-parity.mjs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -921,6 +921,11 @@ function selfTest() {
921921
for (const c of failed) console.error(` ✗ ${c.name}${c.detail ? ` -- ${c.detail}` : ''}`);
922922
if (failed.length) {
923923
console.error(`✗ check-cli-examples-parity self-test: ${failed.length} of ${cases.length} case(s) failed.`);
924+
// A verdict was REACHED and it is red. The handshake below is about a
925+
// `return` that skips this block entirely, not about which way the verdict
926+
// went -- leaving the flag false here would print "never finished" over a
927+
// run that finished and failed, which is a different (and untrue) claim.
928+
selfTestReachedVerdict = true;
924929
return EXIT_FINDINGS;
925930
}
926931
console.log(

0 commit comments

Comments
 (0)