Skip to content

Fix CLI consistency: docs alignment, emoji removal, show_default#266

Merged
danielmeppiel merged 3 commits intomainfrom
copilot/cli-consistency-fix-apm-pack-docs
Mar 12, 2026
Merged

Fix CLI consistency: docs alignment, emoji removal, show_default#266
danielmeppiel merged 3 commits intomainfrom
copilot/cli-consistency-fix-apm-pack-docs

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

Description

Addresses all 6 findings from the CLI Consistency Report — 2 medium, 4 low severity.

Documentation alignment (docs/src/content/docs/reference/cli-commands.md):

  • apm pack --target: [vscode|claude|all][copilot|vscode|claude|all] + alias note
  • apm runtime setup/remove: RUNTIME_NAME [OPTIONS][OPTIONS] {copilot|codex|llm} to match Click output
  • apm unpack --output: --output TEXT-o, --output PATH
  • --parallel-downloads: INTINTEGER

Code fixes:

  • src/apm_cli/commands/mcp.py: Added show_default=True to --limit so [default: 10] appears in help, matching docs
  • src/apm_cli/commands/config.py: Removed ⚙️ from config table title per no-emoji policy

Also updated docs/src/content/docs/guides/pack-distribute.md target tables to include copilot.

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Maintenance / refactor

Testing

  • Tested locally
  • All existing tests pass
  • Added tests for new functionality (if applicable)

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

- Fix pack --target docs to include copilot option and alias note
- Fix runtime setup/remove synopsis to match CLI ({copilot|codex|llm})
- Fix unpack --output docs to show -o shorthand and PATH type
- Add show_default=True to mcp search --limit option
- Fix --parallel-downloads type from INT to INTEGER in docs
- Remove emoji from apm config table title
- Update pack-distribute guide target tables

Co-authored-by: danielmeppiel <51440732+danielmeppiel@users.noreply.github.com>
Copilot AI changed the title [WIP] [cli-consistency] Fix documentation for apm pack --target options Fix CLI consistency: docs alignment, emoji removal, show_default Mar 12, 2026
Copilot AI requested a review from danielmeppiel March 12, 2026 20:23
@danielmeppiel danielmeppiel marked this pull request as ready for review March 12, 2026 20:31
Copilot AI review requested due to automatic review settings March 12, 2026 20:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Aligns CLI documentation with actual Click-generated help output and enforces the repo’s no-emoji CLI output policy in one config display, improving consistency across docs and CLI UX.

Changes:

  • Updated CLI reference docs to match current option metavars/signatures and runtime command usage output.
  • Adjusted apm pack target docs/guides to include copilot and note vscode as an alias.
  • Minor CLI help/UI tweaks: show default for apm mcp search --limit and remove an emoji from the config table title.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/apm_cli/commands/mcp.py Adds show_default=True for --limit so help displays the default value.
src/apm_cli/commands/config.py Removes emoji from the Rich table title for config display.
docs/src/content/docs/reference/cli-commands.md Updates CLI reference signatures/usages to better match Click output (targets, metavars, runtime usage).
docs/src/content/docs/guides/pack-distribute.md Updates pack/distribute guide tables to include copilot target and vscode alias note.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@danielmeppiel danielmeppiel merged commit 9ecd177 into main Mar 12, 2026
8 checks passed
@danielmeppiel danielmeppiel deleted the copilot/cli-consistency-fix-apm-pack-docs branch March 12, 2026 20:40
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.

[cli-consistency] CLI Consistency Report — 2026-03-11

3 participants