fix(plugins): remove broken mcpServers references that broke plugin loading #325

lmiranda · 2026-01-29T23:10:09Z

lmiranda commented

2026-01-29 23:10:09 +00:00

Summary

The MCP consolidation commit (afd4c44) deleted plugin-level .mcp.json files but left references to them in plugin.json and marketplace.json. This caused 7 plugins to fail loading their commands.

Affected plugins: projman, pr-review, cmdb-assistant, data-platform, viz-platform, contract-validator (and indirectly git-flow, clarity-assist)

Changes

Remove mcpServers field from 6 plugin.json files (referenced files no longer exist)
Remove mcpServers field from 6 marketplace.json entries
Add file reference validation to validate-marketplace.sh:
- Validates mcpServers references point to existing files
- Validates hooks references point to existing files
- Validates commands references point to existing paths
Add pre-commit hook to enforce validation on every commit

Prevention

The validation script will now FAIL if any config file references a non-existent file. The pre-commit hook enforces this automatically.

Test plan

Run ./scripts/validate-marketplace.sh - passes
Pre-commit hook executes and validates on commit
Restart Claude Code session and verify all plugin commands appear with /

🤖 Generated with Claude Code

## Summary The MCP consolidation commit (afd4c44) deleted plugin-level `.mcp.json` files but left references to them in `plugin.json` and `marketplace.json`. This caused 7 plugins to fail loading their commands. **Affected plugins:** projman, pr-review, cmdb-assistant, data-platform, viz-platform, contract-validator (and indirectly git-flow, clarity-assist) ## Changes - Remove `mcpServers` field from 6 plugin.json files (referenced files no longer exist) - Remove `mcpServers` field from 6 marketplace.json entries - Add file reference validation to `validate-marketplace.sh`: - Validates `mcpServers` references point to existing files - Validates `hooks` references point to existing files - Validates `commands` references point to existing paths - Add pre-commit hook to enforce validation on every commit ## Prevention The validation script will now **FAIL** if any config file references a non-existent file. The pre-commit hook enforces this automatically. ## Test plan - [x] Run `./scripts/validate-marketplace.sh` - passes - [x] Pre-commit hook executes and validates on commit - [ ] Restart Claude Code session and verify all plugin commands appear with `/` 🤖 Generated with [Claude Code](https://claude.com/claude-code)

lmiranda added the Type/Bug Priority/High labels 2026-01-29 23:10:09 +00:00

lmiranda added 1 commit 2026-01-29 23:10:09 +00:00

fix(plugins): remove broken mcpServers references that broke plugin loading c8b91f6a87

The MCP consolidation commit (afd4c44) deleted plugin-level .mcp.json files
but left references to them in plugin.json and marketplace.json. This caused
7 plugins to fail loading (projman, pr-review, cmdb-assistant, data-platform,
viz-platform, contract-validator, and indirectly git-flow/clarity-assist).

Changes:
- Remove mcpServers field from 6 plugin.json files (file no longer exists)
- Remove mcpServers field from 6 marketplace.json entries
- Add file reference validation to validate-marketplace.sh:
  - Validates mcpServers references point to existing files
  - Validates hooks references point to existing files
  - Validates commands references point to existing paths
- Add pre-commit hook (.git/hooks/pre-commit) to enforce validation

The validation script will now FAIL if any config file references a
non-existent file, preventing this class of bug from happening again.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

lmiranda merged commit e1d7ec46ae into development

2026-01-29 23:10:26 +00:00

lmiranda deleted branch fix/remove-broken-mcp-references

2026-01-29 23:10:26 +00:00

lmiranda referenced this issue from a commit

2026-01-29 23:10:26 +00:00

Merge pull request 'fix(plugins): remove broken mcpServers references that broke plugin loading' (#325) from fix/remove-broken-mcp-references into development

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: personal-projects/leo-claude-mktplace#325