feat: Rename plugins to domain-based names#50
Merged
jpuzz0 merged 8 commits intopatternfly:mainfrom Apr 7, 2026
Merged
Conversation
jpuzz0
force-pushed
the
rename-plugins-to-domain-names
branch
from
5df3d13 to
954a936
Compare
jpuzz0
force-pushed
the
rename-plugins-to-domain-names
branch
from
954a936 to
7d2cc76
Compare
Contributor
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 2 minutes and 42 seconds. ⌛ 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. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (11)
📝 WalkthroughWalkthroughThis PR renames five PatternFly-prefixed plugins to shorter domain names ( Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
🚥 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)
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}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
plugins/react/skills/patternfly-component-structure/SKILL.md (1)
1-96:⚠️ Potential issue | 🔴 CriticalAdd required
name:field to frontmatter.The
SKILL.mdfrontmatter is missing the requiredname:field. Guidelines require: "Frontmatter is required in SKILL.md withnameanddescription. Thenamemust match the directory name." This field controls how the skill is invoked and must be present.Also follow the
pf-prefix convention used by other PatternFly-specific skills in this plugin (pf-bug-triage,pf-unit-test-generator, etc.). The directory and name should bepf-component-structureto match the pattern.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@plugins/react/skills/patternfly-component-structure/SKILL.md` around lines 1 - 96, Add a required frontmatter "name:" field to the SKILL.md frontmatter and set it to the PatternFly-prefixed directory name "pf-component-structure" (so name matches the directory and follows the pf- convention); update the top YAML block (the existing description: line) to include name: pf-component-structure directly under the initial --- frontmatter; ensure there are both name and description keys and no other changes to content.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@PLUGINS.md`:
- Around line 62-63: The PLUGINS.md entry for the skill
write-example-description contains a leaked YAML block scalar (">-") because the
skill's SKILL.md frontmatter has an empty or malformed description; open the
SKILL.md for the skill id write-example-description, fix the frontmatter by
replacing the block scalar with a proper non-empty description string (or remove
the empty description field), save, then re-run the docs/metadata generator that
produces PLUGINS.md so the entry is rebuilt without the raw ">-" artifact.
In `@plugins/react/README.md`:
- Line 39: Update the stale plugin invocation string: locate the occurrence of
"/pf-react:pf-library-test-writer" in the README and replace it with the new
plugin name "/react:pf-library-test-writer" so the skill invocation matches the
renamed plugin; ensure any other occurrences of the old prefix "pf-react:" are
similarly updated to "react:" to keep references consistent.
---
Outside diff comments:
In `@plugins/react/skills/patternfly-component-structure/SKILL.md`:
- Around line 1-96: Add a required frontmatter "name:" field to the SKILL.md
frontmatter and set it to the PatternFly-prefixed directory name
"pf-component-structure" (so name matches the directory and follows the pf-
convention); update the top YAML block (the existing description: line) to
include name: pf-component-structure directly under the initial --- frontmatter;
ensure there are both name and description keys and no other changes to content.
🪄 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: e5092b82-3762-42bd-89a6-d46478c592f3
📒 Files selected for processing (80)
.claude-plugin/marketplace.json.cursor-plugin/marketplace.jsonCONTRIBUTING-SKILLS.mdCONTRIBUTING.mdPLUGINS.mdREADME.mdplugins/a11y/.claude-plugin/plugin.jsonplugins/a11y/.cursor-plugin/plugin.jsonplugins/a11y/README.mdplugins/design-tokens/.claude-plugin/plugin.jsonplugins/design-tokens/.cursor-plugin/plugin.jsonplugins/design-tokens/README.mdplugins/design-tokens/skills/css-var-analyzer/SKILL.mdplugins/design-tokens/skills/css-var-analyzer/css-var-analyzer.jsplugins/design-tokens/skills/css-var-analyzer/format-css-report.jsplugins/design-tokens/skills/css-var-analyzer/scripts/analyze-css-vars.shplugins/design-tokens/skills/pf-analyze-modifiers/SKILL.mdplugins/design-tokens/skills/pf-raw-colors-scan/SKILL.mdplugins/design-tokens/skills/pf-token-auditor/SKILL.mdplugins/design-tokens/skills/pf-token-auditor/token-reference.mdplugins/design-tokens/skills/pf-tokens/SKILL.mdplugins/figma/.claude-plugin/plugin.jsonplugins/figma/.cursor-plugin/plugin.jsonplugins/figma/README.mdplugins/figma/skills/figma-changes/README.mdplugins/figma/skills/figma-changes/SKILL.mdplugins/figma/skills/figma-changes/examples/.figma-sync-cache.jsonplugins/figma/skills/figma-changes/examples/sample-output.mdplugins/figma/skills/figma-changes/examples/token-mappings.jsonplugins/figma/skills/figma-changes/references/figma-api-guide.mdplugins/figma/skills/figma-changes/references/template-changelog.mdplugins/figma/skills/figma-changes/references/template-detailed-report.mdplugins/figma/skills/figma-changes/references/template-release-notes.mdplugins/figma/skills/figma-changes/references/troubleshooting.mdplugins/figma/skills/figma-changes/references/validation-checklists.mdplugins/figma/skills/figma-changes/report-templates.mdplugins/figma/skills/figma-changes/scripts/compare-tokens.jsplugins/figma/skills/figma-changes/scripts/compare-tokens.shplugins/figma/skills/figma-changes/scripts/extract-figma-file-key.shplugins/figma/skills/figma-changes/scripts/figma-sync.shplugins/issue-management/.claude-plugin/plugin.jsonplugins/issue-management/.cursor-plugin/plugin.jsonplugins/issue-management/README.mdplugins/issue-management/skills/duplicate-epic/SKILL.mdplugins/issue-management/skills/duplicate-epic/scripts/duplicate_epic.shplugins/issue-management/skills/pf-create-issue/EXAMPLE.mdplugins/issue-management/skills/pf-create-issue/README.mdplugins/issue-management/skills/pf-create-issue/SKILL.mdplugins/issue-management/skills/quarterly-initiative-report/SKILL.mdplugins/pf-workflow/.claude-plugin/plugin.jsonplugins/pf-workflow/.cursor-plugin/plugin.jsonplugins/pf-workflow/README.mdplugins/react/.claude-plugin/plugin.jsonplugins/react/.cursor-plugin/plugin.jsonplugins/react/README.mdplugins/react/agents/component-structure-audit.mdplugins/react/agents/pf-coding-standards.mdplugins/react/agents/pf-unit-test-standards.mdplugins/react/skills/icon-finder/SKILL.mdplugins/react/skills/patternfly-component-structure/SKILL.mdplugins/react/skills/patternfly-component-structure/references/containers.mdplugins/react/skills/patternfly-component-structure/references/data-components.mdplugins/react/skills/patternfly-component-structure/references/navigation-toolbar.mdplugins/react/skills/patternfly-component-structure/references/page-layout.mdplugins/react/skills/pf-bug-triage/SKILL.mdplugins/react/skills/pf-bug-triage/reference.mdplugins/react/skills/pf-class-migration-scanner/SKILL.mdplugins/react/skills/pf-import-checker/SKILL.mdplugins/react/skills/pf-library-test-writer/SKILL.mdplugins/react/skills/pf-project-scaffolder/SKILL.mdplugins/react/skills/pf-unit-test-generator/SKILL.mdplugins/react/skills/write-example-description/SKILL.MDplugins/repo-management/.claude-plugin/plugin.jsonplugins/repo-management/.cursor-plugin/plugin.jsonplugins/repo-management/README.mdplugins/repo-management/skills/analytics-repo-pruning/SKILL.mdplugins/repo-management/skills/analytics-repo-pruning/reference.mdplugins/repo-management/skills/analytics-repo-pruning/scripts/analytics-repo-pruning.shplugins/repo-management/skills/dependency-recommender/SKILL.mdplugins/repo-management/skills/pf-org-version-update/SKILL.md
💤 Files with no reviewable changes (3)
- plugins/pf-workflow/README.md
- plugins/pf-workflow/.claude-plugin/plugin.json
- plugins/pf-workflow/.cursor-plugin/plugin.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes PF-3908
pf-*prefixed to descriptive, domain-based names so plugin names tell users exactly what they help withpf-workflowinto two focused plugins:issue-managementandrepo-managementRename map
pf-reactreactpf-stylingdesign-tokenspf-a11ya11ypf-figmafigmapf-workflowissue-management+repo-managementWhy
pf-stylingdoesn't tell you what it does —design-tokensdoes.pf-workflowcovered two unrelated domains — issue tracking and repo maintenance — so it's split into two.pf-prefix moves to the skill level where it distinguishes PatternFly-specific skills (pf-unit-test-generator) from generic ones (dependency-recommender). No skill renames in this PR.Summary by CodeRabbit
New Features
Updates
Documentation