fix(vllm): guard discovery URL validation

Author: waleedlatif1

apps/sim/app/api/providers/vllm/models/route.test.ts

@@ -62,4 +62,13 @@ describe('vLLM models route', () => {
       })
     )
   })
+
+  it('returns an empty model list when the configured base URL is unsupported', async () => {
+    setEnv({ VLLM_BASE_URL: 'http://localhost:1234?token=value' })
+
+    const response = await GET(request())
+
+    await expect(response.json()).resolves.toEqual({ models: [] })
+    expect(mockFetch).not.toHaveBeenCalled()
+  })
 })

apps/sim/app/api/providers/vllm/models/route.ts

@@ -27,9 +27,9 @@ export const GET = withRouteHandler(async (_request: NextRequest) => {
     logger.info('VLLM_BASE_URL not configured')
     return NextResponse.json({ models: [] })
   }
-  const apiBaseUrl = getOpenAICompatibleApiBaseUrl(baseUrl)
 
   try {
+    const apiBaseUrl = getOpenAICompatibleApiBaseUrl(baseUrl)
     logger.info('Fetching vLLM models', {
       baseUrl,
     })

apps/sim/providers/vllm/index.ts

@@ -59,9 +59,9 @@ export const vllmProvider: ProviderConfig = {
       logger.info('VLLM_BASE_URL not configured, skipping initialization')
       return
     }
-    const apiBaseUrl = getOpenAICompatibleApiBaseUrl(baseUrl)
 
     try {
+      const apiBaseUrl = getOpenAICompatibleApiBaseUrl(baseUrl)
       const headers: Record<string, string> = {
         'Content-Type': 'application/json',
       }