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

helix-term/src/commands.rs

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