From faa6021c6f300b03160c74fb64e9a678479ac5e1 Mon Sep 17 00:00:00 2001 From: Jason Date: Sat, 2 May 2026 10:28:47 +0800 Subject: [PATCH] fix(ci): drop --max-turns 5 from Claude review args MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Cap was too tight — review tasks need 8-15 turns to read files, analyze, and post results. The first run after the previous prompt change failed with error_max_turns at turn 6. The disallowedTools list already keeps the agent in read-only mode, so an explicit turn cap is redundant. --- .github/workflows/claude.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/claude.yml b/.github/workflows/claude.yml index 8222eb95a..fffbbd3bb 100644 --- a/.github/workflows/claude.yml +++ b/.github/workflows/claude.yml @@ -52,7 +52,6 @@ jobs: claude_args: >- --model claude-opus-4-7 --disallowedTools Edit,Write,MultiEdit,NotebookEdit - --max-turns 5 --append-system-prompt "You are reviewing PRs for cc-switch — a small open-source Tauri 2.0 + React + TypeScript desktop app maintained by a single developer who values pragmatism over perfection. ONLY flag HIGH SIGNAL issues. An issue qualifies only if it meets at least one of: code that fails to compile/parse/run; code that will definitely produce wrong results; a security vulnerability with a concrete trigger (SSRF, injection, unsafe deserialization, credential or data leaks); database/migration correctness (schema mismatch, data loss on upgrade, broken SCHEMA_VERSION transition); cross-platform breakage (Windows/macOS/Linux specific); clear, quotable violations of CLAUDE.md. Each finding must include a confidence score from 0 to 100. Do NOT post any finding scored below 80. Each finding must carry a severity tag — Important (bug that should be fixed before merge), Nit (minor, worth fixing but not blocking), or Pre-existing (exists already, not introduced by this PR).