anthropics / anthropics/financial-services
financial-analysis/.mcp.json is invalid JSON - missing comma + unbalanced braces in egnyte/box block
- Dominant language
- Python
- Stars
- 34.8k
- Forks
- 5.2k
- Avg merge
- 2h 2m
- Merged PRs (30d)
- 3
Description
### Summary
`plugins/vertical-plugins/financial-analysis/.mcp.json` does not parse as valid JSON on the current `main` (`4bbabc7`). Because the file is the plugin's MCP manifest, a fresh install of `financial-analysis` (or any `claude plugin marketplace update`) loads zero MCP servers and the plugin is effectively broken.
### Affected file
`plugins/vertical-plugins/financial-analysis/.mcp.json` (lines 43–51)
### Two defects
1. **Missing comma** after the `egnyte` object (line 46) before `"box"`.
2. **Unbalanced braces** — the `box` object is missing a closing `}`; the file has one fewer `}` than scopes opened.
Current (broken):
```json
"egnyte": {
"type": "http",
"url": "https://mcp-server.egnyte.com/mcp"
}
"box": {
"type": "http",
"url": "https://mcp.box.com"
}
}
```
### Parser output
```
json.decoder.JSONDecodeError: Expecting ',' delimiter: line 47 column 5 (char 1100)
```
(After adding the comma, a second error surfaces for the missing brace.)
### Suggested fix
```json
"egnyte": {
"type": "http",
"url": "https://mcp-server.egnyte.com/mcp"
},
"box": {
"type": "http",
"url": "https://mcp.box.com"
}
}
}
```
### Impact
- New installs of `financial-analysis` get a non-functional plugin (all bundled data-vendor MCP servers — daloopa, morningstar, sp-global, factset, moodys, aiera, lseg, pitchbook, chronograph, egnyte, box — fail to load).
- Existing installs with a cached pre-breakage version are unaffected until they update.
### Suggestion
Add a CI step that runs `python3 -m json.tool` (or `jq empty`) over every `.mcp.json` in the repo to catch manifest syntax errors before merge.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.