Repository navigation
fix: 修复 Anthropic 思考档位与旧预算配置兼容 - #173
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题背景
已有模型绑定保存为 Token Budget 时,adaptive-only Claude 仍可能发出
thinking.type=enabled/budget_tokens,导致上游拒绝请求。旧预算反推 effort 还会让 Auto 变成 Max;手动档位在预算被maxOutputTokens截断后也可能丢失原始语义。修复内容
thinking: { type: 'adaptive' }与output_config.effort,不强制改写用户保存的自定义预算配置。thinkingLevel:手动 low/medium/high/higher/max 按合法能力转换,higher 对应 xhigh;输出预算截断不再覆盖原始选择。runtimeProtocol表达 provider 与 protocol override 的有效协议;保存前剥离,不落库、不参与配置同步。验证
基于同步后的 v2.6.0(936c5f9):
npm run build通过,包含 node/web/plugins 类型检查、plugin-runtime 检查及 main/preload/renderer 构建。git diff --check通过。已知验证限制
npm run test未通过:20 个失败文件、46 项失败测试、7206 项通过、86 项跳过,另有 2 条未处理错误。诊断涉及 Windows/POSIX 路径、权限、命令、zip 缺失及 worker/超时问题;未扩大到无关修复。同步 v2.6.0 后未重跑完整套件,不宣称全仓全绿。npm ci --no-audit --no-fund在 postinstall 的 node-pty 原生重编译阶段失败(MSB8040:缺少 Spectre 缓解库)。后续定向测试和构建通过,但桌面启动与原生终端尚未验证。参考