Skip to content

fix: validate CLI options and output formats - #17

Merged
rogerchappel merged 3 commits into
mainfrom
agent/oss-1055-validate-cli-options
Jul 25, 2026
Merged

fix: validate CLI options and output formats#17
rogerchappel merged 3 commits into
mainfrom
agent/oss-1055-validate-cli-options

Conversation

@rogerchappel

Copy link
Copy Markdown
Owner

Summary

  • reject options that are not supported by the selected command
  • validate required option values and restrict output formats to markdown or json
  • reject inline and separate values supplied to boolean flags
  • document the complete accepted option set

Verification

  • Tests or checks run:
    • npm test
    • bash scripts/validate.sh
    • git diff --check
    • end-to-end invalid format, misspelled option, and boolean-value CLI probes (all exit 1 with actionable stderr)
    • valid init, scan, report, and doctor CLI probes (all exit 0)
  • Manual review completed:

Risk Level

  • Low
  • Medium
  • High

Notes:

This intentionally tightens invalid CLI input handling. Documented invocations remain accepted.

Rollback Plan

  • Revert the three commits in this PR.

Human Decision Needed

  • None
  • Maintainer review
  • Product/design decision
  • Security/privacy review
  • Other:

Commits

  • test: cover invalid CLI options
  • fix: validate command line options
  • docs: document accepted CLI options

@rogerchappel
rogerchappel merged commit e0bc9f9 into main Jul 25, 2026
1 check passed
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.

1 participant