fix(ci): 格式修复防循环索引 + AI审查fail-open（2个bug修复） #1045

Merged

xiaoxia merged 4 commits from fix/auto-fix-loop-detection into main 2026-07-28 09:52:23 +08:00

Conversation 3 Commits 4 Files Changed 2

+3 -3

xiaoxia commented 2026-07-28 08:20:25 +08:00

Owner

Copy Link

Copy Source

Bug修复合集

Bug 1: 防循环检测取错commit

Gitea PR commits API 返回倒序（最新在前），commits[-1] 取到最早的 commit，防循环检测失效。
修复：commits[-1] → commits[0]

Bug 2: AI Code Review fail-open 未生效

代码注释写了fail-open原则，但LLM调用失败和未捕获异常时都是 exit 1，会阻塞PR合并。
修复：两处 exit 1 → exit 0

• LLM调用超时/失败：不阻塞
• 脚本异常崩溃：不阻塞

fail-open原则：AI审查是辅助手段，自身故障不能阻塞正常开发。

## Bug修复合集 ### Bug 1: 防循环检测取错commit Gitea PR commits API 返回倒序（最新在前），`commits[-1]` 取到最早的 commit，防循环检测失效。 修复：`commits[-1]` → `commits[0]` ### Bug 2: AI Code Review fail-open 未生效 代码注释写了fail-open原则，但LLM调用失败和未捕获异常时都是 `exit 1`，会阻塞PR合并。 修复：两处 `exit 1` → `exit 0` - LLM调用超时/失败：不阻塞 - 脚本异常崩溃：不阻塞 fail-open原则：AI审查是辅助手段，自身故障不能阻塞正常开发。

xiaoxia added 1 commit 2026-07-28 08:20:25 +08:00

fix(ci): 防循环检测取最新commit - API返回倒序用[0]而非[-1] ... 2c5281e6d9

Gitea PR commits API返回倒序（最新在前），commits[-1]取到最早的commit，
导致防循环检测失效。修复为commits[0]取最新commit。

auto-approve-bot commented 2026-07-28 08:24:25 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：否
• 阻塞级问题数量：0 个

📊 审查概览

• 整体评价：通过
• 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

无

✅ 良好实践

1. [scripts/ci/auto_fix_formatting.py: 261] 修正了获取最新提交信息的逻辑。GitHub API 返回的 commits 列表默认按时间倒序排列（最新的在前），使用 commits[0] 能够正确获取到最新的提交信息，从而准确判断是否需要跳过格式修复，有效防止了 CI 循环触发问题。

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **[scripts/ci/auto_fix_formatting.py: 261]** 修正了获取最新提交信息的逻辑。GitHub API 返回的 commits 列表默认按时间倒序排列（最新的在前），使用 `commits[0]` 能够正确获取到最新的提交信息，从而准确判断是否需要跳过格式修复，有效防止了 CI 循环触发问题。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-28 00:24:25 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-07-28 08:58:34 +08:00

fix(ci): AI Code Review fail-open - LLM失败/异常时不阻塞合并 ... 09c878621d

修复两处违反fail-open原则的bug：
1. LLM调用失败返回None时 exit 1 → exit 0
2. 未捕获异常时 exit 1 → exit 0

fail-open原则：AI审查是辅助手段，不能因自身故障阻塞正常开发流程。

xiaoxia added 1 commit 2026-07-28 08:59:09 +08:00

fix(ci): AI Code Review fail-open - LLM失败/异常时不阻塞合并 ... 424c736275

修复两处违反fail-open原则的bug：
1. LLM调用失败返回None时 exit 1 → exit 0
2. 未捕获异常时 exit 1 → exit 0

fail-open原则：AI审查是辅助手段，不能因自身故障阻塞正常开发。

xiaoxia added 1 commit 2026-07-28 08:59:42 +08:00

fix(ci): AI Code Review fail-open - LLM失败/异常时不阻塞合并 ... 71d062ba63

修复两处违反fail-open原则的bug：
1. LLM调用失败返回None时 exit 1 → exit 0
2. 未捕获异常时 exit 1 → exit 0

fail-open原则：AI审查是辅助手段，不能因自身故障阻塞正常开发。

xiaoxia changed title from fix(ci): 防循环检测取最新commit - API返回倒序用[0]而非[-1] to fix(ci): 格式修复防循环索引 + AI审查fail-open（2个bug修复） 2026-07-28 09:00:46 +08:00

xiaoxia reviewed 2026-07-28 09:50:35 +08:00

xiaoxia left a comment

Author

Owner

Copy Link

Copy Source

LGTM - bug fixes verified

LGTM - bug fixes verified

xiaoxia merged commit 02e3246f5a into main 2026-07-28 09:52:23 +08:00

gitea-actions bot commented 2026-07-28 09:52:45 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1045 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1045 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

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

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1045