dsh-auto-review
github.com/PerryLink/dsh-auto-review
v0.12.17
模型与账号
写入文件动态执行代码发起网络请求
Second-model AI auto-review for DeepSeek Harness approval requests: a read-only reviewer subagent decides allow/deny on the approval answerer chain, with fail-closed fallback and full session-log audit.
该插件尚未翻译,显示作者原文。
deepseek-harnessdshdsh-plugincordisapprovalauto-reviewsecond-modelai-safety
检测报告
有一定能力。命中了若干能力模式,大多是插件正常工作时会用到的。
5需留意4一般15未命中
高关注需留意一般
能力清单
- 写入文件
writes-files需留意27 处命中lib/types/eval/cli.js:262await mkdir(outDir, { recursive: true });lib/eval-cli.js:424await mkdir(outDir, { recursive: true });lib/report-CGzPAnI6.js:13510await mkdir(options.workspaceRoot, { recursive: true });lib/report-CGzPAnI6.js:13600for (const file of caze.files) await writeFile(join(workspace, file.path), file.content, { flag: "wx" });- 看写入位置。写自己的配置目录是正常的,写系统目录不是。
- 动态执行代码
dynamic-code需留意5 处命中lib/index.js:807new Function("");lib/report-CGzPAnI6.js:6794new Function("");lib/report-CGzPAnI6.js:13889lines.push("| Summary | |", "|---|---|", `| Result | ${s.fail + s.error + s.cancelled === 0 ? "all passed" : "NOT all pa…lib/types/eval/report.js:99lines.push('| Summary | |', '|---|---|', `| Result | ${s.fail + s.error + s.cancelled === 0 ? 'all passed' : 'NOT all pa…- 看 eval / new Function 的入参来自哪里。常量字符串和用户输入是两回事。
- 发起网络请求
network需留意2 处命中README-es.md:423- **Sin mutación global.** No parchea prototipos, ni reescribe `process.env`, ni reemplaza el dispatcher global de fetch…README-pt.md:419- **Sem mutação global.** Não altera protótipos, não reescreve `process.env` nem substitui o dispatcher global de fetch.- 看请求发往哪些域名。发往作者自己的后端是常见做法。
- 启动伪终端
spawns-pty需留意2 处命中CHANGELOG.md:301- Declared the five client `@deepseek-ai/dsh-client-*` services (`dsh-client-connection`, `dsh-client-runtime`, `dsh-cli…README.md:106- **native build scripts**: when `dsh plugin add` stops at `ERR_PNPM_IGNORED_BUILDS` for `koffi` / `node-pty` (pulled in…- 伪终端等于给了一个完整 shell。确认这是插件的核心功能。
- R013 清单可疑
manifest-obfuscation需留意1 处命中- 安装脚本: prepare
- 清单被刻意写乱时,加载行为会难以预测。看原文而不是看渲染结果。
- 接触凭据或密钥
credentials一般49 处命中lib/index.js:6162const SENSITIVE_KEY_PATTERN = /(?:api[_-]?key|token|secret|password|passwd|pwd|authorization|auth|credential|private[_-]…lib/index.js:6349"- deny: the action is destructive, irreversible, credential- or data-exfiltrating, or the",lib/types/review.js:32const SENSITIVE_KEY_PATTERN = /(?:api[_-]?key|token|secret|password|passwd|pwd|authorization|auth|credential|private[_-]…lib/types/review.js:264'- deny: the action is destructive, irreversible, credential- or data-exfiltrating, or the',- 看它读取哪些凭据、发往哪里。用于登录自己的服务是正常的。
- 读取文件
reads-files一般19 处命中lib/types/eval/cli.js:27return existsSync(built) ? built : new URL('../../package.json', import.meta.url);lib/types/eval/cli.js:30const PACKAGE_VERSION = JSON.parse(readFileSync(packageJsonUrl(), 'utf8')).version;lib/types/eval/cli.js:193const stat = statSync(path);lib/types/eval/cli.js:195const entries = await readdir(path, { withFileTypes: true });- 读文件是绝大多数插件的正常工作方式,本身不构成顾虑。
- 读取环境变量
reads-env一般19 处命中lib/types/eval/cli.js:271process.env.DSH_EVAL_SESSIONS_ROOT = join(outDir, 'sessions');lib/types/eval/cli.js:272process.env.DSH_EVAL_WORKSPACE_ROOT = workspaceRoot;lib/eval-cli.js:426process.env.DSH_EVAL_SESSIONS_ROOT = join(outDir, "sessions");lib/eval-cli.js:427process.env.DSH_EVAL_WORKSPACE_ROOT = workspaceRoot;- 环境变量里常混有密钥。看它是否把整个 env 都读走。
- 动态加载模块
dynamic-import一般7 处命中lib/report-CGzPAnI6.js:13347const packChunkRuns = (await import("@deepseek-ai/dsh-session")).packChunkRuns;lib/types/eval/runner.js:34const sessionModule = await import('@deepseek-ai/dsh-session');src/eval/cli.ts:219async function bootComposition(): Promise<import('@deepseek-ai/cordis').Context> {src/eval/cli.ts:261let ctx: import('@deepseek-ai/cordis').Context | undefined- 动态导入本身无害,但会让依赖关系难以静态审阅。
逐条扫描明细
| 能力 | 标识 | 关注度 | 命中次数 |
|---|---|---|---|
| 写入文件 | writes-files | 需留意 | 27 |
| 动态执行代码 | dynamic-code | 需留意 | 5 |
| 发起网络请求 | network | 需留意 | 2 |
| 启动伪终端 | spawns-pty | 需留意 | 2 |
| R013 清单可疑 | manifest-obfuscation | 需留意 | 1 |
| 接触凭据或密钥 | credentials | 一般 | 49 |
| 读取文件 | reads-files | 一般 | 19 |
| 读取环境变量 | reads-env | 一般 | 19 |
| 动态加载模块 | dynamic-import | 一般 | 7 |
这份报告的口径
扫描读的是包里的源码,全程不执行第三方代码。 本次读了 121 个文件,扫描于 2026-10-15。 命中次数只说明源码里出现过这类调用,不说明行为好坏——一个插件读写文件 500 次和 1 次 都会被记为「读取文件」。这是能力清单,不是信任评分,也不是安全结论: 没扫出来的能力可能只是写法没被规则覆盖。
依赖安全公告
这个插件声明的依赖中,有 3 条已公开的安全公告 (最高等级:中):
| 等级 | 依赖包 | 编号 | 摘要 |
|---|---|---|---|
| 中 | yaml | CVE-2023-2251 | Uncaught Exception in yaml 修复版本:2.2.2 详情 → |
| 低 | yaml | CVE-2026-33532 | yaml is vulnerable to Stack Overflow via deeply nested YAML collections 修复版本:1.10.3, 2.8.3 详情 → |
| 低 | zod | CVE-2023-4316 | Zod denial of service vulnerability 修复版本:3.22.3 详情 → |
查询的是依赖声明范围的下界(如 ^4.2.0 查 4.2.0),
不是实际安装到的版本。真正装的时候如果解析到了修复版本,就不受影响——
所以这是「可能受影响」,不是「这个插件有漏洞」。数据来自
OSV.dev。