Add prompt-snippets plugin with /ps command#7
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughAdds a new Prompt Snippets Plugin that provides a Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In
@.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/.claude-plugin/plugin.json:
- Around line 1-5: Open the CLAUDE.md Plugins section and add a new Plugins list
item for the plugin named "prompt-snippets": insert the bullet `-
**prompt-snippets** — Quick behavioral mode switches via /ps command` into the
existing Plugins list so the plugin appears alongside the other 11 entries
(ensure spelling and punctuation match the plugin.json name and description).
In @.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/README.md:
- Around line 7-9: The fenced code blocks showing the command examples (the
blocks containing "/ps [snippet-name]" and "/ps noaction brief") need language
specifiers to silence MD040; edit the README.md usage examples and change the
opening triple-backtick from ``` to ```text for both command blocks so they
become fenced as text code blocks.
In
@.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/skills/ps/SKILL.md:
- Around line 31-33: Add a language tag to the fenced code block containing the
example "/ps noaction brief" so markdownlint MD040 is not triggered; update the
triple-backtick fence that surrounds "/ps noaction brief" to include a language
identifier such as text or shell (e.g., ```text) while leaving the block content
unchanged.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
.devcontainer/CHANGELOG.md.devcontainer/config/defaults/settings.json.devcontainer/plugins/devs-marketplace/.claude-plugin/marketplace.json.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/.claude-plugin/plugin.json.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/README.md.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/skills/ps/SKILL.md
.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/.claude-plugin/plugin.json
Show resolved
Hide resolved
.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/README.md
Outdated
Show resolved
Hide resolved
.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/skills/ps/SKILL.md
Outdated
Show resolved
Hide resolved
12080a7 to
2c56da3
Compare
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.devcontainer/CHANGELOG.md (1)
5-11: LGTM on structure; consider dropping the internal config key from Line 11.Lines 8–10 are cleanly user-facing. Line 11 leaks the implementation detail
disable-model-invocation: trueinto the changelog. Per the coding guidelines, entries should describe what changed for the user, not how it was implemented.✏️ Suggested wording
- - Isolated from skill-engine auto-suggestion (`disable-model-invocation: true`) and independently toggleable via `enabledPlugins` + - Does not appear in skill-engine auto-suggestions; can be independently enabled/disabled via `enabledPlugins`As per coding guidelines: "Write CHANGELOG entries from the user's perspective — what changed, not how it was implemented."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.devcontainer/CHANGELOG.md around lines 5 - 11, The changelog leaks an internal config key—remove the implementation detail `disable-model-invocation: true` from the Prompt Snippets Plugin entry and replace it with user-facing wording that conveys the same behavior (e.g., "isolated from skill-engine auto-suggestions" or "does not trigger model auto-suggestions"); keep the user-visible toggle note about `enabledPlugins` if desired and ensure references to `prompt-snippets` and `/ps` remain unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In @.devcontainer/CHANGELOG.md:
- Around line 5-11: The changelog leaks an internal config key—remove the
implementation detail `disable-model-invocation: true` from the Prompt Snippets
Plugin entry and replace it with user-facing wording that conveys the same
behavior (e.g., "isolated from skill-engine auto-suggestions" or "does not
trigger model auto-suggestions"); keep the user-visible toggle note about
`enabledPlugins` if desired and ensure references to `prompt-snippets` and `/ps`
remain unchanged.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
.devcontainer/CHANGELOG.md.devcontainer/CLAUDE.md.devcontainer/config/defaults/settings.json.devcontainer/plugins/devs-marketplace/.claude-plugin/marketplace.json.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/.claude-plugin/plugin.json.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/README.md.devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/skills/ps/SKILL.md
🚧 Files skipped from review as they are similar to previous changes (4)
- .devcontainer/config/defaults/settings.json
- .devcontainer/plugins/devs-marketplace/.claude-plugin/marketplace.json
- .devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/.claude-plugin/plugin.json
- .devcontainer/plugins/devs-marketplace/plugins/prompt-snippets/README.md
New plugin provides quick behavioral mode switches via a single dispatcher skill. /ps noaction, /ps brief, /ps go, etc. inject short directives that persist for the conversation. Snippets are composable (/ps noaction brief) and isolated from skill-engine auto-suggestion.
- Add prompt-snippets to Plugins list in CLAUDE.md - Add `text` language specifier to fenced code blocks (MD040) in README.md and SKILL.md
6feb1e3 to
a947776
Compare
Summary
New plugin providing quick behavioral mode switches via a single
/psslash command. Derived from session history analysis of 232 user messages across 73 sessions identifying the most frequently typed directives./pscommand with$ARGUMENTSlookup, not 8 separate skillsnoaction,brief,plan,go,review,ship,deep,hold/ps noaction briefapplies multiple snippetsdisable-model-invocation: truekeeps it out of auto-suggestion; own plugin with independentenabledPluginstoggleUsage
Files
plugins/prompt-snippets/(plugin.json, SKILL.md, README.md)Test plan
/psappears as a slash command/ps noaction— Claude acknowledges mode and restricts to investigation/ps brief— Claude responds concisely/pswith no args — lists available snippets/psdoes NOT appear in auto-suggestions during normal promptsenabledPluginstoggleSummary by CodeRabbit
New Features
/psslash command for quick behavioral mode switchesDocumentation
/ps, composing snippets, and extending snippet listChores