tiennm99/ccs
1
0
Fork 0
mirror of https://github.com/tiennm99/ccs.git synced 2026-09-03 06:18:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

hotfix(ci): increase max-turns to 30 and enforce Write tool for review output

Browse Source 
- Increase --max-turns from 20 to 30 (large PRs with 31 files need ~25 turns)
- Explicitly instruct Claude to use Write tool (not Edit/python3 fallback)
- Add READ-ONLY constraint to prevent source code modifications
- Previous run hit error_max_turns at turn 21 despite successfully writing review

Part of #818
This commit is contained in:
Tam Nhu Tran
2026-03-27 14:55:22 -04:00
parent b58c25c10d
commit 179fa338b8
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/ai-review.yml
+5 -3
View File
@@ -242,14 +242,16 @@ jobs:
${{ steps.review-prompt.outputs.content }} ${{ steps.review-prompt.outputs.content }}
## IMPORTANT: Writing the Review ## IMPORTANT: Writing the Review
After completing your analysis, write the final review markdown to `${{ env.REVIEW_OUTPUT_FILE }}`. After completing your analysis, use the `Write` tool to write the final review markdown to `${{ env.REVIEW_OUTPUT_FILE }}`.
Do NOT use `Edit` tool — use `Write` tool directly to create the file in one shot.
Do NOT post any GitHub comments yourself. The workflow will publish the saved file. Do NOT post any GitHub comments yourself. The workflow will publish the saved file.
Do NOT modify any source code files — this is a READ-ONLY review.
End your review with: End your review with:
> 🤖 Reviewed by `${{ env.REVIEW_MODEL }}` > 🤖 Reviewed by `${{ env.REVIEW_MODEL }}`
IMPORTANT RULES: IMPORTANT RULES:
- Overwrite `${{ env.REVIEW_OUTPUT_FILE }}` - Use `Write` tool to overwrite `${{ env.REVIEW_OUTPUT_FILE }}` with the complete review
- Do NOT use shell operators like || or && in bash commands - Do NOT use shell operators like || or && in bash commands
- Do NOT use heredoc (<<) syntax in bash commands - Do NOT use heredoc (<<) syntax in bash commands
- Use simple, single-purpose bash commands only - Use simple, single-purpose bash commands only
@@ -258,7 +260,7 @@ jobs:
--bare --bare
--model ${{ env.REVIEW_MODEL }} --model ${{ env.REVIEW_MODEL }}
--permission-mode bypassPermissions --permission-mode bypassPermissions
--max-turns 20 --max-turns 30
--allowedTools "Glob,Grep,Read,Write,Bash(gh pr diff *),Bash(gh pr view *),Bash(git diff *),Bash(git log *),Bash(git show *),Bash(cat *),Bash(ls *),Bash(wc *),Bash(head *),Bash(tail *),Bash(find *)" --allowedTools "Glob,Grep,Read,Write,Bash(gh pr diff *),Bash(gh pr view *),Bash(git diff *),Bash(git log *),Bash(git show *),Bash(cat *),Bash(ls *),Bash(wc *),Bash(head *),Bash(tail *),Bash(find *)"
# Fallback: if Claude didn't write the review file, extract from execution output # Fallback: if Claude didn't write the review file, extract from execution output