Skip to content

refactor(skills): slim down research skill and fix create-pr branch handling - #541

Merged
baleen37 merged 2 commits into
mainfrom
refactor/slim-research-skill
Feb 26, 2026
Merged

refactor(skills): slim down research skill and fix create-pr branch handling#541
baleen37 merged 2 commits into
mainfrom
refactor/slim-research-skill

Conversation

@baleen37

@baleen37 baleen37 commented Feb 26, 2026

Copy link
Copy Markdown
Owner

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/ directory
  • research/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 ask

Verified

  • SKILL.md reads cleanly end-to-end
  • No conflicts with origin/main

Summary by CodeRabbit

  • Documentation
    • Updated PR creation workflow to include enhanced automation features and clearer stopping conditions for improved efficiency.
    • Streamlined research skill guidance from verbose handbook into action-oriented reference format with simplified tool selection, verification standards, and evidence guidelines.
    • Reorganized research documentation structure for better user navigation and accessibility.

Test User added 2 commits February 27, 2026 00:50
- 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
@baleen37
baleen37 enabled auto-merge (squash) February 26, 2026 15:55
@coderabbitai

coderabbitai Bot commented Feb 26, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1ec67a6 and 9443761.

📒 Files selected for processing (6)
  • plugins/me/skills/create-pr/SKILL.md
  • plugins/me/skills/research/SKILL.md
  • plugins/me/skills/research/references/evidence-standards.md
  • plugins/me/skills/research/references/rationalization-guide.md
  • plugins/me/skills/research/references/subagent-guide.md
  • plugins/me/skills/research/test-scenarios.md

📝 Walkthrough

Walkthrough

Updates 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

Cohort / File(s) Summary
PR Workflow Documentation
plugins/me/skills/create-pr/SKILL.md
Pre-flight step redefined to describe automated branch creation workflow when on main/master instead of a check guideline. Stop conditions changed from "On main/master" and "No changes to commit" to "No changes to commit and no unpushed commits."
Research Skill Restructuring
plugins/me/skills/research/SKILL.md
Major documentation refactor condensing verbose handbook into streamlined reference. Quick Reference and strategy sections replaced with Tool Selection. Extensive tables simplified. Evidence Standards and Rationalization blocks moved to compact Red Flags—STOP section and verification prompts.
Research Reference Consolidation
plugins/me/skills/research/references/evidence-standards.md, rationalization-guide.md, subagent-guide.md, test-scenarios.md
Four reference documents deleted. Content consolidated into main research SKILL.md. Removed: evidence standards definitions, rationalization guidance, subagent strategy guide, and test scenarios documentation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 Branch creation now springs from the commit's wise tale,
Stop conditions tightened, no shortcuts prevail.
Research docs distilled from verbose prose,
To streamlined wisdom—efficiency grows!
Reference files merged with purposeful care, 🌟

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/slim-research-skill

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 87c82b8 into main Feb 26, 2026
4 of 5 checks passed
@baleen37
baleen37 deleted the refactor/slim-research-skill branch February 26, 2026 15:55
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