-
Notifications
You must be signed in to change notification settings - Fork 36
Detect MCP server connection failures in Copilot debug logs #9604
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot when you detect a mcp failure, you should fail the step in parseDebugLogFormat |
…bugLogFormat Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️ |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
|
📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤 |
Smoke Test Results (Claude)Last 2 merged PRs:
Test Results:
Overall Status: PARTIAL PASS (4/6 tests passed)
|
The Copilot log parser was not detecting MCP server connection failures, allowing workflows to continue running with unavailable tools instead of failing fast with actionable error messages.
Changes
Modified
parseDebugLogFormat(): Scans debug logs for MCP connection errors using pattern[ERROR] Failed to start MCP client for remote server <name>: TypeError: fetch failed, extracts server names, and immediately callscore.setFailed()to fail the step when failures are detectedUpdated
parseCopilotLog(): Propagates MCP failures from debug log parsing into result object for consistencyAdded test coverage: 5 test cases covering single/multiple failures, successful connections, deduplication, and verification that
core.setFailed()is called with the correct error messageBehavior
Without MCP failures,
mcpFailuresis undefined and execution continues normally. The step fails immediately during log parsing rather than later in the bootstrap layer, providing faster feedback about MCP connection issues.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.