dsh-diff-approval
github.com/9087/dsh-diff-approval
v0.34.0
其他
可执行外部程序含混淆或编码代码写入文件
DeepSeek Harness plugin: pending-edit review with whole-file diff and Keep/Revert
该插件尚未翻译,显示作者原文。
检测报告
关注度高。命中了不少能力模式,其中可能有需要留意的。逐条看下面的清单。
1高关注4需留意3一般16未命中
高关注需留意一般
能力清单
- 可执行外部程序
spawns-processes高关注1 处命中lib/index.js:8import { spawn } from "node:child_process";- 看它调用哪些程序。调用本机已有的工具是常见做法。
- 含混淆或编码代码
obfuscated-code需留意72 处命中lib/types/annotate-skill.d.ts:19export declare const ANNOTATE_SKILL_CONTENT = "# \u7528\u5361\u7247\u7ED9\u4EE3\u7801\u505A\u6279\u6CE8\n\n\u6539\u52A8\…lib/types/comment-skill.d.ts:24export declare const COMMENT_SKILL_CONTENT = "# \u56DE\u7B54\u4EE3\u7801\u6279\u6CE8\n\n\u6D88\u606F\u4EE5 `[\u8BC4\u8BB…- 混淆本身不是恶意,但公开源码里出现大段混淆值得问一句为什么。
- 写入文件
writes-files需留意4 处命中lib/index.js:1197await io.mkdir(dirname(file), { recursive: true });lib/index.js:1199await io.writeFile(staged, JSON.stringify(value), "utf8");lib/index.js:1201await io.rename(staged, file);lib/index.js:1799await rename(file, quarantined);- 看写入位置。写自己的配置目录是正常的,写系统目录不是。
- 动态执行代码
dynamic-code需留意1 处命中lib/index.js:246schema.callback = new Function("return " + schema.callback)();- 看 eval / new Function 的入参来自哪里。常量字符串和用户输入是两回事。
- R013 清单可疑
manifest-obfuscation需留意1 处命中- 安装脚本: prepare
- 清单被刻意写乱时,加载行为会难以预测。看原文而不是看渲染结果。
- 动态加载模块
dynamic-import一般21 处命中lib/types/client/ColorPicker.d.ts:23}): import("react").JSX.Element;lib/types/client/dsh-icons.d.ts:29export declare const IconListPenOutline16: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;lib/types/client/dsh-icons.d.ts:31export declare const IconChevronDownOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Elemen…lib/types/client/dsh-icons.d.ts:32export declare const IconChevronUpOutline14: ({ size, className }: primitives.IconProps) => import("react").JSX.Element;- 动态导入本身无害,但会让依赖关系难以静态审阅。
- 读取文件
reads-files一般17 处命中lib/index.js:1307raw = await readFile(file, "utf8");lib/index.js:1388names = await readdir(root);lib/index.js:1417names = await readdir(root);lib/index.js:1649raw = await readFile(file, "utf8");- 读文件是绝大多数插件的正常工作方式,本身不构成顾虑。
- 读取环境变量
reads-env一般2 处命中lib/index.js:4503updateCheck ??= createUpdateCheck({ ...process.env.DSH_DIFF_APPROVAL_UPDATE_REGISTRY_URL === void 0 ? {} : { registryUrl…- 环境变量里常混有密钥。看它是否把整个 env 都读走。
逐条扫描明细
| 能力 | 标识 | 关注度 | 命中次数 |
|---|---|---|---|
| 可执行外部程序 | spawns-processes | 高关注 | 1 |
| 含混淆或编码代码 | obfuscated-code | 需留意 | 72 |
| 写入文件 | writes-files | 需留意 | 4 |
| 动态执行代码 | dynamic-code | 需留意 | 1 |
| R013 清单可疑 | manifest-obfuscation | 需留意 | 1 |
| 动态加载模块 | dynamic-import | 一般 | 21 |
| 读取文件 | reads-files | 一般 | 17 |
| 读取环境变量 | reads-env | 一般 | 2 |
这份报告的口径
扫描读的是包里的源码,全程不执行第三方代码。 本次读了 62 个文件,扫描于 2026-10-15。 命中次数只说明源码里出现过这类调用,不说明行为好坏——一个插件读写文件 500 次和 1 次 都会被记为「读取文件」。这是能力清单,不是信任评分,也不是安全结论: 没扫出来的能力可能只是写法没被规则覆盖。
依赖安全公告
这个插件声明的依赖中,有 2 条已公开的安全公告 :
| 等级 | 依赖包 | 编号 | 摘要 |
|---|---|---|---|
| 未分级 | dompurify | CVE-2026-109345 | DOMPurify: IN_PLACE returns a force-removed rawtext root whose text carries attacker markup — pure HTML reparse executes 修复版本:3.4.16 详情 → |
| 未分级 | dompurify | CVE-2026-109344 | DOMPurify: IN_PLACE: node-removing afterSanitize hook leaves detached subtree event handlers armed, causing DOM XSS 修复版本:3.4.16 详情 → |
查询的是依赖声明范围的下界(如 ^4.2.0 查 4.2.0),
不是实际安装到的版本。真正装的时候如果解析到了修复版本,就不受影响——
所以这是「可能受影响」,不是「这个插件有漏洞」。数据来自
OSV.dev。