Repository navigation
Experiment: prepare revised HTTP knowledge A/B gate - #911
Draft
Wenjie Fan (gggdttt) wants to merge 5 commits into
Draft
Wenjie Fan (gggdttt) wants to merge 5 commits into
Wenjie Fan (gggdttt) wants to merge 5 commits into
Conversation
Integrate necessary deterministic adapter support from PR 881 at 8e2946f. Preserve default generic harnesses and all 145 existing code-review entries. Add three separately gated experimental controls and original-report integrity validation; no evaluation is authorized. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Declare engine da9236a6f43d47c37e0bd59ebced33b6fe73f277, which owns BCQuality 130d5de6c4bd72d2158240fe431862b6434c735a. Change only the engine pin and its test constant; shared support and version remain identical. No paid execution is authorized. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Avoid a URL-sanitization-shaped assertion in the experimental fixture test. The check only inspects trusted AL fixture source; fixture bytes and evaluation behavior are unchanged. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Replace the substring URL assertion flagged by CodeQL with an exact whitelist of extracted Label values. No fixture bytes or evaluation behavior change. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Exclude privacy-010 from experimental scope without changing its dataset row. Put the consumed-false success return on a single line, anchor its experimental gold to that line, and promote clean bare GET/Post sites into the target rubric. Preserve all 145 original rows, runtime/scorer settings and version 0.14.0.dev1. Previous stopped smoke remains separate; no new paid execution is authorized. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Wenjie Fan (gggdttt)
added this pull request to stack #939
October 6, 2026 08:45
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.
Current status: v2 preparation only
The revised seven-entry v2 pair is prepared and unrun. It tests the marginal effect of microsoft/BCQuality#196 as a bundle, not an individual HTTP article or routing contribution. A new explicit user approval is required before any reviewer generation, gold-judge call or evaluation dispatch. No retry, replication, result publication, release or product merge is authorized by this preparation.
Final immutable v2 pair
0cbb709caaf92fcb2017bcfa0b1c77185d37cbc6da9236a6f43d47c37e0bd59ebced33b6fe73f277130d5de6c4bd72d2158240fe431862b6434c735a043ddc2c38bd1e71f5dedc5a9487b441d38d13de1e61e1eacbd7e0a55cb63dc4e9418a8ed9059b194287233f80678fd4fb02aca6455d9cf046ca9262Both engines share source
2ad4f65b231707d5222be27f4a7c6ab442662722except the declared BCQuality ref; see microsoft/BC-ALAgents#78. Final Bench trees differ only by one SHA replacement in.github/actions/install-agent-harnesses/action.ymland the matchingDEFAULT_ENGINE_SHAintests/test_review_workflows.py. Every other tracked mode/blob, fixture, workflow, scorer, judge and package version0.14.0.dev1is identical. The candidate incorporated v2 through a normal merge; published history was not rewritten.Dataset SHA-256:
027fcf04cdbf506aeada64ef70498b059006d946e6804162a662ccbba45ef1d8.The shared support selectively integrates the necessary adapter foundation from open, unmerged #881 at
8e2946fd8583dc406d2ecacbaa6ef265326c0db1, not #887. Generic harnesses, unrelated model choices, dependency resolution and runtime/scorer behavior are unchanged by v2. This is not a release.Fixed scope: seven entries, six target sites
synthetic__privacy-008synthetic__privacy-015synthetic__security-clean-02synthetic__errh-tryfunction-swallowed-01synthetic__http-optional-clean-01synthetic__http-consumed-false-01synthetic__http-status-body-01synthetic__privacy-010is removed only from the experimental input, excluding both its historical target and unrelated outbox agent-finding rejection. Its dataset row and scores are untouched. All 145 original rows remain byte-for-byte unchanged. Of the three experimental rows, only the consumed-false row changes.That row now uses
if not Client.Post(HeartbeatUrlTok, Content, Response) then exit(true);, with the expected finding at line 12..12. The successful exchange still falls through toIsSuccessStatusCode; the false path never reads the response. No explanatory steering comments, extra gold or ignored comments were introduced. The compact shape reduces the particular multi-line location ambiguity but does not guarantee future model compliance.All three HTTP fixtures produce zero diagnostics using the already-installed AL extension 30.0.2804007, parser assembly 30.0.42.51579, via
SyntaxFactory.ParseSyntaxTree. A deliberately missing parenthesis in an in-memory sensitivity control produces AL0104. This is syntax only, not symbol resolution, compilation, analyzer execution or runtime validation, and does not prove BC27 application compatibility. No tooling was installed or upgraded. The parent accepted the revised shape for experiment-only use conditional on zero syntax diagnostics; that condition passed. Stochastic Style noise remains possible and must be inventoried rather than hidden.HTTP article annotations remain empty because those articles do not exist at the pre-196 baseline. There are 8 expected comments across 5 judge-eligible entries. Existing gold and scorer behavior remain unchanged.
Proposed paid gate, not execution permission
Both arms retain CLI
1.0.88, rootgpt-5.6-sol, leafgpt-5.6-luna, serial scheduling, configured concurrency 4/effective 1, timeout 30 minutes, Medium severity floors and identical ordered 19 leaves + 1 root.A new approval may authorize one paired smoke: 14 entry generations, at most 280 reviewer processes and 10 gold-judge calls. Five further pairs require another approval: 70 generations, 1400 processes and at most 50 judge calls. Combined conditional maximum: 84 / 1680 / 60. These are invocation bounds, not provider-call, token, credit or dollar caps.
Collect and audit every baseline entry before considering B. Baseline eligibility requires all seven valid reports, all three positive controls and at least one optional-return misread at the six fixed sites. Report the four historical and two controlled sites separately. If both controlled sites are silent but a historical target reproduces, the target-signal condition can still pass. Missing signal, invalid output or a missing control means STOP before B, without retry.
Candidate acceptance requires no target or mixed assertion at any of the six sites in original leaf, root and normalized output, with controls preserved. A leaf misread later removed by the root is not a v2 pass, and filtering losses are not suppression. Use the fixed human rubric, no new AI adjudicator. Inventory unrelated findings; normal Bench P/R/F1 remains secondary. Five replications are not authorized now and are not a powered significance claim.
Integrity and no-publish route
Require exact executed/declared pin equality, source and fixture hashes, all ordered process roles, observed models, complete usage, original schema/role checks and nested outcomes. Manifest completion or JSON schema validity alone is insufficient. Startup CLI must be exactly
1.0.88; optional raw OTel CLI versions remain null rather than being filled from configuration. Raw OTel is deleted after ingestion and is not promised as archived.Any future authorized run must use the exact seven-ID
entriesarray,test-run=true,modified-only=false,repeat=1, and explicitengine-shaequal to its declared pin. Recheck live branch heads before spending. This disables requeue/ephemeral tags and makes summarymock=true, omitting Braintrust/Kusto storage and leaderboard writes. Summary mock does not mock reviewer generation or the gold judge. Preserve the existing concurrency group and matrix cap; no bypass is added.Separate immutable v1 observation
V1 prepared A
32a4c2190f996ef68e696716fed201e0d005cebd/ B385cf73a9aa217692a43af4ec6c4997967efd668. Its only evaluation was baseline run 36839547719: 6/8 complete observations; two original root reports were partial due to consumer-contract rejections. Candidate B was not dispatched. Old inputs, STOP receipt, run artifacts and hashes remain unchanged.V2 is informed by this failed preparation/signal attempt and therefore has a revised scope and fixture. Do not pool v1 observations, reuse its authorization, or describe v2 as an unchanged retry. No causal A/B conclusion follows from v1, and v2 has no evaluation results.
Unpaid preparation evidence
PREPARATION ONLY / NOT_AUTHORIZED.