anthropics / anthropics/financial-services

financial-analysis/.mcp.json is invalid JSON - missing comma + unbalanced braces in egnyte/box block

オープン
#273 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
34.8k
フォーク
5.2k
平均マージ
2時間 2分
マージ済み PR(30日)
3

説明

### 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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。