From c7e30426e6818c0f14b02c6f843ccdca2335ee5f Mon Sep 17 00:00:00 2001 From: Brendan Collins Date: Mon, 18 May 2026 07:46:47 -0700 Subject: [PATCH] rockout: skip CI wait before review-pr and follow-ups Adds an explicit note after Step 8 and an item in Step 9 telling the workflow not to poll CI before running /review-pr or before applying suggestion/nit fixes. CI surfaces failures asynchronously; the review loop runs in parallel and any CI failure becomes a separate follow-up commit on the same branch. --- .claude/commands/rockout.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.claude/commands/rockout.md b/.claude/commands/rockout.md index 8e627f7f0..9b7fde1e5 100644 --- a/.claude/commands/rockout.md +++ b/.claude/commands/rockout.md @@ -149,6 +149,12 @@ plotting conventions, GIS alert boxes, preview images, and humanizer passes. ``` 5. Capture the PR number for the next step. +**Do NOT wait for CI to finish before moving on to Step 9.** Push the PR +and proceed to the review immediately. CI runs asynchronously and the +review-pr / follow-up loop runs in parallel. If CI surfaces a failure +later, address it as a separate follow-up commit on the same branch -- +do not block the review pass on green CI. + ## Step 9 -- Run the Domain-Aware PR Review 1. Invoke the `/review-pr` command against the PR number from Step 8: @@ -161,6 +167,8 @@ plotting conventions, GIS alert boxes, preview images, and humanizer passes. - **Blockers** -- must fix before merge - **Suggestions** -- should fix, not blocking - **Nits** -- optional improvements +4. Run this step regardless of CI status. Do not poll `gh pr checks` or + wait for workflows to finish before invoking `/review-pr`. ## Step 10 -- Follow Up on Review Findings