From edec6f6df242a092545a3c7ffd2856aad4f3f2af Mon Sep 17 00:00:00 2001 From: kaitranntt Date: Tue, 13 Jan 2026 16:37:30 -0500 Subject: [PATCH] fix(ci): update allowedTools pattern to prevent token waste on retries Changed pattern from `Bash(gh pr comment:*)` to `Bash(gh pr comment *)` to match space-separated args. Updated prompt to discourage heredocs and pipes which don't match the pattern, causing 6+ retry attempts. --- .github/workflows/ai-review.yml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ai-review.yml b/.github/workflows/ai-review.yml index b724e730..f95821c1 100644 --- a/.github/workflows/ai-review.yml +++ b/.github/workflows/ai-review.yml @@ -148,15 +148,15 @@ jobs: > 🤖 Reviewed by `${{ env.REVIEW_MODEL }}` ## IMPORTANT: Posting the Review - After completing your analysis, you MUST post the review as a PR comment using: + After completing your analysis, post the review as a PR comment. Use this exact pattern: + ```bash + gh pr comment ${{ github.event.pull_request.number || github.event.issue.number || github.event.inputs.pr_number }} --body "YOUR_REVIEW_HERE" ``` - gh pr comment ${{ github.event.pull_request.number || github.event.issue.number || github.event.inputs.pr_number }} --body "" - ``` - Use a heredoc for the body to handle multi-line content properly. + For multi-line reviews, escape newlines or use simple string. Do NOT use heredocs, cat, echo, or pipes - just put the content directly in --body quotes. claude_args: | --model ${{ env.REVIEW_MODEL }} - --allowedTools "Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Read,Glob,Grep" + --allowedTools "Bash(gh pr comment *),Bash(gh pr diff *),Bash(gh pr view *),Read,Glob,Grep" continue-on-error: true - name: Add success reaction