fix(llm): ensure base_url has protocol prefix for model info fetch when using LiteLLM

frontend/.husky/pre-commit

@@ -1,4 +1,4 @@
 cd frontend
 npm run check-unlocalized-strings
 npx lint-staged
-npm test
+npm test

frontend/__tests__/components/buttons/copy-to-clipboard.test.tsx

@@ -37,4 +37,4 @@ describe("CopyToClipboardButton", () => {
     const button = screen.getByTestId("copy-to-clipboard");
     expect(button).toHaveAttribute("aria-label", "BUTTON$COPIED");
   });
-});
+});

frontend/__tests__/components/features/conversation-panel/conversation-card.test.tsx

@@ -76,11 +76,11 @@ describe("ConversationCard", () => {
     const card = screen.getByTestId("conversation-card");
 
     within(card).getByText("Conversation 1");
-    
+
     // Just check that the card contains the expected text content
     expect(card).toHaveTextContent("Created");
     expect(card).toHaveTextContent("ago");
-    
+
     // Use a regex to match the time part since it might have whitespace
     const timeRegex = new RegExp(formatTimeDelta(new Date("2021-10-01T12:00:00Z")));
     expect(card).toHaveTextContent(timeRegex);

frontend/src/components/features/payment/setup-payment-modal.tsx

@@ -30,7 +30,7 @@ export function SetupPaymentModal() {
           </h1>
           <p>
             <Trans
-              i18nKey="BILLING$CLAIM_YOUR_50"
+              i18nKey={t(I18nKey.BILLING$CLAIM_YOUR_50)}
               components={{ b: <strong /> }}
             />
           </p>

frontend/src/i18n/translation.json

@@ -6089,4 +6089,4 @@
         "tr": "belgelendirme",
         "de": "Dokumentation"
     }
-}
+}

openhands/llm/llm.py

@@ -375,12 +375,17 @@ def init_model_info(self) -> None:
         if self.config.model.startswith('litellm_proxy/'):
             # IF we are using LiteLLM proxy, get model info from LiteLLM proxy
             # GET {base_url}/v1/model/info with litellm_model_id as path param
+            base_url = (self.config.base_url or '').strip()
+            if not base_url.startswith(('http://', 'https://')):
+                base_url = 'http://' + base_url
+
             response = httpx.get(
-                f'{self.config.base_url}/v1/model/info',
+                f'{base_url}/v1/model/info',
                 headers={
                     'Authorization': f'Bearer {self.config.api_key.get_secret_value() if self.config.api_key else None}'
                 },
             )
+
             resp_json = response.json()
             if 'data' not in resp_json:
                 logger.error(

openhands/server/conversation_manager/standalone_conversation_manager.py

@@ -420,22 +420,24 @@ async def _update_conversation_for_event(
         conversation_store = await self._get_conversation_store(user_id, github_user_id)
         conversation = await conversation_store.get_metadata(conversation_id)
         conversation.last_updated_at = datetime.now(timezone.utc)
-        
+
         # Update cost/token metrics if event has llm_metrics
         if event and hasattr(event, 'llm_metrics') and event.llm_metrics:
             metrics = event.llm_metrics
-            
+
             # Update accumulated cost
             if hasattr(metrics, 'accumulated_cost'):
                 conversation.accumulated_cost = metrics.accumulated_cost
-            
+
             # Update token usage
             if hasattr(metrics, 'accumulated_token_usage'):
                 token_usage = metrics.accumulated_token_usage
                 conversation.prompt_tokens = token_usage.prompt_tokens
                 conversation.completion_tokens = token_usage.completion_tokens
-                conversation.total_tokens = token_usage.prompt_tokens + token_usage.completion_tokens
-
+                conversation.total_tokens = (
+                    token_usage.prompt_tokens + token_usage.completion_tokens
+                )
+
         await conversation_store.save_metadata(conversation)
 
 

openhands/server/routes/git.py

@@ -1,7 +1,7 @@
 from fastapi import APIRouter, Depends, status
 from fastapi.responses import JSONResponse
 from pydantic import SecretStr
-from openhands.server.shared import server_config
+
 from openhands.integrations.github.github_service import GithubServiceImpl
 from openhands.integrations.provider import (
     PROVIDER_TOKEN_TYPE,
@@ -16,7 +16,7 @@
     User,
 )
 from openhands.server.auth import get_access_token, get_provider_tokens
-from openhands.server.types import AppMode
+from openhands.server.shared import server_config
 
 app = APIRouter(prefix='/api/user')
 
@@ -33,7 +33,9 @@ async def get_user_repositories(
         )
 
         try:
-            repos: list[Repository] = await client.get_repositories(sort, server_config.app_mode)
+            repos: list[Repository] = await client.get_repositories(
+                sort, server_config.app_mode
+            )
             return repos
 
         except AuthenticationError as e:

openhands/server/routes/manage_conversations.py

@@ -112,7 +112,9 @@ async def _create_new_conversation(
             title=conversation_title,
             user_id=user_id,
             github_user_id=None,
-            selected_repository=selected_repository.full_name if selected_repository else selected_repository,
+            selected_repository=selected_repository.full_name
+            if selected_repository
+            else selected_repository,
             selected_branch=selected_branch,
         )
     )

openhands/storage/conversation/conversation_validator.py

@@ -6,7 +6,9 @@
 class ConversationValidator:
     """Storage for conversation metadata. May or may not support multiple users depending on the environment."""
 
-    async def validate(self, conversation_id: str, cookies_str: str) -> tuple[None, None]:
+    async def validate(
+        self, conversation_id: str, cookies_str: str
+    ) -> tuple[None, None]:
         return None, None
 
 

tests/unit/test_no_protocol_url_test.py

@@ -0,0 +1,42 @@
+from unittest.mock import patch
+
+from openhands.core.config import LLMConfig
+from openhands.llm.llm import LLM
+
+
+def test_base_url_protocol_is_fixed_before_request():
+    """
+    Test that LLM automatically prepends a protocol to the base_url
+    if it is missing, to prevent httpx.UnsupportedProtocol error.
+
+    This avoids runtime crashes when users forget to include 'http://' or 'https://'
+    in the LLMConfig.base_url.
+
+    Steps:
+    1. Create an LLMConfig with a base_url missing the protocol.
+    2. Patch httpx.get to intercept the actual URL used in the request.
+    3. Initialize an LLM instance using this config.
+    4. Call init_model_info() to trigger the request.
+    5. Assert that the URL passed to httpx.get starts with 'http://' or 'https://'.
+    """
+
+    # Create config with base_url missing protocol
+    config = LLMConfig(
+        model='litellm_proxy/test-model', api_key='fake-key', base_url='api.example.com'
+    )
+
+    # Patch httpx.get to intercept the actual request
+    with patch('httpx.get') as mock_get:
+        mock_get.return_value.status_code = 200
+        mock_get.return_value.json.return_value = {'model': 'mock'}
+
+        llm = LLM(config=config)
+
+        # Trigger model info fetch
+        llm.init_model_info()
+
+        # Extract the requested URL and assert protocol is included
+        called_url = mock_get.call_args[0][0]
+        print('Final URL used:', called_url)
+
+        assert called_url.startswith('http://') or called_url.startswith('https://')