Repository navigation
fix: preserve reasoning_content for every assistant message (incl. cancelled/partial turns) - #68
Open
dimassfeb-09 wants to merge 2 commits into
Conversation
dimassfeb-09
force-pushed
the
fix/reasoning-content-backfill-all-assistant
branch
from
October 20, 2026 04:46
a1422ad to
0d63913
Compare
dimassfeb-09
changed the title
qingdeng888
added a commit
to qingdeng888/commandcode-proxy
that referenced
this pull request
Oct 27, 2026
qingdeng888
added a commit
to qingdeng888/commandcode-proxy
that referenced
this pull request
Oct 27, 2026
把 PR MAXeaglet#53 与 MAXeaglet#46 的开关真正接进本 fork 的配置体系，并补上验证与文档： - config.json.example 补上 useProviderModels / useProviderModelsWithPlanFilter / webSearch 三个字段（模板里此前完全没有它们，用户从模板起步就看不到这两个新开关）。 - 后台「设置」页展示这两个开关的当前状态（describeConfig 早就返回了， 前端此前没有渲染，等于新开关在面板上不可见）。 - docs/admin-api.md 的 GET /admin/api/config 示例补上两个字段，并修正两处 还停在 1.1.0 的 version 示例（实际已是 1.3.0）。 - docs/design-notes.md 新增 5.1：记录这次从上游挑着合并的完整取舍表 —— 哪些合、哪些不合、为什么，以及两处与既有实现的语义交叉 （MAXeaglet#68 的 reasoning_effort 扣留 vs 占位符回填；MAXeaglet#46 的搜索循环 vs issue MAXeaglet#20 的对象树回收）。 - test/admin.test.mjs 新增 3 条：两个新开关走通配置分层（可读 / 可热改 / 可落盘且只写改动字段）、 CC_WEB_SEARCH 锁定后后台改不动、CC_USE_PROVIDER_MODELS_WITH_PLAN_FILTER 的来源标记为 env。 全量 238 passed。
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.
Summary
reasoningblocks for assistant history messages that lackreasoning_content. Fabricating reasoning changes semantics and can be rejected by upstream.reasoning_effortas an effective fallback instead of fabricating reasoning.reasoningpart is not duplicated only when noreasoning_contentfield was present.Testing
node --check proxy.mjspasses.Notes
config.jsonis intentionally excluded.