Skip to content

Fix invalid JSON in financial-analysis/.mcp.json#262

Open
mikaumar wants to merge 1 commit into
anthropics:mainfrom
mikaumar:fix/financial-analysis-mcp-json
Open

Fix invalid JSON in financial-analysis/.mcp.json#262
mikaumar wants to merge 1 commit into
anthropics:mainfrom
mikaumar:fix/financial-analysis-mcp-json

Conversation

@mikaumar
Copy link
Copy Markdown

@mikaumar mikaumar commented Jun 2, 2026

Summary

plugins/vertical-plugins/financial-analysis/.mcp.json is not valid JSON and fails to parse. Any agent that loads this vertical's MCP connectors hits the error.

$ python -c "import json; json.load(open('plugins/vertical-plugins/financial-analysis/.mcp.json'))"
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 47 column 5

Cause

Two faults at the bottom of the file:

  1. Missing comma after the egnyte block's closing brace.
  2. The box block is mis-braced, so the root object never closes.

Fix

Three changes only: add the comma after egnyte, correct the box closing brace, and close the root object. No connector entries were added, removed, or altered.

Verification

$ python -c "import json,sys; d=json.load(open('plugins/vertical-plugins/financial-analysis/.mcp.json')); print('valid —', len(d['mcpServers']), 'servers')"
valid — 12 servers

All 12 servers (daloopa, morningstar, sp-global, factset, moodys, mtnewswire, aiera, lseg, pitchbook, chronograph, egnyte, box) parse.

🤖 Generated with Claude Code

The connector manifest failed to parse: a missing comma after the
"egnyte" entry and a mis-braced "box" entry left the root object
unclosed (json.load errored at line 47). Corrected the comma, the
"box" brace, and the root close. All 12 MCP servers now parse.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@ai-integr8tor ai-integr8tor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y

Copy link
Copy Markdown

@ai-integr8tor ai-integr8tor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y

Copy link
Copy Markdown

@ai-integr8tor ai-integr8tor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

y

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