Skip to content

Quote the plugin root in every hook command - #36

Merged
chris-peterson merged 1 commit into
mainfrom
quote-the-plugin-root-in-every-hook-command
Sep 27, 2026
Merged

chris-peterson merged 1 commit into
mainfrom
quote-the-plugin-root-in-every-hook-command

Conversation

@chris-peterson

@chris-peterson chris-peterson commented Sep 24, 2026 •

Copy link
Copy Markdown
Owner

Context

ClaudeWatch's three hook commands expanded ${CLAUDE_PLUGIN_ROOT} without quotes, so a plugin installed under a path containing a space splits into several words and the watchdog never runs. Claude Code 2.1.281's claude plugin validate now warns on each one, and shipyard's projection gate fails on any warning plugin.yml doesn't accept. This quotes all three so the gate stays green once its validator pin moves to 2.1.281.

Review guide

The three hook commands expanded ${CLAUDE_PLUGIN_ROOT} bare, so a
plugin installed under a path with a space split into several words and
the watchdog never ran. Claude Code 2.1.281's `claude plugin validate`
now warns on each one, and shipyard's projection gate fails on any
warning plugin.yml doesn't accept, so the gate goes red once its
validator pin reaches 2.1.281.

SPEC.md, SCHEMA.md and the rules skill quote the same commands, so
they match what the hooks run.
@chris-peterson chris-peterson self-assigned this Sep 24, 2026
@chris-peterson chris-peterson added the bug Something isn't working label Sep 24, 2026
@chris-peterson
chris-peterson marked this pull request as ready for review September 27, 2026 21:20
@chris-peterson
chris-peterson merged commit 086d131 into main Sep 27, 2026
1 check passed
@chris-peterson
chris-peterson deleted the quote-the-plugin-root-in-every-hook-command branch September 27, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant