refactor(create-pr): improve skill per writing-skills guidelines

[baleen37]

Summary

• Fix description to focus on triggers only (CSO compliance)
• Remove unsupported user-invocable frontmatter field
• Add graphviz flowchart for workflow visualization
• Consolidate Quick Reference into table format
• Remove redundant Workflow section, keep Key Decision Points
• Clarify Arguments section
• Remove TDD-VALIDATION.md (not practical, adds maintenance burden)

Word count reduced from ~800 to ~700 words.

Test plan

• BATS tests pass
• Skill loads correctly in Claude Code

Summary by CodeRabbit

• Documentation
  • Enhanced create-pr skill documentation with comprehensive workflow diagrams, detailed decision points for base branch detection and conflict resolution, expanded quick reference guide, and improved step-by-step instructions for PR lifecycle management.
  • Removed validation testing documentation.

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

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This pull request substantially refactors documentation for the create-pr skill. SKILL.md is rewritten with expanded workflow guidance, including new sections for workflow overview, key decision points, and detailed workflow decisions. TDD-VALIDATION.md is removed entirely. No code or API changes are introduced.

Changes

Cohort / File(s)	Summary
Documentation Refactor plugins/me/skills/create-pr/SKILL.md	Substantially rewritten workflow documentation: adds Graphviz workflow diagram, expands "Red Flags - STOP" section, replaces Quick Reference with comprehensive table, introduces Key Decision Points subsection, adds detailed Workflow Decision sections, updates PR Body Template, restructures Common Mistakes section, adds Rationalization Table, clarifies Arguments with supported flags and examples.
Documentation Deletion plugins/me/skills/create-pr/TDD-VALIDATION.md	Entire TDD validation report file removed (previously contained multi-phase validation workflow, metrics, gaps, verification tests, adversarial hardening tests, comparison with original skill, and deployment status).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• PR #195: Overlapping edits to the same create-pr skill documentation files (SKILL.md, TDD-VALIDATION.md) with conflicting changes across both PRs.
• PR #193: Modifies create-pr skill's SKILL.md with reworked base-branch detection, conflict-check/update steps, and PR/merge guidance.
• PR #181: Removes now-redundant command wrapper that depends on the create-pr skill being documented.

Suggested labels

released

Poem

🐰 ✨ A skill reborn in words so clear,
Diagrams guide the workflow dear,
Decision points and red flags bright,
The create-PR shines with new light!
Old tests rest, new docs take flight! 📝

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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