Skip to content

fix: add CLAUDE_PLUGIN_ROOT fallback in hooks.json (fixes #17)#26

Closed
marcodelpin wants to merge 1 commit intoBayramAnnakov:mainfrom
marcodelpin:fix/plugin-root-fallback
Closed

fix: add CLAUDE_PLUGIN_ROOT fallback in hooks.json (fixes #17)#26
marcodelpin wants to merge 1 commit intoBayramAnnakov:mainfrom
marcodelpin:fix/plugin-root-fallback

Conversation

@marcodelpin
Copy link
Copy Markdown
Contributor

Summary

  • All 4 hooks now use shell parameter expansion ${CLAUDE_PLUGIN_ROOT:-$HOME/.claude/plugins/claude-reflect-marketplace/claude-reflect} so hooks degrade gracefully when Claude Code does not expand the variable.
  • Targeted fix for Hooks fail due to CLAUDE_PLUGIN_ROOT not set by Claude Code #17 — smallest possible change.

Note

Split from #20 as requested. This is the easiest to merge and review.

Test plan

  • Install plugin via marketplace, verify hooks fire correctly
  • Unset CLAUDE_PLUGIN_ROOT, verify fallback path is used

…ov#17)

All 4 hooks now use shell parameter expansion
${CLAUDE_PLUGIN_ROOT:-$HOME/.claude/plugins/claude-reflect-marketplace/claude-reflect}
so hooks degrade gracefully when Claude Code does not expand the variable.
@marcodelpin
Copy link
Copy Markdown
Contributor Author

Closing as abandoned — no maintainer activity on this repo for over a month. Feel free to reopen or cherry-pick if/when you pick it back up.

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