chore: delint

Author: anubhav756

packages/toolbox-adk/src/toolbox_adk/toolset.py

@@ -18,12 +18,12 @@
 from google.adk.tools.base_tool import BaseTool
 from google.adk.tools.base_toolset import BaseToolset
 from google.adk.tools.tool_context import ToolContext
+from toolbox_core.utils import validate_unused_requirements
 from typing_extensions import override
 
 from .client import ToolboxClient
 from .credentials import CredentialConfig
 from .tool import ToolboxTool
-from toolbox_core.utils import validate_unused_requirements
 
 
 class ToolboxToolset(BaseToolset):
@@ -131,7 +131,9 @@ async def get_tools(
             # Generate intuitive name for the error string if a specific toolset wasn't used
             validation_name = self.__toolset_name
             if not validation_name:
-                validation_name = ", ".join(self.__tool_names) if self.__tool_names else "default"
+                validation_name = (
+                    ", ".join(self.__tool_names) if self.__tool_names else "default"
+                )
 
             validate_unused_requirements(
                 provided_auth_keys=set(self.__auth_token_getters.keys()),

packages/toolbox-adk/tests/integration/test_integration.py

@@ -572,10 +572,13 @@ async def test_run_multiple_tools_partial_auth_usage(self, auth_token2: str):
         """Tests that when some tokens are used and some aren't across diverse tools, only the truly unused tokens appear in the error."""
         toolset = ToolboxToolset(
             server_url="http://localhost:5000",
-            tool_names=["get-row-by-id-auth", "search-rows"], # first requires 'my-test-auth', second requires nothing
+            tool_names=[
+                "get-row-by-id-auth",
+                "search-rows",
+            ],  # first requires 'my-test-auth', second requires nothing
             auth_token_getters={
-                "my-test-auth": lambda: auth_token2, 
-                "extra-token": lambda: "fake"
+                "my-test-auth": lambda: auth_token2,
+                "extra-token": lambda: "fake",
             },
             credentials=CredentialStrategy.toolbox_identity(),
         )

packages/toolbox-adk/tests/unit/test_toolset.py

@@ -84,9 +84,21 @@ async def test_get_tools_with_auth_token_getters(self, mock_client_cls):
             ({}, [], True),  # No requirements, token is completely unused
             ({"param1": ["service"]}, [], False),  # authn natively consumes it
             ({}, ["service"], False),  # authz natively consumes it
-            ({"param1": ["other"]}, ["service"], False),  # unused by authn, but authz consumes it
-            ({"param1": ["service"]}, ["other"], False),  # authn consumes it, authz doesn't
-            ({"param1": ["other"]}, ["other"], True),  # Requirements exist, but token is unused by both
+            (
+                {"param1": ["other"]},
+                ["service"],
+                False,
+            ),  # unused by authn, but authz consumes it
+            (
+                {"param1": ["service"]},
+                ["other"],
+                False,
+            ),  # authn consumes it, authz doesn't
+            (
+                {"param1": ["other"]},
+                ["other"],
+                True,
+            ),  # Requirements exist, but token is unused by both
         ],
     )
     async def test_get_tools_auth_validation(
@@ -107,7 +119,8 @@ async def test_get_tools_auth_validation(
 
         if should_raise:
             with pytest.raises(
-                ValueError, match="unused auth tokens could not be applied to any tool: service"
+                ValueError,
+                match="unused auth tokens could not be applied to any tool: service",
             ):
                 await toolset.get_tools()
         else:

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

@@ -201,7 +201,9 @@ def validate_unused_requirements(
                     f"unused bound parameters: {', '.join(unused_bound)}"
                 )
 
-        final_target_type = target_type if target_type else ("toolset" if is_toolset else "tool")
+        final_target_type = (
+            target_type if target_type else ("toolset" if is_toolset else "tool")
+        )
         raise ValueError(
             f"Validation failed for {final_target_type} '{name}': {'; '.join(error_messages)}."
         )