chore: replace inspect.stack with sys._getframe

Author: cosmin-staicu

packages/uipath-platform/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "uipath-platform"
-version = "0.0.18"
+version = "0.0.19"
 description = "HTTP client library for programmatic access to UiPath Platform"
 readme = { file = "README.md", content-type = "text/markdown" }
 requires-python = ">=3.11"

packages/uipath-platform/src/uipath/platform/common/_base_service.py

@@ -1,4 +1,5 @@
-import inspect
+import sys
+import types
 from logging import getLogger
 from typing import Any, Literal, Union
 
@@ -32,6 +33,39 @@
     platform_wait_strategy,
 )
 
+_THIS_FILE = __file__
+
+
+def _get_caller_component() -> str:
+    try:
+        current: types.FrameType | None = sys._getframe(1)
+        while current is not None:
+            # Skip frames from this module (_base_service.py)
+            if current.f_code.co_filename == _THIS_FILE:
+                current = current.f_back
+                continue
+            # Skip frames without self/cls or from third-party libraries
+            # (e.g. tenacity retry decorator internals)
+            locals_ = current.f_locals
+            if "self" not in locals_ and "cls" not in locals_:
+                current = current.f_back
+                continue
+            if "site-packages" in current.f_code.co_filename:
+                current = current.f_back
+                continue
+            # Found a meaningful caller
+            if "self" in locals_:
+                module_name = type(locals_["self"]).__name__
+            else:
+                module_name = locals_["cls"].__name__
+            function_name = current.f_code.co_name
+            if module_name and function_name:
+                return f"{module_name}.{function_name}"
+            break
+    except Exception:
+        pass
+    return ""
+
 
 class BaseService:
     def __init__(
@@ -78,26 +112,7 @@ def request(
         self._logger.debug(f"Request: {method} {url}")
         self._logger.debug(f"HEADERS: {kwargs.get('headers', self._client.headers)}")
 
-        try:
-            stack = inspect.stack()
-
-            # use the third frame because of the retry decorator
-            caller_frame = stack[3].frame
-            function_name = caller_frame.f_code.co_name
-
-            if "self" in caller_frame.f_locals:
-                module_name = type(caller_frame.f_locals["self"]).__name__
-            elif "cls" in caller_frame.f_locals:
-                module_name = caller_frame.f_locals["cls"].__name__
-            else:
-                module_name = ""
-        except Exception:
-            function_name = ""
-            module_name = ""
-
-        specific_component = (
-            f"{module_name}.{function_name}" if module_name and function_name else ""
-        )
+        specific_component = _get_caller_component()
 
         kwargs.setdefault("headers", {})
         kwargs["headers"][HEADER_USER_AGENT] = user_agent_value(specific_component)
@@ -181,24 +196,4 @@ def custom_headers(self) -> dict[str, str]:
 
     @property
     def _specific_component(self) -> str:
-        try:
-            stack = inspect.stack()
-
-            caller_frame = stack[4].frame
-            function_name = caller_frame.f_code.co_name
-
-            if "self" in caller_frame.f_locals:
-                module_name = type(caller_frame.f_locals["self"]).__name__
-            elif "cls" in caller_frame.f_locals:
-                module_name = caller_frame.f_locals["cls"].__name__
-            else:
-                module_name = ""
-        except Exception:
-            function_name = ""
-            module_name = ""
-
-        specific_component = (
-            f"{module_name}.{function_name}" if module_name and function_name else ""
-        )
-
-        return specific_component
+        return _get_caller_component()