Skip to content

fix(ls): preserve multiple directory headers#2865

Open
roian6 wants to merge 1 commit into
rtk-ai:developfrom
roian6:fix/ls-multiple-directory-headers
Open

fix(ls): preserve multiple directory headers#2865
roian6 wants to merge 1 commit into
rtk-ai:developfrom
roian6:fix/ls-multiple-directory-headers

Conversation

@roian6

@roian6 roian6 commented Jul 7, 2026

Copy link
Copy Markdown

Summary

  • Preserve per-directory headers while compacting rtk ls output for multiple directory operands.
  • Keep empty directory operands attributed as dir: plus (empty) instead of collapsing all output into one unlabeled empty result.
  • Add regression coverage for populated and empty multi-directory listings.

Fixes #2857.

Testing

  • rustup run 1.91.0 cargo fmt --all -- --check
  • rustup run 1.91.0 cargo clippy --all-targets
  • rustup run 1.91.0 cargo test
  • Manual smoke: rtk ls dir_a dir_b
  • Manual smoke: rtk ls empty_a empty_b
  • Manual smoke: rtk ls root.txt dir_a

Preserve per-directory headers when compacting ls output for multiple directory operands, including empty directories.\n\nFixes rtk-ai#2857.
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.

rtk ls with multiple directory arguments omits per-directory headers

1 participant