Repository navigation
feat(quality): add evidence-backed task review and bounded reviewer evaluation - #194
Open
adithya-s-k wants to merge 10 commits into
Open
adithya-s-k wants to merge 10 commits into
adithya-s-k wants to merge 10 commits into
Conversation
This branch has not been deployed
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.
Task reviews can miss an inactive grader or accept forged test reports even when the reference and baseline checks look correct. This adds an opt-in
evidence-review-v2profile that preserves grounded findings, records the exact evidence and reviewer identity, and derives its disposition from supported defects.The existing reviewer remains the default. This is an independent implementation inspired by micro1's Flow-QC article, not a reproduction of its private code or reported accuracy.
Included
Validation
efde0d53, including Python 3.12–3.14, Windows, docs, runtime contracts and distribution builds. 379 focused local quality, compatibility, migration and pipeline contract tests pass; Ruff, wheel build and documentation checks pass (11,016 internal links).Scope and remaining promotion gates
Static local inventory found the same report-trust pattern in 650/1,330 local research/Tasksmith exports, including all 50 HF ML exports. This is exposure evidence, not 650 executed exploits or current Hub verification. Published datasets and acceptance labels are unchanged. The isolated example has an observable-value contract and does not prove arbitrary Python object identity.
The 20-family independently audited heldout gate is unmet. The prompt-search pilot completed stock and OpenAI-guidance comparisons equally on both models, then stopped a partly started Sonnet-guidance evaluation because its proposal copied an exact benchmark marker. No guidance was selected or promoted. The shipped pre-evaluation guard now rejects exact defect quotations; the stopped run and its uncertain requests remain recorded.
Final pilot ledger: $70.868357 accounted + $27.10 held = $97.968357 committed against the unchanged $150 cap. No paid controllers or validation sandboxes remain running. This PR is ready for review as an experimental, opt-in feature; it does not claim default-promotion readiness or a fully audited 20-family accuracy benchmark.