fix(hooks): conform hooks.json to Claude Code plugin hook schema (fixes plugin failing to load)

[TroyMiner]

Summary

hooks/hooks.json is not valid against the Claude Code plugin hook schema, which causes the entire plugin to fail to load (Status: ✘ failed to load, Hooks: 0). This PR makes the file spec-compliant.

The validation error Claude Code reports:

Hook load failed: [
  { "expected": "array", "code": "invalid_type",
    "path": ["hooks", "PreToolUse", 0, "hooks"] }
]

Root cause

Each matcher entry was written in a flat shape, with type/command/timeout as siblings of matcher. The schema requires each matcher to contain a nested hooks: [] array of actions (see the official superpowers/codex plugins). Three defects, all stemming from not following the hook spec:

#	Defect	Effect
A	Missing per-matcher hooks: [] wrapper	Plugin fails to load (the error above)
B	timeout in ms (5000/3000)	Claude Code timeouts are in seconds — 5000 would block for ~83 min
C	Relative sh scripts/... paths	Hooks run from the user's project dir, so the scripts aren't found

Fix

• Wrap each matcher's action in a hooks: [] array.
• Convert timeouts to seconds (5/3).
• Reference scripts via ${CLAUDE_PLUGIN_ROOT}/scripts/..., matching the official plugins.

Behaviour (which servers/keys are checked, matchers, notes emitted) is unchanged.

Test plan

• claude plugin validate . passes
• hooks.json parses as valid JSON
• After applying, claude plugin list shows remotion-superpowers as ✔ enabled and claude plugin details reports Hooks (2) PreToolUse, PostToolUse (was Hooks (0) + failed to load)

🤖 Generated with Claude Code

[dojo-code-reviewer]

✅ Approved

Approved — 0 blockers, 4 P3. Confidence: 4/5.

🔵 P3 — Minor / nit

• hooks/hooks.json:9 — 🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax (such as pipefail or arrays) is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

            "command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" remotion-media KIE_API_KEY",

• hooks/hooks.json:19 — 🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

            "command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" replicate REPLICATE_API_TOKEN",

• hooks/hooks.json:31 — 🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

            "command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" pexels-attribution",

• hooks/hooks.json:41 — 🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

            "command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" captions-tip",

---

Total findings: 4 business context (4 total)

[dojo-code-reviewer]

🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax (such as pipefail or arrays) is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

Suggested change

	"command": "sh \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" remotion-media KIE_API_KEY",
	"command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" remotion-media KIE_API_KEY",

[dojo-code-reviewer]

🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

Suggested change

	"command": "sh \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" replicate REPLICATE_API_TOKEN",
	"command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/check-mcp-server.sh\" replicate REPLICATE_API_TOKEN",

[dojo-code-reviewer]

🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

Suggested change

	"command": "sh \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" pexels-attribution",
	"command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" pexels-attribution",

[dojo-code-reviewer]

🔵 P3 (minor) — Running the hook with POSIX 'sh' ignores the script's bash shebang ('#!/bin/bash'). If bash-specific syntax is introduced in the script, execution under 'sh' can fail. Running it with 'bash' avoids potential shell compatibility issues.

Suggested change

	"command": "sh \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" captions-tip",
	"command": "bash \"${CLAUDE_PLUGIN_ROOT}/scripts/post-tool-note.sh\" captions-tip",