-
Notifications
You must be signed in to change notification settings - Fork 709
Comparing changes
Open a pull request
base repository: mark3labs/mcp-go
base: v0.37.0
head repository: mark3labs/mcp-go
compare: v0.38.0
- 7 commits
- 14 files changed
- 9 contributors
Commits on Aug 6, 2025
-
fix: resolve stdio transport race condition for concurrent tool calls (…
…#529) * fix: resolve stdio transport race condition for concurrent tool calls Fixes #528 by replacing goroutine-per-request pattern with a worker pool to prevent multiple goroutines from accessing the non-thread-safe bufio.Reader. - Add worker pool pattern with configurable size (default: 5 workers) - Add buffered queue for tool calls (default: 100 capacity) - Protect concurrent writes with mutex - Maintain backward compatibility while fixing the race condition - Add comprehensive test for concurrent tool calls 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> * fix: address golangci-lint errcheck warnings - Add error checking for Write operations in concurrent tool calls test - Ensures all error return values are properly handled 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> * refactor: remove redundant mutex around thread-safe sync.Map sync.Map is already thread-safe and doesn't require additional mutex protection. This simplifies the test code while maintaining the same functionality. 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> * refactor: improve test control flow and add configuration bounds validation - Replace goto statements with labeled break for more idiomatic Go - Reduce test timeout from 5s to 2s for faster test execution - Add upper bounds validation for worker pool size (max 100) and queue size (max 10000) - Add tests to verify configuration bounds are respected - Log warnings when configuration values exceed maximum limits Addresses remaining CodeRabbit review feedback. 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> * fix: use maximum values instead of defaults when limits exceeded When WithWorkerPoolSize or WithQueueSize receive values exceeding their maximum limits, use the maximum allowed value instead of falling back to the default. This better aligns with user expectations when they request large pool/queue sizes. 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> * test: update stdio tests to expect maximum values when limits exceeded Update test expectations to match the new behavior where WithWorkerPoolSize and WithQueueSize use maximum values instead of defaults when the requested size exceeds limits. 🤖 Generated with [opencode](https://opencode.ai) Co-Authored-By: opencode <noreply@opencode.ai> --------- Co-authored-by: opencode <noreply@opencode.ai>
Configuration menu - View commit details
-
Copy full SHA for 9393526 - Browse repository at this point
Copy the full SHA 9393526View commit details
Commits on Aug 11, 2025
-
fix CallToolResult json marshaling and unmarshaling: need structuredC… (
#523) * fix CallToolResult json marshaling and unmarshaling: need structuredContent . * golang json use float to store number type * fix unittest: use NewMetaFromMap to create Result.Meta * mcptest: add a test case to response stuctured content * move after TestServerWithTool * fix unittest: no need check meta
Configuration menu - View commit details
-
Copy full SHA for 9c5d303 - Browse repository at this point
Copy the full SHA 9c5d303View commit details -
feat:add constants for resource content types (#489)
Signed-off-by: FanOne <294350394@qq.com>
Configuration menu - View commit details
-
Copy full SHA for 8a88d01 - Browse repository at this point
Copy the full SHA 8a88d01View commit details
Commits on Aug 12, 2025
-
feat: add missing SetPrompts, DeleteResources, and SetResources metho…
…ds (#445) Complete method parity across Tools, Prompts, and Resources: - SetPrompts/SetResources atomically replace all existing items - DeleteResources removes multiple resources by URI - Add comprehensive tests following existing patterns
Configuration menu - View commit details
-
Copy full SHA for a3d34d9 - Browse repository at this point
Copy the full SHA a3d34d9View commit details -
feat: support creating tools using go-struct-style input schema (#534)
* feat: support creating tools using go-struct-style input schema * docs: add struct-based schema documentation for tools Add comprehensive documentation for the new struct-based schema features introduced in commit b46b0d7, including: - Input schema definition using Go structs with WithInputSchema - Output schema definition using WithOutputSchema - Structured tool handlers with NewStructuredToolHandler - Array output schemas - Schema tags reference - Complete file operations example with structured I/O 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
Configuration menu - View commit details
-
Copy full SHA for 8a18f59 - Browse repository at this point
Copy the full SHA 8a18f59View commit details -
fix: remove duplicate methods server.SetPrompts & server.SetResources (…
…#542) Signed-off-by: Matthis Holleville <matthish29@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for 9f16336 - Browse repository at this point
Copy the full SHA 9f16336View commit details
Commits on Aug 20, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 35ebaa5 - Browse repository at this point
Copy the full SHA 35ebaa5View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v0.37.0...v0.38.0