| name | create-pr |
| description | Create a Pull Request from the current branch — runs local checks, picks target branch, and opens the PR on GitHub |
Create Pull Request
Guides the developer through creating a Pull Request for a Nextcloud app. Confirms the source branch, recommends a target branch based on the branching strategy, optionally runs local quality checks, then creates the PR via the GitHub REST API.
Model Recommendation
This skill involves parsing CI workflows, detecting branch-protection rules, resolving bootstrap deadlocks, and reasoning about code diffs. Mistakes here have real consequences.
First, check the active model from your system context (it appears as "You are powered by the model named…").
"You're on [active-model]. Which model should I use for this PR?"
| Model | Best for |
|---|
| Sonnet | Most PRs — handles CI parsing and branch logic well |
| Opus | Repos with reusable CI workflows, branch-protection rulesets, or a complex branching strategy — that's where it pays off most |
If the chosen model differs from the active model, tell the user:
"You're on [active-model] but chose [chosen-model]. To switch: use /model [chosen-model] in the chat input, or open the model picker in the Claude Code UI. Then re-run this command."
Then stop.
Hard Rules
NEVER modify .github/workflows/ files. This skill reads workflow files to understand what CI runs, but must never edit, create, or delete any workflow file or change any job definition — regardless of what checks fail or what the branch protection requires. If a workflow mismatch is detected (e.g. wrong job name for a required status check), report the issue to the user and stop — do not attempt to fix it.
Step 0: Select Repository
Scan the workspace for available git repositories:
WORKSPACE_ROOT="$(git rev-parse --show-toplevel 2>/dev/null)/.."
for dir in "$WORKSPACE_ROOT"/*/; do
if [ -d "$dir/.git" ] || git -C "$dir" rev-parse --git-dir > /dev/null 2>&1; then
echo "$dir"
fi
done
For each found repo, also get its remote URL and current branch:
git -C {dir} remote get-url origin 2>/dev/null
git -C {dir} branch --show-current 2>/dev/null
Ask the user using AskUserQuestion:
"Which repository do you want to create a PR for?"
List each repo as: {app-name} [{current-branch}] ({remote-url})
Store the selected repo's absolute path as {REPO_ROOT}.
Step 1: Detect Current Branch & Repo
Run within {REPO_ROOT}:
git -C {REPO_ROOT} branch --show-current
git -C {REPO_ROOT} remote get-url origin
Store:
{CURRENT_BRANCH} — active branch name
{REMOTE_URL} — GitHub repo URL
Step 2: Confirm Source Branch
Ask the user using AskUserQuestion:
"Your current branch is {CURRENT_BRANCH}. Is this the correct branch to create a PR from?"
- Yes — proceed with
{CURRENT_BRANCH}
- No, let me specify → ask them for the correct branch name and store it as
{CURRENT_BRANCH}
Step 3: Recommend Target Branch
Based on {CURRENT_BRANCH}, determine the recommended target using the project branching strategy:
| Source branch pattern | Recommended target | Allowed targets |
|---|
feature/*, bugfix/*, or other non-standard branch | development | development |
development | beta | beta |
beta | main | main |
hotfix/* | main | main, beta, development |
Fetch available remote branches:
git -C {REPO_ROOT} fetch --prune
git -C {REPO_ROOT} branch -r | grep -v HEAD | sed 's|origin/||' | sort
Ask the user using AskUserQuestion:
"Which branch should this PR target?"
List the allowed target branches, marking the recommended one with (recommended). If there is only one valid option, pre-select it and ask for confirmation instead.
Store the answer as {TARGET_BRANCH}.
Step 3.2: Validate Target Branch Against Branch-Protection Workflow
Read references/branch-protection-guide.md for the full procedure covering: branch-protection workflow validation (Step 3.2), required-check bootstrap deadlock detection (Step 3.2b), and bootstrap deadlock resolution post-push (Step 3.2c). Apply all three steps now.
Step 3.4: Verify Global Settings Version (ConductionNL/.github only)
Only run this step if {REMOTE_URL} contains ConductionNL/.github.
Invoke the /verify-global-settings-version skill now and follow its steps completely.
-
If it reports Case A or B (no changes, or correctly bumped) → proceed silently to Step 3.5.
-
If it reports Case C (VERSION BUMP MISSING) → pause the PR flow. Ask using AskUserQuestion:
"The global-settings VERSION has not been bumped. How do you want to proceed?"
- Apply a patch bump now — increment to next patch, show the new value, then continue to Step 3.5
- Apply a minor bump now — increment minor, show new value, then continue to Step 3.5
- I'll fix it manually — stop here
-
If it reports Case D (VERSION bumped but no file changes) → warn the user but allow the PR to proceed.
Step 3.5: Check for Existing PR
Before running any checks, look up whether a PR already exists for this source → target combination:
gh pr list --repo "{REMOTE_URL}" --head "{CURRENT_BRANCH}" --base "{TARGET_BRANCH}" --state open --json number,title,url,createdAt,author
Also check for a closed/merged PR to give full context:
gh pr list --repo "{REMOTE_URL}" --head "{CURRENT_BRANCH}" --base "{TARGET_BRANCH}" --state merged --json number,title,url,mergedAt --limit 1
If an open PR already exists:
Inform the user clearly:
"An open PR already exists for {CURRENT_BRANCH} → {TARGET_BRANCH}:
#{number}: {title}
{url}
Opened by {author} on {createdAt}"
Then ask using AskUserQuestion:
"A PR for this branch already exists. What would you like to do?"
- View the existing PR — open the URL and stop here
- Update the existing PR (push new commits + update description) — store
{EXISTING_PR_NUMBER} and proceed to Step 3.7 as normal; in Step 7 use PATCH instead of POST
- Continue anyway (create a duplicate — not recommended) — proceed to Step 3.7 as normal
If a merged PR was found but no open PR:
Inform the user:
"Note: A previous PR for {CURRENT_BRANCH} → {TARGET_BRANCH} was already merged (#{number}: {title}, merged {mergedAt}). You may be re-opening the same work."
Then proceed normally to Step 4.
If no PR exists: proceed silently to Step 4.
Step 3.7: Check for Uncommitted or Unpushed Changes
Check the working tree and push status of {CURRENT_BRANCH}:
git -C {REPO_ROOT} status --short
git -C {REPO_ROOT} log origin/{CURRENT_BRANCH}...HEAD --oneline 2>/dev/null || echo "(branch not yet on remote)"
If there are uncommitted changes (modified, untracked, or staged files):
Before listing all uncommitted changes, specifically check for lock files that are untracked or modified:
git -C {REPO_ROOT} status --short | grep -E "composer\.lock|package-lock\.json"
If composer.lock or package-lock.json appear as untracked (??) or modified (M), warn prominently:
"⚠️ {filename} is not committed. CI installs dependencies from the lock file — without it, dependency versions may differ between local and CI, causing check failures. This should be committed before creating the PR."
Inform the user of all uncommitted changes, listing the files. Then ask using AskUserQuestion:
"There are uncommitted changes on {CURRENT_BRANCH}. What would you like to do?"
- Commit them now — ask for a commit message, run
git -C {REPO_ROOT} add -A && git -C {REPO_ROOT} commit -m "{message}", then continue
- Stash them — run
git -C {REPO_ROOT} stash, continue, and remind user to git stash pop afterwards
- Continue without committing — proceed (these changes will not be in the PR)
If the branch has commits not yet pushed to origin:
Inform the user. Then ask using AskUserQuestion:
"Branch {CURRENT_BRANCH} has unpushed commits. Push them now before continuing?"
- Yes, push now — run
git -C {REPO_ROOT} push -u origin {CURRENT_BRANCH}, then continue
- No, push later — note that unpushed commits won't be in the PR until pushed; continue
Step 4: Run Local Checks (optional)
Ask the user using AskUserQuestion:
"Do you want to run local quality checks before creating the PR? This mirrors exactly what CI will run and ensures the PR checks pass."
- Yes, run checks — proceed to Step 4a
- No, skip checks — proceed to Step 5
Step 4a: Read CI Workflows — the Source of Truth
The workflow files define exactly what to run locally. Do not hardcode assumptions.
Find all workflow files triggered on push or pull_request:
find {REPO_ROOT}/.github/workflows -name "*.yml" -o -name "*.yaml" 2>/dev/null | sort
If no .github/workflows directory exists:
"No GitHub Actions workflows found — skipping local checks."
Then proceed to Step 5.
Read each workflow file. For each job in a triggered workflow:
If the job runs steps directly — read and record every run: step in order.
If the job delegates to a reusable workflow (uses: org/repo/.github/workflows/file.yml@ref) — fetch and read that workflow too:
gh api "repos/{org}/{repo}/contents/{path}?ref={ref}" --jq '.content' | base64 -d
Then read the reusable workflow's jobs and their run: steps, also noting any inputs: the calling workflow passes (e.g. enable-eslint: true) — these control which jobs/steps actually execute.
Build a complete ordered list of every run: step the CI executes for this repo's push/PR trigger.
Step 4b: Determine Check Working Directory
For the Nextcloud workspace, checks run inside the app subdirectory. Detect from the changed files:
git -C {REPO_ROOT} diff --name-only origin/{TARGET_BRANCH}...HEAD
Look for which app directory has the most changed files. If ambiguous, ask:
"Which app directory should we run checks in?" — list the changed app directories.
Store as {CHECK_DIR}.
Step 4c: Categorise Steps and Build Execution Plan
Read references/ci-step-categorisation-guide.md for the full categorisation rules (Install / Check / Docker check / Skip), mixed-job handling, "requires Nextcloud" detection signals, Docker check adaptations (PHPUnit, Newman), lock file gate, and execution plan display format. Apply the full procedure now.
Step 4d: Run Install Steps
Run each install step exactly as it appears in the CI workflow, in CI order. If any install step fails, stop immediately and show the full error — do not proceed to checks.
Step 4d-verify: Verify Check Tools Are Available
Read references/check-tool-verification.md for the full verification procedure: tool checklist by command pattern, local vs Docker probe commands, missing-tool report format, and the three-option resolution prompt. Apply the full procedure now.
Step 4e: Run Check Steps
Run each check step exactly as it appears in the CI workflow, in CI order. Run them one by one, show output as each completes, and record pass/fail.
For steps flagged as optional (e.g. slow test suites with phpunit/pytest), ask first:
"Run {command} too? (this may be slow)"
Step 4f: Report & Decide
Display a results table with one row per check step:
CI check results:
[{job name}] {command} ✅ PASS / ❌ FAIL
[{job name}] {command} ✅ PASS / ❌ FAIL
...
-
If all checks pass → proceed to Step 5 with a success note.
-
If any check fails → show the full output and ask using AskUserQuestion:
"Some checks failed. How do you want to proceed?"
- Fix issues first, then re-run — stop here; let the user fix and re-invoke the skill
- Create PR anyway — proceed to Step 5 with a warning note in the PR body listing which checks failed
Step 5: Analyse Branch Changes
Collect the full picture of what is on this branch before drafting anything:
git -C {REPO_ROOT} log origin/{TARGET_BRANCH}...HEAD --oneline
git -C {REPO_ROOT} log origin/{TARGET_BRANCH}...HEAD --format="%H %s%n%b" --no-merges
git -C {REPO_ROOT} diff --stat origin/{TARGET_BRANCH}...HEAD
git -C {REPO_ROOT} diff origin/{TARGET_BRANCH}...HEAD -- "*.php" "*.js" "*.ts" "*.vue" | head -300
Read each changed file's diff to understand what actually changed (not just filenames). This is the basis for the title and description — derive them from the actual code changes, not just commit messages.
Step 5.5: Check for OpenSpec issue link
Check if a plan.json exists for the current change:
find {REPO_ROOT}/openspec/changes -maxdepth 2 -name "plan.json" ! -path "*/archive/*" 2>/dev/null
If found, read it and extract tracking_issue and repo. Store as {TRACKING_ISSUE} (e.g. 42) or null if not found.
This will be included in the PR description to auto-close the issue on merge.
Step 6: Draft PR Title & Description
Using the commit log, diff stat, and file-level diffs from Step 5, draft:
Title: Concise, action-verb sentence describing the main purpose (e.g. Add full-text search to registers). Do not use the branch name verbatim. Do not include app names or ticket numbers unless the commits reference them.
Description:
## Summary
{3–6 bullet points derived from the actual commits and diffs — what changed and why}
## Checks
{one of:}
- ✅ All local checks passed (`composer check:strict`)
- ⚠️ Some checks failed — see CI for details
- ⏭️ Checks skipped
## Test plan
- [ ] CI passes
- [ ] Tested locally
- [ ] Reviewed for regressions
{if TRACKING_ISSUE is set:}
Closes #{TRACKING_ISSUE}
Present the draft to the user in the chat — show both the title and the full description as they would appear on GitHub.
Then ask using AskUserQuestion:
"Does this PR title and description look good?"
- Yes, proceed — proceed to Step 7 (will create or update depending on whether
{EXISTING_PR_NUMBER} is set)
- Change something → ask: "What would you like to change or improve?" — apply the feedback, show the updated draft, and ask again
- Let me write my own title → ask for a new title, update the draft, show it, and ask again
Repeat the review loop until the user approves.
Store the approved title as {PR_TITLE} and description as {PR_BODY}.
Step 7: Create or Update the PR
Push the branch to origin if not already pushed:
git -C {REPO_ROOT} push -u origin {CURRENT_BRANCH}
Parse {OWNER} and {REPO} from {REMOTE_URL} (e.g. https://github.com/ConductionNL/myapp.git → owner=ConductionNL, repo=myapp).
Always use the GitHub REST API directly — never use gh pr create or gh pr edit (they use GraphQL and may trigger deprecation errors).
If updating an existing PR ({EXISTING_PR_NUMBER} is set):
gh api repos/{OWNER}/{REPO}/pulls/{EXISTING_PR_NUMBER} \
--method PATCH \
-f title="{PR_TITLE}" \
-f body="{PR_BODY}" \
--jq '{number: .number, title: .title, url: .html_url}'
If creating a new PR:
gh api repos/{OWNER}/{REPO}/pulls \
--method POST \
-f title="{PR_TITLE}" \
-f head="{CURRENT_BRANCH}" \
-f base="{TARGET_BRANCH}" \
-f body="{PR_BODY}" \
--jq '{number: .number, title: .title, url: .html_url}'
Store the returned PR number as {PR_NUMBER} and URL as {PR_URL}.
Step 8: Confirm & Report
After the PR is created, display:
- The PR URL
- Source → target branch
- Check status summary
- Next steps (e.g., "Request a review", "Watch CI status")
Capture Learnings
After execution, review what happened and append new observations to learnings.md under the appropriate section:
- Patterns That Work — approaches that produced good results
- Mistakes to Avoid — errors encountered and how they were resolved
- Domain Knowledge — facts discovered during this run
- Open Questions — unresolved items for future investigation
Each entry must include today's date. One insight per bullet. Skip if nothing new was learned.
💡 If you switched models to run this command, don't forget to switch back to your preferred model with /model <name> (e.g. /model default or /model sonnet).