Ship PR
This is the orchestration layer for an explicitly requested full lifecycle through
merge. Individual push, PR, review, or merge requests belong to
../push-pr/SKILL.md. Full-cycle authorization permits merge, but evidence still
gates it. If the repository has no quality gate, report that gap instead of
inventing CI or claiming a clean full cycle.
Loop
- Inspect authored scope and the requested/default base. Create a branch and
scoped conventional commits without unrelated changes. Run the repository's
gates on the committed tree and
git diff --check <base>..HEAD. A red local
gate blocks push.
- Load
../push-pr/SKILL.md and use its evidence, template, PR creation/update,
and metadata procedure. Do not duplicate endpoint mechanics here.
- Watch required CI to a final state. Read review findings, not just review-bot
status. For each feedback cycle, use
../push-pr/references/review-threads.md: verify findings against source, fix
and test valid issues, commit/push, reply in-thread, then resolve appropriately.
Rebut invalid findings with evidence in-thread; leave them open for reviewer
confirmation when needed. A reply alone never resolves a thread.
- For conflicts, integrate the current base and rerun gates before pushing.
Repeat verification after every fix; stop and report findings requiring a
human decision.
- Merge only with
mergeStateStatus: CLEAN, green local gates and required CI,
and every review thread resolved. Use a repository-allowed merge method;
never bypass protections. Auto-merge is not implied by repository capability.
- Delete the merged task branch and sync the local base without discarding user
work. If local changes prevent safe cleanup, report the remaining cleanup
rather than reset them.
Verify completion with gh pr view <n> --json state,mergedAt,mergeCommit,statusCheckRollup
(MERGED, a merge SHA, and every rollup check passing) plus paginated GraphQL
reviewThreads.isResolved all true. Report the PR link, merge SHA, and any
cleanup limitation. Commit/merge conventions belong to
../git-workflow-and-versioning/SKILL.md.
1---2name: ship-pr3description: Use when the user wants a change shipped autonomously end to end: commit, push, open the PR, watch CI, triage and address bot/human review comments in-thread, resolve threads, and merge when everything is green ('ship it', 'run the full PR cycle', 'commit, push, PR, and merge if clean'). Runs to merge without per-step asks. NOT for stepwise asks: pushing only, opening a PR only, addressing review feedback only, or merging an existing PR only (push-pr owns those).4---56# Ship PR78This is the orchestration layer for an explicitly requested full lifecycle through9merge. Individual push, PR, review, or merge requests belong to10`../push-pr/SKILL.md`. Full-cycle authorization permits merge, but evidence still11gates it. If the repository has no quality gate, report that gap instead of12inventing CI or claiming a clean full cycle.1314## Loop15161. Inspect authored scope and the requested/default base. Create a branch and17 scoped conventional commits without unrelated changes. Run the repository's18 gates on the committed tree and `git diff --check <base>..HEAD`. A red local19 gate blocks push.202. Load `../push-pr/SKILL.md` and use its evidence, template, PR creation/update,21 and metadata procedure. Do not duplicate endpoint mechanics here.223. Watch required CI to a final state. Read review findings, not just review-bot23 status. For each feedback cycle, use24 `../push-pr/references/review-threads.md`: verify findings against source, fix25 and test valid issues, commit/push, reply in-thread, then resolve appropriately.26 Rebut invalid findings with evidence in-thread; leave them open for reviewer27 confirmation when needed. A reply alone never resolves a thread.284. For conflicts, integrate the current base and rerun gates before pushing.29 Repeat verification after every fix; stop and report findings requiring a30 human decision.315. Merge only with `mergeStateStatus: CLEAN`, green local gates and required CI,32 and every review thread resolved. Use a repository-allowed merge method;33 never bypass protections. Auto-merge is not implied by repository capability.346. Delete the merged task branch and sync the local base without discarding user35 work. If local changes prevent safe cleanup, report the remaining cleanup36 rather than reset them.3738Verify completion with `gh pr view <n> --json state,mergedAt,mergeCommit,statusCheckRollup`39(`MERGED`, a merge SHA, and every rollup check passing) plus paginated GraphQL40`reviewThreads.isResolved` all true. Report the PR link, merge SHA, and any41cleanup limitation. Commit/merge conventions belong to42`../git-workflow-and-versioning/SKILL.md`.
ryan-brosas