fix: fix platform tempdir problem

Author: zakahan

veadk/evaluation/eval_set_recorder.py

@@ -21,7 +21,7 @@
 from google.adk.sessions import BaseSessionService
 
 from veadk.utils.logger import get_logger
-from veadk.utils.misc import formatted_timestamp
+from veadk.utils.misc import formatted_timestamp, get_temp_dir
 
 logger = get_logger(__name__)
 
@@ -30,7 +30,7 @@ class EvalSetRecorder(LocalEvalSetsManager):
     def __init__(
         self, session_service: BaseSessionService, eval_set_id: str = "default"
     ):
-        super().__init__(agents_dir="/tmp/")
+        super().__init__(agents_dir=get_temp_dir())
         self.eval_set_id = eval_set_id if eval_set_id != "" else "default"
         self.session_service: BaseSessionService = session_service
 

veadk/tracing/telemetry/opentelemetry_tracer.py

@@ -36,6 +36,7 @@
 )
 from veadk.utils.logger import get_logger
 from veadk.utils.patches import patch_google_adk_telemetry
+from veadk.utils.misc import get_temp_dir
 
 logger = get_logger(__name__)
 
@@ -159,7 +160,7 @@ def dump(
         self,
         user_id: str = "unknown_user_id",
         session_id: str = "unknown_session_id",
-        path: str = "/tmp",
+        path: str = get_temp_dir(),
     ) -> str:
         def _build_trace_file_path(path: str, user_id: str, session_id: str) -> str:
             return f"{path}/{self.name}_{user_id}_{session_id}_{self.trace_id}.json"

veadk/utils/misc.py

@@ -148,3 +148,18 @@ def set_envs(config_yaml_path: str) -> tuple[dict, dict]:
         os.environ[k] = str(v)
 
     return config_dict, veadk_environments
+
+
+def get_temp_dir():
+    """
+    Return the corresponding temporary directory based on the operating system
+    - For Windows systems, return the system's default temporary directory
+    - For other systems (macOS, Linux, etc.), return the /tmp directory
+    """
+    # First determine if it is a Windows system
+    if sys.platform.startswith("win"):
+        # Windows systems use the temporary directory from environment variables
+        return os.environ.get("TEMP", os.environ.get("TMP", "/tmp"))
+    else:
+        # Non-Windows systems (macOS, Linux, etc.) uniformly return /tmp
+        return "/tmp"