Skip to content

refactor(skills): remove create-branch and create-commit skills#212

Merged
ryzizub merged 3 commits into
mainfrom
claude/trusting-nobel-6e712d
Jul 3, 2026
Merged

refactor(skills): remove create-branch and create-commit skills#212
ryzizub merged 3 commits into
mainfrom
claude/trusting-nobel-6e712d

Conversation

@ryzizub

@ryzizub ryzizub commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Description

Removes the create-branch and create-commit skills. They duplicated git capabilities agents already have. Their genuinely useful, non-obvious behavior is preserved rather than dropped:

  • create-branch — its base-branch guard is inlined into brainstorm, plan, and debrief. Each checks the current branch and prompts to create a feature branch only when on main/master/develop, otherwise stays silent. No script dependency (the shared detect-base-branch.sh and the rebase symlink are untouched).
  • create-commit — its logic (secret-file staging guard, Conventional Commits format, plan-type inference from docs/plan/, ticket Refs: extraction) is absorbed into create-pr Step 3. conventional-commits.md moved into create-pr/references/.

create-pr is kept and hardened. It enforces the repo PR-template convention that agents otherwise tend to skip — the main reason to keep it. Its template detection and Type-of-Change checkbox mapping are unchanged.

Docs updated: README skill table and CLAUDE.md supporting-skills list. CHANGELOG left as history.

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Code refactor (refactor)
  • Documentation (docs)
  • CI change (ci)
  • Chore (chore)

🤖 Generated with Claude Code

@ryzizub ryzizub requested a review from a team as a code owner July 2, 2026 15:07
@ryzizub ryzizub force-pushed the claude/trusting-nobel-6e712d branch from 5aa2bb2 to f4d2b85 Compare July 2, 2026 15:10
These duplicated capabilities agents already have on their own. Their
unique value is preserved rather than lost:

- create-branch's base-branch guard is inlined into brainstorm, plan, and
  debrief: each checks the current branch and prompts to branch only when
  on main/master/develop, otherwise stays silent.
- create-commit's logic (secret-file staging guard, Conventional Commits
  format, plan-type inference, ticket Refs) is absorbed into create-pr
  Step 3. conventional-commits.md moved into create-pr/references/.

create-pr is kept and hardened. It enforces the repo PR-template
convention that agents otherwise tend to skip. Docs updated: README skill
table and CLAUDE.md supporting-skills list.

CI: the Detect Changed Skills step listed deleted SKILL.md paths, spawning
validate jobs for directories that no longer exist. Added --diff-filter=d
so deletions are excluded from the matrix.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@ryzizub ryzizub force-pushed the claude/trusting-nobel-6e712d branch from f4d2b85 to 18dbd90 Compare July 3, 2026 08:27
Comment thread skills/create-pr/SKILL.md
Addresses review: the removed "When to use" body block had one trigger the
when_to_use frontmatter lacked — "work on a branch is complete and ready to
publish for review." Merged it into the frontmatter field.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@ryzizub ryzizub merged commit 7691c77 into main Jul 3, 2026
8 checks passed
@ryzizub ryzizub deleted the claude/trusting-nobel-6e712d branch July 3, 2026 12:44
@vgvbot vgvbot mentioned this pull request Jul 3, 2026
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.

2 participants