Improve deployment and error handling#15
Merged
Merged
Conversation
Configure HttpServer to write logs to /data/logs instead of ephemeral /app/logs directory. This ensures logs survive pod restarts and redeployments, enabling better debugging of intermittent issues. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add ability to enable or disable registered MCP servers through both
MCP tools and REST API. This allows temporarily disabling problematic
servers without unregistering them.
Changes:
- Add ServerRegistry.SetEnabledAsync() method
- Add MCP tools: enable_service, disable_service
- Add REST endpoints: POST /api/admin/services/{name}/enable
POST /api/admin/services/{name}/disable
- Disconnect when disabling to free resources
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add documentation for the new enable_service and disable_service tools, including when to use them, how they work, and examples. Changes: - Add enable_service and disable_service to tool reference table - Add new "Enabling and Disabling Servers" section - Update error handling section to mention disabled servers - Add example usage Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Previously, when a downstream MCP server returned IsError=true, the aggregator would return a successful string response prefixed with [Error]. This meant the MCP protocol's error flag was lost, and LLM clients couldn't distinguish between error messages and actual failures. Now, downstream errors throw ToolExecutionException, making error handling consistent with timeouts and properly propagating error state through the MCP protocol. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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
1 participant
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
Changes
Log Persistence
/data/logsin deployment for log persistence across restartsServer Management
enable_serviceanddisable_serviceMCP tools/api/servers/{name}/enableand/disable)Error Handling
ToolExecutionExceptionfor downstream tool errorsIsError=trueinstead of returning error stringsTest plan
🤖 Generated with Claude Code