output `stderr` in `:sh` popup if shell commands fail (#11239)

* refactor(commands): output `stderr` in `:sh` popup

* refactor(commands): switch to `from_utf8_lossy`

This way something is always displayed.

* refactor: no longer log stderr output on failure

helix-term/src/commands.rs

@@ -5716,27 +5716,24 @@ async fn shell_impl_async(
         process.wait_with_output().await?
     };
 
-    if !output.status.success() {
+    let output = if !output.status.success() {
-        if !output.stderr.is_empty() {
+        if output.stderr.is_empty() {
-            let err = String::from_utf8_lossy(&output.stderr).to_string();
+            match output.status.code() {
-            log::error!("Shell error: {}", err);
+                Some(exit_code) => bail!("Shell command failed: status {}", exit_code),
-            bail!("Shell error: {}", err);
+                None => bail!("Shell command failed"),
-        }
+            }
-        match output.status.code() {
-            Some(exit_code) => bail!("Shell command failed: status {}", exit_code),
-            None => bail!("Shell command failed"),
         }
+        String::from_utf8_lossy(&output.stderr)
+        // Prioritize `stderr` output over `stdout`
     } else if !output.stderr.is_empty() {
-        log::debug!(
+        let stderr = String::from_utf8_lossy(&output.stderr);
-            "Command printed to stderr: {}",
+        log::debug!("Command printed to stderr: {stderr}");
-            String::from_utf8_lossy(&output.stderr).to_string()
+        stderr
-        );
+    } else {
-    }
+        String::from_utf8_lossy(&output.stdout)
+    };
 
-    let str = std::str::from_utf8(&output.stdout)
+    Ok(Tendril::from(output))
-        .map_err(|_| anyhow!("Process did not output valid UTF-8"))?;
-    let tendril = Tendril::from(str);
-    Ok(tendril)
 }
 
 fn shell(cx: &mut compositor::Context, cmd: &str, behavior: &ShellBehavior) {

helix-term/src/commands/typed.rs

@@ -2308,7 +2308,7 @@ fn run_shell_command(
                     ));
                     compositor.replace_or_push("shell", popup);
                 }
-                editor.set_status("Command succeeded");
+                editor.set_status("Command run");
             },
         ));
         Ok(call)