From f76830ab1210e29b70ca91b51ed527a94c6c3b8b Mon Sep 17 00:00:00 2001
From: Yuya Nishihara <yuya@tcha.org>
Date: Sat, 2 Mar 2024 14:07:41 +0900
Subject: [PATCH] cli: translate last-minute EPIPE internally in
 handle_command_result()

---
 cli/src/cli_util.rs      | 5 ++---
 cli/src/command_error.rs | 8 ++++++--
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/cli/src/cli_util.rs b/cli/src/cli_util.rs
index fa12f2155..dc5aa065e 100644
--- a/cli/src/cli_util.rs
+++ b/cli/src/cli_util.rs
@@ -74,7 +74,7 @@ use tracing_subscriber::prelude::*;
 
 use crate::command_error::{
     handle_command_result, internal_error, internal_error_with_message, user_error,
-    user_error_with_hint, user_error_with_message, CommandError, BROKEN_PIPE_EXIT_CODE,
+    user_error_with_hint, user_error_with_message, CommandError,
 };
 use crate::commit_templater::CommitTemplateLanguageExtension;
 use crate::config::{
@@ -2562,8 +2562,7 @@ impl CliRunner {
         let mut ui = Ui::with_config(&layered_configs.merge())
             .expect("default config should be valid, env vars are stringly typed");
         let result = self.run_internal(&mut ui, layered_configs);
-        let exit_code = handle_command_result(&mut ui, result)
-            .unwrap_or_else(|_| ExitCode::from(BROKEN_PIPE_EXIT_CODE));
+        let exit_code = handle_command_result(&mut ui, result);
         ui.finalize_pager();
         exit_code
     }
diff --git a/cli/src/command_error.rs b/cli/src/command_error.rs
index e50fa0a4b..2621fb0ad 100644
--- a/cli/src/command_error.rs
+++ b/cli/src/command_error.rs
@@ -436,9 +436,13 @@ impl From<GitIgnoreError> for CommandError {
     }
 }
 
-pub(crate) const BROKEN_PIPE_EXIT_CODE: u8 = 3;
+const BROKEN_PIPE_EXIT_CODE: u8 = 3;
 
-pub(crate) fn handle_command_result(
+pub(crate) fn handle_command_result(ui: &mut Ui, result: Result<(), CommandError>) -> ExitCode {
+    try_handle_command_result(ui, result).unwrap_or_else(|_| ExitCode::from(BROKEN_PIPE_EXIT_CODE))
+}
+
+fn try_handle_command_result(
     ui: &mut Ui,
     result: Result<(), CommandError>,
 ) -> io::Result<ExitCode> {