chore(toolbox-core): Standardize on ValueError for improved error consistency

This PR updates instances where generic `Exception` objects were thrown in `toolbox-core` to use `ValueError` instead.

This aligns with the prevalent use of `ValueError` for argument-related issues elsewhere in the package, leading to more specific, predictable, and robust error handling for consumers of `toolbox-core`.

Author: anubhav756

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

@@ -176,7 +176,7 @@ async def load_tool(
         # parse the provided definition to a tool
         if name not in manifest.tools:
             # TODO: Better exception
-            raise Exception(f"Tool '{name}' not found!")
+            raise ValueError(f"Tool '{name}' not found!")
         tool, used_auth_keys, used_bound_keys = self.__parse_tool(
             name,
             manifest.tools[name],

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

@@ -184,7 +184,7 @@ async def __call__(self, *args: Any, **kwargs: Any) -> str:
             req_auth_services = set()
             for s in self.__required_authn_params.values():
                 req_auth_services.update(s)
-            raise Exception(
+            raise ValueError(
                 f"One or more of the following authn services are required to invoke this tool"
                 f": {','.join(req_auth_services)}"
             )
@@ -301,7 +301,7 @@ def bind_params(
                 )
 
             if name not in param_names:
-                raise Exception(f"unable to bind parameters: no parameter named {name}")
+                raise ValueError(f"unable to bind parameters: no parameter named {name}")
 
         new_params = []
         for p in self.__params:

packages/toolbox-core/tests/test_client.py

@@ -266,7 +266,7 @@ async def test_load_tool_not_found_in_manifest(aioresponses, test_tool_str):
     )
 
     async with ToolboxClient(TEST_BASE_URL) as client:
-        with pytest.raises(Exception, match=f"Tool '{REQUESTED_TOOL_NAME}' not found!"):
+        with pytest.raises(ValueError, match=f"Tool '{REQUESTED_TOOL_NAME}' not found!"):
             await client.load_tool(REQUESTED_TOOL_NAME)
 
     aioresponses.assert_called_once_with(
@@ -474,7 +474,7 @@ async def test_bind_params_fail(self, tool_name, client):
         assert len(tool.__signature__.parameters) == 2
         assert "argA" in tool.__signature__.parameters
 
-        with pytest.raises(Exception) as e:
+        with pytest.raises(ValueError) as e:
             tool.bind_params({"argC": lambda: 5})
         assert "unable to bind parameters: no parameter named argC" in str(e.value)
 
@@ -605,7 +605,7 @@ async def test_bind_param_fail(self, tool_name, client):
         assert len(tool.__signature__.parameters) == 2
         assert "argA" in tool.__signature__.parameters
 
-        with pytest.raises(Exception) as e:
+        with pytest.raises(ValueError) as e:
             tool.bind_param("argC", lambda: 5)
         assert "unable to bind parameters: no parameter named argC" in str(e.value)
 

packages/toolbox-core/tests/test_e2e.py

@@ -188,7 +188,7 @@ async def test_run_tool_param_auth_no_auth(self, toolbox: ToolboxClient):
         """Tests running a tool with a param requiring auth, without auth."""
         tool = await toolbox.load_tool("get-row-by-email-auth")
         with pytest.raises(
-            Exception,
+            ValueError,
             match="One or more of the following authn services are required to invoke this tool: my-test-auth",
         ):
             await tool()

packages/toolbox-core/tests/test_sync_e2e.py

@@ -156,7 +156,7 @@ def test_run_tool_param_auth_no_auth(self, toolbox: ToolboxSyncClient):
         """Tests running a tool with a param requiring auth, without auth."""
         tool = toolbox.load_tool("get-row-by-email-auth")
         with pytest.raises(
-            Exception,
+            ValueError,
             match="One or more of the following authn services are required to invoke this tool: my-test-auth",
         ):
             tool()