Skip to content

Fail early with click.UsageError when hwqueue pipeline receives no valid mode inputs#135

Draft
Copilot wants to merge 2 commits intoprosenj_cli_hq_eval_report_phase_2from
copilot/sub-pr-99-another-one
Draft

Fail early with click.UsageError when hwqueue pipeline receives no valid mode inputs#135
Copilot wants to merge 2 commits intoprosenj_cli_hq_eval_report_phase_2from
copilot/sub-pr-99-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 6, 2026

When neither --input nor --baseline-dir/--test-dir were provided to the hwqueue pipeline command, the CLI fell through to an else branch that printed Baseline: None / Test: None before failing at pipeline validation — misleading output with no actionable guidance.

Changes

  • Early validation: Added a click.UsageError check before any output is displayed when the invocation doesn't satisfy either mode's requirements.
  • Removed dead branch: Eliminated the unreachable "Unknown Mode" string and simplified the mode-determination logic to the two valid branches.
Error: Must provide either --input for single input mode, or both --baseline-dir and --test-dir for comparison mode.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…eue pipeline

Co-authored-by: prosenjitdhole <239307697+prosenjitdhole@users.noreply.github.com>
Copilot AI changed the title [WIP] Update blank pipeline command and CLI changes Fail early with click.UsageError when hwqueue pipeline receives no valid mode inputs Mar 6, 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