chore: Remove the decline_responses_websockets translation after a deprecation release

未关闭
#1,625 0 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

维护者通常 1 天内回复

@Alan-Cha 已经在做这个了。

开始于 2026年11月3日。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
55/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
活跃
技术栈
go
领域
backend

调研方向

First confirm that a release has shipped the warning and that the project is ready for a breaking change. Then inspect core/config/config.go and core/config/validate.go, and review the history of #1467 for the earlier refusal. Update TestLoad_DeprecatedDeclineKeyStillApplies into a refusal test and remove the old-key references from the three named docs; done means the old key is refused with a message naming its replacement mode and the release notes mark the change as breaking.

由索引模型根据 Issue 内容生成。

描述

What

v0.9.4 shipped listener.decline_responses_websockets, a boolean. #1467 replaces it with listener.responses_websockets: parse | decline | forward. To keep v0.9.4 configs working, #1467 still reads the old key when the new one is unset (true → decline, false → forward), logs a WARN naming the new key, and refuses a config that sets both.

This issue tracks removing that translation.

When

Not before both of these:

  • At least one release has shipped the warning. A user who set the old key should have had a release that told them to change it before one that refuses it.
  • A release the project is willing to label a breaking change. Before 1.0, semver allows a breaking change in any release, and this repo has done it: #1429 shipped in v0.9.2, a patch release, listed under "Breaking changes" in its notes. So this does not have to wait for 1.0, but it needs a deliberate release-notes entry, not a silent removal.

What removal means

Do not delete the field outright. If the field were simply deleted, the YAML would ignore the key, and a config that set decline_responses_websockets: false would quietly get the default, parse, with no word said. Replace the translation with a startup refusal that names the mode the old value maps to. That refusal is what #1467 did before v0.9.4 shipped the key, and it is in that PR's history.

  • core/config/config.go: DeclineResponsesWebSockets and the fallback in ResponsesWebSocketsMode.
  • core/config/validate.go: the WARN and the both-set refusal become a single refusal.
  • TestLoad_DeprecatedDeclineKeyStillApplies becomes a refusal test.
  • Docs: docs/agents/codex.md, docs/framework-architecture.md and CLAUDE.md describe the old key; drop those lines.

PR title prefix: Breaking change:, so the release notes say so.

主要语言
Go
星标
23
派生
44
平均合并
16 小时 32 分钟
30 天内合并 PR
227

环境准备

  • 没有 Dockerfile 或 Docker Compose 文件
  • 没有 Pull Request 模板
  • 阅读贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

rossoctl/cortex 的其他 Issue

查看 rossoctl/cortex 的全部 Issue

相似的 Issue

更多 Go Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。