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

   ### What problem does this PR solve?
   
   Issue Number: N/A
   
   Related PR: #66959
   
   Problem Summary:
   
   A converged local review using GPT-6 Astra or Claude Fable 5.1 is rejected 
by the PASS receipt validator with "model is not allowed", even when the review 
meets the existing effort, commit, reviewer, and findings requirements.
   
   Extend the exact model allowlist:
   - `gpt-6-astra`: `xhigh`, `max`, or `ultra`.
   - `claude-fable-5-1` and its Claude Code `[1m]` form: `xhigh` or `max`.
   
   Existing model eligibility and receipt checks are preserved. The hosted 
runner continues to use GPT-5.6 Sol. The `apache/doris-skills` allowlist needs 
a companion update; land the receiver support first.
   
   Model references:
   - [GPT-6 Astra](https://developers.openai.com/api/docs/guides/latest-model)
   - [Fable 5.1 migration and effort 
levels](https://platform.claude.com/docs/en/models/fable-5-1/migration-guide)
   - [Claude Code model 
configuration](https://code.claude.com/docs/en/model-config)
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test
       - [ ] Regression test
       - [x] Unit Test
       - [x] Manual test (add detailed scripts or steps below)
           - `python3 .github/scripts/test_validate_review_pass_comment.py` — 
25 tests passed, including the expanded model/effort subcases.
           - Expanded tests fail against the previous validator and pass with 
the allowlist extension.
           - Negative cases cover insufficient effort, Claude `ultra`, unlisted 
model aliases, and an unlisted future Fable version.
           - Whitespace checks passed for both changed files.
       - [ ] No need to test or manual test. Explain why:
   
   - Behavior changed:
       - [ ] No.
       - [x] Yes. Qualified Astra and Fable 5.1 receipts can satisfy the 
existing code-review check.
   
   - Does this need documentation?
       - [x] No.
       - [ ] Yes.
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label
   
   


-- 
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