fix(ci): auto-fix checkout加--force，避免tarball untracked文件冲突 #643

Merged

xiaoxia merged 1 commits from fix/ci-autofix-checkout-force into develop 2026-07-20 14:28:42 +08:00

Conversation 2 Commits 1 Files Changed 1

+1 -1

xiaoxia commented 2026-07-20 14:16:59 +08:00

Owner

Copy Link

Copy Source

问题

Validate使用tarball方式checkout代码，auto-fix脚本执行git checkout -B切换到源分支时，tarball留下的untracked文件会导致checkout失败。

修复

git checkout -B 加 --force 参数，强制覆盖untracked文件。

关联

• 替换#638（与#636冲突，#636已合并push重试+rebase核心功能）
• 属于Validate black格式不一致问题（失败率45%）的补充修复

## 问题 Validate使用tarball方式checkout代码，auto-fix脚本执行`git checkout -B`切换到源分支时，tarball留下的untracked文件会导致checkout失败。 ## 修复 `git checkout -B` 加 `--force` 参数，强制覆盖untracked文件。 ## 关联 - 替换#638（与#636冲突，#636已合并push重试+rebase核心功能） - 属于Validate black格式不一致问题（失败率45%）的补充修复

xiaoxia added 1 commit 2026-07-20 14:16:59 +08:00

fix(ci): auto-fix checkout加--force，避免tarball untracked文件冲突 7593c6cde3

auto-approve-bot commented 2026-07-20 14:18:58 +08:00

Collaborator

Copy Link

Copy Source

📊 审查概览

• 整体评价：需修改
• 严重问题数量：1 个
• 一般建议数量：0 个

❌ 需修改的问题（严重）

1. [scripts/ci/auto_fix_formatting.py: 65] Git 命令参数顺序错误
   • 问题描述：git checkout -B 命令的语法要求紧跟其后的参数是新分支的名称。当前代码将 --force 放在 -B 之后，Git 会将 --force 解析为分支名，而将 {head_branch} 解析为起始点（start_point），导致命令执行失败或产生错误的分支。
   • 修改建议：将 --force（或 -f）选项移到 -B 之前。修改为：

     run(f"git checkout --force -B {head_branch} FETCH_HEAD")
     

     或

     run(f"git checkout -f -B {head_branch} FETCH_HEAD")
     

💡 改进建议（一般）

无

✅ 良好实践

• 意识到了在 CI 环境中切换分支可能遇到冲突或脏状态，试图使用 force 标志来增强脚本的健壮性，方向是正确的。

---

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

### 📊 审查概览 - 整体评价：需修改 - 严重问题数量：1 个 - 一般建议数量：0 个 ### ❌ 需修改的问题（严重） 1. **[scripts/ci/auto_fix_formatting.py: 65] Git 命令参数顺序错误** - 问题描述：`git checkout -B` 命令的语法要求紧跟其后的参数是新分支的名称。当前代码将 `--force` 放在 `-B` 之后，Git 会将 `--force` 解析为分支名，而将 `{head_branch}` 解析为起始点（start_point），导致命令执行失败或产生错误的分支。 - 修改建议：将 `--force`（或 `-f`）选项移到 `-B` 之前。修改为： ```python run(f"git checkout --force -B {head_branch} FETCH_HEAD") ``` 或 ```python run(f"git checkout -f -B {head_branch} FETCH_HEAD") ``` ### 💡 改进建议（一般） 无 ### ✅ 良好实践 - 意识到了在 CI 环境中切换分支可能遇到冲突或脏状态，试图使用 force 标志来增强脚本的健壮性，方向是正确的。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-20 14:18:58 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 30f125bc78 into develop 2026-07-20 14:28:42 +08:00

gitea-actions bot commented 2026-07-20 14:28:52 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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