fix(bake): use baked tool name and description in help output - #66
Open
syf2211 wants to merge 1 commit into
Open
fix(bake): use baked tool name and description in help output#66syf2211 wants to merge 1 commit into
syf2211 wants to merge 1 commit into
Conversation
Thread the baked config name and description through BakeConfig into build_argparse so `mcp2cli @<name> -h` shows the tool's prog and custom description instead of the generic mcp2cli banner. Fixes knowsuchagency#63
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.
Summary
Use the baked tool's configured
nameanddescriptionin argparse help output when runningmcp2cli @<name> -h, instead of always showing the genericmcp2clibanner.Motivation
Fixes #63
bake create --descriptionandbake update --descriptionalready persist a description into the baked config, but_run_baked()never passed it through to the CLI parser. As a result,mcp2cli @petstore -hshowedusage: mcp2cliand the default tagline even when a custom description was configured.Changes
BakeConfigwith optionalnameanddescriptionfields_run_baked()from the baked configprog/descriptionparameters tobuild_argparse()handle_mcp()and_handle_openapi_mode()when building the subcommand parserTests
Result: 354 passed
Notes
BakeConfig.bake installwrappers still execmcp2cli @<name>, so this fix also improvespetstore -hfor installed wrappers.