Skip to content

Add an evidence-based performance improvement reviewer - #20619

Merged
T-Gro merged 15 commits into
mainfrom
idea/dm-943
Oct 21, 2026
Merged

T-Gro merged 15 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
@T-Gro
T-Gro marked this pull request as ready for review October 21, 2026 10:38
@T-Gro

T-Gro commented Oct 21, 2026

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

Co-authored-by: T-Gro <46543583+T-Gro@users.noreply.github.com>

Copilot AI commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merged main into the PR branch in commit 61a8783. Git found no conflicts, and the merge commit has both expected parents.

@github-project-automation github-project-automation Bot moved this from New to In Progress in F# Compiler and Tooling Oct 21, 2026
@T-Gro
T-Gro merged commit d85eebc into main Oct 21, 2026
2 checks passed
@T-Gro
T-Gro deleted the idea/dm-943 branch October 21, 2026 16:20
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

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants