Skip to content

Conversation

@burmudar
Copy link
Contributor

@burmudar burmudar commented Dec 9, 2025

  • go run ./cmd/src mcp would panic because we didn't check the arg length before using it - now we check it and print usage text
  • make usage consistent and remove the dot

Test plan

tested locally

go run ./cmd/src mcp
NOTE: This command is still experimental
The 'mcp' command exposes MCP tools as subcommands for agents to use.

USAGE:
  src mcp list-tools              List available tools
  src mcp <tool-name> schema      View the input/output schema of a tool
  src mcp <tool-name> <flags>     Invoke a tool with the given flags
  src mcp <tool-name> -h          List the available flags of a tool

@burmudar burmudar requested a review from a team December 9, 2025 11:49
@burmudar burmudar self-assigned this Dec 9, 2025
if no subcommands given print usage
@burmudar burmudar merged commit 81fac14 into main Dec 9, 2025
8 checks passed
@burmudar burmudar deleted the wb/fix-mcp-panic branch December 9, 2025 15:07
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.

3 participants