feat: add waza models command to list available models#208
Merged
Conversation
- Create focused 5-minute Quick Start page at site/src/content/docs/quick-start.mdx - Add installation options (binary, from source, azd extension) - Include authentication, first skill creation, minimal eval YAML - Add Mermaid workflow diagram - Include workflow steps: install → auth → create → write → run → view - Place Quick Start as first item in sidebar navigation - Update homepage to prominently link Quick Start guide - Site builds successfully with 17 pages Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
spboyer
requested review from
chlowell,
richardpark-msft and
wbreza
as code owners
Add a new `waza models` command that queries the Copilot SDK for
available models and displays them as a formatted table (or JSON with
--json flag). The table shows model ID, name, vision support, and
context window size.
Changes:
- Add ListModels to CopilotClient interface and copilotClientWrapper
- Add ListModels method on CopilotEngine
- Regenerate gomock mocks for both internal and cmd packages
- Register `models` subcommand in root.go
- Handle auth errors gracefully ("run copilot login first")
- Add 7 tests covering table output, JSON, empty list, auth errors,
backend errors, and token formatting
- Update CLI reference docs (site/src/content/docs/reference/cli.mdx)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
spboyer
force-pushed
the
squad/141-models-command
branch
from
f597ac5 to
9e6f107
Compare
Run gofmt to fix formatting issues flagged by CI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add //nolint:errcheck for fmt.Fprintln/Fprintf calls matching existing patterns in cmd_check.go. Run gofmt on both files. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #141
What
Adds a new
waza modelscommand that queries the Copilot SDK for available models and displays them as a formatted table.Table output
JSON output (
--jsonflag)For scripting and piping.
Changes
internal/execution/copilot_client_wrappers.go— AddedListModelstoCopilotClientinterface + wrapper implementationcmd/waza/cmd_models.go— New command with table/JSON output, auth error handlingcmd/waza/cmd_models_test.go— 7 tests: table, JSON, empty, auth error, not authenticated, backend error, token formattingcmd/waza/root.go— Registermodelssubcommandinternal/executionandcmd/wazasite/src/content/docs/reference/cli.mdx— CLI reference updatedAuth handling
If not authenticated, the command returns:
Testing
go test ./...— all passgo vet ./...— cleancd site && npm run build— cleanWorking as Linus (Backend Developer)
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com