fix: strip trailing stderr redirects before rewrite matching (#530)

[pszymkowiak]

Closes #530

Problem

rtk rewrite "git status 2>&1" returned nothing because 2>&1 was part of the string passed to pattern matching.

Fix

Strip trailing redirects (2>&1, 2>/dev/null, etc.) before matching, re-append after rewrite.

git status 2>&1       → rtk git status 2>&1
cargo test 2>/dev/null → rtk cargo test 2>/dev/null
git log 2> /dev/null  → rtk git log 2> /dev/null

Test plan

• 1111 tests passing
• Tested with rtk rewrite on all redirect patterns
• Existing rewrites unaffected

[pszymkowiak]

[w] wshm · Automated triage by AI

📊 Automated PR Analysis

🐛 Type	bug-fix
🟡 Risk	medium

Summary

Fixes a bug where trailing stderr/stdout redirects (e.g., 2>&1, 2>/dev/null) were included in the pattern matching string during command rewriting, causing rewrites to fail silently. The fix strips trailing redirects before matching and re-appends them after rewriting.

Review Checklist

• Tests present
• Breaking change
• Docs updated

Linked issues: #530

---

Analyzed automatically by wshm · This is an automated analysis, not a human review.

[FlorianBruniaux]

ok