chore: Make sync tool member variables protected

This change modifies the visibility of sync tool member variables to `protected`.

This allows for direct and type-safe access by subclasses, such as those in orchestration-specific packages like `toolbox-langchain`'s `ToolboxTool` implementations.

Making these variables protected also ensures they are read-only from outside the class hierarchy, enhancing encapsulation.

Author: anubhav756

packages/toolbox-core/src/toolbox_core/sync_tool.py

@@ -52,9 +52,9 @@ def __init__(
         if not isinstance(async_tool, ToolboxTool):
             raise TypeError("async_tool must be an instance of ToolboxTool")
 
-        self.__async_tool = async_tool
-        self.__loop = loop
-        self.__thread = thread
+        self._async_tool = async_tool
+        self._loop = loop
+        self._thread = thread
 
         # NOTE: We cannot define __qualname__ as a @property here.
         # Properties are designed to compute values dynamically when accessed on an *instance* (using 'self').
@@ -65,27 +65,39 @@ def __init__(
         # a string value immediately, not a descriptor that evaluates later.
         self.__qualname__ = f"{self.__class__.__qualname__}.{self.__async_tool._name}"
 
+    @property
+    def _async_tool(self) -> ToolboxTool:
+        return self._async_tool
+    
+    @property
+    def _loop(self) -> AbstractEventLoop:
+        return self._loop
+
+    @property
+    def _thread(self) -> Thread:
+        return self._thread
+
     @property
     def __name__(self) -> str:
-        return self.__async_tool.__name__
+        return self._async_tool.__name__
 
     @property
     def __doc__(self) -> Union[str, None]:  # type: ignore[override]
         # Standard Python object attributes like __doc__ are technically "writable".
         # But not defining a setter function makes this a read-only property.
         # Mypy flags this issue in the type checks.
-        return self.__async_tool.__doc__
+        return self._async_tool.__doc__
 
     @property
     def __signature__(self) -> Signature:
-        return self.__async_tool.__signature__
+        return self._async_tool.__signature__
 
     @property
     def __annotations__(self) -> dict[str, Any]:  # type: ignore[override]
         # Standard Python object attributes like __doc__ are technically "writable".
         # But not defining a setter function makes this a read-only property.
         # Mypy flags this issue in the type checks.
-        return self.__async_tool.__annotations__
+        return self._async_tool.__annotations__
 
     @property
     def _name(self) -> str:
@@ -129,8 +141,8 @@ def __call__(self, *args: Any, **kwargs: Any) -> str:
         Returns:
             The string result returned by the remote tool execution.
         """
-        coro = self.__async_tool(*args, **kwargs)
-        return asyncio.run_coroutine_threadsafe(coro, self.__loop).result()
+        coro = self._async_tool(*args, **kwargs)
+        return asyncio.run_coroutine_threadsafe(coro, self._loop).result()
 
     def add_auth_token_getters(
         self,
@@ -149,8 +161,8 @@ def add_auth_token_getters(
             getters registered.
         """
 
-        new_async_tool = self.__async_tool.add_auth_token_getters(auth_token_getters)
-        return ToolboxSyncTool(new_async_tool, self.__loop, self.__thread)
+        new_async_tool = self._async_tool.add_auth_token_getters(auth_token_getters)
+        return ToolboxSyncTool(new_async_tool, self._loop, self._thread)
 
     def bind_params(
         self, bound_params: Mapping[str, Union[Callable[[], Any], Any]]
@@ -166,8 +178,8 @@ def bind_params(
             A new ToolboxSyncTool instance with the specified parameters bound.
         """
 
-        new_async_tool = self.__async_tool.bind_params(bound_params)
-        return ToolboxSyncTool(new_async_tool, self.__loop, self.__thread)
+        new_async_tool = self._async_tool.bind_params(bound_params)
+        return ToolboxSyncTool(new_async_tool, self._loop, self._thread)
 
     def bind_param(
         self,