fix(ci): handle "No issues found" comment + restore version-check [skip-regression-check]

1. Handle Claude's "No issues found" comment as a clean pass instead
   of treating empty BODY as blocking. Checks for this before looking
   for "Found N issues" comments.

2. Restore version-check to run on all PRs (not just staging-promotion).
   It's 4-8 seconds and catches missing version bumps early.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
This commit is contained in:
Henry Park
2026-03-09 15:33:15 -07:00
co-authored by Claude Sonnet 4.6
parent b5b9934df0
commit fb8ac7455e
2 changed files with 10 additions and 3 deletions
+9
View File
@@ -298,6 +298,15 @@ jobs:
exit 0
fi
# Check for "No issues found" first (clean pass)
NO_ISSUES=$(gh api "repos/${REPO}/issues/${PR_NUMBER}/comments" \
--jq '[.[] | select(.user.login == "claude[bot]") | select(.body | test("No issues found"))] | length' 2>/dev/null || echo "0")
if [ "$NO_ISSUES" -gt 0 ]; then
echo "Claude review found no issues — gate passes"
echo "has_blocking=false" >> "$GITHUB_OUTPUT"
exit 0
fi
# Get the last Claude comment that contains findings
JQ_FILTER='[.[] | select(.user.login == "claude[bot]") | select(.body | test("Found [0-9]+ issue"))] | last'
BODY=$(gh api "repos/${REPO}/issues/${PR_NUMBER}/comments" \
+1 -3
View File
@@ -117,9 +117,7 @@ jobs:
version-check:
name: Version Bump Check
runs-on: ubuntu-latest
if: >
github.event_name == 'pull_request' &&
contains(github.event.pull_request.labels.*.name, 'staging-promotion')
if: github.event_name == 'pull_request'
steps:
- name: Checkout repository
uses: actions/checkout@v6