[提案] 把 ui-deliverables 的 FileDiff(带行号 diff 审阅渲染)提升为 ui-primitives 的可复用组件

[KannaKuron]

背景

插件生态里展示 git diff 是常见需求。以 dsh-ide-git(JetBrains 风格 Git 工具窗)为例:变更文件点击后的 diff 预览,希望能和产品自家的 diff 审阅视图(deliverables 的「本轮改动」)保持同一视觉与能力。

现状是两个组件各管一头:

• ui-primitives 的 DiffBlock:插件可用的官方 diff 渲染(seed 表内),但没有行号 gutter;
• ui-deliverables 的 FileDiff:双列旧/新行号、± 标记列、@@ hunk 头、Shiki 逐行高亮、unified/split——体验是标杆,但它是该包私有组件,bundle purity 阻止任何插件 require 它。

于是插件只有两个都不理想的选择:用 DiffBlock(与官方审阅视图割裂、无行号),或者照着 FileDiff 重写一份布局(临时可用,官方演进后会漂移)。dsh-ide-git v0.14.2 就走了第二条路,所以对这个漂移有切身体会。

观察

FileDiff 组合的积木其实已经在 seed 表里:useCodeHighlighter(共享懒加载 Shiki)、languageForPath,以及 --dsw-alias-file-diff-added/deleted-gutter/marker/bg 这组 file-diff alias token。缺的只是组件本体的布局层。

提议

把 FileDiff 的行号 diff 渲染提升为 ui-primitives 的可复用组件(比如 LineDiffBlock,或给 DiffBlock 增加行号模式),deliverables 自身改为消费它:

• 插件与宿主所有 diff 视图统一为同一渲染器,主题由既有 alias token 保证跟随;
• deliverables 从「实现者」变成「消费者」,单一实现;
• unified 视图即可满足绝大多数场景,split 可以留在 deliverables 作为扩展。

愿意提 PR 承担迁移与测试;如果方向上更倾向于给 DiffBlock 加行号模式而不是新组件,也完全可以按维护者的偏好来。

[KannaKuron]

补充一个实证结论(实测 dsh 0.1.7-rc.2 隔离实例):从插件 client bundle require('@deepseek-ai/dsh-client-ui-deliverables') 不会抛错——packages/client/modules/src/client/manifest.ts 的解析契约写明 boot 图行可被解析(seed 词 → 图行 → 已注册工厂,throw 只留给图外的名字),实测返回 { apply, inject }。

所以这里的技术卡点比原帖说的更小:不是「跨包禁止」,而是 ui-deliverables 的包入口没有 re-export 组件——FileDiff / hunkRows / splitRows 这些只存在于包内模块,插件拿到的入口只有 Cordis 插件面。

据此把请求收窄为更轻的选项(任一即可,按维护者偏好):

• 最小改动:在 ui-deliverables 的 client 入口 re-export FileDiff(以及 hunkRows / splitRows / renderedHunks 这几个纯函数)。插件侧配合 dsh.client.inject 声明到达顺序即可直接消费,零迁移成本。
• 更彻底(原帖方案):把行号 diff 渲染提升为 ui-primitives 组件。

如果方向 OK,两种方案我都可以出 PR。