* 🐛 Pass the base branch and the project when opening a PR The create-pr skill resolves the target branch with scripts/detect-target-branch in step 1 and then never passes it, so gh opens the PR against the repository default. A branch cut from staging therefore targets develop. mem:workflow/creating-prs also requires --project "Main", which the skill's command omitted. Two checks were missing as well. A branch behind its base produces a verdict about a tree nobody will ship, because checks run against head merged into base. A draft PR runs no checks at all, since every test workflow is gated on the pull_request.draft flag, so the skill now asks before opening one and states that cost. Finally, the AI-assisted-by trailer belongs in the PR body: Penpot squash-merges, so the landed commit message comes from the PR title and body, and a trailer that lives only on branch commits disappears at merge. AI-assisted-by: claude-opus-5 Signed-off-by: Álvaro Tejero Cantero <alvorithm@teje.ro> * ♻️ Drop the behind-the-base check and fix the trailer note Review found both claims wrong. The behind-the-base check said the checks would report on a tree nobody ships, but every test workflow runs actions/checkout with no ref, so a pull_request event builds refs/pull/N/merge, which is head already merged into base. Being behind the base therefore changes nothing about the verdict, and the check only forced a rebase for no gain. The trailer note claimed a trailer on branch commits disappears at merge. It does not: GitHub's squash body lists the commit messages, and 91b433985a carries three AI-assisted-by lines from the two commits of #11677. The paragraph now asks for the trailer in the body as a courtesy to the reviewer, once per model, and says the commits keep theirs. AI-assisted-by: claude-opus-5 Signed-off-by: Álvaro Tejero Cantero <alvorithm@teje.ro> --------- Signed-off-by: Álvaro Tejero Cantero <alvorithm@teje.ro>
5.1 KiB
name, description
| name | description |
|---|---|
| create-pr | PR flow — open a new PR for the current task branch (validates base branch, commits, issue and push state) or update an existing PR's title or description to match Penpot conventions. Use it when the user asks to open or create a PR, in any phrasing. |
Create PR
Two modes. Open mode takes the current task branch to a new, validated PR. Update mode rewrites an existing PR's title or description. Gather information, validate, and act in one pass. If validation fails, STOP with a single coherent message that lists every problem and states exactly what information is missing — never fix or work around problems silently.
Both modes require an authenticated gh CLI (gh auth status) and never
push — the user pushes from their own shell.
When to use
- The user asks to open or create a NEW PR for the current task branch, in
any phrasing ("open a PR", "create the pull request", "put this up for
review") — or runs
/create-pr. → Open mode. - The user asks to fix or update an EXISTING PR's title or description to match conventions. → Update mode.
If the running agent cannot write (for example, the plan agent), say so and stop — this skill needs the build agent.
Open mode
1. Gather context (read-only)
- Current branch:
git rev-parse --abbrev-ref HEAD. - Target base branch: run
./scripts/detect-target-branchfrom the repo root. It prints the nearest ancestor branch of HEAD (exit 0) or fails (exit 1). - Commits:
git log --oneline <base>..HEAD. - Push state (local):
git rev-parse --verify origin/<branch>and compare with HEAD. It reads the local remote-tracking ref — no network, no SSH. It reflects the last push or fetch this clone knows about. - Issue: from the session context, or from the branch name —
issue-NNNNmaps to issue NNNN; recover its title and body withgh issue view NNNN.
2. Validate — stop with one message if anything fails
Run all checks before reporting, then report every failure together:
- Base branch not usable. If the script fails (exit 1), or its output —
after stripping an optional
remotes/origin/prefix — is not one of the canonical branches (develop,staging,main), stop and ask the user to re-run with more context — for example, passing the base branch explicitly in their invocation. An explicit base given by the user overrides the script's output. - On a base branch. There is no task branch to merge — say so and stop.
- No commits. The branch has no commits ahead of the base — say so and stop.
- No clear issue. There is no issue in the session context, and the
branch name has no
issue-NNNNpattern (orgh issue viewfinds nothing) — say so and stop. Exception: the user's invocation saysno issue/without issue— then continue without an issue reference. - Branch not pushed. The remote-tracking ref
origin/<branch>is missing, orgit rev-parse origin/<branch>differs from HEAD — the branch was never pushed, or has commits the remote does not have. Never push yourself; ask the user to push and to run/create-pragain afterwards, then stop.
3. Already-open PR
Check whether a PR already exists for this branch (gh pr list --head <branch>). If one exists, report its URL and stop — do not create a second
one. Title or description fixes belong to Update mode.
4. Write and create the PR
Write the title and body following mem:workflow/creating-prs (title format,
description structure, writing principles) and mem:workflow/creating-commits
(commit type emojis). Derive the title and body from the commits and, when
there is one, from the issue body. Reference the issue with Closes #NNNN.
Repeat the AI-assisted-by: trailer in the body, once per model that worked
on the branch, so the PR states the assistance where a reviewer reads it. The
branch commits keep their own trailers, and a squash merge carries every one
of them into the landed message.
Before offering or accepting a draft PR, warn that CI doesn't run on them. Add
--draft only when the user agrees to that.
gh pr create --repo penpot/penpot --base "<BASE>" --title "<TITLE>" \
--project "Main" --body-file /tmp/pr-body.md
--base is the branch resolved in step 1: without it the PR opens against the
repository default, which is wrong for a branch cut from staging. --project "Main" is required by mem:workflow/creating-prs.
5. Report
Report the PR URL and stop.
Update mode
- Identify the PR: the number given by the user, or
gh pr list --head <branch>. - Write the new title and/or body following
mem:workflow/creating-prs. - Apply and verify:
gh pr edit <NUMBER> --repo penpot/penpot --title "<TITLE>" --body-file /tmp/pr-body.md
gh pr view <NUMBER> --repo penpot/penpot --json title,body
- Report and stop.
User context
Extra context in the user's invocation (the message that triggered this skill)
plays the role command arguments play elsewhere: overrides such as no issue /
without issue, an explicit base branch (from origin/staging), a PR number
for Update mode, and so on.