Skip to content

Use exit code 2 for errors, reserve exit code 1 for check failures#15

Merged
simonw merged 2 commits intomainfrom
claude/rodney-error-codes-Li6kP
Feb 17, 2026
Merged

Use exit code 2 for errors, reserve exit code 1 for check failures#15
simonw merged 2 commits intomainfrom
claude/rodney-error-codes-Li6kP

Conversation

@simonw
Copy link
Owner

@simonw simonw commented Feb 17, 2026

Rodney now uses distinct exit codes: 0 for success, 1 for check/assertion
failures (exists, visible, ax-find), and 2 for errors (bad arguments, no
browser session, timeouts, etc.). This lets CI scripts distinguish between
"the check reported false" and "something broke" using set -e.

Adds exit codes documentation to help.txt and a new "Using Rodney for
checks" section to the README with examples of each check command and a
shell script showing how to combine them for CI.

https://claude.ai/code/session_01Fpw4mXBew12PXPuMcbPpoQ

Rodney now uses distinct exit codes: 0 for success, 1 for check/assertion
failures (exists, visible, ax-find), and 2 for errors (bad arguments, no
browser session, timeouts, etc.). This lets CI scripts distinguish between
"the check reported false" and "something broke" using set -e.

Adds exit codes documentation to help.txt and a new "Using Rodney for
checks" section to the README with examples of each check command and a
shell script showing how to combine them for CI.

https://claude.ai/code/session_01Fpw4mXBew12PXPuMcbPpoQ
Demonstrates all three exit codes against a sample checkout page:
exit 0 for passing checks (exists, visible, ax-find), exit 1 for
failing checks (missing element, hidden element, no a11y match),
and exit 2 for errors (unknown command, missing args, no session).
Includes a combined script showing how to collect failures in CI.

https://claude.ai/code/session_01Fpw4mXBew12PXPuMcbPpoQ
@simonw simonw merged commit 107d385 into main Feb 17, 2026
2 checks passed
@simonw simonw mentioned this pull request Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants