Skip to content

Mcp protocol#167

Open
skatkov wants to merge 21 commits intomainfrom
mcp-protocol
Open

Mcp protocol#167
skatkov wants to merge 21 commits intomainfrom
mcp-protocol

Conversation

@skatkov
Copy link
Owner

@skatkov skatkov commented Feb 1, 2026

No description provided.

Stanislav (Stas) Katkov added 21 commits January 23, 2026 01:10
1. Tool filtering (internal/mcp/tools.go, internal/mcp/executor.go)
   - Blocks devtui.completion, devtui.version, devtui.mcp, devtui.serve
   - Tests verify filtered tools don't appear in tools/list and can't be executed
2. Nested MCP params (internal/mcp/server.go)
   - Added ToolCallParams and ToolCallArguments types to handle MCP's nested arguments structure
   - Maps arguments.input → CallParams.Input, arguments.args → CallParams.Args, arguments.flags → CallParams.Flags
3. Structured count output (cmd/count.go, internal/mcp/executor.go)
   - Added --json flag to count command
   - Auto-injects --json in MCP executor when calling devtui.count
   - Output: {"characters":2231,"spaces":350,"words":294} with labeled fields
4. Cleaned up tool schema (internal/mcp/tools.go)
   - Excludes --json from being exposed to MCP clients (since it's auto-injected)
@skatkov
Copy link
Owner Author

skatkov commented Feb 5, 2026

Interesting ideas could be found here:
https://github.com/goreleaser/mcp/blob/main/mcp.go

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.

1 participant