Skip to content

Conversation

built-by-as
Copy link
Owner

Summary

  • Updated MCP server regex patterns to recognize SSE connection failures
  • Made SSE pattern matching case-insensitive
  • Added support for failed connection status symbol (✗)

Problem

The MCP server list parser was only matching servers with successful (✓) or warning (⚠) status. This caused SSE servers with failed connections to be ignored, as shown in:

jira: https://mcp.atlassian.com/v1/see (SSE) - ✗ Failed to connect

Solution

Updated both regex patterns in main.ts:

  1. Line 148: MCP server line detection regex
  2. Line 220: Server parsing regex

Changes:

  • Added symbol to the status character class
  • Made pattern case-insensitive with i flag to handle "SSE", "sse", etc.
  • Updated comments to reflect new behavior

Test plan

  • Verify failed SSE connections now appear in MCP server list
  • Confirm status indicator shows disconnected state
  • Test with various case combinations (SSE, sse, Sse)

Update regex patterns to recognize SSE servers (case-insensitive) with failed connection status (✗).
Previously only matched successful (✓) or warning (⚠) states, causing failed SSE connections to be ignored.
@built-by-as built-by-as merged commit 26ab453 into main Oct 9, 2025
@built-by-as built-by-as deleted the failed-to-connect-regex-mcp branch October 9, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant