Skip to content

Commit 7691c77

Browse files
ryzizubclaude
andauthored
refactor(skills): remove create-branch and create-commit skills (#212)
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 4b0759c commit 7691c77

11 files changed

Lines changed: 15 additions & 263 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ jobs:
4444
run: |
4545
BASE=${{ github.event.pull_request.base.sha }}
4646
HEAD=${{ github.event.pull_request.head.sha }}
47-
SKILLS=$(git diff --name-only $BASE $HEAD | \
47+
SKILLS=$(git diff --name-only --diff-filter=d $BASE $HEAD | \
4848
{ grep 'SKILL.md' || true; } | \
4949
xargs -I {} dirname {} | \
5050
sort -u | \

CLAUDE.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ Each phase persists its output to `docs/` so the next phase can discover it from
3535
Supporting skills:
3636

3737
- `/create` (project creation — routes to companion plugins)
38-
- `/create-branch` (workspace setup)
39-
- `/create-commit` (propose and create conventional commit messages for staged changes)
4038
- `/create-pr` (generate a PR title and description from branch commits and optionally open it on GitHub or GitLab)
4139
- `/plan-technical-review` (validate plans)
4240
- `/refine-approach` (iterative document improvement)

README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,7 @@ Wingspan operates at a higher level, orchestrating agentic workflows across the
9898
| [**Build**](skills/build/SKILL.md) | `/build <plan file path>` | Execute a plan — write code and tests, run quality review, ship a PR |
9999
| [**Review**](skills/review/SKILL.md) | `/review [path]` | Run quality review agents on demand — assess code quality and identify issues |
100100
| [**Hotfix**](skills/hotfix/SKILL.md) | `/hotfix <bug description>` | Apply a minimal, targeted fix for emergency bugs — enforces review and testing without brainstorm or planning |
101-
| [**Create Branch**](skills/create-branch/SKILL.md) | `/create-branch` | Set up a workspace (branch or worktree) before writing artifacts |
102101
| [**Create**](skills/create/SKILL.md) | `/create <what to create>` | Scaffold a new project by routing to the right companion plugin |
103-
| [**Create Commit**](skills/create-commit/SKILL.md) | `/create-commit` | Stage and commit changes using conventional commit messages |
104102
| [**Create PR**](skills/create-pr/SKILL.md) | `/create-pr` | Validate (formatter, linter, tests, and CI checks), stage, commit, push, and open a pull request on the project's Git hosting platform — aborts on any failure |
105103
| [**Rebase**](skills/rebase/SKILL.md) | `/rebase` | Rebase the current feature branch onto the base branch to stay up-to-date |
106104
| [**Debrief**](skills/debrief/SKILL.md) | `/debrief <incident or context>` | Produce a structured post-incident analysis — timeline, root cause, and actionable follow-ups |

skills/brainstorm/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,9 @@ Use **AskUserQuestion tool** to ask which approach the user prefers.
101101

102102
#### 1.4. Set up workspace
103103

104-
Before writing any files, ensure the session is on a feature branch:
104+
Before writing any files, ensure the session is not on the base branch:
105105

106-
- Call @create-branch to check and optionally create a working branch or worktree.
106+
- Run `git rev-parse --abbrev-ref HEAD`. If the current branch is a base branch (`main`, `master`, or `develop`), use **AskUserQuestion** to offer creating a feature branch — `git checkout -b <type>/<kebab-topic>`, name under 60 characters — before writing. If already on a feature branch, continue without prompting.
107107

108108
### 2. Capture the design document
109109

skills/create-branch/SKILL.md

Lines changed: 0 additions & 108 deletions
This file was deleted.

skills/create-branch/scripts/detect-base-branch.sh

Lines changed: 0 additions & 1 deletion
This file was deleted.

skills/create-commit/SKILL.md

Lines changed: 0 additions & 133 deletions
This file was deleted.

skills/create-pr/SKILL.md

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
name: create-pr
33
description: Stage, commit, push, and open a pull request following project conventions and the Conventional Commits spec. Accepts optional skip-checks argument to bypass validation when called from /build.
4-
when_to_use: Use when user says "create a PR", "open a PR", "ship it", "submit a pull request", or "open a merge request".
4+
when_to_use: Use when user says "create a PR", "open a PR", "ship it", "submit a pull request", or "open a merge request", or when work on a branch is complete and ready to publish for review.
55
argument-hint: "[optional: skip-checks | ticket/issue number e.g. VGV-123 | short description]"
66
disable-model-invocation: true
77
allowed-tools: Bash(git push *) Bash(git add *) Bash(git commit *) Bash(gh *) Bash(glab *)
@@ -27,14 +27,6 @@ Stage uncommitted changes, commit them, push the branch, and open a pull request
2727
- Do not push before the user confirms the commit.
2828
- Use the current branch as the source; target is `BASE_BRANCH` (determined in Step 2).
2929

30-
## When to use
31-
32-
Use this skill when:
33-
34-
- The user asks to open or create a pull request.
35-
- The user asks to "create a PR", "open a PR", "submit a PR", or similar.
36-
- Work on a branch is complete and the user wants to publish it for review.
37-
3830
## Context
3931

4032
<context>$ARGUMENTS</context>
@@ -86,7 +78,13 @@ Store as `BASE_BRANCH`.
8678

8779
## Step 3: Stage and commit
8880

89-
Use the **create-commit** skill with the `single-commit` argument to stage files and produce a single conventional commit message.
81+
Produce a single conventional commit covering all changes.
82+
83+
- **Never stage** secret files: `.env`, `*.key`, `*.pem`, `*secret*`, `*credential*`, `*.p12`, `*.jks`.
84+
- Stage the remaining changes with `git add`.
85+
- Propose one commit message following Conventional Commits (`type(scope): subject`). Consult `references/conventional-commits.md` for the full spec. Infer `type` from a plan file in `docs/plan/` if present; otherwise infer from the diff. Extract the ticket number from the branch name (e.g. `feat/VGV-59-...``VGV-59`) into a `Refs:` line.
86+
87+
Use **AskUserQuestion** to confirm before committing (**Yes** / **No** / **Edit**). On confirm, commit with a HEREDOC to preserve formatting. Do not push yet.
9088

9189
## Step 4: Push
9290

File renamed without changes.

skills/debrief/SKILL.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,9 @@ Action items are recorded in the document only — they become separate tickets.
7474

7575
### 5. Set up workspace
7676

77-
Before writing the debrief file, ensure the session is on a feature branch:
77+
Before writing the debrief file, ensure the session is not on the base branch:
7878

79-
- Call @create-branch to check and optionally create a working branch or worktree.
79+
- Run `git rev-parse --abbrev-ref HEAD`. If the current branch is a base branch (`main`, `master`, or `develop`), use **AskUserQuestion** to offer creating a feature branch — `git checkout -b <type>/<kebab-topic>`, name under 60 characters — before writing. If already on a feature branch, continue without prompting.
8080

8181
### 6. Write the debrief document
8282

0 commit comments

Comments
 (0)