Repository navigation
Replies: 5 comments
|
你这条诉求成立,而且我确认了变更与文档出自同一个提交 —— 所以这不是"漏写文档",而是目录归属问题。但你有一处硬错要先改掉,否则维护者会拿它否掉整条报告。 1. ✅ 核心事实:变更与文档同属一个提交
⇒ 指南不是忘了写,而是被放进了
2.
|
|
感谢核对,尤其是第 2 节那处 —— 你是对的,我那句话写错了,我先更正,再把你标为「无法核实」的两点补上可复核的取数方式。 0. 更正:tag
|
|
这次轮到我更正两处,而且都是我的错。另外你补的取数方式让我把一个原本标为「无法核实」的点真正核实了 —— 结论对你有利。 1.
|
| 关键词 | 命中 |
|---|---|
webRuntime |
0 |
webStartup |
0 |
web-runtime |
0 |
trustedHosts |
0 |
⇒ 「alpha.2 的 release notes 完全没有提到这个破坏性变更」—— 这一条现在是已验证的 TRUE,不再是我上一条说的"无法核实"。
建议你把它写进报告,并附上取数方法(这比任何叙述都硬):
GET https://api.github.com/repos/deepseek-ai/deepseek-harness/releases/tags/dsh-v0.2.1-alpha.2
→ body 中 grep "webRuntime" / "webStartup" / "web-runtime" / "trustedHosts" 均为 0
我上一条之所以说"不在 git 里",是因为 release body 确实不在仓库对象里 —— 它存在 GitHub Release 一侧,用 Releases API 取,而不是 git。这是我说错的地方,取数路径你找对了。
4. ✅ 你对 tag 与 commit 的对应也核实无误
git/ref/tags/dsh-v0.2.1-alpha.2 → tag object 6851e496… → commit d743267388641bc76f17c45ce8b4c231aed1d32c
commits/master → d743267388641bc76f17c45ce8b4c231aed1d32c
⇒ dsh-v0.2.1-alpha.2 与 master 同点,这也正是我核对时用的基准(origin/master = d743267388)。所以我们两边所有 tag 级结论都建立在同一个 commit 上,不存在版本漂移的可能 —— 这一点值得写进报告,它能一次性挡掉"是不是版本不同"的质疑。
5. 关于你的诉求排序(我认可,并补一句)
你把三条按 首选/其次/备选 排好、并把"目录名 = 变更首次落地的 release"这一约定与它的代价写清 —— 这正是这条报告应该有的形态。
补一句让首选更容易被采纳:把那一行写成两个版本位的关系,而不是只写一个日期,例如:
该变更自
dsh-v0.2.1-alpha.2起生效(本指南位于v0.2.1-alpha.1/,因为改动首次落地于该 tag 之后、alpha.2 发布之前)。
这样维护者既看到生效版本,也看到"为什么它挂在 alpha.1 目录下"——把最可能被反问的那一点先答掉。
6. 边界
- 我重新核了:
git ls-tree -r dsh-v0.2.1-alpha.1 -- docs/upgrade-guide(18 行,已贴出明细)、以及上表那次 Releases API 调用(status 200、body长度9594、四个关键词 0 命中)。 - 我没有核实你引用的
@imccyu/@Dudu-0223那两条 release 条目属于哪个范围(你判断它们与本变更无关,我采信)—— 那需要逐条比对 PR,超出本次核对范围。 - 本文第 1、2 节是对我上一条的更正,请以本文为准。
|
三条更正互相抵消掉两条 —— 净值只剩「文件数 18」,而这一点你已经自己更正了。逐项对齐:
我按你的意见收紧了一处措辞你提醒「9594 疑似帖子字数」之后我重测了口径,发现自己那句表述松:原写「逐行匹配 你建议的「附上取数方法」我照做了 —— 现在这条不是叙述,是一次可重跑的调用 + 可核对的数字。 已写进正文的两处
需要的话我可以把最小 diff 直接贴上来三份文件各加 1 行,我可以在这里贴完整 diff 供你们内部挑一份合入。我不擅自开 PR —— 按 需要我现在就贴 diff 吗?(另: |
0.2.1-rc.2 fallout: plugins that read
|
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
0.2.1-alpha.2相对0.2.1-alpha.1删除了 Web 侧的公开服务webRuntime(连同web-runtime行的trustedHosts配置面)。这个破坏性变更有迁移正文、但没有自己的 upgrade-guide 条目,release notes 里也没有任何指向,于是第三方插件在升级后集体静默失效。下面是我在本机(Windows 10 19045 / DSH0.2.1-alpha.2+ nightly 桌面壳)做逐 tag 比对后的取证结论。1. 结论:变更本身有文档,只是被挂在了 alpha.1 名下
迁移指南正文存在且完整:
docs/upgrade-guide/v0.2.1-alpha.1/web-listener-trust-config/{guide.md,guide.zh.md,guide.i18n.yaml},description 逐字:正文给出 5 步迁移(
inject换webStartup/ 表达式换ctx.webStartup.trustedHosts并删lanAddresses/ connection overlay 重写 / wildcard host 换具体地址 /--dump-config预览)。docs/upgrade-guide/在 tagdsh-v0.2.1-alpha.2与 master 上都只有v0.1.7-rc.2/v0.2.0-rc.2/v0.2.1-alpha.1三个目录,没有v0.2.1-alpha.2;在 tagdsh-v0.2.1-alpha.1上目录存在(18 个文件、2 个版本位),但还没有v0.2.1-alpha.1/子目录 —— 即这份指南是 tag 之后(10-04 的 commit)才加进去的。(★ 更正:原文曾写作「该目录根本不存在」;2026-10-14 经 PerryLink 指出后按git/trees/…?recursive=1复核修正 —— alpha.1 tag = 18 blobs,alpha.2 tag = 78 blobs。)GET https://api.github.com/repos/deepseek-ai/deepseek-harness/releases/tags/dsh-v0.2.1-alpha.2→published_at2026-10-09T11:09:26Z,body长度 9594 字符)按词频逐项统计:webRuntime/webStartup/web-runtime/trustedHosts均 0 次;宽松正则webRuntime|webStartup|web-runtime|web-startup|upgrade-guide|迁移|trustedHost只命中 2 行,全部与本变更无关:迁移出现 3 次但只落在上述 2 行内(L52 一行两次);「体验优化」里关于 Web 监听只有一行「Web 支持绑定指定的 IPv4/IPv6 地址,并通过 --tls-cert 和 --tls-key 配置原生 HTTPS 监听」,与本变更无关。也就是说:维护者写了正确的文档,但按「目录名 = 变更首次落地的 release」的命名约定,读者要在 alpha.2 上遇到问题,得先去翻 alpha.1 的目录,而 release notes 不会告诉他要翻。
2. 归因:逐 tag 比对 + 唯一破坏性 commit
packages/bundle/web-app/src/index.ts中const WEB_RUNTIME_SERVICE = 'webRuntime'+ctx.provide(WEB_RUNTIME_SERVICE, runtime)resolveLanTrust(ctx.webServer.host, config.trustedHosts)(绑定地址隐含信任)docs/upgrade-guide/下是否有该变更的指南唯一破坏性 commit:
cb9a590c8"Support concrete Web listener bind addresses"(作者时间 2026-10-04T12:30:28Z,80 个文件)。命中点:packages/bundle/web-app/src/index.ts +57/−66、packages/bundle/web-app/cordis.patch.yml +14/−14(web-runtime行新增inject: [webStartup])、packages/client/connection/src/api-request-trust.ts +34/−13、packages/client/connection/src/index.ts +3/−5、packages/client/connection/src/rpc-host.ts +7/−2、packages/host/webserver/src/index.ts +101/−5packages/bundle/web-app/tests/trusted-hosts.spec.tsdocs/upgrade-guide/v0.2.1-alpha.1/web-listener-trust-config/guide.md +36/−0(+guide.zh.md/ +guide.i18n.yaml)即「删服务」和「写迁移指南」是同一次提交完成的 —— 文档不是漏写,而是目录名标在了 alpha.1 上,而变更实际第一次落地在 alpha.2。另外
webStartup并不是新名字:它在 rc.1/rc.2/alpha.1/alpha.2 四个 tag 上都是packages/bundle/web-app/src/startup.ts里同一行export const WEB_STARTUP_SERVICE = 'webStartup'。所以对外看起来像「改名」,实际是删掉一个聚合服务并改变信任语义,而 release notes 没有给出这个信号。3. 这个缺口在用户侧的放大形态(第三方插件集体静默不挂载)
插件
inject里的webRuntime永远等不到提供者 ⇒ 条目停在pending (waiting for service: webRuntime)⇒ 宿主只打一条dsh: warning: 1 entry did not activate,HTTP 侧表现为空体 405/404(serve-static 兜底),插件自己的 UI 仍在(客户端 half 独立加载),用户看到的是「侧边栏能开、文件树/git 全 405」。跨平台独立报告(都是同一个根因):
omdsh-dev/DSH-better-sidebar#906(macOS / Win11 / Linux 至少 4 台机器)、#931(要求给挂载门禁pnpm test:mount的 DSH pin 加 alpha 通道 lane)、修复 PR #905 / #932;另有@huanlin/dsh-plugin-better-sidebar-plugin-office0.2.0 受影响。4. 请求
docs/upgrade-guide/v0.2.1-alpha.2/(或在v0.2.1-alpha.1/web-listener-trust-config条目里注明「该变更自 alpha.2 起生效」),并在 alpha.2 release notes 的「其他变更」加一行 breaking 提示。仅此一条就能把上面整类故障的排查时间从「翻源码」压到「读一行」。English summary
On
0.2.1-alpha.2, the publicwebRuntimeservice (and theweb-runtimerow'strustedHostsconfig surface) was removed by commitcb9a590c8"Support concrete Web listener bind addresses" (2026-10-04, 80 files). The migration guide written by that same commit lives atdocs/upgrade-guide/v0.2.1-alpha.1/web-listener-trust-config/— butdocs/upgrade-guide/has onlyv0.1.7-rc.2,v0.2.0-rc.2andv0.2.1-alpha.1on both master and tagdsh-v0.2.1-alpha.2, so the release that actually broke third-party plugins has no entry of its own, and the alpha.2 release notes contain no line about it.webStartupwas never a new name (it is the sameWEB_STARTUP_SERVICEexport in rc.1/rc.2/alpha.1/alpha.2); the change deleted a service and altered trust semantics rather than renaming one. Effect on third parties: any plugin that injectswebRuntimestalls atpending (waiting for service: webRuntime)with onlydsh: warning: 1 entry did not activate, and its HTTP routes answer an empty-body 405 while its client half still renders. Independent reports:omdsh-dev/DSH-better-sidebar#906 (macOS/Win11/Linux), #931, PRs #905/#932. Request: publish the guide underdocs/upgrade-guide/v0.2.1-alpha.2/(or cross-reference it from the alpha.1 entry) and add one breaking-change line to the alpha.2 release notes.All reactions