Skip to content

Delete dead v4_slice parity test pipeline - #5457

Merged
briansrls merged 5 commits into
mainfrom
session/tidy-deer-261
Jun 21, 2026
Merged

briansrls merged 5 commits into
mainfrom
session/tidy-deer-261

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes the dead dag_emit_from_resolved_matches_compile_sources_for_v4_slice() test from src/v1/tests/src/pipeline.rs.

The test depends on the fixtures/v2-mvp1 fixture which has been permanently deleted and no longer exists in git history or the working tree. This test was previously hidden by the old rust-gate allowlist but was exposed by #5427 widening, revealing it as a permanent red failure.

Test plan

  • ✓ Test function removed cleanly
  • ✓ No remaining references to the fixture or test
  • ✓ Verified deletion doesn't break other tests

briansrls and others added 3 commits June 21, 2026 07:57
dag_emit_from_resolved_matches_compile_sources_for_v4_slice reads fixture
fixtures/v2-mvp1 which was deleted and no longer exists in git history or the
working tree. The test was hidden by the old rust-gate allowlist but exposed
by #5427, and cannot pass. (Refs: #5427)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
The v1_compiler_lib_test module was solely used by the now-deleted
dag_emit_from_resolved_matches_compile_sources_for_v4_slice test to
compile-check the v1-compiler lib test harness. With that parity test
gone, this module is dead code. (Addresses review feedback on #5457)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
…rnal_authority_transport to exception roster

The extdeps_external_authority_transport.dag (added in #5418) imports
extdeps.languages.bash.program but was not included in the exception roster,
causing the realization_vocab_clean_tree_holds test to fail. The file is a
transport module in dsl/tools/ following the same pattern as other rostered
transports, so it requires roster entry per DESIGN §3.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@briansrls
briansrls force-pushed the session/tidy-deer-261 branch from 51bed66 to 32838bc Compare June 21, 2026 08:19
@briansrls
briansrls merged commit aa9ada0 into main Jun 21, 2026
1 check passed
@briansrls
briansrls deleted the session/tidy-deer-261 branch June 21, 2026 14:43
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