dsh-edit-diff 0.2.0 → 0.2.1

package/README.md

@@ -1,4 +1,4 @@
-# dsh·去重复 diff 展示插件
+# dsh·修改显示优化插件
 
 接管 DSH 浏览器的 `edit` / `write` 工具卡片，用近线性行级 diff 做真正的差异展示，消除内置 DiffBlock 把「未变化的相同行」在删除区红 `-` 行和新增区绿 `+` 行各渲染一遍的重复。
 
@@ -6,13 +6,11 @@
 
 ## 引言
 
-DeepSeek Harness 内置的 DiffBlock 展示工具改动时有个毛病：它拿到旧文件全文和新文件全文，就把旧的全染成红色 `-` 行、新的全染成绿色 `+` 行——**完全不做行级匹配**。
+原版 diff 的显示相当离谱：一次只修改一行，它却把这行连同上下各三行一起带进 diff，共十四行，其中七行被折叠，正好红区和绿区完全不重叠。不点开二级展开根本看不出到底改了什么——点开了也照样看花眼。
 
-结果呢？一次只改了一小段的 edit，一堆根本没变的注释、空行、样板代码，在红绿两区各出现一遍，仿佛同一句话被说了两次。大文件改起来更是重复加刷屏，想扫一眼真正改了什么，得靠肉眼在一大摊重复里找不同。
+这个插件把无关的相同行全部去除。只修改一行时，就只显示一行红一行绿，真正的修改加粗划线，一眼就能看清改在哪里。
 
-这哪行。改个文件就该像看 git diff 一样：**只亮出真正的差异，相同行不该凑热闹**。
-
-于是这个插件接管了 `edit` 和 `write` 两个工具卡片，用近线性算法算出真实差异，相同行一行不渲染，替换行还把改动到的具体字符用下划线标出来。一眼就知道到底改了什么。
+另外，PTC 模式下默认只显示调用成功与否，本插件也做了适配，能正常显示修改内容。
 
 ## 功能
 
@@ -24,18 +22,14 @@ DeepSeek Harness 内置的 DiffBlock 展示工具改动时有个毛病：它拿
 - **对齐原生观感**：字号、圆角、配色复用内置 DiffBlock/ToolRow 同一批 `--dsw-*` 主题变量，视觉上就是原生卡片。
 - **复制与统计**：复制只含差异行，`- `/`+ ` 前缀；底部 `└ +A -R · N file(s)` 变更统计。
 - **单文件与多文件**：`edit` / `write` 双双接管，覆盖新建只显示绿色新增，纯删、多 hunk 等场景。
-- **PTC 模式兼容**：`run_code` 子调用中的 `edit` / `write` 同样显示去重 diff，通过 `callId` 中的 `:code:` 标记识别子调用。注意 `write` 是全文件覆写，无旧内容，只能显示全新增；想看精确 diff 应使用 `edit`。
+- **PTC 模式兼容**：`run_code` 子调用中的 `edit` / `write` 同样显示优化后的修改 diff，通过 `callId` 中的 `:code:` 标记识别子调用。注意 `write` 是全文件覆写，无旧内容，只能显示全新增；想看精确 diff 应使用 `edit`。
 
 ## 它长什么样
 
-`edit` 只改动一小段后的实际效果：
-
-![去重复 diff 效果](docs/去重复diff.webp)
-
-- **顶部行**：`✎ Edit · dsh-cache-billing\src\client.ts`，右侧 chevron 可点开收起。
-- **diff 体**：只显示真实变化的删除行红色 `-` 与新增行绿色 `+`，完全相同行不重复渲染。
-- **行内高亮**：把 `峰谷价标注` 改成 `峰谷价标注附模型名`，改动到的字符 `附模型名` 以下划线加粗标出；右下角为复制按钮。
-- **底部统计**：`└ +1 -1 · 1 file`，列明净增删行数与文件数。
+|  | 效果图 |
+| --- | --- |
+| 原版 diff | ![原版离谱显示](docs/原版diff.webp) |
+| 本插件 | ![插件优化显示](docs/优化diff.webp) |
 
 ## 安装
 

package/lib/client.js

@@ -1,14 +1,13 @@
 // dsh-edit-diff 浏览器端，标准 dsh client bundle 形态。
 //
-// 通过 window.__ModuleLoader__.load 注册模块表条目，factory 从注入的 require
-// 解析 react 等外部依赖。默认导出 Cordis 插件对象，由浏览器内核挂载。
+// 通过 window.__ModuleLoader__.load 注册模块表条目，factory 从注入的 require 解析 react 等外部依赖。
+// 默认导出 Cordis 插件对象，由浏览器内核挂载。
 //
-// 接管 edit/write 工具卡片：内置 DiffBlock 对新旧全文逐行染色，不做行级匹配，
-// 相同行在红绿两区各渲染一遍。本插件接管 tool.call.toolview 的 edit/write key，
-// 用近线性行级 diff 公共前缀/后缀收缩 + Myers 跳过相同行，替换行做字符级
-// 下划线高亮。视觉对齐内置 DiffBlock/ToolRow。卡片默认收起，长 diff 二次折叠。
-// PTC 模式下 run_code 子调用中的 edit/write 同样显示去重 diff，
-// 通过 callId 中的 :code: 标记识别子调用，从 argsRaw 动态构建 diff。
+// 接管 edit/write 工具卡片：内置 DiffBlock 对新旧全文逐行染色且不做行级匹配，相同行在红绿两区各渲染一遍。
+// 本插件接管 tool.call.toolview 的 edit/write key，用近线性行级 diff 跳过相同行，替换行做字符级下划线高亮。
+// 视觉对齐内置 DiffBlock/ToolRow。卡片默认收起，长 diff 二次折叠。
+//
+// PTC 模式下 run_code 子调用中的 edit/write 同样显示去重 diff，按 callId 的 :code: 标记识别，从 argsRaw 动态构建。
 
 window.__ModuleLoader__.load({
   id: 'dsh-edit-diff',
@@ -160,32 +159,49 @@ window.__ModuleLoader__.load({
     }
 
     function extractDiffs(block) {
-      // PTC 模式：从 argsRaw 动态构建 diffs
-      // callId 包含 ':code:' 表示这是 run_code 的子调用
-      if (typeof block.callId === 'string' && block.callId.includes(':code:')) {
-        // RunningToolCall 的 argsRaw 在 block 上，ToolResultNode 的 argsRaw 在 block.call 上
-        const argsRaw = block.call ? block.call.argsRaw : block.argsRaw
-        const name = block.call ? block.call.name : block.name
-        if (name === 'edit' || name === 'write') {
-          try {
-            const args = JSON.parse(argsRaw)
-            if (args && typeof args === 'object') {
-              const filePath = args.file_path
-              if (typeof filePath === 'string' && filePath !== '') {
-                if (name === 'write' && typeof args.content === 'string') {
-                  return narrowDiffs([{ path: filePath, oldText: null, newText: args.content }])
-                }
-                if (name === 'edit' && typeof args.old_string === 'string' && typeof args.new_string === 'string') {
-                  return narrowDiffs([{ path: filePath, oldText: args.old_string || null, newText: args.new_string }])
-                }
+      const done = 'kind' in block
+      // PTC 子调用标记：新版 DSH 用 parentCallId；旧版 callId 含 ':code:'，双兼容
+      const subCall = block.parentCallId !== void 0 ||
+        (typeof block.callId === 'string' && block.callId.includes(':code:'))
+
+      // settled 且非错误：取真实应用的 diff，新版 DSH 存放在 block.meta.diffs。
+      // 结构与内置 diffCardModel/appliedDiffs 同构，支持多 hunk。
+      if (done && !subCall && !block.isError) {
+        const meta = block.meta
+        if (meta !== null && typeof meta === 'object' && !Array.isArray(meta)) {
+          const diffs = meta.diffs
+          if (Array.isArray(diffs) && diffs.length > 0) {
+            const n = narrowDiffs(diffs)
+            if (n !== null) return n
+          }
+        }
+      }
+
+      // 子调用 / running / 兜底：没有真实 diff，改为从参数意图推断。
+      // running 的 argsRaw/name 在 block 顶层，settled 的挪到 block.call 上，两处兼容。
+      // settled 失败同样不显示意图 diff，与内置 diffCardModel 一致。
+      if (done && !subCall && block.isError) return null
+      const argsRaw = (block.call ? block.call.argsRaw : block.argsRaw) ?? ''
+      const name = block.call ? block.call.name : block.name
+      if (name === 'edit' || name === 'write') {
+        try {
+          const args = JSON.parse(argsRaw)
+          if (args && typeof args === 'object') {
+            const filePath = args.file_path
+            if (typeof filePath === 'string' && filePath !== '') {
+              if (name === 'write' && typeof args.content === 'string') {
+                return narrowDiffs([{ path: filePath, oldText: null, newText: args.content }])
+              }
+              if (name === 'edit' && typeof args.old_string === 'string' && typeof args.new_string === 'string') {
+                return narrowDiffs([{ path: filePath, oldText: args.old_string || null, newText: args.new_string }])
               }
             }
-          } catch {}
-        }
+          }
+        } catch {}
       }
-      
-      // 标准模式：从 callView 或 resultView 提取 diffs
-      if (!('kind' in block)) {
+
+      // 旧版 DSH 在 0.1.x 重构前没有以上字段，回退从 callView/resultView 读 diff。
+      if (!done) {
         const call = block.callView && block.callView.card === 'diff' ? block.callView : null
         return call ? narrowDiffs(call.diffs) : null
       }
@@ -421,7 +437,7 @@ window.__ModuleLoader__.load({
         })
 
         slots.inject('tool.call.toolview', () => {
-          // priority 要低于默认 0，遮蔽 file-mutation-toolview 的 edit/write，最低者渲染；
+          // priority 要低于默认 0，遮蔽 file-mutation-toolview 的 edit/write，最低者渲染。
           // 若也传 0 会在同一 key 上 clash，而非替换。
           slots.register({ name: 'tool.call.toolview', key: 'edit', priority: -1 }, DiffCard)
           slots.register({ name: 'tool.call.toolview', key: 'write', priority: -1 }, DiffCard)

package/lib/index.js

@@ -1,9 +1,8 @@
 /**
- * 主机端：空操作 Cordis 插件。所有行为在浏览器端 lib/client.js，
- * 通过 package.json dsh.client 声明被 web 客户端模块系统发现。
- * 本半身存在是为了让插件根目录成为完整的双面包，在宿主 Loader 中
- * 显示为一个条目；通过 dsh.bundle 成为自挂载 bundle 层，
- * 一条 dsh plugin --profile web add 即可完整激活。
+ * 主机端：空操作 Cordis 插件。
+ * 所有行为在浏览器端 lib/client.js，通过 package.json dsh.client 声明被 web 客户端模块系统发现。
+ * 本半身存在是为了让插件根目录成为完整的双面包，在宿主 Loader 中显示为一个条目。
+ * 通过 dsh.bundle 成为自挂载 bundle 层，一条 dsh plugin --profile web add 即可完整激活。
  */
 /** 插件名，即配置条目 id。 */
 const name = "dsh-edit-diff";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-edit-diff",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "质量 diff：接管 edit/write 工具卡片，行级去重与行内高亮，消除相同内容在删除与新增两区的重复；兼容 PTC 模式，标准可安装的 dsh 客户端插件",
   "keywords": [
     "deepseek-harness",