fix(ci): 修复3个CI架构bug防止再次停摆 #722

Merged

xiaoxia merged 3 commits from fix/ci-architecture-bugs-3items into develop 2026-07-22 18:15:03 +08:00

Conversation 5 Commits 3 Files Changed 6

+8 -4

xiaoxia commented 2026-07-22 17:10:16 +08:00

Owner

Copy Link

Copy Source

修复三个会导致CI停摆的架构性bug:

1. pr-auto-scan定时扫描不工作: 脚本下载到/tmp但调用scripts/下的，路径不一致
2. auto_fix_formatting.py变量顺序bug: repo/token先使用后定义NameError
3. auto-fix推送不触发新CI: 用GITHUB_TOKEN推送被Gitea忽略

修复三个会导致CI停摆的架构性bug: 1. pr-auto-scan定时扫描不工作: 脚本下载到/tmp但调用scripts/下的，路径不一致 2. auto_fix_formatting.py变量顺序bug: repo/token先使用后定义NameError 3. auto-fix推送不触发新CI: 用GITHUB_TOKEN推送被Gitea忽略

xiaoxia added 2 commits 2026-07-22 17:10:16 +08:00

fix(ci): 修复pr-auto-scan脚本路径不一致导致定时扫描失败 09697f4230

fix(ci): auto_fix变量顺序+REVIEW_TOKEN推送触发新CI fde3ab3063

auto-approve-bot commented 2026-07-22 17:13:47 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #722

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

1. .gitea/workflows/pr-auto-scan.yml 第43行：脚本执行路径从 scripts/ci/pr_auto_scan.py 修改为 /tmp/pr_auto_scan.py，但 diff 中未见任何将文件复制或下载到 /tmp 目录的前置步骤。这将导致 CI 流程执行时报错 No such file or directory。除非该文件是通过容器挂载或其他未展示的步骤预置的，否则这是一个会导致 CI 失败的逻辑错误。

💡 建议（0个可选）

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能良好

---

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

## 代码审查结果 - PR #722 ### ⚠️ 问题（1个需要修改） 1. **.gitea/workflows/pr-auto-scan.yml 第43行**：脚本执行路径从 `scripts/ci/pr_auto_scan.py` 修改为 `/tmp/pr_auto_scan.py`，但 diff 中未见任何将文件复制或下载到 `/tmp` 目录的前置步骤。这将导致 CI 流程执行时报错 `No such file or directory`。除非该文件是通过容器挂载或其他未展示的步骤预置的，否则这是一个会导致 CI 失败的逻辑错误。 ### 💡 建议（0个可选） --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-22 17:13:47 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia added 1 commit 2026-07-22 17:43:49 +08:00

fix(ci): 修复CI rerun时数据库命名冲突问题 ... 34e5b18a30

在CREATE DATABASE前先执行DROP DATABASE IF EXISTS WITH (FORCE)，
防止rerun同一run_id时因数据库已存在而失败。

影响：validate_migration.sh / run_integration_tests.sh / run_validate.sh

auto-approve-bot approved these changes 2026-07-22 17:51:03 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-22 17:51:03 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

gitea-actions bot commented 2026-07-22 17:51:43 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#722
预览链接	https://pr-722.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #722 | | 预览链接 | [https://pr-722.preview.xiaoxiajianji.com](https://pr-722.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia merged commit 7fc68e797b into develop 2026-07-22 18:15:03 +08:00

gitea-actions bot commented 2026-07-22 18:15:12 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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#722