Skip to content

Fix malformed JSON in financial-analysis .mcp.json (Box entry)#259

Open
leslieqy901023-prog wants to merge 1 commit into
anthropics:mainfrom
leslieqy901023-prog:fix/mcp-json-box-syntax
Open

Fix malformed JSON in financial-analysis .mcp.json (Box entry)#259
leslieqy901023-prog wants to merge 1 commit into
anthropics:mainfrom
leslieqy901023-prog:fix/mcp-json-box-syntax

Conversation

@leslieqy901023-prog
Copy link
Copy Markdown

Problem

The Adding Box to MCP Integrations (#187) change left plugins/vertical-plugins/financial-analysis/.mcp.json as invalid JSON:

  • Missing comma after the egnyte server block
  • Box server object's closing brace was missing (only the mcpServers brace was present)

Because all connectors are centralized in this one file and shared across the other plugins, any fresh install or claude plugin update of financial-analysis would fail to parse the config and load none of the MCP connectors.

    "egnyte": {
      "type": "http",
      "url": "https://mcp-server.egnyte.com/mcp"
    }                  // <- missing comma
    "box": {
      "type": "http",
      "url": "https://mcp.box.com"
  }                    // <- box object never closed
}

Fix

Add the missing comma after egnyte and properly close the box object.

Validation

  • python3 -m json.tool now parses cleanly → 12 servers
  • python3 scripts/check.pyOK — 80 file(s) checked, 0 issues.
  • Pre-commit hook bumped financial-analysis 0.1.1 → 0.1.2 per repo convention.

🤖 Generated with Claude Code

The "Adding Box" change (anthropics#187) left the Box server entry with a missing
comma after the egnyte block and an unclosed brace, making the entire
mcpServers config invalid JSON. Any fresh install or update of the
financial-analysis plugin would fail to load all connectors.

Add the missing comma and close the Box object brace. Validated with
`python3 scripts/check.py` (0 issues).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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