fix(plugin): 修复 GitHub 插件官方信任回填判据 - #3238
Conversation
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
| Filename | Overview |
|---|---|
| apps/desktop/src/main/plugin-market/service.ts | 将官方信任回填资格切换到 Host 权威信任投影,现有来源、审批、摘要匹配及安装锁门禁保持不变。 |
| apps/desktop/src/main/plugin-market/tests/service.test.ts | 为已有官方 receipt 场景补充完整的官方信任投影夹具,使测试符合新的完成判据。 |
Reviews (2): Last reviewed commit: "修复插件市场测试夹具信任类型" | Re-trigger Greptile
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
这条 PR 改到了插件市场的官方信任回填判据(看运行时 trust 信息,而不再看安装目录元数据),属于插件基座,需要维护者确认后再合。 讨论 issue:#3243 维护者请直接在本 PR 上 Approve;若要修改请 Request Changes。作者这边先不用再推,除非维护者点名要改。 |
|
@nothingyuancando 👋 这个 PR 目前与 请在本地 merge 最新的 |
Signed-off-by: nothingyuancando <1579035512@qq.com>
|
No reviewable files after applying ignore patterns. |
MagicLizi
left a comment
There was a problem hiding this comment.
对照当前 head 54d8be5fdd66186e461838ebb460a750341e043f 与 main,本 PR 的 diff 为空(0 files / 0 lines),但描述仍声称修复了 GitHub 插件官方信任回填判据。
- [P1] (无改动文件)— 当前 head 相对 base 没有任何代码变化。合并只会留下空 squash,无法实现描述中的修复。请确认修复是否已在 main(若已进入主干请关闭本 PR);若尚未进入,请重新把改动提交到本分支后再请求审查。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查结论:请关闭本 PR,不要合并。
- apps/desktop/src/main/plugin-market/service.ts:455 — 相对当前 main,本 PR 的 unique diff 为空(files=0 / additions=0)。描述里的官方信任回填已由 #3094 合入主干。合并本 PR 是空操作。
- 不要在当前 main 上重放旧 commit
0c4156b2:那会丢掉 #3094 的|| !hasCindyOfficialTrustMetadata(installed.dir)半边,导致缺.cindy-trust.json镜像时不再自愈。
请 close as superseded。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
MagicLizi
left a comment
There was a problem hiding this comment.
审查结论
当前 head 54d8be5fdd66186e461838ebb460a750341e043f 相对 main 0 files / 0 additions / 0 deletions。PR 描述仍声称要修 GitHub 插件官方信任回填判据(installed.trust / #3041),但这份改动已经不在当前 diff 里。
合并空 PR 没有修复效果,还会制造一次无内容的 squash。请二选一:
- 如果修复已经随其它 PR 进主干:关闭本 PR;
- 如果修复仍需要:把判据改动重新提交到本分支后再请求审查。
在 diff 重新出现并经过审查之前,不合并。
这些问题已在当前 head 的后续 commit 中修复(格式门/安全门已通过),自动 dismiss 旧的 CHANGES_REQUESTED 以解除合并阻塞。
这次改了什么
摘要
修复 Cindy GitHub 插件官方信任回填使用安装目录
.cindy-trust.json作为完成判据的问题。现在以 Host 投影的installed.trust作为权威事实:当目录镜像看似完整、但权威信任仍未完成时,仍会按既有校验和安装锁流程触发安全回填。变更类型
fix缺陷修复feat新功能refactor/perf重构或性能优化docs/test/chore文档、测试或工程维护范围
isCindyOfficialTrustInfo(installed.trust);更新官方 receipt 已存在的测试夹具。UI 变化
怎么验证的
自动验证
手工验证
不涉及;该修复覆盖插件市场后台回填逻辑。
未执行的验证
未执行 packaged macOS 升级现场验证;CI 将覆盖跨平台构建与相关单测。
风险
风险分类
影响与回滚
cindy-github的存量 trust 回填资格判断;既有下载 SHA、manifest digest、approval、来源和安装锁校验保持不变。提交前检查
git commit -s)