Remove MCP protocol version display from session output#81
Open
Remove MCP protocol version display from session output#81
Conversation
The "MCP: 2025-11-25" info shown in session lines (e.g. `mcpc`) and logTarget output is redundant noise. Remove it from human-readable output while keeping protocolVersion in JSON session detail output. https://claude.ai/code/session_01C3RfAyxwnFqemmd8Mvzj9u
The connect command was catching and silently discarding non-AuthError exceptions from showServerDetails. This caused a race condition where fast-failing bridges (e.g. OAuth servers returning 401) would mark the session as expired before showServerDetails ran, resulting in a ClientError instead of AuthError, which was then swallowed — making the connect command succeed (exit 0) when it should have failed. Fix by letting all showServerDetails errors propagate. Since the health check blocks until the bridge has definitively connected or failed, there's no reason to swallow errors. Also fix bridge's sendError to properly map AuthError to exit code 4 (was falling through to default code 2). https://claude.ai/code/session_01C3RfAyxwnFqemmd8Mvzj9u
Restore try/catch around showServerDetails but keep auth error propagation working correctly: - Bridge now marks auth failures as 'unauthorized' (not 'expired'), so ensureBridgeReady throws AuthError which connectSession re-throws - Non-auth failures (TLS errors, timeouts) are still swallowed so connect succeeds and the session can be used later (fixes insecure test) - Session expiry (404) stays as 'expired' with ClientError (swallowed) https://claude.ai/code/session_01C3RfAyxwnFqemmd8Mvzj9u
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
2 participants
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
This PR removes the display of MCP protocol version information from session and target output formatting. The protocol version is no longer shown in the CLI output for both session listings and direct server connections.
Key Changes
formatSessionLine()functionprotocolVersionfield fromLogTargetOptionsinterfacelogTarget()function for both stdio and HTTP transportsImplementation Details
https://claude.ai/code/session_01C3RfAyxwnFqemmd8Mvzj9u