unbridled-41 opened a new pull request, #6013:
URL: https://github.com/apache/rocketmq-dashboard/pull/6013

   ### Which Issue(s) This PR Fixes
   Fixes #6012
   ### Problem / Evidence
   `web/src/pages/studio/llmModelOptions.ts` vs 
`LlmConfigService.PROVIDER_MODELS` (the source of truth for `GET /llm/models` 
and the home-page picker):
   ```
   LlmSettings: mirrors the server curated model list per provider
   AssertionError: expected [ 'qwen3.8-max', ... ] (7) to deeply equal [...] 
(12)
   ```
   ### Root cause / Fix
   Two sources of truth for one fact; the frontend mirror was not updated when 
the server's curated set grew (and gained gemma2 on ollama in the other 
direction). Align the fallback with the server list and pin both entries in the 
existing test.
   ### Priority and scoring
   **PRIORITY 52** - impact 18/40 (five models unreachable in the settings 
dialog), blast radius 10/20 (the AI settings dialog and the composer's model 
list), reproducibility 18/20 (pinned by the new test), maintenance value 6/20. 
**FIX_CONFIDENCE 88**.
   ### Tests
   `cd web && npx vitest run src/pages/studio/__tests__/LlmSettings.test.ts` -> 
`Tests 5 passed`; the new case fails before the change and passes after it; 
`npx eslint` and `npx tsc -b` are clean.
   ### Risk
   The fallback list is longer for tongyi and one entry shorter for ollama; the 
endpoint's answer still wins when it succeeds.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to