Add ASP.NET Core adversarial fix workflows - #68625
Draft
PureWeen wants to merge 5 commits into
Draft
Conversation
This was referenced Aug 18, 2026
PureWeen
force-pushed
the
pureween-publish-reviewer-skills-d85
branch
from
August 19, 2026 19:19
ecdb736 to
abf5ab2
Compare
Synchronize the public-safe try-fix and fix-challenge reviewer runtime and deterministic evaluation assets from the reviewed private source. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 57b92326-1091-49d1-8d5c-8db75e6b994f
Keep installed reviewer skills runtime-only while preserving the specialized Vally suites, fixtures, policies, and deterministic support tooling under eng/skill-evals. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 57b92326-1091-49d1-8d5c-8db75e6b994f
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 57b92326-1091-49d1-8d5c-8db75e6b994f
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 57b92326-1091-49d1-8d5c-8db75e6b994f
PureWeen
force-pushed
the
pureween-publish-reviewer-skills-d85
branch
from
August 22, 2026 21:40
abf5ab2 to
e474a15
Compare
Key changes from v1 (208 lines) to v2 (185 lines): - Add producer-layer recording before candidates launch - Add candidate-independent assertion freezing - Add root-cause locality gate (fix layer vs producer layer) - Add role-specific packet/envelope separation for candidates - Restore reference file pointers (output-contract, empirical-proof, proof-calibration, evidence-and-orchestration) - Fix step 3/4 contradiction for carrying both candidates to proof - Restore runtime identity (unverified/not comparable) language - Add model-superiority prohibition and default merge readiness 4-model reviewed (MAI, Terra, Opus 5, Sonnet 5). All blocking issues resolved. 9/9 deterministic tests pass. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 57b92326-1091-49d1-8d5c-8db75e6b994f
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.
Add ASP.NET Core adversarial fix workflows
Description
This adds two discoverable Copilot skills for ASP.NET Core fix work:
fix-challengereviews an existing PR, issue fix, or local diff and produces one evidence-backed recommendation.fix-issuestarts from an issue with no existing fix, develops independent candidate mechanisms, and requires candidate-independent red/green proof before recommending a production fix.Both workflows use a shared, non-discoverable candidate contract under
eng/fix-workflows/candidate. This keeps candidate behavior consistent without exposing a third user-facing skill.For local, stateless, low-risk changes,
fix-challengeuses a bounded two-candidate path. Lifecycle, concurrency, compatibility, serialization, interop, security, protocol, performance, and other material risks use a deeper path with adversarial narrowing and focused empirical proof.Candidate sessions are always read-only. Both workflows are local-only by default.
fix-issuepermits the final orchestrator to commit, push, or open a PR only when the caller explicitly requests those actions after synthesis and validation. Issue comments or mutations require separate explicit authorization.Broader goal
The goal is to learn where repository-specific guidance, independent alternatives, better evidence collection, and focused validation can make agents more reliable when reviewing or developing ASP.NET Core fixes. The workflows calibrate effort to risk while preserving disagreement and proof limitations.
See the other ASP.NET Core PRs created by @PureWeen with the
agentic-workflowslabel for related work.How to use it
Review an existing fix:
Develop a fix from an issue:
Use
fix-challengewhen a fix or diff already exists. Usefix-issuewhen starting from a natural issue without exposing candidates to a known solution.What's included
fix-challengeandfix-issueHosted automation and retained run output are not part of this contribution.
Validation
No model or judge calls were rerun for this update.
Current limitations
The checked-in model policy is provisional and publicly anchored to the commit that first published its selection summary. It is based on one trial per model and case and should not be read as population-level evidence that one model is generally better than another. Authoritative runtime-model and cost telemetry were unavailable, so hosted runs remain explicitly non-comparable for model-performance claims.
Procedural candidate isolation with withheld peer outputs is not a filesystem, network, credential, or security boundary.
PRs used during development
#63826, #65578, #66246, #66894, #67589, #68037, #68078, #68081, #68110, #68114, #68116, #68146, #68222, #68254, #68279, #68281, #68317, #68344, #68365, #68453, #68471, #68504, #68540