Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 7, 2025

The Claude Test Failure Analysis workflow was failing with "Invalid MCP configuration: MCP config is not a valid JSON" due to a JSON comment in the MCP server configuration.

Changes

  • Removed invalid inline comment from github-research MCP server env configuration
  • JSON spec doesn't support # comments; the comment broke the entire config parser
 "env": {
-  "DISABLE_SUMMARIES": "true",  # Disable verbose summaries for faster analysis
+  "DISABLE_SUMMARIES": "true",
   "GITHUB_PERSONAL_ACCESS_TOKEN": "${{ secrets.GITHUB_TOKEN }}"
 }
Original prompt

This section details on the original issue you should resolve

<issue_title>Claude Analyze Failure not working</issue_title>
<issue_description>error: Invalid MCP configuration:
MCP config is not a valid JSON</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 7, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 7, 2025

Copilot AI changed the title [WIP] Fix Claude Analyze failure due to invalid MCP configuration Fix invalid JSON comment in claude-on-test-failure.yml MCP configuration Nov 7, 2025
Copilot AI requested a review from strawgate November 7, 2025 19:07
Copilot finished work on behalf of strawgate November 7, 2025 19:07
@strawgate strawgate marked this pull request as ready for review November 7, 2025 19:11
@strawgate strawgate merged commit cae92d8 into main Nov 7, 2025
1 of 2 checks passed
@strawgate strawgate deleted the copilot/fix-claude-analyze-issue branch November 7, 2025 19:11
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.

Claude Analyze Failure not working

2 participants