feat(cli): add clients command to list supported AI agent clients#9
feat(cli): add clients command to list supported AI agent clients#9
Conversation
- Add `agent-scanner clients` subcommand with platform availability table - Support --json flag for machine-readable output - Support --current-os flag to filter by current platform - Add cross-platform client catalog independent of build tags - Add bidirectional sync tests to catch catalog drift Closes #8 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds a new agent-scanner clients CLI subcommand to improve discoverability of supported AI agent clients and their OS availability.
Changes:
- Introduces a cross-platform client catalog (
GetAllSupportedClients) including per-OS availability flags. - Adds
agent-scanner clientscommand with--jsonand--current-osoutput modes (table/JSON + current-platform filtering). - Adds tests for catalog/platform sync and CLI formatting/filtering helpers.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| internal/discovery/clients_all.go | Adds the canonical cross-platform catalog of supported clients and platform availability. |
| internal/discovery/clients_all_test.go | Adds sync/consistency tests between the catalog and the platform-specific registries. |
| internal/cli/root.go | Registers the new clients subcommand on the root CLI. |
| internal/cli/flags.go | Adds ClientsFlags and a global clientsFlags instance for the new command. |
| internal/cli/clients.go | Implements agent-scanner clients output (table/JSON) and --current-os filtering. |
| internal/cli/clients_test.go | Tests platform filtering and table/JSON formatting helpers. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Reject unexpected positional args in clients command - Use Cobra OutOrStdout instead of hardcoded os.Stdout Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Use make() instead of nil var so empty results encode as [] not null Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
agent-scanner clientssubcommand that lists all supported AI agent clients with platform availability (macOS/Linux/Windows)--jsonflag for machine-readable output and--current-osflag to filter by current platformclients_all.go) with bidirectional sync tests to catch drift against platform-specific registriesCloses #8
Test plan
make lintpasses with 0 issuesmake testpasses all tests including new onesagent-scanner clientsshows correct table outputagent-scanner clients --jsonoutputs valid JSONagent-scanner clients --current-osfilters correctly🤖 Generated with Claude Code