anthropics / anthropics/financial-services

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

Đang mở
#273 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Python
Star
34.8k
Fork
5.2k
Merge trung bình
2 giờ 2 phút
Pull request đã merge (30 ngày)
3

Mô tả

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

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.