Skip to content

Fix invalid financial-analysis .mcp.json; gate .mcp.json in check.py … - #384

Open
xinfenguc-glitch wants to merge 5 commits into
anthropics:mainfrom
xinfenguc-glitch:China-Macro
Open

xinfenguc-glitch wants to merge 5 commits into
anthropics:mainfrom
xinfenguc-glitch:China-Macro

Conversation

@xinfenguc-glitch

Copy link
Copy Markdown

…(#1)

Commit 120a31d (Adding Box to MCP Integrations) left the egnyte entry without a trailing comma and never closed the box object, so the whole file failed to parse and none of the plugin's MCP servers could load.

  • Restore valid JSON, keeping all 12 servers including box.
  • scripts/check.py: parse every git-tracked *.mcp.json and require a top-level mcpServers object.
  • Add a check workflow that runs scripts/check.py on PRs and main, since no CI job ran it before.

Claude-Session: https://claude.ai/code/session_01PCNeNGNJnLwcBn8ubnSS7N

xinfenguc-glitch and others added 5 commits September 27, 2026 14:46
)

Commit 120a31d (Adding Box to MCP Integrations) left the egnyte entry
without a trailing comma and never closed the box object, so the whole
file failed to parse and none of the plugin's MCP servers could load.

- Restore valid JSON, keeping all 12 servers including box.
- scripts/check.py: parse every git-tracked *.mcp.json and require a
  top-level mcpServers object.
- Add a check workflow that runs scripts/check.py on PRs and main, since
  no CI job ran it before.


Claude-Session: https://claude.ai/code/session_01PCNeNGNJnLwcBn8ubnSS7N

Co-authored-by: Claude <noreply@anthropic.com>
Keeps original source files outside the plugin tree so they never ship
with a plugin; matches the china-macro-sources/ inbox on
claude/awesome-babbage-f70biu.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JT6VVHLHQoyPMeqJcBmq3k
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.

2 participants