Skip to content

refactor(me): optimize create-pr skill for token efficiency - #172

Merged
baleen37 merged 3 commits into
mainfrom
chore/create-pr
Jan 23, 2026
Merged

refactor(me): optimize create-pr skill for token efficiency#172
baleen37 merged 3 commits into
mainfrom
chore/create-pr

Conversation

@baleen37

@baleen37 baleen37 commented Jan 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • Reduce SKILL.md from 276 to 153 lines (44% token reduction)
  • Move detailed documentation to references/ directory structure
  • Optimize conflict-check.sh to accept BASE as argument (eliminates redundant calculation)
  • Consolidate duplicate auto-merge sections into single source of truth
  • Apply 2026 best practices: concise, imperative language, no watermarks

Test plan

  • Tested skill workflow end-to-end
  • Verified conflict detection works with BASE argument
  • Confirmed all references resolve correctly

Summary by CodeRabbit

  • Documentation
    • Reorganized create-pr workflow documentation with streamlined quick start guide
    • Updated checklist items, pre-flight checks, and workflow step sequences
    • Enhanced conflict resolution process with improved error messages and resolution guidance
    • Refined PR workflow documentation with state-based PR action tables and consolidated guidelines

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

baleen37 and others added 3 commits January 24, 2026 00:01
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
@baleen37
baleen37 enabled auto-merge (squash) January 23, 2026 15:14
@coderabbitai

coderabbitai Bot commented Jan 23, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Documentation restructuring
plugins/me/skills/create-pr/QUICK_START.md, plugins/me/skills/create-pr/SKILL.md
Significant rewrite of SKILL.md from "Overview + Implementation (6 Steps)" format to "Core Principles + Workflow (5 Steps)" structure. Updated QUICK_START.md with renumbered workflow steps, refined conflict check guidance referencing script invocation, and rewritten PR state action tables. Consolidated verbose sections into compact tables for error handling and guardrails.
Script logic refinement
plugins/me/skills/create-pr/scripts/conflict-check.sh
Changed base branch handling from remote derivation to explicit argument with "main" default. Added fetch error handling, merge-base calculation and validation, and updated conflict detection to use computed merge-base instead of direct origin merge. Enhanced error messaging with explicit resolution steps and guide references.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

Suggested labels

released

Poem

🐰 A rabbit hops through the workflow,
Restructured neat, from chaos to flow,
Conflicts now checked with merge-base delight,
Fast reference guides make PR paths bright!
Create-pr skill bounces—refined and tight!

✨ Finishing touches
  • 📝 Generate docstrings

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 b5b9eb6 into main Jan 23, 2026
1 of 2 checks passed
@baleen37
baleen37 deleted the chore/create-pr branch January 23, 2026 15:15
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