Skip to content

fix(review): let the review model read the skill's references - #137

Merged
jmarek41 merged 1 commit into
mainfrom
fix/ai-review-plugin-path
Oct 27, 2026
Merged

jmarek41 merged 1 commit into
mainfrom
fix/ai-review-plugin-path

Conversation

@jmarek41

@jmarek41 jmarek41 commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Claude Code reads a directory marketplace's plugins from that directory (readFromFolder in claude plugin list --json), not from its plugin cache. The marketplace was moved to $RUNNER_TEMP, which the review model is denied, so the skill's reference files (zone table, severity rules) could not be read and the review ran without them.

The marketplace now lives in ~/.claude/plugins/ai-review-marketplace, covered by the existing Read(~/.claude/plugins/**) allow rule and still outside the checkout. Its .git stays denied by Read(**/.git/**).

Verified on a consumer PR pointed at this branch: the "permission denied" note on the reference file is gone and the review now names a zone that only the reference table defines.

Claude Code reads a directory marketplace's plugins from that directory,
not from its plugin cache. The marketplace sat in RUNNER_TEMP, which the
model may not read, so the skill's reference files were denied. It now
lives under ~/.claude/plugins, which the model may read.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@jmarek41
jmarek41 merged commit af990d6 into main Oct 27, 2026
2 checks passed
@jmarek41
jmarek41 deleted the fix/ai-review-plugin-path branch October 27, 2026 10:27
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