Skip to content

feat(discover): treat yadm commands like git#940

Merged
FlorianBruniaux merged 1 commit intortk-ai:developfrom
niklasmarderx:fix/yadm-support-567
Apr 13, 2026
Merged

feat(discover): treat yadm commands like git#940
FlorianBruniaux merged 1 commit intortk-ai:developfrom
niklasmarderx:fix/yadm-support-567

Conversation

@niklasmarderx
Copy link
Copy Markdown
Contributor

Problem

yadm (https://yadm.io) is a dotfile manager that wraps git. Running
yadm status, yadm diff, etc. produces the same output as the
equivalent git commands, but rtk doesn't recognize them.

Fix

Extended the git discovery rule to also match yadm prefixes:

  • rules.rs: pattern now matches ^(?:git|yadm)\s+..., added "yadm" to rewrite prefixes
  • 3 new tests: classify yadm status, classify yadm diff, rewrite yadm status

After:

yadm status  →  rtk git status  (70% savings)
yadm diff    →  rtk git diff    (80% savings)

Closes #567

yadm is a dotfile manager that wraps git — same subcommands, same
output format. Extending the git rule to also match `yadm` gives
yadm users the same token savings without any extra CLI support.

Closes #567
Copy link
Copy Markdown
Collaborator

@pszymkowiak pszymkowiak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @niklasmarderx — clean addition. yadm is a thin git wrapper so routing through rtk git makes sense. The regex change is minimal and the 3 tests cover classify + rewrite.

LGTM.

@niklasmarderx
Copy link
Copy Markdown
Contributor Author

Just a gentle ping — this has been approved for a while. Let me know if there's anything else needed before merge.

@FlorianBruniaux FlorianBruniaux merged commit 3606bce into rtk-ai:develop Apr 13, 2026
10 checks passed
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.

3 participants