Skip to content

feat(harness): Explain multi-select keys in harness setup - #133

Open
ShivainVij wants to merge 1 commit into
mainfrom
shivain/harness-select-instruction
Open

ShivainVij wants to merge 1 commit into
mainfrom
shivain/harness-select-instruction

Conversation

@ShivainVij

@ShivainVij ShivainVij commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Adding a small comment as part of Validation Error to explain how to use the multi-select harness picker.

Before
image

After:
image

@ShivainVij
ShivainVij requested review from cretz and a balanced review from Copilot October 6, 2026 21:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The focused message update is accurate and introduces no functional risk.

Review effort: Balanced
Findings: None

What changed in this PR

Clarifies keyboard controls when harness setup requires at least one selection.

Changes:

  • Adds Space and Enter usage hints to the validation error.
File Description
internal/​cmd/​command.harness.go Clarifies multi-select validation guidance.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants