sync(ci): 同步缺失的CI监控脚本到main分支 #941

Merged

xiaoxia merged 1 commits from sync/ci-scripts-to-main-20260726 into main 2026-07-26 18:13:27 +08:00

Conversation 2 Commits 1 Files Changed 2

+926

xiaoxia commented 2026-07-26 16:51:51 +08:00

Owner

Copy Link

Copy Source

问题

main分支缺少2个CI脚本文件，导致schedule类workflow假成功：

脚本	被哪个workflow引用	假成功原因
scripts/ci_trigger_monitor.py	ci-trigger-monitor.yml	set +e + exit 0，文件不存在也报success
scripts/ci_code_review.py	code-review.yml	continue-on-error，文件不存在也不影响

修复

从develop同步这2个脚本文件到main。

注意：workflow YAML已经在PR #933中同步过了，本PR只同步缺失的脚本依赖。

## 问题 main分支缺少2个CI脚本文件，导致schedule类workflow假成功： | 脚本 | 被哪个workflow引用 | 假成功原因 | |------|------------------|-----------| | scripts/ci_trigger_monitor.py | ci-trigger-monitor.yml | set +e + exit 0，文件不存在也报success | | scripts/ci_code_review.py | code-review.yml | continue-on-error，文件不存在也不影响 | ## 修复 从develop同步这2个脚本文件到main。 注意：workflow YAML已经在PR #933中同步过了，本PR只同步缺失的脚本依赖。

xiaoxia added 1 commit 2026-07-26 16:51:51 +08:00

sync(ci): 同步CI监控脚本到main分支 ... 17dd9a54d4

同步2个缺失的CI脚本文件：
- scripts/ci_trigger_monitor.py (ci-trigger-monitor.yml依赖)
- scripts/ci_code_review.py (code-review.yml依赖)

main分支缺少这两个脚本导致对应workflow假成功

auto-approve-bot commented 2026-07-26 16:58:34 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #941

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

1. scripts/ci_trigger_monitor.py 第184行：将 from datetime import datetime 放在了 for 循环内部。虽然 Python 模块有缓存机制，但这是严重的代码规范错误，且在循环体中重复执行 import 语句（即使开销很小）属于逻辑冗余，应移至文件顶部。
2. scripts/ci_code_review.py 第411行：使用 line.split(" ") 解析 Git diff 头部。如果文件路径中包含空格（虽然少见但合法），解析结果 parts 的长度和索引会错位，导致 current_file 提取错误，进而影响文件过滤逻辑。建议使用 line.split(" ", 3) 限制分割次数或使用正则表达式提取。
3. scripts/ci_trigger_monitor.py 第39行：except Exception: 捕获了所有异常（包括 json.JSONDecodeError）。如果 API 返回了非 JSON 格式的数据（如 5xx 错误页 HTML），重试 3 次不仅徒劳，还会延长故障发现时间。应区分网络超时/连接错误（可重试）与数据解析错误（应立即失败）。

💡 建议（1个可选）

1. scripts/ci_trigger_monitor.py 第136行：get_env("GITEA_URL", ...) 的返回值未被赋值给任何变量，属于无效代码（Dead Code），建议删除以保持整洁。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-26 08:58:34 | 模型:}

## 代码审查结果 - PR #941 ### ⚠️ 问题（3个需要修改） 1. **scripts/ci_trigger_monitor.py 第184行**：将 `from datetime import datetime` 放在了 `for` 循环内部。虽然 Python 模块有缓存机制，但这是严重的代码规范错误，且在循环体中重复执行 import 语句（即使开销很小）属于逻辑冗余，应移至文件顶部。 2. **scripts/ci_code_review.py 第411行**：使用 `line.split(" ")` 解析 Git diff 头部。如果文件路径中包含空格（虽然少见但合法），解析结果 `parts` 的长度和索引会错位，导致 `current_file` 提取错误，进而影响文件过滤逻辑。建议使用 `line.split(" ", 3)` 限制分割次数或使用正则表达式提取。 3. **scripts/ci_trigger_monitor.py 第39行**：`except Exception:` 捕获了所有异常（包括 `json.JSONDecodeError`）。如果 API 返回了非 JSON 格式的数据（如 5xx 错误页 HTML），重试 3 次不仅徒劳，还会延长故障发现时间。应区分网络超时/连接错误（可重试）与数据解析错误（应立即失败）。 ### 💡 建议（1个可选） 1. **scripts/ci_trigger_monitor.py 第136行**：`get_env("GITEA_URL", ...)` 的返回值未被赋值给任何变量，属于无效代码（Dead Code），建议删除以保持整洁。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-26 08:58:34 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 5ae6c33bf6 into main 2026-07-26 18:13:27 +08:00

xiaoxia referenced this pull request 2026-07-26 18:39:01 +08:00

[CI] 铁证级核查 - 全面修复主工单 #980

gitea-actions bot commented 2026-07-26 18:40:21 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #941 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#941