refactor(me): optimize create-pr skill for token efficiency - #172
Conversation
Rename skill and command from "commit-push-pr" to "create-pr" for better clarity and user experience. Changes: - Rename skill directory: commit-push-pr → create-pr - Rename command file: commit-push-pr.md → create-pr.md - Update all internal references in documentation - Update test assertion for renamed command file - Fix inconsistency: commit-push-pull-request → create-pr Rationale: - "create-pr" better reflects user intent (outcome vs process) - More concise and intuitive naming - Aligns with common git workflow terminology No functional changes - pure rename with documentation updates. Co-Authored-By: Claude <noreply@anthropic.com>
- Reduce SKILL.md from 276 to 153 lines (44% reduction) - Move detailed docs to references/ directory - Optimize conflict-check.sh to accept BASE as argument - Consolidate duplicate auto-merge sections - Standardize file naming (lowercase_with_underscores) - Remove unnecessary watermarks and Co-Authored-By - Apply 2026 best practices: concise, imperative language
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis PR restructures the create-pr skill documentation from a procedural implementation guide to a principle-driven workflow, updates the quick-start checklist with renumbered steps and refined guidance, and improves the conflict-check.sh script with explicit base branch argument handling and merge-base calculations instead of direct remote merge operations. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes Possibly related PRs
Suggested labels
Poem
✨ Finishing touches
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 |
Summary
Test plan
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.