-
Notifications
You must be signed in to change notification settings - Fork 147
Description
Issue Description
Type: Documentation inconsistency
Priority: Low
Current State
The documentation lists secrets as a Setup Command group, but this is not reflected in the command group tests.
Evidence
Documentation (cli.md line 525):
secretsCmd.GroupID = "setup"Test file (command_groups_test.go):
The secrets command is not included in the test cases that verify command group assignments (lines 12-48). The test covers:
- Setup: init, new, add, remove, update
- Development: compile, mcp, status, mcp-server
- Execution: run, enable, disable, trial
- Analysis: logs, audit
- Utilities: pr
The secrets command is missing from this list.
Actual Impact
Minor discrepancy between documentation and test coverage. The command may be properly assigned to the "setup" group in the code, but the test doesn't verify this, which could lead to regressions if the group assignment is accidentally removed.
Expected Behavior
The secrets command should be included in the command group assignment tests in cmd/gh-aw/command_groups_test.go.
Suggested Fix
Add test case in command_groups_test.go:
{name: "secrets command in setup group", commandName: "secrets", expectedGroup: "setup", shouldHaveGroup: true},This should be added to the "Setup Commands" section of the test (around line 24).
AI generated by CLI Consistency Checker