Skip to content

fix(cli): update help examples binary name#98

Open
sarvesh1327 wants to merge 3 commits into
bug-ops:masterfrom
sarvesh1327:fix/help-examples-binary-name
Open

fix(cli): update help examples binary name#98
sarvesh1327 wants to merge 3 commits into
bug-ops:masterfrom
sarvesh1327:fix/help-examples-binary-name

Conversation

@sarvesh1327
Copy link
Copy Markdown

Summary

  • Update clap help examples for introspect, generate, and skill to use the published mcp-execution-cli binary name.
  • Add a regression test that renders long help for those subcommands and rejects the old mcp-cli examples.

Test Plan

  • cargo test -p mcp-execution-cli test_cli_help_examples_use_published_binary_name -- --nocapture
  • cargo test -p mcp-execution-cli
  • cargo fmt --all --check
  • cargo clippy -p mcp-execution-cli --all-targets -- -D warnings
  • Manual help checks for introspect, generate, and skill

Closes #93

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   89.82%   89.84%   +0.01%     
==========================================
  Files          33       33              
  Lines        7817     7830      +13     
==========================================
+ Hits         7022     7035      +13     
  Misses        795      795              
Flag Coverage Δ
mcp-cli 89.84% <100.00%> (+0.01%) ⬆️
mcp-codegen 89.84% <100.00%> (+0.01%) ⬆️
mcp-core 89.84% <100.00%> (+0.01%) ⬆️
mcp-files 89.84% <100.00%> (+0.01%) ⬆️
mcp-introspector 89.84% <100.00%> (+0.01%) ⬆️
mcp-server 89.84% <100.00%> (+0.01%) ⬆️
mcp-skill 89.84% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
crates/mcp-cli/src/cli.rs 94.55% <100.00%> (+0.37%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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: --help examples reference old binary name mcp-cli instead of mcp-execution-cli

2 participants