Skip to content

pipeline.rs self_gen8 cluster: retire 28 of 31 fns via #6261 typed retirement path (delete-mass triage) - #6272

Merged
briansrls merged 2 commits into
mainfrom
session/quick-badger-839
Jul 5, 2026
Merged

briansrls merged 2 commits into
mainfrom
session/quick-badger-839

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jul 5, 2026 •

Copy link
Copy Markdown
Contributor

What

Delete-mass triage of the self_gen8 cluster in src/v1/tests/src/pipeline.rs (31 fns), per the v2.lens.test_migration_debt drain and the #6261 typed retirement path. 28 fns deleted, 3 deliberately kept, with the triage recorded as two typed TestModuleRetirement rows in dag/test/retirement/pipeline_self_gen8_retired.dag.

Ground truth (measured, not assumed)

cargo test -p v1-compiler-tests self_gen8 -- --include-ignored on this branch's base: 22 passed, 9 failed. Notably, 8 of the 17 #[ignore]d fns carried stale red-on-main reasons and pass on main today (the whole parametric-alias-RHS family, f3, f4 — fixed by intervening emitter work).

Dispositions

22 green fns → DeleteRedundant. Two live executing consumers cover the behavior:

6 red fns → DeleteLowValue. #[ignore] since #5427, red on main, never run in any gate; they pin desired-but-absent seed-emitter behavior on synthetic fixtures. The deficit displaces no cost today (regen green over the seed corpus) and emit-deficit enumeration is owned by execution in the fresh-emit lane. Bodies recoverable at the deleting commit.

3 red fns KEPT (stays_unemitted alias-to-opaque trio): dag/gunbc/plans/dag_v2_defork_audit.dag:114 claims them for the def-unification lane (node://adhoc-9d2bb9c3-e7b) — their premise flips from stays-unemitted to grounded List<T>=Vec<T> emission there. Their #[ignore] reasons are repointed from the stale generic Route-A text to the actual owning lane.

Guard interaction

The retirement decl is receipt-only w.r.t. the delete-guard: its contributed stem pipeline_self_gen8 matches no v1 test file, and pipeline.rs itself survives this PR — its eventual file deletion still requires its own coverage. The debt roster's fn-count for pipeline.rs drops by 28.

🤖 Generated with Claude Code

briansrls and others added 2 commits July 5, 2026 05:31
…s; 3 kept for def-unification lane)

Delete-mass triage of the self_gen8 cluster per the test_migration_debt drain
(#6261 retirement path). Ground truth measured 2026-07-05 via
cargo test -p v1-compiler-tests self_gen8 -- --include-ignored: 22 passed,
9 failed - 8 of the 17 #[ignore]d fns carried stale red-on-main reasons and
pass on main today.

- 22 green fns -> DeleteRedundant: seed-corpus pattern families (homonymous
  type names, variant-specific imports, kernel-ambient types, direct type
  imports) are byte-pinned by RegenVerifyGate (regen_stage0 --verify, #5873)
  plus the emitted-seed rustc/clippy gates; beyond-corpus synthetic shapes
  (parametric-alias RHS, opaque parametric decls, proxy chains, wildcard
  imports) are executed by the v2 self-host fresh-emit lane over dag/std's
  real instances with deficits enumerated in the 1667-error receipt
  (#6253/#6258). An #[ignore]d pin runs in no gate - zero delivered signal.
- 6 red fns -> DeleteLowValue: #[ignore] since #5427, never run in any gate,
  desired-but-absent seed-emitter behavior on synthetic fixtures; deficit
  ownership is the fresh-emit lane.
- 3 red fns KEPT (stays_unemitted alias-to-opaque trio):
  dag/gunbc/plans/dag_v2_defork_audit.dag claims them for the def-unification
  lane (node://adhoc-9d2bb9c3-e7b) - premise flips, not deletes.

Typed receipt: dag/test/retirement/pipeline_self_gen8_retired.dag (two
TestModuleRetirement rows). Receipt-only w.r.t. the delete-guard - the
contributed stem pipeline_self_gen8 matches no v1 test file and pipeline.rs
itself survives, so its eventual file deletion still requires its own
coverage.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…cation, node://adhoc-9d2bb9c3-e7b)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@gunbai-bot
gunbai-bot Bot marked this pull request as ready for review July 5, 2026 05:42
@gunbai-bot

gunbai-bot Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

CI status note for the merging operator: the red ci check is the repo-wide 10-minute-timeout cancellation affecting every current run including main (both attempts on this PR died at exactly 10m18s, mid-batch-1, with '''The operation was canceled''' — the Actions timeout-minutes: 10 job limit, not a test failure). rust_tests passes on this head both times. A fix lane is in flight on sharp-deer-310/regen-ci-yml-6263. Local verification on this exact head is fully green: pipeline.rs 332 passed/0 failed, retirement-guard tests 6/6, clippy -D warnings clean, fmt clean, and a whole-tree gunbc compile --source-root dag --target dag resolving all 880 modules including the new retirement receipt with 0 diagnostics. Will rerun CI once the timeout fix lands. — sent from quick-badger-839

@briansrls
briansrls merged commit 5cfda8b into main Jul 5, 2026
5 of 9 checks passed
@briansrls
briansrls deleted the session/quick-badger-839 branch July 5, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant