Add bot-review mode for CI/CD non-interactive PR code review.#3300
Merged
Add bot-review mode for CI/CD non-interactive PR code review.#3300
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3300 +/- ##
==========================================
- Coverage 75.29% 75.21% -0.09%
==========================================
Files 504 509 +5
Lines 33422 35211 +1789
Branches 10673 11270 +597
==========================================
+ Hits 25166 26483 +1317
- Misses 6032 6422 +390
- Partials 2224 2306 +82 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
domchen
reviewed
Mar 8, 2026
…/libpag into feature/kevingpqi_cr_bot * 'feature/kevingpqi_cr_bot' of https://github.com/Tencent/libpag: Add NativeWindow ownership tracking to prevent double-free on HarmonyOS. (#3309) Merge spec publish into playground publish script and add skill-creator skill (#3307) Improve skills page content and optimize publish script (#3306) Add PAGX AI Skills documentation page with bilingual support (#3304) Switch libxml2 to static vcpkg build on all platforms. (#3303) Add pagx CLI npm package release workflow and playground improvements (#3301) Replace ffaudio with ffmovie for audio encoding. (#3297) Fix codeformat script for PEP 668 and apply code formatting. (#3299)
…dary clarifications.
shlzxjp
reviewed
Mar 24, 2026
Comment on lines
+152
to
+154
| After forwarding the last reviewer's report to the verifier, send a separate | ||
| message stating: "All N reviewer reports have been forwarded. Please finalize | ||
| your verdicts for all issues above." (Replace N with the actual reviewer count.) |
Collaborator
There was a problem hiding this comment.
teams-review.md中也有“Completion signal”，内容是重复的
|
|
||
| --- | ||
|
|
||
| ## Exit Codes |
Collaborator
There was a problem hiding this comment.
退出码机制是否必要，skill 是 markdown 指令而非可执行程序，需确认框架是否支持退出码。
Collaborator
Author
There was a problem hiding this comment.
这里的退出码机制是给流水线使用的，如果有一些异常情况，可以及时通知出来
|
|
||
| If diff is empty → print "No changes to review." and stop. | ||
|
|
||
| If diff exceeds 200 lines, first run `git diff --stat` for overview, then read |
Collaborator
There was a problem hiding this comment.
缺少 diff 上限保护，超大 PR 可能导致上下文爆炸
shlzxjp
reviewed
Mar 24, 2026
| @@ -0,0 +1,344 @@ | |||
| # Bot Review | |||
Collaborator
Author
There was a problem hiding this comment.
Phase 1 中 git fetch origin pull/{number}/head:pr-{number} 创建的本地分支名带了 PR 编号（pr-{number}），所以不同 PR 的分支名不会冲突， 即便是通过一个 PR 被触发多次，流水线中有排队策略，也没有问题
…cleanup clarification.
shlzxjp
approved these changes
Mar 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
为 /cr 技能添加 bot 模式，支持在 CI/CD 流水线中进行非交互式 PR 代码审查。
主要变更：