Repository navigation
fix(scoring): verify saved responses and revision evidence - #7
Merged
Merged
Conversation
This was referenced Oct 20, 2026
lanyue-llk
commented
Oct 22, 2026
lanyue-llk
commented
Oct 22, 2026
lanyue-llk
commented
Oct 22, 2026
lanyue-llk
commented
Oct 22, 2026
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
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.
Problem and result
Live and saved responses could receive different verdicts, and a task revision could name regression controls without executing them. This PR gives Browser Eval and other callers the same deterministic scoring and revision-export contract.
Changes
inspect-revisionexposes those checks.The branch includes current main's article acceptance contracts. Expanded main-content references and unified mixed requirements remain in draft #8. Execution, retained evidence and all three report pipelines are handled by Browser Eval #56.
Validation
make check: 2,541 tests and 321 subtests passed after integrating current main.6bd263ed; the PR has no merge conflict.Historical report scope
The published 430-task report retains its frozen snapshot and scorer binding. Its complete comparison against the audited
e1b6efc1definitions contains 286 matching contracts, 91 retrieval-hint-only differences and 53 other differences. Original/revised-definition pass counts are 378/360/229/223/217/190 and 385/367/233/227/222/194 respectively. LZNOIP's eight canonical English navigation aliases are preserved by the report's bound revision. These saved results are not recomputed when this package's source identity changes.The full scope and source-copy audit are attached to the Notion review. The existing report keeps its normal version presentation and permissions.
Scoring and partial reruns
Reproduction and canonical-contract comparison
The historical scorer is preserved at report-scorer-20261022, paired with harness report-renderer-20261022. The full frozen task snapshot, its parent, both annotation revisions, canonical contracts and verdict inventories are attached to the v8 review. Report data remains outside Git and retains its existing private access.
Both offline comparisons use the same 430 case IDs and 2,580 hash-verified saved responses, including the previously declared eight recovered originals and 199 public projections. Scorer
6bd263edreproduces all published verdicts with zero changes. Applying the canonical base contracts ate1b6efc1gives:All 97 verdict differences occur in 27 of the 53 tasks with contract/reference differences. The 286 identical tasks and 91 retrieval-hint-only tasks have no verdict differences. These canonical scores are a diagnostic under different task requirements; no new provider calls were made.
The differences include concrete historical reference corrections: the canonical retail task requires “Returning & Exchanging Online Orders” while the report requires the observed “Returning or Exchanging Online Orders”; the canonical Arch task combines navigation and heading into “Arch Linux Official Package Search” while the corrected heading is “Package Search”. The canonical example-page task also requires the title and a different body region; the saved Moli output reaches 66.67% against that region, below its 80% requirement. Changing to those base contracts would change the benchmark definition, so this comparison does not replace the declared report snapshot.