Repository navigation
docs(readme): add missing built-in plugins to install list - #294
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Up to standards ✅🟢 Issues
|
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
 [Low impact] This documentation change appears safe to merge. SummaryUpdates
Reviews (3) · Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." · Reviewed by Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 1 file
Confidence score: 5/5
README.mdstill listsaskunder Built-in Plugins, which conflicts with the install list’s External Plugins classification. Move the ASK entry to External Plugins.
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. When an issue isn't valid or won't be fixed in this PR, reply in its thread with the reason and then resolve the thread. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="README.md">
<violation number="1" location="README.md:599">
P3: `ask` is still documented under the earlier `### Built-in Plugins` catalog, while this install list now classifies it as external. Move the ASK entry into `### External Plugins` there as well so the README has one classification.</violation>
</file>
Architecture diagram
sequenceDiagram
participant User as Plugin User
participant CLI as Claude Code CLI
participant Marketplace as Marketplace Config
participant Registry as Plugin Registry (@pleaseai)
Note over User,Registry: Plugin Discovery and Installation Flow
User->>CLI: Reads README "Available Plugins" section
CLI->>Marketplace: References .claude-plugin/marketplace.json
Marketplace->>Marketplace: Lists built-in plugins (source starts with ./plugins/)
Marketplace->>Marketplace: Lists external plugins (source starts with github:)
User->>CLI: /plugin install <name>@pleaseai
CLI->>Registry: Request plugin package
Registry-->>CLI: Return plugin
CLI-->>User: Confirmation
Note over Marketplace,Registry: Plugin Classification Boundary
alt Built-in plugin (source = ./plugins/xxx)
Marketplace->>Marketplace: Categorize under "Built-in plugins"
else External plugin (source = github:pleaseai/xxx)
Marketplace->>Marketplace: Categorize under "External plugins"
end
Note over CLI: README install list mirrors<br/>marketplace classification
CLI->>Marketplace: Verify listed plugins match actual registry entries
Marketplace-->>CLI: 77 built-in plugins available
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
Dismissed because a newer commit was pushed; Greptile will re-review the current head.
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
Dismissed because a newer commit was pushed; Greptile will re-review the current head.
…ll-list # Conflicts: # README.md
|
Summary
The
# Built-in pluginspart of the install list under Available Plugins inREADME.mdwas missing 19 built-in plugins, and two plugins were in the wrong section. This adds one/plugin install <name>@pleaseailine for each, after the current last line. Existing lines keep their order and nothing is removed.Added:
ast-grep,axi,codegraph,edgeparse,eve,git-ai,graphify,greptile,nostics,nuxt-seo,orpc,port,rtk,semble,shadcn-ui,skillopt-sleep,tiptap,tosspayments,vercel-sandbox.How the list was derived
Against
origin/mainat 240fa35:.claude-plugin/marketplace.jsonentry whosesourceis a string starting with./plugins/. There are 77 of them.jq -r '.plugins[] | select((.source|type)=="string" and (.source|startswith("./plugins/"))) | .name' .claude-plugin/marketplace.json/plugin install <name>@pleaseailine in thebashblock under### Available Plugins.comm -23 built-in listedis empty and the block has no duplicate lines.Section fixes
playwright-cliis built-in (./plugins/playwright-cli), so it moved from# External pluginsto the end of# Built-in plugins.askcomes fromgithub:pleaseai/ask, so it moved from# Built-in pluginsto the end of# External plugins.After both moves, the names under
# Built-in pluginsmatch the 77 built-in entries exactly, and# External pluginslists no built-in plugin.Not included
evlogis being added on a separate branch, so it is not in this PR.Related issue
None.
Checklist
bun run test): not applicable, docs onlybun run lint): not run, docs onlyBREAKING CHANGE:note is included