Skip to content

fix: auto-review 只在第1轮派回修复避免循环#5

Merged
liuxiaotong merged 5 commits into
mainfrom
fix-auto-review-loop
Mar 6, 2026
Merged

fix: auto-review 只在第1轮派回修复避免循环#5
liuxiaotong merged 5 commits into
mainfrom
fix-auto-review-loop

Conversation

@liuxiaotong

Copy link
Copy Markdown
Owner

Summary

  • auto-review 只在第 1 轮审查不通过时派 ceo-assistant 修复
  • 避免"审查→派单→修复→再审查→再派单"无限循环
  • 第 4 轮熔断通知保持不变

@liuxiaotong liuxiaotong enabled auto-merge (squash) March 6, 2026 08:36

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto Review (Round 1) — Changes Requested

Review task timed out

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto Review (Round 4) — Approved

第 4 轮审查通过。核心改进:(1) 用文件传递 diff 避免 shell 变量截断特殊字符;(2) Python heredoc 构建 JSON payload 消除命令注入风险;(3) auto-review 只在第 1 轮派单修复,避免无限循环;(4) gh pr review --request-changes 实现原生审查状态;(5) HTTP 状态码检查 + CREW_API_TOKEN 校验 + 临时文件清理。所有安全问题已修复,逻辑正确。

@liuxiaotong liuxiaotong merged commit 0ce28e6 into main Mar 6, 2026
5 of 6 checks passed
@liuxiaotong liuxiaotong deleted the fix-auto-review-loop branch March 6, 2026 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant