Skip to content

fix: validate plugin hooks.json on install#230

Merged
backnotprop merged 3 commits intomainfrom
feat/install-hook-validation
Mar 6, 2026
Merged

fix: validate plugin hooks.json on install#230
backnotprop merged 3 commits intomainfrom
feat/install-hook-validation

Conversation

@backnotprop
Copy link
Owner

Summary

  • Install scripts now check if the plugin's hooks.json exists at ~/.claude/plugins/marketplaces/plannotator/apps/hook/hooks/hooks.json
  • If found, overwrites it with the canonical config (timeout: 345600) to fix stale values (e.g. 1800)
  • Added to all three install scripts: install.sh, install.ps1, install.cmd

Test plan

  • Run install.sh with plugin installed — verify hooks.json is updated
  • Run install.sh without plugin installed — verify no error
  • Verify timeout value is 345600 after install

🤖 Generated with Claude Code

backnotprop and others added 3 commits March 5, 2026 18:47
The Claude Code plugin marketplace can leave stale hooks.json with
incorrect timeout values (e.g. 1800 instead of 345600). Install scripts
now check for and overwrite the plugin's hooks.json if it exists.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…figure plannotator

Detects "command".*plannotator in settings.json (only matches hook configs, not
enabledPlugins) and only warns when the plugin is also installed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The duplicate hook warning referenced !CLAUDE_SETTINGS! but the variable
was never set, causing the warning to silently never fire for CMD users.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@backnotprop backnotprop merged commit 64d2f29 into main Mar 6, 2026
1 check passed
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.

1 participant