From 93056bb276d1d37e151fc9594ce04e4e2abc22e7 Mon Sep 17 00:00:00 2001 From: Charlie Bailey Date: Tue, 9 Jun 2026 11:17:20 -0700 Subject: [PATCH] fixed path for bundled avatar URIs. --- multi_llm_chatbot_backend/app/config.py | 3 +-- .../app/tests/unit/test_avatar_resolution.py | 10 ++++------ 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/multi_llm_chatbot_backend/app/config.py b/multi_llm_chatbot_backend/app/config.py index c8154837..2331a235 100644 --- a/multi_llm_chatbot_backend/app/config.py +++ b/multi_llm_chatbot_backend/app/config.py @@ -154,8 +154,7 @@ def _resolve_image(self) -> str: self.avatar, self.id, ) return f"icon://{self.icon}" - base = os.getenv("REACT_APP_API_URL", "http://localhost:8000").rstrip("/") - return f"{base}/api/avatars/bundled/{self.avatar}" + return f"/api/avatars/bundled/{self.avatar}" def to_frontend_config(self) -> dict: return { diff --git a/multi_llm_chatbot_backend/app/tests/unit/test_avatar_resolution.py b/multi_llm_chatbot_backend/app/tests/unit/test_avatar_resolution.py index 43da8a95..22937fe2 100644 --- a/multi_llm_chatbot_backend/app/tests/unit/test_avatar_resolution.py +++ b/multi_llm_chatbot_backend/app/tests/unit/test_avatar_resolution.py @@ -1,6 +1,5 @@ import unittest import tempfile -import os from unittest.mock import patch, MagicMock from pathlib import Path @@ -113,11 +112,10 @@ def test_bundled_avatar_exists_returns_path(self, mock_get_path): persona = PersonaItemConfig( id="test", name="Test", icon="Brain", avatar="advisor1.png", ) - with patch.dict(os.environ, {"REACT_APP_API_URL": "http://localhost:8000"}): - self.assertEqual( - persona._resolve_image(), - "http://localhost:8000/api/avatars/bundled/advisor1.png", - ) + self.assertEqual( + persona._resolve_image(), + "/api/avatars/bundled/advisor1.png", + ) @patch("app.utils.avatar_helpers.get_bundled_avatar_path") def test_bundled_avatar_missing_falls_back_to_icon(self, mock_get_path):