From 44b63f07671a2a9cec9a17bc174ea0ceeb7db275 Mon Sep 17 00:00:00 2001
From: Soff <891708+soffchen@users.noreply.github.com>
Date: Mon, 29 Dec 2025 18:46:15 +0800
Subject: [PATCH] fix: Return an error if the user home directory cannot be
 determined for the fallback log path.

---
 internal/logging/global_logger.go | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/internal/logging/global_logger.go b/internal/logging/global_logger.go
index e5bd6721..290849e0 100644
--- a/internal/logging/global_logger.go
+++ b/internal/logging/global_logger.go
@@ -99,9 +99,11 @@ func ConfigureLogOutput(loggingToFile bool, logsMaxTotalSizeMB int) error {
 		// When logging to file is enabled but WRITABLE_PATH is not set,
 		// use a default writable location to avoid errors on read-only filesystems
 		// (e.g., Homebrew installations on macOS).
-		if home, err := os.UserHomeDir(); err == nil {
-			logDir = filepath.Join(home, ".cliproxyapi", "logs")
+		home, err := os.UserHomeDir()
+		if err != nil {
+			return fmt.Errorf("logging: failed to determine user home directory for fallback log path: %w", err)
 		}
+		logDir = filepath.Join(home, ".cliproxyapi", "logs")
 	}
 
 	protectedPath := ""