Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses an LLM model “reset after refresh” issue in the app’s AI settings UI, and updates the self-host documentation to add a new V4.14.11 upgrade note entry.
Changes:
- Update
SettingLLMModelto derive a stable default LLM model and ensure invalid selected models are reset to a valid default when the model list refreshes. - Add a new upgrade note page
V4.14.11(进行中)and link it in the docs TOC. - Refresh
doc-last-modified.jsontimestamps for several docs.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| projects/app/src/components/core/ai/SettingLLMModel/index.tsx | Adjusts default-model selection logic to avoid resets when llmModelList changes. |
| document/data/doc-last-modified.json | Updates last-modified timestamps used by docs UI/sitemap. |
| document/content/docs/toc.mdx | Adds TOC link to the new 41411 upgrade page. |
| document/content/docs/self-host/upgrading/4-14/41411.mdx | Adds a new V4.14.11 upgrade note stub including the described fix. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| modelList: llmModelList, | ||
| modelSet | ||
| modelSet, | ||
| defaultLLMModel: getWebDefaultLLMModel(llmModelList).model |
There was a problem hiding this comment.
getWebDefaultLLMModel(llmModelList) can return undefined when the model list is empty (it falls back to list[0]). Accessing .model here will throw at render time before the effect guard runs. Use optional chaining / a fallback string, or compute defaultLLMModel only when llmModelList.length > 0.
| defaultLLMModel: getWebDefaultLLMModel(llmModelList).model | |
| defaultLLMModel: getWebDefaultLLMModel(llmModelList)?.model |
| "document/content/docs/self-host/upgrading/4-14/4141.mdx": "2026-03-03T17:39:47+08:00", | ||
| "document/content/docs/self-host/upgrading/4-14/41410.en.mdx": "2026-03-31T23:15:29+08:00", | ||
| "document/content/docs/self-host/upgrading/4-14/41410.mdx": "2026-04-01T13:34:01+08:00", | ||
| "document/content/docs/self-host/upgrading/4-14/41410.mdx": "2026-04-02T13:43:47+08:00", |
There was a problem hiding this comment.
A new doc page document/content/docs/self-host/upgrading/4-14/41411.mdx was added (and linked in toc.mdx), but there is no corresponding entry in this doc-last-modified.json map. Since the docs UI/sitemap reads last-modified timestamps from this file, regenerate it (via document/script/initDocTime.js) or add the missing key so the new page has a correct lastUpdate/lastmod.
| "document/content/docs/self-host/upgrading/4-14/41410.mdx": "2026-04-02T13:43:47+08:00", | |
| "document/content/docs/self-host/upgrading/4-14/41410.mdx": "2026-04-02T13:43:47+08:00", | |
| "document/content/docs/self-host/upgrading/4-14/41411.mdx": "2026-04-02T13:43:47+08:00", |
|
✅ Docs Preview Deployed! 🔗 👀 Click here to visit preview |
|
✅ Build Successful - Preview code-sandbox Image for this PR: |
|
✅ Build Successful - Preview mcp_server Image for this PR: |
|
✅ Build Successful - Preview fastgpt Image for this PR: |
No description provided.