Skip to content

test(eve): clarify stale-ask-question-selection judge criterion (both variants)#923

Merged
prha merged 2 commits into
mainfrom
prha/deflake_hitl_stale
Jul 18, 2026
Merged

test(eve): clarify stale-ask-question-selection judge criterion (both variants)#923
prha merged 2 commits into
mainfrom
prha/deflake_hitl_stale

Conversation

@prha

@prha prha commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

What

hitl/stale-ask-question-selection's closedQA judge flipped a valid reply to 0 on ~22% of runs. The root cause is the judge, not the model.

Evidence

Instrumented repro (16 runs against a deployed preview) showed 16/16 replies correctly engage with the stale selection — e.g. "That response is stale and no longer relevant; the earlier prompt was already answered with 'Use current context.'" Every reply satisfies the intended criterion, yet the real eval still scored 0% ~22% of the time. So the binary closedQA grader was intermittently answering "no" for good replies.

The likely trigger: the old criterion asked the grader to confirm the reply "acknowledges a late selection of the STALE-CANDIDATE-7Q4M option" — the option-specific phrasing made it flip for replies that engage in general terms ("stale / earlier prompt / already answered") without repeating the option text.

Fix

Reword the criterion to match how models reliably respond: treat the message as a late response to an earlier question; apply it or explain it's already-answered/stale; answer "yes" unless the reply instead answers the current pending question or ignores it.

Verified against a deployed preview: 0/15 flips post-change vs ~22% before. The deterministic HITL mechanics (stale selection becomes a new user turn, fresh request ID, etc.) already pass and are unchanged — the judge was the only nondeterministic gate. Model behavior was never the problem, so no prompt/instruction change.

The eval's closedQA judge flipped a valid reply to 0 ~22% of runs. Root cause
is the judge, not the model: instrumented repro showed 16/16 replies correctly
engage with the stale selection ('that response is stale and no longer
relevant; the earlier prompt was already answered'). The old criterion asked
the binary grader to confirm the reply 'acknowledges a late selection of the
STALE-CANDIDATE-7Q4M option' — the option-specific phrasing made it
intermittently answer 'no' for replies that engage in general terms.

Reword the criterion to match how models reliably respond (treat it as a late
response to an EARLIER question; apply or explain it's already-answered/stale;
'yes' unless the reply answers the current pending question or ignores it).
Verified against a deployed preview: 0/15 flips post-change vs ~22% before.

Signed-off-by: prha <prha@vercel.com>
@prha
prha marked this pull request as draft July 17, 2026 22:28
@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
eve-docs Ready Ready Preview, Comment, Open in v0 Jul 18, 2026 3:12am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
eve-docs-1644 Skipped Skipped Open in v0 Jul 18, 2026 3:12am
eve-docs-4759 Skipped Skipped Open in v0 Jul 18, 2026 3:12am

The -idle sibling carries the identical closedQA criterion and the identical
flake — run 29618944115/attempt-4 failed on BOTH stale-ask-question-selection
and -idle scoring 0%, and the earlier commit only fixed the non-idle one.

Apply the same clarification, adapted for the idle flow (nothing is pending
when the stale selection arrives, so no 'current pending question' clause).
Verified against a deployed preview: 11/11 closedQA=100% post-change.

Signed-off-by: prha <prha@vercel.com>
@prha prha changed the title test(eve): clarify stale-ask-question-selection judge criterion test(eve): clarify stale-ask-question-selection judge criterion (both variants) Jul 18, 2026
@vercel
vercel Bot temporarily deployed to Preview – eve-docs-1644 July 18, 2026 03:11 Inactive
@vercel
vercel Bot temporarily deployed to Preview – eve-docs-4759 July 18, 2026 03:11 Inactive
@prha
prha merged commit 477762e into main Jul 18, 2026
149 of 151 checks passed
@prha
prha deleted the prha/deflake_hitl_stale branch July 18, 2026 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants