feat: added design comments skill#113
Conversation
📝 WalkthroughWalkthroughA new ChangesPF Design Comments Skill Documentation
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
⚔️ Resolve merge conflicts
Warning Review ran into problems🔥 ProblemsLinked repositories: Your configuration references 1 linked repositories, but your current plan allows 0. Analyzed ``, skipped Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@plugins/react/skills/pf-design-comments/SKILL.md`:
- Around line 1-4: The pf-design-comments skill is missing the required
disable-model-invocation: true flag in its frontmatter. Add this flag to the
frontmatter section in the SKILL.md file alongside the existing name and
description fields to properly indicate that this skill has file-system side
effects and should only be invoked by users, not by the AI model
programmatically.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 01d243ea-8cdd-4558-a64e-405d6aa6d72e
📒 Files selected for processing (3)
PLUGINS.mdplugins/react/README.mdplugins/react/skills/pf-design-comments/SKILL.md
| --- | ||
| name: pf-design-comments | ||
| description: Integrate @patternfly/design-comments into React apps for on-page design feedback, pinned comment threads, GitHub Issues sync, and Jira linking. Use when adding design comments, review overlays, or removing the commenting system from a PatternFly React project. | ||
| --- |
There was a problem hiding this comment.
Add disable-model-invocation: true to frontmatter — this skill has side effects.
The skill executes npx design-comments init, which modifies multiple project files (src/app/index.tsx, src/app/routes.tsx, src/app/AppLayout/AppLayout.tsx, webpack.dev.js), creates new files (src/app/Comments/Comments.tsx, env templates), and adds webpack middleware. These are file-system mutations and therefore side effects that require the disable-model-invocation: true flag per PatternFly ai-helpers conventions (CONTRIBUTING-SKILLS.md and retrieved learnings on PR 77). This ensures only the user can invoke the skill, not the AI model programmatically.
🔧 Proposed frontmatter fix
---
name: pf-design-comments
description: Integrate `@patternfly/design-comments` into React apps for on-page design feedback, pinned comment threads, GitHub Issues sync, and Jira linking. Use when adding design comments, review overlays, or removing the commenting system from a PatternFly React project.
+disable-model-invocation: true
---📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| --- | |
| name: pf-design-comments | |
| description: Integrate @patternfly/design-comments into React apps for on-page design feedback, pinned comment threads, GitHub Issues sync, and Jira linking. Use when adding design comments, review overlays, or removing the commenting system from a PatternFly React project. | |
| --- | |
| --- | |
| name: pf-design-comments | |
| description: Integrate `@patternfly/design-comments` into React apps for on-page design feedback, pinned comment threads, GitHub Issues sync, and Jira linking. Use when adding design comments, review overlays, or removing the commenting system from a PatternFly React project. | |
| disable-model-invocation: true | |
| --- |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@plugins/react/skills/pf-design-comments/SKILL.md` around lines 1 - 4, The
pf-design-comments skill is missing the required disable-model-invocation: true
flag in its frontmatter. Add this flag to the frontmatter section in the
SKILL.md file alongside the existing name and description fields to properly
indicate that this skill has file-system side effects and should only be invoked
by users, not by the AI model programmatically.
Sources: Coding guidelines, Learnings
1 participant
Added ability to run patternfly design comment system via a skill.
Summary by CodeRabbit
New Features
Documentation