roykoand opened a new pull request, #73105: URL: https://github.com/apache/airflow/pull/73105
`ARG_LIMIT` (used by `airflow dags list-jobs --limit`) had no `type=` validation, unlike its siblings `ARG_JOB_LIMIT` and `ARG_NUM_EXECUTIONS`, which both use `type=positive_int(...)`. A non-numeric `--limit` value (e.g. `airflow dags list-jobs --limit abc`) was passed through argparse as a raw string and crashed deep inside SQLAlchemy's limit-clause coercion with an unhandled `ValueError`/traceback, instead of the standard argparse usage error every other typed numeric flag produces. This adds the same `type=positive_int(allow_zero=False)` to `ARG_LIMIT`, plus a regression test asserting `--limit abc` now exits with argparse's usage-error code (2) instead of crashing. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Sonnet 5) Generated-by: Claude Code (Sonnet 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
