Skip to content

简化 context builder 的防御性兼容逻辑#710

Open
fennoai[bot] wants to merge 1 commit into
mainfrom
fennoai/issue-709-1780675838
Open

简化 context builder 的防御性兼容逻辑#710
fennoai[bot] wants to merge 1 commit into
mainfrom
fennoai/issue-709-1780675838

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented Jun 5, 2026

变更内容

  • 删除 DefaultBuilder 中旧版 promptSources 兼容字段和回退路径。
  • 移除 NewConfiguredBuilder 对 nil section source 的跳过逻辑,改为依赖调用链传入有效 source。
  • 调整相关测试,覆盖当前 stable/dynamic source 构造路径。

验证

  • git diff --check

未运行 go test ./...:当前执行环境中 go 不在 PATH。

Generated with [FennoAI](https://github.com/apps/fennoai)
Co-authored-by: minorcell <120795714+minorcell@users.noreply.github.com>
@fennoai fennoai Bot mentioned this pull request Jun 5, 2026
Copy link
Copy Markdown
Author

@fennoai fennoai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

未发现需要反馈的问题。

已检查本次变更和相关调用点,并执行了 git diff --check origin/main...HEAD。未能运行 go test ./internal/context:当前环境中 go 不在 PATH。

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 5, 2026

Codecov Report

❌ Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/context/builder.go 77.77% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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.

0 participants