Repository navigation
dataset(code-review): add reaction gold answers from BCApps PR 10990-neg-3926331607-source-faithful-20261005 - #922
Closed
Wenjie Fan (gggdttt) wants to merge 1 commit into
Conversation
…neg-3926331607-source-faithful-20261005
Contributor
Collaborator
Author
|
Closing this source-faithful Gold candidate because both evaluation arms emitted the same unrelated Performance finding and the selected Upgrade finding did not reproduce in the baseline. The immutable Gold commit, A/B runs, and artifacts remain preserved as evidence. |
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.
Summary
Adds code-review gold answers from explicit BCApps PR feedback.
Negative guards use exact source-file patches at immutable reviewed revisions, not invented synthetic code. Their targets are omitted from both expected and ignored comments; new negative-only entries have no expected findings.
Existing eligible gold answers are reused without changing their payloads.
Accepted findings
Rejected findings
Knowledge dependency
Validation
See the gate-owned offline evaluation section below.
Generated by the BC-ALAgentsInternal self-improvement workflow.
Offline evaluation: regression
Candidate correctness failed: unexpected or missed findings remain. Existing ignored gold comments retain their neutral scoring semantics.
Preparation attempt 1: validated.
Selection: new. New:
synthetic__upgrade-contoso-demo-data-no-upgrade-wiring-01. Reused (complete payloads): ``.synthetic__upgrade-contoso-demo-data-no-upgrade-wiring-01/false_positive_guard/ Add-Expense-VAT-settings-to-Contoso-demo-tool BCApps#10990 (comment): Exact pipeline-validated negative snapshot (base_commit 1e3872168af58518ac2a02e8cf5934cbc28eee32) reproducing the reviewed hunk: Codeunit.Run(Codeunit::"Create Expense VAT Rates") added inside CreateMasterData() of a codeunit implementing "Contoso Demo Data Module". The maintainer rejected the request for an Upgrade-subtype entry point because this is Preview demo data invoked only through the explicit Contoso Demo Data tool, never automatically on install or version upgrade -- no subscriber or Subtype=Install/Upgrade exists anywhere in this module for any of its demo-data procedures, so the alleged upgrade-coverage gap is not a defect introduced by this change. Limitation: this guard covers only Contoso demo-data-module seeding helpers invoked exclusively through the demo-data tool; it does not generalize to codeunits that genuinely participate in the install/upgrade pipeline, nor does it certify every Preview-labelled feature as upgrade-exempt.Coverage references and outcome shapes are validated mechanically. Semantic equivalence, severity calibration and recommendation quality are NOT proved by these checks or a matching F1.
Common engine code:
ecf8e31759d6ddd6d78e3a0b7836b40134368009; dataset SHA256:39B118C1CDE2A4BA879BA544288C777E80D99B14F7882870045CC993BBE830B5.Model:
gpt-5.6-luna; judge:gpt-5.3-codex. Exact D:synthetic__upgrade-contoso-demo-data-no-upgrade-wiring-01.Dataset base:
6fe2d36319d9198f64696f027eaaef495916531d; candidate:f74ae0f8e65da48e1c6aece9233c0e9d1cbf9efd.226ccd7565c8966564abeb04599679e069e467440df2f2c5eaf3e51f289571c68c6d0cb55212dd09ac249ba4c95eaef14edbe9818fcdd65561bf6a2f78b9b0fa1864bef3f9c2dc8404e1765603bf88c055248fe87e4adfca5a6e276f32e95e1d7b89f5870d728298efa0dc112bae858aab792e7c6fc43d46Baseline
Run: https://github.com/microsoft/BC-Bench/actions/runs/37281893633; conclusion: success; wall clock: 6.7 minutes.
Raw findings and runtime pin proofs are retained in the self-improvement-feedback artifact and the linked evaluation run.
synthetic__upgrade-contoso-demo-data-no-upgrade-wiring-01Observed AI credits: unavailable; coverage 0/1 entries. Missing entries are not extrapolated into a total; reported amounts do not attest full credit coverage.
Candidate
Run: https://github.com/microsoft/BC-Bench/actions/runs/37282634446; conclusion: success; wall clock: 6.4 minutes.
Raw findings and runtime pin proofs are retained in the self-improvement-feedback artifact and the linked evaluation run.
synthetic__upgrade-contoso-demo-data-no-upgrade-wiring-01Observed AI credits: unavailable; coverage 0/1 entries. Missing entries are not extrapolated into a total; reported amounts do not attest full credit coverage.
Missing telemetry is unavailable, not zero. Evaluation-only metrics exclude candidate generation and are not the full-cycle cost.
Human review must verify source-patch fidelity, gold correctness, and target attribution. No automatic merge or branch-protection claim is made.
Cycle usage and elapsed time (running, as of 2026-10-05T08:23:34.0216801+00:00)
Orchestrator work: 1395.519 seconds. This is not the completed GitHub workflow duration.
Subtotals are CLI/Bench-reported usage, not invoices. Missing values are not zero. Emitted-span coverage does not prove complete child/background billing; judge usage is not exported. Premium requests are not AI credits or currency.
Nested stage, agent and API durations are not added into cycle wall clock. Root queue/setup, telemetry export, artifact upload and later dashboard publication are excluded.
PR evidence is an as-of snapshot before publication completes. Final-for-orchestrator metrics are emitted to self-improvement-cycle.json in the root self-improvement-feedback artifact and its Actions summary when persistence succeeds; a running checkpoint is not final evidence.
Root run: https://github.com/microsoft/BC-ALAgentsInternal/actions/runs/37280881649/attempts/1