zclllyybb opened a new pull request, #65626:
URL: https://github.com/apache/doris/pull/65626

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: #65178
   
   The review runner previously obtained the aggregate pull-request diff 
through the GitHub API. GitHub rejects aggregate diffs larger than 20,000 
lines, causing large pull requests to fail before Codex could review them. The 
runner now checks out complete history and locally generates the changed-file 
list and the verified three-dot diff from the live base and head SHAs. The 
review prompt also states those diff semantics, and context-preparation 
failures are reported separately from Codex failures.
   
   Each run now randomly selects `oss://doris-community-ci/codex/auth.json.1` 
or `oss://doris-community-ci/codex/auth.json.2`, downloads it locally as 
`auth.json`, and syncs refreshed credentials back to that same selected object.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: `git diff --check upstream/master...HEAD`
   - Test: Fork workflow run on a recreated 280-file PR (+26,082/-452), 
completed review successfully.
   - Test: Fork workflow run on a small PR, completed review successfully; 
selected `auth.json.1` and uploaded refreshed credentials back to `auth.json.1`.
   - Behavior changed: Yes
   - 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]

Reply via email to