Skip to content

ci: @claude 审核默认改用 Opus 4.8,复杂时主 Agent 自行派发子 Agent#669

Merged
appergb merged 1 commit into
mainfrom
ci/claude-review-default-opus48
Jun 15, 2026
Merged

ci: @claude 审核默认改用 Opus 4.8,复杂时主 Agent 自行派发子 Agent#669
appergb merged 1 commit into
mainfrom
ci/claude-review-default-opus48

Conversation

@appergb

@appergb appergb commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

User description

背景

@claude 云端审核(.github/workflows/claude.yml)原来默认用 claude-sonnet-4-6,只在评论写 --opus 时升级。本次按需求把默认审核模型改为 Opus 4.8,并保留主 Agent 在复杂改动时自行派发子 Agent的能力(不强制多 Agent 架构)。

改动(仅 claude.yml,+17/-8)

  1. 默认模型 claude-sonnet-4-6claude-opus-4-8。开关反转:原 --opus 升级开关改成 --sonnet 降级开关 —— 评论写 --sonnet(或含 claude-sonnet)才降回 sonnet,作为简单任务的省额度逃生通道。
  2. 单主 Agent 为默认:--append-system-prompt 指示默认由单个主 Agent 完成审核;仅当改动较大/复杂时,主 Agent 才用 Task 工具自行派发子 Agent 协助并汇总。
  3. 放行派发能力:--allowedTools "Task,Read,Grep,Glob" —— 允许派发子 Agent 及只读审码。不放行 Bash,审核保持只读、不执行 PR 代码。

为什么 base 是 main 而不是 beta

claude.ymlissue_comment / issues / pull_request_review 等事件触发,GitHub Actions 对这类事件一律使用默认分支(main)上的 workflow 文件。所以此改动必须落到 main 才会真正改变 @claude 的行为;合并到 beta 不会生效。本 PR 为纯 CI 配置改动,不含应用代码,直接进 main 安全。

取舍

Opus 4.8 的 token 消耗高于 sonnet。但触发面未变 —— 仍是只有组织成员主动 @claude 才跑、不自动 review,总量可控;急用省钱时评论带 --sonnet 即可降级。

测试计划

  • ruby -ryaml 校验 YAML 合法,claude_args 三个 flag(--model / --allowedTools / --append-system-prompt)解析正确(append-system-prompt 为单行、双引号包裹、内部无双引号)
  • pick_model 逻辑:默认输出 claude-opus-4-8,命中 --sonnet/claude-sonnet 时输出 claude-sonnet-4-6
  • 合并后,在某 issue / PR 评论 @claude 跑一次,确认用 Opus 4.8、单主 Agent 正常出审核结论;再用 @claude --sonnet 确认能降级

PR Type

Enhancement


Description

  • Default model switched to claude-opus-4-8 with opt-down via --sonnet comment

  • Allowed Task, Read, Grep, Glob tools for complex sub-agent dispatch

  • Added cross-platform compatibility checks in system prompt


Diagram Walkthrough

flowchart LR
  A["Comment contains --sonnet?"] -->|Yes| B["Use claude-sonnet-4-6"]
  A -->|No| C["Use claude-opus-4-8"]
  C --> D["Single main agent; can dispatch sub-agents if complex"]
  D --> E["System prompt: cross-platform checks"]
Loading

File Walkthrough

Relevant files
Enhancement
claude.yml
Switch default model and enhance agent capabilities           

.github/workflows/claude.yml

  • Changed default model from sonnet-4-6 to opus-4-8 with opt-down via
    comment
  • Added allowedTools: Task, Read, Grep, Glob (no Bash) for sub-agent
    dispatch
  • Added append-system-prompt with cross-platform compatibility and
    single-agent guidance
+20/-8   

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit 4a986fe)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

- 默认模型 claude-sonnet-4-6 → claude-opus-4-8;评论写 --sonnet(或含 claude-sonnet)降级省 OAuth 额度
- claude_args 放行 Task + Read/Grep/Glob(不放行 Bash,保持只读、不执行 PR 代码)
- append-system-prompt:默认单主 Agent 审核,仅改动较大/复杂时才自行派发子 Agent 再汇总
- append-system-prompt 内置全平台适配性要求:扫描/审核时核对 macOS/Windows/Linux 三端
  #[cfg(target_os=...)] 分支是否同步(Linux 多处 allow(dead_code) 最易漏),识别仅单平台生效或破坏他端的改动
@appergb appergb force-pushed the ci/claude-review-default-opus48 branch from d003f07 to 4a986fe Compare June 15, 2026 13:40
@github-actions

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit 4a986fe

@appergb appergb merged commit f5fd598 into main Jun 15, 2026
4 checks passed
@appergb appergb deleted the ci/claude-review-default-opus48 branch June 15, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant