Skip to content

docs(pst): rule 5 -- PR creation always includes CI watch+merge, no asking#68

Merged
pstaylor-patrick merged 1 commit into
mainfrom
feat/pst-rule5-auto-ci-watch
Jun 21, 2026
Merged

docs(pst): rule 5 -- PR creation always includes CI watch+merge, no asking#68
pstaylor-patrick merged 1 commit into
mainfrom
feat/pst-rule5-auto-ci-watch

Conversation

@pstaylor-patrick

Copy link
Copy Markdown
Owner

Summary

Fixes the failure mode shown in screenshot: after creating a PR and CI starts, Claude asked "Want me to watch CI and merge when ready?" instead of just doing it.

  • In admin-bypass mode, PR creation + CI watch + merge is one atomic operation (rule 13: run to completion)
  • Immediately queue gh pr merge --auto --squash after PR creation once review is recorded -- GitHub auto-merges when CI goes green
  • Never ask whether to watch CI; that answer is always yes
  • Fallback: gh pr checks <PR#> --watch then merge, if --auto is disabled on the repo

Test plan

  • Docs-only, SKILL.md 141 lines (under 200 budget)

🤖 Generated with Claude Code

https://claude.ai/code/session_019BCr5Qxi8jXnG2Rr7zdkw1

…sking

The "Want me to watch CI?" question is the failure mode. In admin-bypass
mode, PR creation and CI watch-then-merge are one atomic operation. Queue
gh pr merge --auto --squash immediately after PR creation (once adversarial
review is recorded); never split this into a question. Fallback to
--watch loop if --auto is unavailable on the repo.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019BCr5Qxi8jXnG2Rr7zdkw1
@pstaylor-patrick pstaylor-patrick merged commit 07b859f into main Jun 21, 2026
1 check passed
@pstaylor-patrick pstaylor-patrick deleted the feat/pst-rule5-auto-ci-watch branch June 21, 2026 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant