Repository navigation
fix(rewrite): route yadm to rtk yadm instead of the project's git repository - #20
Merged
Merged
Conversation
yadm manages a separate dotfiles repository, so rewriting its subcommands to rtk git ran git on whatever project the shell was in. Remove yadm from the git rule; the yadm filter already routes it to the yadm binary as rtk yadm. Adapted from kylehgc/rtk 1d67272 by RawNuke (Apache-2.0), including the rule change and tests. Its bare git add change is not included. See rtk-ai#3408.
Say yadm left the git rewrite rule rather than claiming it can never be rewritten, and that discover lists it as unhandled like other commands with only a TOML filter.
ryanduguid
enabled auto-merge (squash)
October 30, 2026 13:13
|
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.
Summary
yadmcommands now rewrite tortk yadminstead ofrtk git, so they act on the dotfiles repository rather than the current project's repository.Credit: kylehgc/rtk
1d672725by RawNuke (Apache-2.0).Evidence
rtk rewritegivesrtk yadmfor the cd, env-prefix, compound and-Cforms.core::stream::tests::test_read_lines_lossy_preserves_lines_after_invalid_utf8failed the same way on Windows before this change.Merge danger
Door: two-way. Blast radius: yadm users
Unverified
yadmon PATH.sudo yadmis not rewritten.rtk discoverlistsyadm statusas unhandled, as it does for every TOML-filter-only command such asjq. The README states this.