Skip to content

fix(plugins): correct empty hooks.json schema in 3 vertical plugins#162

Closed
dongjia1982 wants to merge 1 commit into
anthropics:mainfrom
dongjia1982:fix/empty-hooks-json
Closed

fix(plugins): correct empty hooks.json schema in 3 vertical plugins#162
dongjia1982 wants to merge 1 commit into
anthropics:mainfrom
dongjia1982:fix/empty-hooks-json

Conversation

@dongjia1982

Copy link
Copy Markdown

Summary

Three vertical plugins ship a hooks/hooks.json containing an empty JSON array ([]), which fails to load in current Claude Code versions. The expected schema is an object with a hooks key — matching what investment-banking and financial-analysis already use.

Affected files:

  • plugins/vertical-plugins/equity-research/hooks/hooks.json
  • plugins/vertical-plugins/private-equity/hooks/hooks.json
  • plugins/vertical-plugins/wealth-management/hooks/hooks.json

Each is changed from [] to:

{
  "hooks": {}
}

Repro

With one of the affected plugins enabled, Claude Code fails to load the plugin's commands (e.g. /comps, /dcf) because the marketplace loader rejects the malformed hooks.json.

Test plan

  • cat plugins/vertical-plugins/{equity-research,private-equity,wealth-management}/hooks/hooks.json returns valid JSON matching the investment-banking format
  • Reload plugin in Claude Code and verify commands appear

🤖 Generated with Claude Code

The hooks.json files in equity-research, private-equity, and
wealth-management contained an empty array `[]`, which fails to load
in current Claude Code versions that expect an object with a `hooks`
key. This matches the format already used by investment-banking and
financial-analysis.
@mihilmy

mihilmy commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Thanks for catching this and taking the time to send a fix! 🙏 This was reported by many contributors — the underlying bug is now resolved in #232, which repairs all four broken hooks.json files ([]{"hooks": {}}) and adds a plugin-validate CI check (the official claude plugin validate) so a malformed manifest can't regress again. Closing as superseded by #232. Really appreciate the contribution.

@mihilmy mihilmy closed this May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants