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

scripts/ci/auto_fix_formatting.py

@@ -1,8 +1,10 @@
 #!/usr/bin/env python3
 """CI中自动修复代码格式（Python: black + isort | Frontend: prettier），并推送回原分支。
 
-- PR事件：自动修复并push回PR源分支（Agent提交的PR自动修，人提交的仅诊断）
+- PR事件：所有PR只要Code Quality因格式问题失败，自动修复并push回源分支
 - Push事件（develop/main）：自动修复并push回原分支，保持主干格式永远正确
+- 防循环：修复commit带 [skip ci-format-check] 标记，检测到该标记则跳过修复
+- 只修格式（black/isort/prettier），ruff逻辑类错误不动
 当code quality检查因格式问题失败时触发。
 """
 
@@ -239,7 +241,7 @@ def main():
             print("无法获取PR号，跳过自动修复")
             return
 
-        # 获取PR作者信息，判断是人还是Agent提交的
+        # 获取PR信息
         pr_info_url = f"{api_url}/repos/{repo}/pulls/{pr_number}"
         req_pr = urllib.request.Request(pr_info_url, headers={"Authorization": f"token {token}"})
         with urllib.request.urlopen(req_pr) as resp:
@@ -247,17 +249,26 @@ def main():
         pr_author = pr_info.get("user", {}).get("login", "")
         print(f"PR作者: {pr_author}")
 
-        # 判断是否为Agent提交的PR
-        agent_authors = {"actions", "auto-approve-bot", "gitea-actions"}
-        is_agent_pr = pr_author in agent_authors or "bot" in pr_author.lower()
+        # 防循环检测：检查最新commit是否已经是格式修复commit
+        # 修复commit message 带 [skip ci-format-check] 标记，检测到则跳过
+        head_branch_tmp = pr_info.get("head", {}).get("ref", "")
+        skip_marker = "[skip ci-format-check]"
+        try:
+            commits_url = f"{api_url}/repos/{repo}/pulls/{pr_number}/commits?limit=3"
+            req_commits = urllib.request.Request(commits_url, headers={"Authorization": f"token {token}"})
+            with urllib.request.urlopen(req_commits) as resp_commits:
+                commits = json.loads(resp_commits.read())
+            latest_msg = commits[-1].get("commit", {}).get("message", "") if commits else ""
+            if skip_marker in latest_msg:
+                print(f"检测到最新commit包含 {skip_marker} 标记，跳过格式修复（防循环）")
+                print("本次格式检查失败是格式修复commit触发的CI回跑，属正常现象")
+                sys.exit(0)
+        except Exception as e:
+            print(f"⚠️  防循环检测失败，继续执行: {e}")
 
-        if is_agent_pr:
-            print(f"检测到Agent提交的PR（作者: {pr_author}），将自动修复并推送")
-            fix_mode = "auto_fix_and_push"
-        else:
-            print(f"检测到人提交的PR（作者: {pr_author}），仅诊断不自动修改")
-            print("（如需自动修复，请用Agent账号提交PR，或手动运行格式化脚本）")
-            fix_mode = "diagnose_only"
+        # 所有PR都自动修复格式（不再区分人/Agent）
+        print("检测到格式问题，将自动修复并推送回分支")
+        fix_mode = "auto_fix_and_push"
 
         print("=== 检测到代码格式问题，尝试自动修复 ===")
         print(f"PR #{pr_number}")
@@ -315,26 +326,6 @@ def main():
         print("没有需要提交的格式改动")
         return
 
-    # 诊断模式：只报告问题，不修改不推送
-    if fix_mode == "diagnose_only":
-        print()
-        print("=" * 50)
-        print("📋 格式问题诊断报告（人提交的PR，仅诊断不自动修复）")
-        print("=" * 50)
-        print()
-        print("以下文件存在格式问题，建议手动修复：")
-        for line in result.stdout.strip().split("\n"):
-            print(f"  {line}")
-        print()
-        print("修复方式：")
-        print("  后端(Python): 运行 black + isort")
-        print("  前端: 运行 prettier --write")
-        print("  或使用 scripts/agent-commit.sh 提交（自动格式化）")
-        print()
-        print("=" * 50)
-        # 以非0状态码退出，让CI继续报失败（因为问题没修）
-        sys.exit(1)
-
     print()
     print("变更文件:")
     for line in result.stdout.strip().split("\n"):
@@ -342,7 +333,7 @@ def main():
 
     # 提交修复
     run("git add -A")
-    run('git commit -m "style: auto-format with black + isort + prettier"')
+    run('git commit -m "style: auto-format with black + isort + prettier [skip ci-format-check]"')
 
     # 推送（head_branch已从ensure_git_repo获取）
     print(f"\nPR来源分支: {head_branch}")