refactor(skills): slim down research skill and fix create-pr branch handling - #541
Conversation
- Reduce SKILL.md from 1957 to ~180 words (90% reduction) - Remove redundant references/ directory (content was duplicated in SKILL.md) - Remove test-scenarios.md (internal testing artifact, not needed at runtime) - Drop stale mgrep reference from subagent guide
|
Caution Review failedThe pull request is closed. ℹ️ Recent review infoConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughUpdates PR workflow documentation to automate branch creation from commit messages when on main/master, revises stop conditions, and consolidates research skill documentation from verbose handbook format to streamlined reference style. Multiple research reference documents are deleted and their content merged into the main SKILL.md. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
✨ 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 |
Slims the research skill and fixes a UX issue in create-pr.
Changes
research/SKILL.md: 1957 → ~180 words (90% reduction) — removed duplicated content, merged redundant sections, dropped references/ directoryresearch/references/: deleted (content was duplicated in SKILL.md)research/test-scenarios.md: deleted (internal testing artifact)create-pr/SKILL.md: auto-create branch when on main/master instead of stopping to askVerified
Summary by CodeRabbit