From 52e832d39e07b16eba70cd43ed7b0fa5e5751eb4 Mon Sep 17 00:00:00 2001 From: openhands Date: Sat, 8 Nov 2025 14:26:42 +0530 Subject: [PATCH] Ensure log directory is created if missing Replaces the previous check for log directory existence with automatic creation using os.makedirs. This allows telemetry logging to proceed even if the log directory does not already exist. --- openhands-sdk/openhands/sdk/llm/utils/telemetry.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/openhands-sdk/openhands/sdk/llm/utils/telemetry.py b/openhands-sdk/openhands/sdk/llm/utils/telemetry.py index 6d4174ce1f..bff695738b 100644 --- a/openhands-sdk/openhands/sdk/llm/utils/telemetry.py +++ b/openhands-sdk/openhands/sdk/llm/utils/telemetry.py @@ -217,10 +217,8 @@ def log_llm_call( if not self.log_dir: return try: - # Only log if directory exists and is writable. - # Do not create directories implicitly. - if not os.path.isdir(self.log_dir): - raise FileNotFoundError(f"log_dir does not exist: {self.log_dir}") + # Create log directory if it doesn't exist + os.makedirs(self.log_dir, exist_ok=True) if not os.access(self.log_dir, os.W_OK): raise PermissionError(f"log_dir is not writable: {self.log_dir}")