Skip to content

feat(cargo): add cargo install filtering - #70

Merged
pszymkowiak merged 1 commit into
rtk-ai:masterfrom
FlorianBruniaux:feat/cargo-install-filtering
Feb 9, 2026
Merged

feat(cargo): add cargo install filtering#70
pszymkowiak merged 1 commit into
rtk-ai:masterfrom
FlorianBruniaux:feat/cargo-install-filtering

Conversation

@FlorianBruniaux

@FlorianBruniaux FlorianBruniaux commented Feb 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add rtk cargo install with 80-90% token reduction by stripping dep compilation, downloading, locking noise
  • Keep only actionable lines: installed/replaced/errors/PATH warnings
  • Extract format_crate_info helper to eliminate duplication
  • 11 unit tests covering success, replace, errors, already-installed, edge cases

Output examples

Success:

✓ cargo install (rtk v0.11.0, 130 deps compiled)
  Replaced package `rtk v0.9.4` with `rtk v0.11.0` (/Users/user/.cargo/bin/rtk)

Error:

cargo install: 2 errors (rtk v0.11.0, 45 deps compiled)
═══════════════════════════════════════
error[E0308]: mismatched types
...

Already installed:

✓ cargo install: rtk v0.11.0 already installed

Closes #69

Test plan

  • cargo fmt --all --check passes
  • cargo clippy --all-targets — 0 new warnings
  • 260 tests pass (254 existing + 6 new)
  • Manual test: cargo install --path . --force via hook

🤖 Generated with Claude Code

@FlorianBruniaux @claude
feat(cargo): add cargo install filtering with 80-90% token reduction
645a773 
Strip dep compilation, downloading, locking noise from cargo install output.
Keep only installed/replaced/error lines with compact summary format.

Closes rtk-ai#69

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 8, 2026 10:36
Copilot AI reviewed Feb 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@pszymkowiak
pszymkowiak merged commit 447002f into rtk-ai:master Feb 9, 2026
2 checks passed
ahundt pushed a commit to ahundt/rtk that referenced this pull request Feb 23, 2026
@pszymkowiak
Merge pull request rtk-ai#70 from FlorianBruniaux/feat/cargo-install-…
3beb2bc 
…filtering

feat(cargo): add cargo install filtering
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: add cargo install filtering

3 participants

@FlorianBruniaux @pszymkowiak