fix: STDIO transport stdout flush and MCP notification handling #37

Merged
michal merged 1 commits from fix/stdio-flush-and-notifications into main 2026-02-24 19:10:04 +00:00
Owner

Summary

  • Fix STDIO transport truncating large responses by waiting for stdout flush before exit
  • Handle MCP notification methods in router instead of returning errors
  • Use -p shorthand in config claude output

Test plan

  • mcpd tests pass (487)
  • mcplocal tests pass (164)
  • E2E: test-mcp.sh homeautomation shows upstream tools
## Summary - Fix STDIO transport truncating large responses by waiting for stdout flush before exit - Handle MCP notification methods in router instead of returning errors - Use -p shorthand in config claude output ## Test plan - [x] mcpd tests pass (487) - [x] mcplocal tests pass (164) - [ ] E2E: test-mcp.sh homeautomation shows upstream tools
michal added 1 commit 2026-02-24 19:09:58 +00:00
fix: STDIO transport stdout flush and MCP notification handling
Some checks failed
CI / lint (pull_request) Has been cancelled
CI / typecheck (pull_request) Has been cancelled
CI / test (pull_request) Has been cancelled
CI / build (pull_request) Has been cancelled
CI / package (pull_request) Has been cancelled
9cb0c5ce24
- Wait for stdout.write callback before process.exit in STDIO transport
  to prevent truncation of large responses (e.g. grafana tools/list)
- Handle MCP notification methods (notifications/initialized, etc.) in
  router instead of returning "Method not found" error
- Use -p shorthand in config claude output

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
michal merged commit f5a902d3e0 into main 2026-02-24 19:10:04 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: michal/mcpctl#37