xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 8 Pull Requests 8 Actions 1 Packages Projects Releases 7 Wiki Activity

feat: 格式自动修复对所有PR开放，添加防循环机制 #1037

Closed
xiaoxia wants to merge 2 commits from feature/auto-fix-all-prs into main
pull from: feature/auto-fix-all-prs
merge into: xiaoxia:main
Conversation 1 Commits 2 Files Changed 1
+24 -33
xiaoxia commented 2026-07-27 15:57:53 +08:00
Owner
Copy Link
Copy Source

变更内容

修改 scripts/ci/auto_fix_formatting.py：

  1. 去掉Agent限制：所有PR只要Code Quality因格式问题失败，都自动修复并push回分支，不再区分人/Agent提交
  2. 添加防循环机制：修复commit message带 [skip ci-format-check] 标记，检测到最新commit含此标记则跳过修复
  3. 删除diagnose_only模式：不再有仅诊断不修复的模式

防循环原理

  • 格式修复commit push回分支后，会重新触发Code Quality检查
  • 检查失败时auto_fix_formatting.py会先检查最新commit message
  • 如果包含 [skip ci-format-check]，说明是修复commit触发的回跑，直接exit 0
  • 但此时Code Quality检查本身还是失败的（因为修复的是格式问题，而ruff等其他错误可能仍然存在）
  • 纯格式问题的PR：修复后重新触发的Code Quality应该会pass（因为格式已修好）

待验证

  • 人提的格式问题PR，自动修复并push回分支
  • 防循环生效，不会无限重试
## 变更内容 修改 `scripts/ci/auto_fix_formatting.py`： 1. **去掉Agent限制**：所有PR只要Code Quality因格式问题失败，都自动修复并push回分支，不再区分人/Agent提交 2. **添加防循环机制**：修复commit message带 `[skip ci-format-check]` 标记，检测到最新commit含此标记则跳过修复 3. **删除diagnose_only模式**：不再有仅诊断不修复的模式 ## 防循环原理 - 格式修复commit push回分支后，会重新触发Code Quality检查 - 检查失败时auto_fix_formatting.py会先检查最新commit message - 如果包含 `[skip ci-format-check]`，说明是修复commit触发的回跑，直接exit 0 - 但此时Code Quality检查本身还是失败的（因为修复的是格式问题，而ruff等其他错误可能仍然存在） - **纯格式问题的PR**：修复后重新触发的Code Quality应该会pass（因为格式已修好） ## 待验证 - [ ] 人提的格式问题PR，自动修复并push回分支 - [ ] 防循环生效，不会无限重试
xiaoxia added 1 commit 2026-07-27 15:57:53 +08:00
feat: 格式自动修复对所有PR开放，添加防循环机制
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 10s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m11s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 27s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Failing after 1m30s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Has been cancelled
Details
CI/CD Pipeline / Unit Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been cancelled
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been cancelled
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been cancelled
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been cancelled
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been cancelled
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
AI Code Review / AI Code Review (pull_request) Has been cancelled
Details
Preview Deploy / Deploy Preview Environment (pull_request) Has been cancelled
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Has been skipped
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 36s
Details
5dc6c367e7
auto-approve-bot added 1 commit 2026-07-27 18:03:03 +08:00
style: auto-format with black + isort + prettier [skip ci-format-check]
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 11s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 28s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 49s
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 42s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 53s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 16s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m15s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m17s
Details
AI Code Review / AI Code Review (pull_request) Successful in 6m16s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m58s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m37s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Failing after 0s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 17s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
20736c1f04
xiaoxia closed this pull request 2026-07-27 20:30:14 +08:00
gitea-actions bot commented 2026-07-27 21:02:35 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1037 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Some checks are pending
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 11s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 28s
Details
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 49s
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 42s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 53s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 16s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m15s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m17s
Details
AI Code Review / AI Code Review (pull_request) Successful in 6m16s
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m58s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m37s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Failing after 0s
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 17s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request)
Required

Pull request closed

Please reopen this pull request to perform a merge.
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1037
Delete Branch "feature/auto-fix-all-prs"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?