Skip to content

chore(pr-automation): include matched critical path files in review comment#1994

Merged
kaizhou-lab merged 1 commit intomainfrom
zynx/chore/critical-path-files-in-comment
Mar 31, 2026
Merged

chore(pr-automation): include matched critical path files in review comment#1994
kaizhou-lab merged 1 commit intomainfrom
zynx/chore/critical-path-files-in-comment

Conversation

@piorpua
Copy link
Copy Markdown
Contributor

@piorpua piorpua commented Mar 31, 2026

Summary

  • pr-review 的 automation-result 块新增 CRITICAL_PATH_FILES 字段,列出命中 CRITICAL_PATH_PATTERN 的具体文件
  • pr-automation 的 APPROVED + NEEDS_HUMAN_REVIEW 评论中追加 📂 命中核心路径的文件列表
  • 所有早退出路径(CI_NOT_READY / CI_FAILED)同步加上 CRITICAL_PATH_FILES: (none) 保持格式一致

Test plan

  • 对含有 docs/ 改动的 PR 运行 /pr-review --automation,确认 CRITICAL_PATH_FILES 正确列出匹配文件
  • 对不含核心路径改动的 PR 运行,确认 CRITICAL_PATH_FILES: (none)
  • 验证 pr-automation 评论中正确显示文件列表

@kaizhou-lab kaizhou-lab merged commit dadc505 into main Mar 31, 2026
8 checks passed
@kaizhou-lab kaizhou-lab deleted the zynx/chore/critical-path-files-in-comment branch March 31, 2026 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants