refactor(commands): trim end of pipe-like output

helix-term/src/commands.rs

@@ -5760,6 +5760,7 @@ fn shell(cx: &mut compositor::Context, cmd: &str, behavior: &ShellBehavior) {
  let fragment = range.slice(text);
  match shell_impl(shell, cmd, pipe.then(|| fragment.into())) {
  Ok(result) => {
+ let result = Tendril::from(result.trim_end());
  if !pipe {
  shell_output = Some(result.clone());
  }

helix-term/tests/test/commands.rs

@@ -209,13 +209,10 @@ async fn test_multi_selection_shell_commands() -> anyhow::Result<()> {
  "},
  "|echo foo<ret>",
  indoc! {"\
- #[|foo\n]#
- 
- #(|foo\n)#
- 
- #(|foo\n)#
- 
- "},
+ #[|foo]#
+ #(|foo)#
+ #(|foo)#"
+ },
  ))
  .await?;
 
@@ -228,12 +225,9 @@ async fn test_multi_selection_shell_commands() -> anyhow::Result<()> {
  "},
  "!echo foo<ret>",
  indoc! {"\
- #[|foo\n]#
- lorem
- #(|foo\n)#
- ipsum
- #(|foo\n)#
- dolor
+ #[|foo]#lorem
+ #(|foo)#ipsum
+ #(|foo)#dolor
  "},
  ))
  .await?;
@@ -247,12 +241,9 @@ async fn test_multi_selection_shell_commands() -> anyhow::Result<()> {
  "},
  "<A-!>echo foo<ret>",
  indoc! {"\
- lorem#[|foo\n]#
- 
- ipsum#(|foo\n)#
- 
- dolor#(|foo\n)#
- 
+ lorem#[|foo]#
+ ipsum#(|foo)#
+ dolor#(|foo)#
  "},
  ))
  .await?;