chore: Remove unused strict flag + fix default values + fix docstring

Author: anubhav756

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

@@ -56,7 +56,6 @@ def __init__(
         async def create_client():
             return ToolboxClient(url, client_headers=client_headers)
 
-        # Ignoring type since we're already checking the existence of a loop above.
         self.__async_client = asyncio.run_coroutine_threadsafe(
             create_client(), self.__class__.__loop
         ).result()

packages/toolbox-langchain/src/toolbox_langchain/async_client.py

@@ -103,7 +103,7 @@ async def aload_toolset(
         auth_tokens: Optional[dict[str, Callable[[], str]]] = None,
         auth_headers: Optional[dict[str, Callable[[], str]]] = None,
         bound_params: dict[str, Union[Any, Callable[[], Any]]] = {},
-        strict: bool = True,
+        strict: bool = False,
     ) -> list[AsyncToolboxTool]:
         """
         Loads tools from the Toolbox service, optionally filtered by toolset
@@ -118,9 +118,11 @@ async def aload_toolset(
             auth_headers: Deprecated. Use `auth_token_getters` instead.
             bound_params: An optional mapping of parameter names to their
                 bound values.
-            strict: If True, raises a ValueError if any of the given bound
-                parameters are missing from the schema or require
-                authentication. If False, only issues a warning.
+            strict: If True, raises an error if *any* loaded tool instance fails
+                to utilize at least one provided parameter or auth token (if any
+                provided). If False (default), raises an error only if a
+                user-provided parameter or auth token cannot be applied to *any*
+                loaded tool across the set.
 
         Returns:
             A list of all tools loaded from the Toolbox.
@@ -170,7 +172,6 @@ def load_tool(
         auth_tokens: Optional[dict[str, Callable[[], str]]] = None,
         auth_headers: Optional[dict[str, Callable[[], str]]] = None,
         bound_params: dict[str, Union[Any, Callable[[], Any]]] = {},
-        strict: bool = True,
     ) -> AsyncToolboxTool:
         raise NotImplementedError("Synchronous methods not supported by async client.")
 
@@ -181,6 +182,6 @@ def load_toolset(
         auth_tokens: Optional[dict[str, Callable[[], str]]] = None,
         auth_headers: Optional[dict[str, Callable[[], str]]] = None,
         bound_params: dict[str, Union[Any, Callable[[], Any]]] = {},
-        strict: bool = True,
+        strict: bool = False,
     ) -> list[AsyncToolboxTool]:
         raise NotImplementedError("Synchronous methods not supported by async client.")

packages/toolbox-langchain/src/toolbox_langchain/tools.py

@@ -63,7 +63,7 @@ async def _arun(self, **kwargs: Any) -> dict[str, Any]:
 
 
     def add_auth_token_getters(
-        self, auth_token_getters: dict[str, Callable[[], str]], strict: bool = True
+        self, auth_token_getters: dict[str, Callable[[], str]]
     ) -> "ToolboxTool":
         """
         Registers functions to retrieve ID tokens for the corresponding
@@ -131,7 +131,6 @@ def bind_param(
         self,
         param_name: str,
         param_value: Union[Any, Callable[[], Any]],
-        strict: bool = True,
     ) -> "ToolboxTool":
         """
         Registers a value or a function to retrieve the value for a given bound