hello-stephen opened a new pull request, #66003:
URL: https://github.com/apache/doris/pull/66003
### What problem does this PR solve?
Issue Number: None
Related PR: #65813
Automated `/review` currently waits for the complete multi-agent convergence
before publishing any result. Recent review traffic shows that blocking
findings are common, while the median final-review latency is much longer than
the point at which contributors often start `run buildall`. This makes
expensive CI fan-out likely to start before actionable review feedback is
available.
This PR keeps TeamCity triggering manual and adds an explicit fast-feedback
checkpoint to the existing review runner:
- The main agent performs a time-bounded scan of the complete diff before
spawning subagents.
- A confirmed blocker is published immediately as a `REQUEST_CHANGES` review.
- If no blocker is confirmed in the initial scan, the workflow posts a
clearly provisional PR comment while the complete review continues.
- Fast and final review outputs are bound to the reviewed HEAD SHA and
verified through GitHub APIs.
- A dismissed fast review still counts as successfully delivered feedback,
but it is never interpreted as a clean result.
- The workflow reports the fast-feedback latency and warns when it misses
the 120-second target after Codex starts.
- Ripgrep installation is skipped when the hosted runner already provides it.
The complete multi-agent review and final GitHub review are still required.
This PR does not automatically trigger `run buildall`; the PR author decides
whether to start CI after reading the provisional feedback.
### Validation
- `actionlint` on `.github/workflows/code-review-runner.yml`
- `git diff --check`
- Focused `jq` fixtures for:
- blocker fast review
- dismissed fast review
- provisional no-blocker comment
- stale HEAD rejection
- final review actor/state/marker validation
- latency calculation
- Two independent read-only reviews of the GitHub Actions and review-state
semantics
A live end-to-end run is not claimed because the workflow depends on
repository-only OSS/Codex secrets. The 120-second target starts when Codex
starts; checkout and runner setup time are reported separately by Actions and
remain outside this first change.
### Release note
None
### Check List (For Author)
- Test: Manual test
- Behavior changed: Yes. Automated review publishes provisional fast
feedback before the complete review finishes.
- Does this need documentation: No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
