Repository navigation
t27f: deps view answers why A imports B -- one shortest use chain (Closes #10705) - #10715
Merged
Merged
Conversation
…Closes #10702) deps_unknown reads the focus spec's source next to the use-edges list and writes one `unknown<TAB>path` line per use line with no edge to its whole path or its single fallback, mirroring bootstrap/src/use_resolve.rs. Measured on the Railway lab, master f33f269: 2489 use lines, 23 unknown in 13 specs, byte for byte the `t27c gen` "resolves to no spec" warnings; the drop-one and empty controls both fail. Spec 25/25 tests, 0 vacuous; corpus ratchet CLEAN. The t27f skill gains the oracle recipe. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…oses #10705) deps_why a b writes one shortest chain of edge-list lines from a to b, each exact, a tie taken in edge-list order; `none` when a does not reach b; refused whole when the walk or the output does not fit. It answers the question cargo tree -i, npm explain and pnpm why answer. deps_find is split out of deps_in_range so the backtrack can pick the first edge line in a range whose importer sits one layer closer. Six tests: the shortest chain over DEPS_FIXTURE, a tie in edge-list order, a spec to itself in no hops, none against the edges and off the graph, a cycle crossed both ways, and an output one byte short refused whole. Lab: test-report 31/31, 0 vacuous; a scratch corpus check over 2466 edge lines held hop count = layer and joined hops for every spec reached from lesson_reel, spec_deps and t27f_gate; the one-hop-more control and the last-edge mutation fail. Seal re-saved and verified; corpus ratchet CLEAN. The t27f skill gets the why recipe, the finding that all 13 specs with a dropped use line fail for other reasons first, and the pkill self-match trap. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # .claude/skills/t27f/SKILL.md # .trinity/seals/t27f_t27f_spec_deps.json # specs/t27f/spec_deps.t27
gHashTag
enabled auto-merge (squash)
October 18, 2026 23:20
Contributor
This was referenced Oct 18, 2026
gHashTag
added a commit
that referenced
this pull request
Oct 18, 2026
Master's SKILL.md is the squash of #10715, byte-identical to this branch's base, so the branch copy is kept. 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
1 participant
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.
Closes #10705
What
deps_why a binspecs/t27f/spec_deps.t27answers the question the dependencies view could not: why does spec A import spec B. It writes one shortest chain oft27c use-edgeslines from A to B:why A B nonewhen A does not reach B.refused full|deep Awhen the walk does not fit, andcap + 1when the output does not fit. Nothing is ever cut.This is the query
cargo tree -i,npm explain,pnpm whyand Bazelsomepathanswer for their ecosystems.deps_findis split out ofdeps_in_rangeso the backtrack can take the first edge line in a range whose importer sits one layer closer to A.Tests (6 new, 31 total)
DEPS_FIXTURE: f to d in 4 hops;noneagainst the direction of the edges and for a spec off the graph;refused full.Evidence (Railway lab)
t27c test-report specs/t27f/spec_deps.t27: 31 pass, 0 FAIL, 0 vacuous (md5 9aef3a30).lesson_reel,spec_depsandt27f_gate. Each chain was as long as the spec's layer, every hop was an edge-list line, and the hops joined up. The one-hop-more control failed, and dropping the last edge failed two tests.t27c seal ... --verify: all hashes MATCH.t27c suite --ratchet --corpus-only: RATCHET CLEAN, 2319 specs.Skill
.claude/skills/t27f/SKILL.mdgets three additions:useline failtest-reportfor other reasons first, so repairing an import alone fixes none of them;pkill -fmatches its own ssh session.The merge commit only takes master's squash of #10704. Master's copies of the three files were byte-identical to the branch's base.
🤖 Generated with Claude Code