From d090edd5c6d85fa02b54462f067e14b406b8b2b7 Mon Sep 17 00:00:00 2001
From: Orion <orionc1226@gmail.com>
Date: Thu, 23 Jul 2026 08:45:15 +0800
Subject: [PATCH] =?UTF-8?q?refactor(homebrew):=20=E7=BB=9F=E4=B8=80=20Home?=
 =?UTF-8?q?brew=20=E8=BE=93=E5=87=BA=E7=9D=80=E8=89=B2=E5=A4=84=E7=90=86?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- 移除交互式终端的特殊处理逻辑，禁用 Homebrew 自带动态进度显示
- 统一使用语义着色处理所有场景下的 Homebrew 输出
- 删除 BSD script 伪终端分配功能，简化日志记录流程
- 更新 README 文档说明输出着色的一致性改进
---
 homebrew/README.md               |  2 +-
 homebrew/brew-upgrade-manager.sh | 39 ++++++++++----------------------
 2 files changed, 13 insertions(+), 28 deletions(-)

diff --git a/homebrew/README.md b/homebrew/README.md
index 5bfbe77..0e02b32 100644
--- a/homebrew/README.md
+++ b/homebrew/README.md
@@ -325,7 +325,7 @@ brew upgrade --cask --greedy visual-studio-code
 ## 注意事项
 
 - 脚本启用了 `set -euo pipefail`，但 Formula/Cask 升级阶段会捕获错误、继续隔离其他失败项，并在最后统一返回状态。
-- 交互式终端中直接保留 Homebrew 的 TTY 输出，因此 Formula/Cask 下载会显示原生实时进度；输出被重定向或用于 CI 时会自动回退到按行输出。
+- Homebrew 输出在交互式终端、重定向和 CI 中都会经过同一套语义着色，避免版本变化行因原生 TTY 输出而丢失颜色。
 - `brew upgrade --cask --greedy` 可能退出正在运行的 GUI 应用，建议先保存重要工作。
 - 不建议日常启用 `HB_CASK_FORCE=1`；它会允许 Homebrew 覆盖已有 Cask 文件。
 - 远程启动器属于“下载后执行”模式，只应从可信仓库使用。
diff --git a/homebrew/brew-upgrade-manager.sh b/homebrew/brew-upgrade-manager.sh
index 4218737..f8be73c 100644
--- a/homebrew/brew-upgrade-manager.sh
+++ b/homebrew/brew-upgrade-manager.sh
@@ -573,14 +573,8 @@ colorize_brew_output() {
 run_brew_colored() {
     local pipeline_status=()
 
-    # Homebrew 只有在 stdout 连接 TTY 时才会让 curl 显示动态下载进度。
-    # 交互式终端中保留原始文件描述符，并使用 Homebrew 原生配色。
-    if [[ -t 1 ]]; then
-        brew "$@"
-        return $?
-    fi
-
-    # 非交互式输出无法显示动态进度，继续按行进行语义着色。
+    # 禁用 Homebrew 自带的不完整配色，统一按语义着色。这里不能在
+    # 交互式终端直接运行 brew，否则版本变化行会绕过着色器。
     if env -u HOMEBREW_COLOR brew "$@" 2>&1 | colorize_brew_output; then
         return 0
     fi
@@ -704,26 +698,17 @@ run_brew_with_retries() {
         fi
 
         : > "$ACTIVE_LOG_FILE"
-        if [[ -t 0 && -t 1 && -t 2 && "$(uname -s 2>/dev/null || true)" == "Darwin" && -x /usr/bin/script ]]; then
-            # BSD script 为 brew 分配伪终端：既保留实时进度，又记录完整输出供错误分类。
-            if /usr/bin/script -q -e "$ACTIVE_LOG_FILE" brew "$@"; then
-                status=0
-            else
-                status=$?
-            fi
+        if env -u HOMEBREW_COLOR brew "$@" 2>&1 | tee "$ACTIVE_LOG_FILE" | colorize_brew_output; then
+            status=0
         else
-            if env -u HOMEBREW_COLOR brew "$@" 2>&1 | tee "$ACTIVE_LOG_FILE" | colorize_brew_output; then
-                status=0
-            else
-                pipeline_status=("${PIPESTATUS[@]}")
-                status="${pipeline_status[0]}"
-                if ((${pipeline_status[1]:-0} != 0)); then
-                    print_error "failed to write retry log."
-                    status="${pipeline_status[1]}"
-                elif ((${pipeline_status[2]:-0} != 0)); then
-                    print_error "failed to colorize Homebrew output."
-                    status="${pipeline_status[2]}"
-                fi
+            pipeline_status=("${PIPESTATUS[@]}")
+            status="${pipeline_status[0]}"
+            if ((${pipeline_status[1]:-0} != 0)); then
+                print_error "failed to write retry log."
+                status="${pipeline_status[1]}"
+            elif ((${pipeline_status[2]:-0} != 0)); then
+                print_error "failed to colorize Homebrew output."
+                status="${pipeline_status[2]}"
             fi
         fi