-
Notifications
You must be signed in to change notification settings - Fork 262
Description
Issue Description
Type: Inconsistency in command capitalization
Priority: Low
Current State
The CLI commands have inconsistent capitalization in their Short descriptions when referring to "agentic workflows" or similar terms.
Evidence from Source Code
Commands using lowercase "agentic workflows":
init_command.go(line 17): "Initialize repository for agentic workflows"add_command.go(line 23): "Add agentic workflows from repositories to .github/workflows"trial_command.go(line 45): "Trial one or more agentic workflows as if..."fix_command.go(line 34): "Apply automatic codemod-style fixes to agentic workflow files"update_command.go(line 17): "Update agentic workflows from their source repositories..."logs_command.go(line 27): "Download and analyze agentic workflow logs..."status.go(line 12): "Show status of all agentic workflows in the repository"mcp_add.go(line 306): "Add an MCP tool to an agentic workflow"mcp_list.go(line 224): "List MCP servers defined in agentic workflows"
Commands using capitalized terms:
enable_cmd(main.go line 132): "Enable agentic workflows"disable_cmd(main.go line 151): "Disable agentic workflows and cancel..."
Issue
The product name "GitHub Agentic Workflows" should be consistently capitalized according to the capitalization guidelines in specs/capitalization.md which states:
Product Name: "GitHub Agentic Workflows" (always capitalize)
Feature Names: Use sentence case (e.g., "safe output messages")
When referring to the workflows themselves (not the product), it appears the codebase uses lowercase "agentic workflows" in most places, but enable and disable commands use this as if it were part of the product name.
Actual Impact
Minor inconsistency in CLI help text. Users may perceive this as lack of polish.
Expected Behavior
Consistent capitalization throughout. Since these are referring to workflow artifacts (not the product itself), they should use lowercase: "agentic workflows"
Suggested Fix
Update the Short descriptions in main.go:
- Line 132:
"Enable agentic workflows"(lowercase 'agentic') - Line 151:
"Disable agentic workflows and cancel..."(lowercase 'agentic')
This matches the pattern used in all other command descriptions.
AI generated by CLI Consistency Checker