refactor(context): 引入 MicroCompactConfig 聚合结构,收敛 Builder 构造函数链#447
refactor(context): 引入 MicroCompactConfig 聚合结构,收敛 Builder 构造函数链#447Yumiue wants to merge 2 commits into1024XEngineer:mainfrom
Conversation
- 新增 MicroCompactConfig 结构体,将 MicroCompactPolicySource、 MicroCompactSummarizerSource、MicroCompactPinChecker 三个接口 打包为单一配置对象,简化 Builder 构造参数 - 新增 NewConfiguredBuilder 作为统一构造入口,支持可变 SectionSource - newPromptSources 改为接受 ...SectionSource 可变参数,nil 自动跳过 - 旧构造函数保留并标记 Deprecated,委托到新入口 - 更新 bootstrap.go 和 runtime.go 使用新构造 API - 补充 TestNewConfiguredBuilder 覆盖 5 个子场景 关联: 1024XEngineer#416
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
| @@ -22,55 +20,78 @@ func newDefaultBuilder( | |||
| summarizers MicroCompactSummarizerSource, | |||
There was a problem hiding this comment.
newDefaultBuilder appears to be unused after introducing NewConfiguredBuilder (no call sites found). Keeping an unreferenced constructor increases maintenance surface and can confuse future call-path tracing. Consider removing it or routing one public constructor through it to keep a single assembly path.
|
/simplify |
Result已完成你在 已执行内容:
提交与推送:
未执行项:
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: Yumiue <188874804+Yumiue@users.noreply.github.com>
|
@xgopilot 审查性review,先确认有没有完成目标任务,再看看有没有引入新问题或是行为上的退化 |
Review Findings
目标完成度检查
结论
|
目标
回应 #416 中关于「context 模块微压缩策略配置复杂」的痛点,引入 MicroCompactConfig 聚合结构,收敛 DefaultBuilder 的 5 个构造函数为统一入口。
改动内容
关于 #416 其余提案的评估
关联: #416