From 95e6b6b0631adc2ae1a7281b7097da90c7f70774 Mon Sep 17 00:00:00 2001 From: JacobPEvans <20714140+JacobPEvans@users.noreply.github.com> Date: Fri, 6 Mar 2026 06:39:41 -0500 Subject: [PATCH 1/2] fix(finalize-pr): remove pre-PR steps from workflow diagram The workflow section showed /init-worktree, implementation, and gh pr create steps before /finalize-pr, causing AI to recommend this skill at the point of committing and opening a PR. The skill only applies post-PR-creation. (claude) --- github-workflows/skills/finalize-pr/SKILL.md | 22 +++++++++++--------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/github-workflows/skills/finalize-pr/SKILL.md b/github-workflows/skills/finalize-pr/SKILL.md index 462bcc7..3541ba5 100644 --- a/github-workflows/skills/finalize-pr/SKILL.md +++ b/github-workflows/skills/finalize-pr/SKILL.md @@ -217,17 +217,19 @@ Wait for explicit user merge commands. ## Workflow +Use this skill ONLY after a PR already exists. To create a PR first, use `gh pr create`. + ```text -Single PR: -/init-worktree → [implement] → gh pr create → /finalize-pr [number] - ↓ - Phase 1: Resolve PR number - Phase 2: Resolution Loop (automatic fixes) - Phase 3: Pre-Handoff Verification - Phase 4: Update PR Metadata - Phase 5: Report ready (wait for user) - ↓ - User invokes: /squash-merge-pr or /rebase-pr +Single PR (PR already created): +/finalize-pr [number] + ↓ +Phase 1: Resolve PR number +Phase 2: Resolution Loop (automatic fixes) +Phase 3: Pre-Handoff Verification +Phase 4: Update PR Metadata +Phase 5: Report ready (wait for user) + ↓ +User invokes: /squash-merge-pr or /rebase-pr Repo-wide: /finalize-pr all From b85d17f8dc3633003416310fe71ead8de8b8bbf8 Mon Sep 17 00:00:00 2001 From: JacobPEvans <20714140+JacobPEvans@users.noreply.github.com> Date: Fri, 6 Mar 2026 07:04:02 -0500 Subject: [PATCH 2/2] docs(finalize-pr): show both single-PR invocation forms in workflow diagram Workflow diagram showed only `/finalize-pr [number]` but the skill also supports no-arg current-branch mode. Updated to show both forms explicitly. (claude) --- github-workflows/skills/finalize-pr/SKILL.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/github-workflows/skills/finalize-pr/SKILL.md b/github-workflows/skills/finalize-pr/SKILL.md index 3541ba5..83f7ffe 100644 --- a/github-workflows/skills/finalize-pr/SKILL.md +++ b/github-workflows/skills/finalize-pr/SKILL.md @@ -221,7 +221,8 @@ Use this skill ONLY after a PR already exists. To create a PR first, use `gh pr ```text Single PR (PR already created): -/finalize-pr [number] +/finalize-pr (current branch) +/finalize-pr (by number) ↓ Phase 1: Resolve PR number Phase 2: Resolution Loop (automatic fixes)