fix: redirect logger output to stderr for MCP compatibility #2
Conversation
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
In MCP servers, stdout must only contain JSON-RPC messages. Pino logger was writing to stdout by default, causing JSON parsing errors when the log output mixed with protocol messages. This fix: - Configures Pino to write all logs to stderr (fd 2) - Applies to both development (pino-pretty) and production modes - Prevents "Expected ',' or ']' after array element" JSON parsing errors Fixes the issue where MCP server fails to communicate with Claude Desktop due to corrupted JSON-RPC message stream. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Added comprehensive PageSpeed Insights API v5 data extraction tools: Visual Analysis: - get_visual_analysis: Screenshots and visual timeline of page load Performance Analysis: - get_element_analysis: DOM elements causing performance issues - get_network_analysis: Network waterfall with request details - get_javascript_analysis: JS execution breakdown and unused scripts - get_image_optimization_details: Detailed image optimization opportunities - get_render_blocking_details: Resources blocking initial render - get_third_party_impact: Third-party script performance impact - get_full_audit: Complete Lighthouse audit with all categories Implementation Details: - Created response-parser.ts for safe data extraction - Enhanced type definitions for all audit data structures - Each tool provides both formatted text and raw JSON data - Added comprehensive test suite and documentation Testing: - All tools verified working with real websites - Added test-tools.js for automated testing - Added TESTING.md with testing guide - Added manual-test.sh for developer testing This update exposes previously inaccessible data from the PageSpeed Insights API, providing developers with deeper performance insights. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Author
|
🚀 Major Feature Update: 8 New Advanced Analysis Tools Added While working on the initial fixes, I've implemented a significant enhancement to the MCP server by adding 8 new tools that 📊 New Tools Implemented: Visual Analysis
Performance Deep Dive
Comprehensive Auditing
🔧 Technical Implementation:
✅ Testing:
💡 Example Use Cases: Find out why images are slowing down your site"Show me image optimization opportunities for my website" Identify render-blocking resources"What resources are blocking the initial render of my page?" Analyze third-party impact"How are third-party scripts affecting my site's performance?" This enhancement significantly expands the capabilities of the PageSpeed Insights MCP server, making it a more comprehensive Note: This feature addition is backward compatible and doesn't affect existing functionality. |
All MCP tools now work correctly with Claude Code by removing resource attachments from responses. Claude Code only supports text, image, audio, and resource_link content types, not the "resource" type. This fix ensures all 8 new tools and existing tools work properly: - Visual analysis - Element analysis - Network analysis - JavaScript analysis - Image optimization details - Render-blocking details - Third-party impact - Full audit - Recommendations All tools now return only text content without breaking functionality. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
In MCP servers, stdout must only contain JSON-RPC messages. Pino logger was writing to stdout by default, causing JSON parsing errors when the log output mixed with protocol messages.
This fix:
Fixes the issue where MCP server fails to communicate with Claude Desktop due to corrupted JSON-RPC message stream.
🤖 Generated with Claude Code