Skip to content

docs(design-system): DS-5 对齐执行路线与双端设计合同 - #4022

Merged
MagicLizi merged 2 commits into
makecindy:mainfrom
kirozeng:ds/5-roadmap-and-platform-contract
Sep 7, 2026
Merged

MagicLizi merged 2 commits into
makecindy:mainfrom
kirozeng:ds/5-roadmap-and-platform-contract

Conversation

@kirozeng

@kirozeng kirozeng commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

设计系统改造系列 DS-5,路线图见 docs/design-rules/design-governance.md §12

这次改了什么

摘要

仓内旧路线仍将 DS-5 指向聊天、DS-6 指向 Permission,台账还等待已落地的 DS-4。本次对齐 DS-5—12,补 DS-4b/#4010,并用 G1—G4 分别验收。Token README 补当前数值来源、DS-8/10 接管合同与两条真实双端消费路径,治理集中登记待决项。

变更类型

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

范围

  • 关联需求:设计系统 DS-5(治理 §12)。
  • 包含:五份文档、台账生成器四处静态说明及一条已有断言。
  • 不包含:产品组件/颜色/尺寸、Token JSON、依赖、主题写盘、Mobile 发现或 CI 能力。
  • 用户可见变化:无;零视觉,仅合同与台账说明同步。
  • breaking change:无。

UI 变化

不涉及产品 UI。合同依据:docs/design-rules/DESIGN.md §4/5/10 的组件、圆角与主题边界;design-governance.md §4/6/7 的兼容、证据与风险规则。既有批准不扩大、pending 不追认为已批准。

怎么验证的

自动验证

  • pnpm design:inventory 连续两次:退出 0,字节一致,人工区保留。
  • pnpm check:design-inventory:退出 0;35 个 Desktop surface,生成事实未变。
  • pnpm test:unit:related:退出 0;根 test:runner 504 通过、1 原有跳过、0 失败;无受影响 workspace 单测。
  • pnpm --filter @cindy/design-tokens run --if-present typecheck:退出 0。
  • git diff --check:通过。

手工验证

基于 main 36638ff 核对两端输入提交与用户/助手正文实际消费链、当前引用和平台派生;三路只读复核后纠正名称与定位。84 个本地文档链接目标存在;inventory 35 个 ID、owner、状态与保护含义保持。仅文档可达性检查,不冒充 G2 独立试用。

未执行的验证

未做 Desktop/iOS/Android Light/Dark 实机目检,本批未修改产品 UI。related 按已有调度不运行未受影响的 workspace 单测;根 runner 的 1 项跳过因 platform-tools 二进制未就位。验证日志保存在本地,尚未上传公开附件。DCO 签名与实际提交范围检查在提交预检中执行。

风险

风险分类

  • 其他:零视觉基建;未来接管与待决登记不代表当前生产能力。

影响与回滚

  • 影响范围:文档路线/合同、生成器展示说明,产品、权限和用户数据无变化。
  • 回滚方式:整体回退本批文档及对应静态模板/断言;不涉及用户数据回写。
  • Permission 三项仍阻塞 DS-11;DS-4/4b 证据缺口由 DS-6 补齐。Mobile 实际接管时按 fingerprint 判断冷更,不预先宣称无风险。

提交前检查

  • 已 review 完整 diff
  • 每个 commit 都带 DCO 签名(git commit -s)
  • UI 变化已说明不涉及
  • 未包含凭证、令牌或授权文件
  • 已补充必要文档
  • 已确认测试结果并说明未执行原因

目标:完成 DS-5 路线、数值权威、双端消费样本与待决合同对齐,让贡献者仅凭仓内文档能够接手 DS-6。
范围:五份文档、inventory 四处静态说明、一条已有断言;保留旧 ID/覆盖作用域/用户主题磁盘不变、保护族、两模式与管理员审核合同。
非目标:产品 UI、生产值、DTCG JSON、依赖、发现逻辑、CI、Permission 业务或 DS-6 实施。
验收:SC-01—10;两条真实双端链/平台静态覆盖与动态分界;台账状态不虚增;生成两次一致、freshness、related、受影响包 typecheck 与 diff 检查通过;pending 与未验证项如实记录。

MUST-FIX(本轮共识确认,按不变量归族去重)

本轮无 MUST-FIX 项。

ARCHIVE(已登记接受的残余风险)

本轮无 ARCHIVE 项。

Signed-off-by: kirozeng <kirozeng@gmail.com>
group g1 tip c2d89fd0cf4c90ab38fa63dd39b3fd50c3eba3de

Signed-off-by: kirozeng <kirozeng@gmail.com>
@kirozeng
kirozeng requested a review from a team as a code owner September 7, 2026 02:18
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@greptile-apps

greptile-apps Bot commented Sep 7, 2026

Copy link
Copy Markdown

Greptile Summary

本 PR 将设计系统后续工作统一为 DS-5—12,补充 DS-4b 状态,并明确 Desktop/Mobile Token 接管边界、真实消费样本、待裁决事项和 G1—G4 验收合同。

  • 路线调整为 DS-8 接管 Desktop、DS-9 完成聊天呈现、DS-10 接管 Mobile、DS-11 处理 Permission。
  • inventory 的人工下一动作、生成器静态说明及相关测试断言同步更新。
  • Token README 记录现行数值来源、未来生成落点、平台覆盖和双端真实消费链。
  • 未修改产品 UI、生产 Token 数值、依赖或运行时接线。

Confidence Score: 5/5

本 PR 的文档、台账说明和对应断言保持一致,未发现影响合并的具体问题。

重新编号后的治理路线与 inventory 的逐 surface 动作能够对应,关键双端消费链说明与实际代码一致,生成器也不会覆盖现有人工台账;没有运行时、安全或依赖层面的变更。

Important Files Changed

Filename Overview
docs/design-rules/design-governance.md 重构 DS-5—12 路线、待裁决合同及 G1—G4 验收标准,内容与当前台账分工一致。
packages/design-tokens/README.md 明确当前与目标数值来源、双端接管边界及真实消费样本;已核对的关键源码链准确。
docs/design-rules/design-inventory.md 在不改变 surface 状态和保护含义的前提下,同步各入口后续批次与人工下一动作。
scripts/shared/design-inventory.mjs 更新默认台账说明和 Permission/Mobile 批次文本;既有人工区保留机制不受影响。
scripts/tests/design-inventory.test.mjs 将默认 seed 的 Mobile 发现阶段断言同步到 DS-7。
docs/design-rules/cindy-design-system.md 新增贡献者阅读路径和 Token 合同入口,链接目标有效。
docs/dev-rules/repo-map.md 登记 design-tokens 影子包及 DS-8/DS-10 未来接管关系。

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  DS5["DS-5<br/>路线与合同对齐"] --> DS6["DS-6<br/>设置表单与普通确认"]
  DS6 --> DS7["DS-7<br/>复用守卫与 Mobile 发现"]
  DS7 --> DS8["DS-8<br/>Desktop Token 生产接管"]
  DS8 --> DS9["DS-9<br/>聊天呈现迁移"]
  DS9 --> DS10["DS-10<br/>Mobile Token 接管"]
  DS10 --> DS11["DS-11<br/>双端 Permission 呈现"]
  DS11 --> DS12["DS-12<br/>扩大门禁与交接"]
  DS8 -.验收.-> G1["G1 集中改风格"]
  DS6 -.验收.-> G2["G2 独立贡献者可用"]
  DS7 -.验收.-> G3["G3 防持续退化"]
  DS10 -.验收.-> G4["G4 兼容与双端成立"]
Loading

Reviews (1): Last reviewed commit: "fix-run ds5-anchor wave1 squash" | Re-trigger Greptile

@MagicLizi MagicLizi added status:ci-running CI 还在跑(review-pr 自动维护,仅展示) touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示) touches:rules 改动碰到规则 / 规范文档(review-pr 自动维护,仅展示) labels Sep 7, 2026
@kirozeng

kirozeng commented Sep 7, 2026

Copy link
Copy Markdown
Contributor Author

本轮反馈已核实:两条新评论都来自机器人,不需要改代码,分支也没有新提交(head 仍为 6e386a3)。

逐条说明:

  • comment 5564130078(chatgpt-codex-connector):Codex code review 机器人报告自己的用量配额到上限,是机器人自身状态通知,不含任何审查意见或修改要求。
  • comment 5564154663(greptile-apps):Greptile 自动摘要,置信度 5/5,明确写明「未发现影响合并的具体问题」,未提出任何修改请求。

顺手交叉核对了摘要与 PR 实际状态,两边一致:

  • 摘要列出的 7 个文件与 PR 实际改动完全一致(docs/design-rules/ 三份、docs/dev-rules/repo-map.md、packages/design-tokens/README.md、scripts/shared/design-inventory.mjs、scripts/tests/design-inventory.test.mjs),全部是文档 / 脚本 / 测试,未触碰产品 UI、生产 Token 数值、依赖或运行时接线。
  • CI 全绿:DCO、Linux / Windows unit tests、check:pr-design-basis、verify、verify-checks、Desktop Git integration、Greptile Review 全部 pass。

dispatch:eba528f996bc440e

@MagicLizi MagicLizi removed the status:ci-running CI 还在跑(review-pr 自动维护,仅展示) label Sep 7, 2026

@MagicLizi MagicLizi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

代码重审通过:当前 head 未发现 P0/P1。DS-5 文档合同、inventory 生成器与测试、repo-map / Token README 对齐一致;规则改写未放宽兼容红线。全部 conversation 已 resolve。

@MagicLizi
MagicLizi merged commit 63e9072 into makecindy:main Sep 7, 2026
11 checks passed
@MagicLizi

Copy link
Copy Markdown
Contributor

合了。把 DS-5 从聊天 Pattern 改成文档合同、并把 inventory / repo-map / Token README 一起对齐,后面双端设计不会再各写各的编号。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

touches:product-ui 改动碰到产品 / UI 面(review-pr 自动维护,仅展示) touches:rules 改动碰到规则 / 规范文档(review-pr 自动维护,仅展示)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants