refactor(cyclopts): take the tokens to parse as an argument

[tsvikas]

main() read sys.argv[1:] itself, so a test had to monkeypatch
sys.argv to reach it. tokens defaults to None, which cyclopts already
resolves to sys.argv[1:], so the entry point is unaffected. This is the
shape the upstream recipe settled on in BrianPugh/cyclopts#875.

The unhandled-error test stubbed app with a keyword-only signature, so
the call raised TypeError over the signature rather than the RuntimeError
the test is about. Both exit 70, so it passed either way.

The CycloptsError clause now records that cyclopts >=5 exits 2 on parse
errors by itself, per BrianPugh/cyclopts#874, so it and
exit_on_error=False can both go once the dependency requires it.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01AHLfbxCaEcA1NzUEHCHPpK

[sourcery-ai]

Hey - I've reviewed your changes and they look great!

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}