feat(ci): 新增AI代码审查Bot（扣子原生API） #426

Merged

xiaoxia merged 9 commits from feature/ai-code-review into develop 2026-07-17 01:24:59 +08:00

Conversation 1 Commits 9 Files Changed 2

+680

auto-approve-bot commented 2026-07-16 23:14:10 +08:00

Collaborator

Copy Link

Copy Source

变更内容

1. 新增 scripts/ci_code_review.py - AI代码审查脚本，支持扣子原生Bot API
2. 新增 .gitea/workflows/code-review.yml - 独立的Code Review workflow

功能

• PR触发时自动拉取diff，调用扣子Bot进行代码审查
• 审查结果以评论形式回写到PR
• continue-on-error，不阻断CI流水线
• 支持diff截断（默认30000字符），避免超大PR超时

前置依赖

需要在仓库Secrets中配置：

• GITEA_TOKEN - Gitea访问令牌（用于回写评论）
• LLM_API_KEY - 扣子SAT token
• COZE_BOT_ID - 扣子Bot ID

## 变更内容 1. 新增 `scripts/ci_code_review.py` - AI代码审查脚本，支持扣子原生Bot API 2. 新增 `.gitea/workflows/code-review.yml` - 独立的Code Review workflow ## 功能 - PR触发时自动拉取diff，调用扣子Bot进行代码审查 - 审查结果以评论形式回写到PR - continue-on-error，不阻断CI流水线 - 支持diff截断（默认30000字符），避免超大PR超时 ## 前置依赖 需要在仓库Secrets中配置： - `GITEA_TOKEN` - Gitea访问令牌（用于回写评论） - `LLM_API_KEY` - 扣子SAT token - `COZE_BOT_ID` - 扣子Bot ID

auto-approve-bot added 2 commits 2026-07-16 23:14:10 +08:00

feat(ci): add AI code review script 143d11fe07

feat(ci): add code review workflow 6b1b2e710e

auto-approve-bot added 1 commit 2026-07-17 00:23:01 +08:00

fix: rename GITEA_TOKEN to GITEA_API_TOKEN to avoid conflict with built-in variable e1db127123

xiaoxia added 1 commit 2026-07-17 00:29:42 +08:00

fix: rename secret to REVIEW_GITEA_TOKEN (GITEA_ prefix is reserved) 5abc6552f4

xiaoxia added 1 commit 2026-07-17 00:36:24 +08:00

fix: remove setup-python action, use system python3 (not available in runner cache) aeef4011de

xiaoxia added 1 commit 2026-07-17 00:45:00 +08:00

fix: Coze v3 API异步轮询 - GET方法+query参数，等待answer消息 ac1ae892ac

auto-approve-bot commented 2026-07-17 00:48:54 +08:00

Author

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #426

⚠️ 问题（3个需要修改）

1. scripts/ci_code_review.py 第 475-481 行：过滤逻辑未生效，代码中识别了需要跳过的文件（如 .lock, .png）并记录日志，但并未从 diff_text 中实际移除这些文件的内容。后果是 LLM 仍然会消耗 Token 审查这些二进制或非关键文件，甚至可能产生幻觉。
2. scripts/ci_code_review.py 第 157-166, 580 行：评论去重逻辑未连接。虽然定义了 get_existing_review_comments 函数用于查找旧评论，但在 main 函数中并未调用它。后果是每次 PR 更新都会发布一条新的评论，导致 PR 页面充斥着重复的审查记录，而不是更新旧评论。
3. scripts/ci_code_review.py 第 518, 533, 544, 609 行：异常处理掩盖错误。代码在多处异常或配置缺失时执行 sys.exit(0)（正常退出），配合 YAML 中的 continue-on-error: true，会导致 CI 任务永远显示绿色（成功）。后果是当配置错误或 API 调用失败时，开发者难以从 CI 状态中发现机器人已失效。建议脚本在遇到错误时应返回非 0 状态码，由 YAML 层面控制是否阻断 CI。

💡 建议（3个可选）

1. .gitea/workflows/code-review.yml 第 26 行：建议将 actions/checkout@v3 升级到 v4，以获得最新的安全补丁和功能支持。
2. scripts/ci_code_review.py 第 36 行：get_env_or_fail 函数定义后未被使用，建议删除或在 main 函数中复用该逻辑以简化代码。
3. scripts/ci_code_review.py 第 587 行：判断是否有严重问题的逻辑（"问题" in review_result）较为脆弱，如果 LLM 返回“没有发现问题”，该判断也会误判为包含问题。建议结合正则表达式或更严格的格式解析来判断。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-17 00:48:54 | 模型:}

## 代码审查结果 - PR #426 ### ⚠️ 问题（3个需要修改） 1. **scripts/ci_code_review.py 第 475-481 行**：过滤逻辑未生效，代码中识别了需要跳过的文件（如 .lock, .png）并记录日志，但并未从 `diff_text` 中实际移除这些文件的内容。后果是 LLM 仍然会消耗 Token 审查这些二进制或非关键文件，甚至可能产生幻觉。 2. **scripts/ci_code_review.py 第 157-166, 580 行**：评论去重逻辑未连接。虽然定义了 `get_existing_review_comments` 函数用于查找旧评论，但在 `main` 函数中并未调用它。后果是每次 PR 更新都会发布一条新的评论，导致 PR 页面充斥着重复的审查记录，而不是更新旧评论。 3. **scripts/ci_code_review.py 第 518, 533, 544, 609 行**：异常处理掩盖错误。代码在多处异常或配置缺失时执行 `sys.exit(0)`（正常退出），配合 YAML 中的 `continue-on-error: true`，会导致 CI 任务永远显示绿色（成功）。后果是当配置错误或 API 调用失败时，开发者难以从 CI 状态中发现机器人已失效。建议脚本在遇到错误时应返回非 0 状态码，由 YAML 层面控制是否阻断 CI。 ### 💡 建议（3个可选） 1. **.gitea/workflows/code-review.yml 第 26 行**：建议将 `actions/checkout@v3` 升级到 `v4`，以获得最新的安全补丁和功能支持。 2. **scripts/ci_code_review.py 第 36 行**：`get_env_or_fail` 函数定义后未被使用，建议删除或在 `main` 函数中复用该逻辑以简化代码。 3. **scripts/ci_code_review.py 第 587 行**：判断是否有严重问题的逻辑（`"问题" in review_result`）较为脆弱，如果 LLM 返回“没有发现问题”，该判断也会误判为包含问题。建议结合正则表达式或更严格的格式解析来判断。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-17 00:48:54 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-07-17 01:01:25 +08:00

fix(ci): 修复AI代码审查3个bug - 过滤生效/评论去重/异常退出码 ... cdb68636bf

- Bug1: 过滤逻辑实际从diff中移除跳过的文件（lock/图片等），节省token
- Bug2: 评论去重接上，每次新审查先删除旧的AI评论，避免刷屏
- Bug3: 异常情况sys.exit(1)替代sys.exit(0)，配合continue-on-error也能看到失败

xiaoxia added 1 commit 2026-07-17 01:13:20 +08:00

style: black format ci_code_review.py d83d0a5d69

xiaoxia added 1 commit 2026-07-17 01:17:46 +08:00

style: isort fix imports d3c8b5f1a2

xiaoxia merged commit b891eced23 into develop 2026-07-17 01:24:59 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#426