Skip to content

feat(scp): add compact transfer output filter - #1261

Open
officialasishkumar wants to merge 1 commit into
rtk-ai:developfrom
officialasishkumar:feat/scp-filter-1257
Open

feat(scp): add compact transfer output filter#1261
officialasishkumar wants to merge 1 commit into
rtk-ai:developfrom
officialasishkumar:feat/scp-filter-1257

Conversation

@officialasishkumar

Copy link
Copy Markdown

Summary

  • Add a native rtk scp wrapper that filters combined transfer output.
  • Strip intermediate progress updates while preserving final byte summaries, warnings, and errors.
  • Register scp for hook/discover rewrites and document command coverage.

Closes #1257

Test plan

  • cargo fmt --all --check
  • cargo test scp
  • cargo clippy --all-targets && cargo test
  • Manual testing: fake scp binary on PATH, rtk scp file.txt deploy@host:/tmp/file.txt output inspected

Important: All PRs must target the develop branch (not master).
See CONTRIBUTING.md for details.

Add a native scp command wrapper that filters combined transfer output, strips intermediate progress updates, keeps final byte summaries, and preserves warnings and errors. Register the hook rewrite and update command coverage docs.

Signed-off-by: Asish Kumar <officialasishkumar@gmail.com>
@CLAassistant

CLAassistant commented Apr 13, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@pszymkowiak pszymkowiak added effort-medium 1-2 jours, quelques fichiers enhancement New feature or request labels Apr 13, 2026
@pszymkowiak

Copy link
Copy Markdown
Collaborator

[w] wshm · Automated triage by AI

📊 Automated PR Analysis

Type feature
🟢 Risk low

Summary

Adds a new rtk scp command that wraps the native scp binary, filtering out intermediate transfer progress updates while preserving final byte summaries, warnings, and error lines. The command is registered in the discovery/rewrite system and documented in the coverage guide.

Review Checklist

  • Tests present
  • Breaking change
  • Docs updated

Linked issues: #1257


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort-medium 1-2 jours, quelques fichiers enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants