fix: rename test agent alias to test-mcp #202
Merged
Conversation
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
The 'test' identifier was both a subcommand (for running test scenarios) and a built-in agent alias, causing ambiguous parsing. For example, `adcp test get_adcp_capabilities` would incorrectly route to the test subcommand instead of calling the tool on the test agent. Renamed the MCP test agent alias to 'test-mcp' to match the 'test-a2a' convention and eliminate the conflict. The test subcommand now has a clear, unambiguous namespace. - Renamed BUILT_IN_AGENTS.test → test-mcp - Updated all help text and examples - Verified both test-mcp and test-a2a agents work with actual tool calls
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.
The 'test' identifier was both a subcommand (for running test scenarios) and a built-in agent alias, causing ambiguous CLI parsing. For example,
adcp test get_adcp_capabilitiesincorrectly routed to the test subcommand.Renamed the MCP test agent alias to 'test-mcp' to match the 'test-a2a' naming convention and eliminate the conflict. The test subcommand now has a clear namespace.