Skip to content

refactor(me): convert create-pr skill to standalone command - #318

Merged
baleen37 merged 2 commits into
mainfrom
chore/fix-create-pr
Feb 2, 2026
Merged

refactor(me): convert create-pr skill to standalone command#318
baleen37 merged 2 commits into
mainfrom
chore/fix-create-pr

Conversation

@baleen37

@baleen37 baleen37 commented Feb 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Converted plugins/me/skills/create-pr/ skill to standalone command
  • Removed skill directory structure, merged into commands/create-pr.md
  • Simplified content by removing redundant sections (Graphviz diagram, When to Use, Rationalization Table)
  • Updated documentation to reflect Skills count: 6→5

Changes

  • Before: 141 lines with skill indirection
  • After: 92 lines (35% reduction) with direct workflow definition

Test plan

  • Command structure follows other commands pattern
  • All references to create-pr skill removed from documentation
  • No merge conflicts with main branch

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Reorganized and enhanced create-pr skill documentation with comprehensive guide sections including core principles, quick reference, decision points, and PR templates.
    • Updated skill references and listings across documentation files.

✏️ Tip: You can customize this high-level summary in your review settings.

baleen37 and others added 2 commits February 2, 2026 11:49
- Move plugins/me/skills/create-pr/SKILL.md → plugins/me/commands/create-pr.md
- Remove skill directory structure (skills/create-pr/)
- Update documentation (README.md, CLAUDE.md) to reflect Skills count: 6→5
- Simplify command content:
  - Remove Graphviz diagram (30 lines) - Quick Reference table is sufficient
  - Remove "When to Use" section (10 lines) - already in description
  - Remove Rationalization Table (9 lines) - duplicates Common Mistakes
- Result: 141 lines → 92 lines (35% reduction)

Command now directly defines workflow without skill indirection.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@baleen37
baleen37 enabled auto-merge (squash) February 2, 2026 02:50
@coderabbitai

coderabbitai Bot commented Feb 2, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Restructuring of create-pr skill documentation: removing the skill definition file (skills/create-pr/SKILL.md) and its manifest entries, while simultaneously expanding the command documentation (commands/create-pr.md) with comprehensive multi-section content. This consolidates create-pr documentation from skill manifests to the command layer.

Changes

Cohort / File(s) Summary
Manifest & Index Files
plugins/me/CLAUDE.md, plugins/me/README.md, plugins/me/skills/CLAUDE.md
Removed create-pr skill references from inventory files and skill count reduction from 6 to 5.
Command Documentation
plugins/me/commands/create-pr.md
Expanded with comprehensive multi-section documentation including core principles, red flags, quick reference, decision points, PR body template, common mistakes, and argument specifications.
Skill Definition
plugins/me/skills/create-pr/SKILL.md
Deleted entire skill definition file previously containing Git workflow documentation, diagrams, and comprehensive reference material.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

released

Poem

🐰 Files reorganized with care,
Old docs bundled elsewhere,
Commands shine bright and true,
Skills consolidated anew! ✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/fix-create-pr

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@baleen37
baleen37 merged commit d94df80 into main Feb 2, 2026
1 check passed
@baleen37
baleen37 deleted the chore/fix-create-pr branch February 2, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant