Implement CodeRabbit feedback

Author: max-svistunov

src/app/endpoints/rags.py

@@ -11,7 +11,7 @@
 from authentication.interface import AuthTuple
 from authorization.middleware import authorize
 from client import AsyncLlamaStackClientHolder
-from configuration import configuration
+from configuration import LogicError, configuration
 from models.config import Action
 from models.responses import (
     ForbiddenResponse,
@@ -119,7 +119,7 @@ def _resolve_rag_id_to_vector_db_id(rag_id: str) -> str:
     """
     try:
         byok_rags = configuration.configuration.byok_rag
-    except (AttributeError, RuntimeError):
+    except (AttributeError, RuntimeError, LogicError):
         return rag_id
 
     for brag in byok_rags:

tests/unit/app/endpoints/test_rags.py

@@ -292,7 +292,7 @@ def _make_byok_config(tmp_path: Any) -> AppConfig:
 
 @pytest.mark.asyncio
 async def test_rags_endpoint_returns_rag_ids_from_config(
-    mocker: MockerFixture, tmp_path: str
+    mocker: MockerFixture, tmp_path: Path
 ) -> None:
     """Test that /rags endpoint maps llama-stack IDs to user-facing rag_ids."""
     byok_config = _make_byok_config(str(tmp_path))
@@ -333,7 +333,7 @@ def __init__(self) -> None:
 
 @pytest.mark.asyncio
 async def test_rag_info_endpoint_accepts_rag_id_from_config(
-    mocker: MockerFixture, tmp_path: str
+    mocker: MockerFixture, tmp_path: Path
 ) -> None:
     """Test that /rags/{rag_id} accepts a user-facing rag_id and resolves it."""
     byok_config = _make_byok_config(str(tmp_path))
@@ -375,7 +375,7 @@ def __init__(self) -> None:
 
 
 def test_resolve_rag_id_to_vector_db_id_with_mapping(
-    mocker: MockerFixture, tmp_path: str
+    mocker: MockerFixture, tmp_path: Path
 ) -> None:
     """Test that _resolve_rag_id_to_vector_db_id maps rag_id to vector_db_id."""
     byok_config = _make_byok_config(str(tmp_path))
@@ -385,7 +385,7 @@ def test_resolve_rag_id_to_vector_db_id_with_mapping(
 
 
 def test_resolve_rag_id_to_vector_db_id_passthrough(
-    mocker: MockerFixture, tmp_path: str
+    mocker: MockerFixture, tmp_path: Path
 ) -> None:
     """Test that unmapped IDs are passed through unchanged."""
     byok_config = _make_byok_config(str(tmp_path))