Skip to content

fix(ai): change default provider to intern and fix error handling#253

Open
LynPtl wants to merge 2 commits intoInvolutionHell:mainfrom
LynPtl:fix/ai-assistant-provider
Open

fix(ai): change default provider to intern and fix error handling#253
LynPtl wants to merge 2 commits intoInvolutionHell:mainfrom
LynPtl:fix/ai-assistant-provider

Conversation

@LynPtl
Copy link
Contributor

@LynPtl LynPtl commented Feb 14, 2026

1.默认 AI 提供商更改为 Intern (书生)。
2.修复了切换模型后旧错误信息残留的问题。
3.修复了 Token 读取的竞态条件问题。
4.代码注释已更新为英文。

@vercel
Copy link

vercel bot commented Feb 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
website-preview Ready Ready Preview, Comment Feb 14, 2026 3:32pm

@vercel
Copy link

vercel bot commented Feb 14, 2026

@LynPtl is attempting to deploy a commit to the longsizhuo's projects Team on Vercel.

A member of the Team first needs to authorize it.

@LynPtl
Copy link
Contributor Author

LynPtl commented Feb 14, 2026

额githubactions CI报错,可能应该用回归标准写法

@longsizhuo
Copy link
Member

我觉得代码注释是中文的比较好?如果可以的话双语更欢迎!毕竟大多数开发者还是中文母语的

@LynPtl
Copy link
Contributor Author

LynPtl commented Feb 14, 2026

我觉得代码注释是中文的比较好?如果可以的话双语更欢迎!毕竟大多数开发者还是中文母语的

收到关于注释的建议。
我越改越糟了...感觉还是要用Ref,我再研究一下。

@longsizhuo
Copy link
Member

有问题很正常,这里迭代很多次了,慢慢来。随时叫我和@Crokily

@Crokily
Copy link
Contributor

Crokily commented Feb 14, 2026

辛苦啦!

默认模型确认为书生

文件内有个 useRef 引入了但没有使用,没用的话需要去掉

如果选了OpenAI但是没有填入token,我和他对话报错。然后我选书生,会依旧报错,需要刷新网页:
但似乎这个问题依旧存在:
image
复现:选openapi,不配置 key,保存并刷新,进来后切换成书生,发消息,失败,要刷新才可用

原因应该是:这个问题的本质可能是 useChat 这个 Hook的实例,对话报错后创建了实例(这个问题在没对话报错时不会出现可能是因为没对话则没有创建实例),但更换了 key 以后实例没有更新,依旧读取旧配置,用 useEffect 的写法可以把错误信息刷掉,不过内部的问题没有解决。

解决思路可以试着看看:让 transport 保持单实例(在 DocsAssistant.tsx (line 37) 定义),body() 从 useRef 读取最新 provider/key(这样切换后立刻生效)。
还有一种更暴力省事的:useChat 这个 hook 默认只在 id 或 chat 变了才会“重建聊天实例”,所以给 useChat 的 id 加上 provider(例如 id: assistant-${provider}),让切 provider 时强制重建 chat。代价是切换会清掉当前对话状态。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants