feat(skills): sync ship-category skills with addyosmani upstream - #642
Conversation
Replace 5 ship-category skills with full upstream versions from addyosmani/agent-skills, and remove two skills that are now subsumed by the richer upstream content: - git-workflow-and-versioning (38 → 300 lines) - ci-cd-and-automation (33 → 390 lines) - deprecation-and-migration (46 → 206 lines) - documentation-and-adrs (58 → 278 lines) - shipping-and-launch (63 → 309 lines) - remove: commit, project-setup
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughSeven skill documentation files were restructured and substantially expanded. Two skills (commit and project-setup) were entirely removed. The remaining five skills—CI/CD automation, deprecation/migration, documentation/ADRs, Git workflow, and shipping/launch—were expanded with detailed overviews, decision frameworks, structured workflows, concrete code examples, and verification checklists. Updates follow a consistent pattern: metadata, applicability guidance, core principles, step-by-step procedures, code templates, patterns/automation, and quality verification sections. ChangesSkill Documentation Expansion and Consolidation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 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 |
# [17.1.0](v17.0.0...v17.1.0) (2026-05-10) ### Features * **skills:** sync ship-category skills with addyosmani upstream ([#642](#642)) ([bfeaac5](bfeaac5))
Summary
plugins/me/skills/with the full upstream versions from addyosmani/agent-skills. Local versions had been heavily abridged (33-63 lines) and lost the upstream's strategic context — Common Rationalizations, Red Flags, Verification sections, and concrete templates (fullci.yml, ADR-001 template, deprecation notice, rollback runbook).commitandproject-setup.Changes
No conflicts with existing local assets —
/ship,/verify,commit-guard.shhook and its BATS tests are untouched and complementary to the new strategic skills.Test plan
bats tests/— 71/71 passingcommit/project-setupskillsSummary by CodeRabbit