Repository navigation
Test audit 1: delete digest pins, type-size recomputations, unrun tests and scalar-oracle pairing tests - #1723
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…sses (TA1 a, b) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Test audit 1: delete digest pins, type-size recomputations, unrun tests and scalar-oracle pairing tests
Closes #1722
The owner approved four test categories for deletion without mutation proof (2026-10-18), as the first phase-B cut of the 2026-10 test audit (owner: "Don't you think almost 50% of codebase being tests is overboard?"). Each deleted test is listed with its category and line count in
.github/ISSUE_SPECS/1722-delete-digest-type-size-and-unrun-tests.md.What changed
size_ofrecomputations: 9 tests trimmed in place. Real resource ceilings and ABI layouts stay.Review
One adversarial verifier: READY WITH FIXES, with 2 majors. Two real estimator relations had been cut together with the
size_ofcode: the graph-compiler runtime-metadata charge and the graph activity-offsets charge (#1652). Both are restored as relations, and each one is red on its mutant (448 vs 492; 4096 vs 4124). Four minors are fixed. The re-check gave READY.Evidence
fmt, clippy
-D warnings, the tests of the 11 touched packages (1,786 passed; 359 again after the fold) andcheck-workspace-policy.shall pass locally. CI is the first full independent build.Kept for an owner decision (not cut here)
SCALAR_INITIAL_MATRIX_TRACE.limiter_linked_sessionPIN, which declares itself a cross-target corpus owner.phase_profile: no workflow runs it, but it is the only user of the graph phase-profile probes.size_ofcounts inside larger tests.🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.