feat(please-plugins): add find-plugin skill for marketplace discovery#156
feat(please-plugins): add find-plugin skill for marketplace discovery#156
Conversation
Searches marketplace.json catalog and installed plugin skill files to recommend relevant plugins based on user queries.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Code Review
This pull request introduces the 'Find Plugin' skill, which enables users to discover and install Claude Code plugins from the pleaseai marketplace by searching the plugin catalog and skill index. Feedback includes suggestions to optimize search commands by removing redundant find calls and using jq for more efficient JSON parsing, as well as a recommendation to ensure the final list of plugin recommendations is deduplicated.
There was a problem hiding this comment.
1 issue found across 1 file
Confidence score: 4/5
- This PR is likely safe to merge, but there is a concrete behavior gap:
plugins/please-plugins/skills/find-plugin/SKILL.mddocuments/discovers only one skill layout, so valid skills under.agents/skillsmay never be found. - The issue is moderate in impact (missing plugin discovery for some installs) but limited in scope to skill lookup coverage rather than a broad stability or security regression.
- Pay close attention to
plugins/please-plugins/skills/find-plugin/SKILL.md- ensure discovery/search includes.agents/skillsso both supported layouts are handled.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="plugins/please-plugins/skills/find-plugin/SKILL.md">
<violation number="1" location="plugins/please-plugins/skills/find-plugin/SKILL.md:39">
P2: Search `.agents/skills` as well; this only covers one of the supported skill layouts, so some installed skills will never be discovered.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
There was a problem hiding this comment.
1 issue found across 1 file (changes from recent commits).
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="plugins/please-plugins/skills/find-plugin/SKILL.md">
<violation number="1" location="plugins/please-plugins/skills/find-plugin/SKILL.md:39">
P2: Keep the skill search case-insensitive; dropping `-i` can miss valid matches when the query casing differs from the skill text.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Make skill search case-insensitive by adding -i flag to grep command in find-plugin SKILL.md
Summary
Adds a
find-pluginskill to the please-plugins plugin that helps users discover and install Claude Code plugins from the pleaseai marketplace. The skill searches the marketplace catalog (marketplace.json) and installed plugin skill files (SKILL.md) to recommend relevant plugins based on user queries.Summary by cubic
Adds the
find-pluginskill toplease-pluginsto help users discover and install Claude Code plugins from thepleaseaimarketplace. It searches the marketplace catalog and installed plugin skill files, deduplicates matches, and recommends relevant plugins with install commands.skillsand.agents/skills) case-insensitively and dedupes results.claude plugin install {plugin}@pleaseai, and flags installed viainstalled_plugins.json.Written for commit bd85704. Summary will update on new commits.