Conversation
…skill
Add explanatory note before the Complete Plugin Entry Example in
schema-reference.md to explain that ${CLAUDE_PLUGIN_ROOT} resolves to
the plugin's installation directory and should be used for portable paths.
Fixes #95
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
|
Claude finished @sjnims's task in 1m 3s —— View job PR Review Complete ✅Reviewing PR #97: docs: add ${CLAUDE_PLUGIN_ROOT} explanation to marketplace-structure skill SummaryLooks good! This is a valuable documentation improvement that addresses a real gap in the marketplace-structure skill. What's Good
SuggestionsNone - this PR is ready to merge as-is. Checklist Verification
Recommendation: Approve and merge. This is a focused, helpful documentation improvement that addresses user confusion without introducing unnecessary complexity. |
Summary
Add explanatory note for the
${CLAUDE_PLUGIN_ROOT}environment variable in the marketplace-structure skill's schema reference documentation.Problem
Fixes #95
The
${CLAUDE_PLUGIN_ROOT}environment variable is used in the hooks and mcpServers examples within the Complete Plugin Entry Example, but wasn't explicitly explained. Users unfamiliar with plugin development may not understand what this variable represents.Solution
Added a brief note directly before the Complete Plugin Entry Example that explains:
${CLAUDE_PLUGIN_ROOT}resolves to (plugin's absolute installation path)This approach was chosen over adding a new subsection because:
Alternatives Considered
Changes
plugins/plugin-dev/skills/marketplace-structure/references/schema-reference.md: Added note before Complete Plugin Entry Example explaining${CLAUDE_PLUGIN_ROOT}Testing
🤖 Generated with Claude Code