Skip to content

Add an evidence-based performance improvement reviewer - #20619

Draft
T-Gro wants to merge 14 commits into
mainfrom
idea/dm-943
Draft

T-Gro wants to merge 14 commits into
mainfrom
idea/dm-943

Conversation

@T-Gro

@T-Gro T-Gro commented Sep 22, 2026

Copy link
Copy Markdown
Member

Add a read-only reviewer for performance-improvement PRs, grounded in source-attributed review precedents and calibrated cases, to assess measurement quality, behavior preservation, and implementation risk.

Copilot and others added 4 commits September 22, 2026 12:21
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add the read-only eight-group performance reviewer and a minimal discovery reference. Record blind development replays, calibration corrections, compact candidate identity and invocation fallback without consuming holdout feedback.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Reconcile source attribution and coverage, preserve failed calibration and isolation runs, and record the verified C6-S acceptance and delivery audit.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label Sep 22, 2026
@T-Gro
T-Gro requested a review from abonie September 23, 2026 07:57
Copilot and others added 7 commits September 23, 2026 12:04
Reconcile the original 485 PR identities and query receipts without new discovery or inspection credit. Add offline cache recovery and a sealed, fail-closed inventory gate with normal and optimized regression coverage.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Exercise temporary archive reconciliation, repeat import reuse, invalid receipts, and byte-only source seal changes without granting inspection credit. All 16 tests pass normally and under Python optimization; the exact 485-member inventory passes and completion remains rejected.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Record evidence-backed scope decisions for the original 485 PRs, immutable six-way shards, connected-family exposure, and predeclared authorship and evaluation policies. Preserve real-source availability blockers and leave reviewer semantics unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Start freshness tests from a valid metadata-only holdout and synchronize family summaries and test-only seals before injecting unknown access. Check the specific exposure error and independently exercise both audit requirements with distinct diagnostics. Preserve all frozen corpus records and reviewer semantics.

Validation: 27 Python tests pass normally and optimized; four broken freshness guards are detected in both modes; inventory, scope, shard and whitespace checks pass. Release solution build and all 19,941 executed .NET tests pass (537 skipped) using local FSharp.Core and the desktop SDK-test prerequisites.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Join the central-package-management variants without losing historical aliases or access events, and retain prior freeze hashes as additive provenance. Exercise all disqualifying exposure categories and a resealed connected-family split.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Parameterize copied-source and account-identity attestations without losing either scenario. Require distinct diagnostics with consistent test-only exposure seals so unrelated validation failures cannot satisfy the tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
No holdout or reviewer invocation was attempted without an enforced isolation boundary. Require owner runner enrollment before resuming sprint 3.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@T-Gro
T-Gro marked this pull request as draft September 25, 2026 07:23
Copilot and others added 2 commits September 29, 2026 11:43
Keep the original sealed485-PR universe and exposure evidence; bind frozen-note validation to its unchanged original entry. All485 substantive inspections still remain pending.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 20b10bf6-6a7c-483a-8d09-d521e7ddd73d

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

1 participant