diff --git a/pyproject.toml b/pyproject.toml index e7a9f51ab3aa9815bfba06140e39b36687cf87c5..376cdc1fdfa0bc48ca0eedeaa9f7c1c5f23ca882 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -1,6 +1,6 @@ [tool.poetry] name = "semantic-router" -version = "0.0.47" +version = "0.0.48" description = "Super fast semantic router for AI decision making" authors = [ "James Briggs <james@aurelio.ai>", diff --git a/semantic_router/layer.py b/semantic_router/layer.py index 6b9ee1d212aadbb35bdc338f205722131a3a2db7..c87c79358b291caaf5fc7d24eafde67830fcbae7 100644 --- a/semantic_router/layer.py +++ b/semantic_router/layer.py @@ -433,9 +433,9 @@ class RouteLayer: :type str: """ if route_name not in [route.name for route in self.routes]: - err_msg = f"Route `{route_name}` not found" - logger.error(err_msg) - raise ValueError(err_msg) + err_msg = f"Route `{route_name}` not found in RouteLayer" + logger.warning(err_msg) + self.index.delete(route_name=route_name) else: self.routes = [route for route in self.routes if route.name != route_name] self.index.delete(route_name=route_name) diff --git a/tests/unit/test_layer.py b/tests/unit/test_layer.py index c6c36b7ee21d6769170721404e2e3ea619f16991..4566401d0e47c19a03fb03210e775c738dfb894b 100644 --- a/tests/unit/test_layer.py +++ b/tests/unit/test_layer.py @@ -244,11 +244,8 @@ class TestRouteLayer: ) # Attempt to remove a route that does not exist non_existent_route = "non-existent-route" - with pytest.raises(ValueError) as excinfo: - route_layer.delete(non_existent_route) - assert ( - str(excinfo.value) == f"Route `{non_existent_route}` not found" - ), "Attempting to remove a non-existent route should raise a ValueError." + route_layer.delete(non_existent_route) + # we should see warning in logs only (ie no errors) def test_add_multiple_routes(self, openai_encoder, routes, index_cls): route_layer = RouteLayer(encoder=openai_encoder, index=index_cls())