Skip to content

fix(financial-analysis): resolve .mcp.json syntax and remove dead doc link (#273, #299) - #324

Open
AbhiPra24 wants to merge 1 commit into
anthropics:mainfrom
AbhiPra24:fix/mcp-manifest-json-and-doc-links
Open

fix(financial-analysis): resolve .mcp.json syntax and remove dead doc link (#273, #299)#324
AbhiPra24 wants to merge 1 commit into
anthropics:mainfrom
AbhiPra24:fix/mcp-manifest-json-and-doc-links

Conversation

@AbhiPra24

Copy link
Copy Markdown

Resolves #273
Resolves #299

Summary of Changes

  1. Fix .mcp.json syntax in financial-analysis:
    • Added missing comma after the egnyte connector object.
    • Added missing closing brace } to the box connector object.
  2. Prevent future JSON syntax regressions in CI:
    • Added plugins/**/.mcp.json and plugins/**/hooks/*.json to json_globs in scripts/check.py so all manifest and MCP files are validated automatically before merge.
  3. Remove dead file reference in documentation:
    • Removed nonexistent mcp-categories.json reference from CLAUDE.md.
  4. Version bump:
    • Applied standard patch bump to financial-analysis (0.1.1 -> 0.1.2) via scripts/version_bump.py.

Verification

  • Ran python3 scripts/check.py — verified 93/93 files parsed and passed with 0 issues.
  • Ran python3 -m json.tool plugins/vertical-plugins/financial-analysis/.mcp.json — verified valid JSON format.
  • Ran python3 scripts/version_bump.py --check — verified version bump validation passes cleanly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant