Skip to content

chore(protocol): remove unused shared packages - #2324

Merged
xd-bobo merged 1 commit into
mainfrom
chore/trim-protocol-packages
Aug 10, 2026
Merged

chore(protocol): remove unused shared packages#2324
xd-bobo merged 1 commit into
mainfrom
chore/trim-protocol-packages

Conversation

@xd-bobo

@xd-bobo xd-bobo commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

这次改了什么

摘要

cindy-protocol submodule 更新到 dbbf169703(稳定标签 client-baseline-dbbf169,上游 PR makecindy/cindy-protocol#49),移除已经没有消费方的 skill/voice protocol workspace 记录,并明确协议仓只维护、持续精简现有四个 package。

同时修正 submodule forward guard 在 Git 2.54 下的测试夹具:通过临时可见分支获取基线 commit 后删除该分支,继续验证缺失的已发布 head 可以补取,不改变生产逻辑。

变更类型

  • docs / test / chore 文档、测试或工程维护

范围

  • 关联需求:精简 cindy-protocol
  • 本 PR 包含:submodule gitlink、workspace 测试清单、锁文件、协议规则文档、Git 2.54 测试兼容
  • 明确不包含:剩余四个 protocol package 的业务实现修改
  • 用户可见变化:无
  • 是否存在 breaking change:无;被删除的两个 package 在客户端和服务端均无引用

UI 变化

不涉及。

怎么验证的

自动验证

corepack pnpm@10.33.2 install --frozen-lockfile
结果:通过

corepack pnpm@10.33.2 --filter desktop run --if-present typecheck
结果:通过

node --test scripts/__tests__/check-submodule-forward.test.mjs
结果:13/13 通过

corepack pnpm@10.33.2 test:unit
结果:最终完整复跑通过;runner 392/392,Desktop 23763 tests、Mobile 及全部必跑 workspace 通过

corepack pnpm@10.33.2 check:submodule-forward
结果:56d9d9f54e -> dbbf169703,forward 且已发布

corepack pnpm@10.33.2 check:dco
结果:通过

首次完整单测中 iOS Simulator discovery 用例在并发负载下发生一次 5 秒超时;单独复跑 787ms 通过,随后完整门禁复跑全绿。

手工验证

不涉及。

未执行的验证

无。

风险

风险分类

  • 协议兼容

影响与回滚

  • 影响范围:客户端 protocol workspace 与 submodule pin;不改变 wire contract。
  • 存量插件影响:无。
  • 回滚:将 gitlink 与锁文件恢复至上一 pin;旧 checkout 仍可通过历史 commit 访问已删除 package。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名
  • 未提交凭证、令牌或授权文件
  • 已补充必要文档
  • 已确认测试结果

@xd-bobo
xd-bobo requested review from a team and zqchris as code owners August 10, 2026 03:31
Signed-off-by: xd-bobo <caojianbo@xd.com>
@xd-bobo
xd-bobo force-pushed the chore/trim-protocol-packages branch from 67b69ee to b0513fd Compare August 10, 2026 03:34
@greptile-apps

greptile-apps Bot commented Aug 10, 2026

Copy link
Copy Markdown

Greptile Summary

本 PR 更新 cindy-protocol 子模块并移除已无消费方的 skill/voice protocol workspace 记录,同时调整协议仓规则文档和 Git 2.54 兼容测试夹具。

  • 从测试工作区清单和锁文件中删除 skill/voice protocol
  • 将协议仓文档明确为仅维护并持续精简现有四个 package
  • 通过临时远端分支构造 submodule forward guard 的测试基线

Confidence Score: 5/5

未发现由本次变更引入的具体可达故障,PR 看起来可以安全合并。

删除的两个协议工作区在主仓中没有消费引用,锁文件与测试清单同步更新,测试夹具调整也未改变生产检查行为。

Important Files Changed

Filename Overview
cindy-protocol 子模块指针从 56d9d9f54e 更新为 dbbf1697037;未发现可证实的兼容性或发布可达性缺陷。
scripts/test-workspaces.config.mjs 移除两个已删除且主仓无引用的 protocol 测试工作区,剩余四个工作区与文档声明一致。
pnpm-lock.yaml 仅删除 skill/voice protocol 对应的两个 importer,未改变共享依赖版本。
scripts/tests/check-submodule-forward.test.mjs 使用临时可见分支获取 baseline commit 后删除该分支,未改变生产检查逻辑。
docs/dev-rules/protocol-and-submodules.md 更新协议仓边界、兼容升级策略、发布要求及 review 清单。
docs/dev-rules/repo-map.md 同步记录协议仓当前保留的四个 package 及封闭精简策略。

Reviews (1): Last reviewed commit: "chore(protocol): remove unused shared pa..." | Re-trigger Greptile

@xd-bobo
xd-bobo merged commit e118350 into main Aug 10, 2026
19 checks passed
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.

1 participant