From 1ec38567908fab86d1cccd3e476c5a3010435c71 Mon Sep 17 00:00:00 2001 From: matthieugouel Date: Thu, 18 Jun 2026 21:25:01 +0200 Subject: [PATCH] ci: make Claude Review advisory (never block PRs) The review step exited non-zero when claude-code-action hit --max-turns, turning the PR check red even though the review ran and posted findings. Add continue-on-error so the review is advisory: it posts what it finds but never blocks a merge. Also raise --max-turns 5 -> 20 so the review has room to finish its analysis (with track_progress the 5-turn budget was exhausted before completing); timeout-minutes: 15 remains the wall-clock guard. Co-Authored-By: Claude Opus 4.8 (1M context) --- .github/workflows/claude-review.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index b8fbf10..ba499d0 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -31,6 +31,11 @@ jobs: - name: Run Claude Review uses: anthropics/claude-code-action@v1 + # The review is advisory: it posts findings but must never block a PR. + # claude-code-action exits non-zero when it hits --max-turns, so without + # this the job goes red even though the review ran. Keep CI green and let + # the posted review speak for itself. + continue-on-error: true with: claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} github_token: ${{ secrets.CLAUDE_GITHUB_PAT }} @@ -41,4 +46,4 @@ jobs: conventions. Report important findings only; skip nits. Use inline comments on the relevant lines where it helps. claude_args: | - --max-turns 5 + --max-turns 20