dsh-plugin-security-review

检测报告

能力清单

• 以 shell 执行命令shell-exec高关注3 处命中
  • lib/index.js:454{ id: 'PROC_EXEC_UNSAFE', severity: 'critical', category: '任意命令执行', message: '不安全进程执行(exec/execSync 字符串形式, 或 spawn 搭配 sh…
  • README.en.md:91- **Host rules**: unsafe process execution (exec/execSync/shell:true — critical), VM escape, Node internal APIs, constru…
  • README.md:93- **Host 规则**:不安全进程执行(exec/execSync/shell:true — critical)、VM 逃逸、Node 内部 API、构造器链逃逸、宿主进程终止(critical);子进程执行能力(spawn/fork …
  • 看命令字符串是拼出来的还是写死的。拼接用户输入才是真正的风险。
• 访问原生绑定native-access高关注1 处命中
  • lib/index.js:457{ id: 'CTOR_ESCAPE', severity: 'critical', category: '构造器逃逸', message: 'constructor.constructor 链式逃逸', src: "constructor…
  • 访问原生绑定或走 constructor 链，等于绕开 JS 层的一切限制。确认这是插件的核心机制，而不是顺手拿到的便利。
• 写入文件writes-files需留意4 处命中
  • lib/index.js:79mkdirSync(dir, { recursive: true })
  • lib/index.js:80writeFileSync(file, JSON.stringify(payload), 'utf8')
  • lib/index.js:181mkdirSync(dir, { recursive: true })
  • lib/index.js:182writeFileSync(file, JSON.stringify(struct), 'utf8')
  • 看写入位置。写自己的配置目录是正常的，写系统目录不是。
• 发起网络请求network需留意2 处命中
  • client/client.js:189fetch(PENDING_URL)
  • client/client.js:201fetch(DECIDE_URL, {
  • 看请求发往哪些域名。发往作者自己的后端是常见做法。
• 动态执行代码dynamic-code需留意1 处命中
  • lib/index.js:690new Function(code)
  • 看 eval / new Function 的入参来自哪里。常量字符串和用户输入是两回事。
• 读取文件reads-files一般10 处命中
  • lib/index.js:51const parsed = JSON.parse(readFileSync(file, 'utf8'))
  • lib/index.js:132try { parsed = JSON.parse(readFileSync(file, 'utf8')) } catch (error) { return empty }
  • lib/index.js:217const parsed = JSON.parse(readFileSync(file, 'utf8'))
  • lib/index.js:967if (statSync(candidates[i]).isDirectory()) return candidates[i]
  • 读文件是绝大多数插件的正常工作方式，本身不构成顾虑。
• 读取环境变量reads-env一般9 处命中
  • lib/index.js:39try { home = String(process.env.DSH_HOME || '') } catch (error) { /* ignore */ }
  • lib/index.js:119try { home = String(process.env.DSH_HOME || '') } catch (error) { /* ignore */ }
  • lib/index.js:206try { home = String(process.env.DSH_HOME || '') } catch (error) { /* ignore */ }
  • lib/index.js:474{ id: 'ENV_READ', severity: 'low', category: '环境读取', message: 'process.env 环境变量读取', src: "process\\.env" },
  • 环境变量里常混有密钥。看它是否把整个 env 都读走。
• 动态加载模块dynamic-import一般4 处命中
  • lib/index.js:483{ id: 'C_MODULE_LOAD', severity: 'medium', category: '模块加载', message: '浏览器端 import()/require()', src: "import\\s*\\(|req…
  • README.en.md:93- **Client rules**: `document.cookie`, `innerHTML` XSS, browser `eval`, dynamic module loading (high); import()/require,…
  • README.md:95- **Client 规则**:`document.cookie`、`innerHTML` XSS、浏览器 `eval`、动态模块加载(high);import()/require、存储、网络、跳转、postMessage、Service …
  • 动态导入本身无害，但会让依赖关系难以静态审阅。

这份报告的口径

扫描读的是包里的源码，全程不执行第三方代码。 本次读了 7 个文件，扫描于 2026-10-16。 命中次数只说明源码里出现过这类调用，不说明行为好坏——一个插件读写文件 500 次和 1 次 都会被记为「读取文件」。这是能力清单，不是信任评分，也不是安全结论： 没扫出来的能力可能只是写法没被规则覆盖。

可信度

• -10声明与实物不符：peer-static-import-unsatisfiedmajor：{'@deepseek-ai/dsh-tools': '^0.1.1-rc.2'}