Roll out reliable AI review evidence in OAuth - #210
Merged
Conversation
This comment has been minimized.
This comment has been minimized.
Contributor
|
Reviewed; no blockers found. |
kriszyp's review on harper#2256: the validator was left on 224c2ad while the claude/gemini callers moved to 4b59dc0 — the #86 delta changes the workflow contract the validator checks callers against, so the pins move together. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01T4VjVDt6egBUxmGiASuahs
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
1 participant
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.
Roll out the reliable review-evidence plumbing from HarperFast/ai-review-prompts#86 in OAuth by moving the Claude, Gemini, and caller-validator workflows to its verified merge commit. This keeps the review and validation paths on one immutable workflow, prompt, and script version.
For the human reviewer
No open judgment calls. The three caller pins move together to avoid validating the new review callers against a different upstream revision. Mention, issue-to-PR, permissions, triggers, secrets, model selection, layers, and OAuth-specific checks are unchanged.
Verification
git diff --checkpassed.4b59dc0ddb15aff517884127b58204f79193b4f2as the signed, verified merge commit for Make AI review calibration evidence reliable ai-review-prompts#86.d58736f…head and reported no blockers; all ordinary CI passed.Review coverage
Authored by Codex, with the validator alignment follow-up authored by Claude Fable 5 from Kris Zyp's canary feedback. Independent Claude review covered the initial pin bump; Gemini reviewed the final head with no findings. Cursor and further domain adjudication were pruned by the minimal low-risk policy. Receipt @ d58736f.
Human-Review-Need: 2 @ d58736f