dsh-code-security

检测报告

能力清单

• 以 shell 执行命令shell-exec高关注3 处命中
  • lib/rules.js:16{ id: 'SEC-003', title: 'Node.js spawn shell: true', cwe: 'CWE-78', severity: 'high', confidence: 'high', languages: JS,…
  • rules/semgrep.yml:18- pattern: execSync($CMD, ...)
  • rules/semgrep.yml:20- pattern: $CP.execSync($CMD, ...)
  • 看命令字符串是拼出来的还是写死的。拼接用户输入才是真正的风险。
• 写入文件writes-files需留意17 处命中
  • lib/engines.js:59await fs.mkdir(parent, { recursive: true });
  • lib/engines.js:73await fs.mkdir(path.dirname(file), { recursive: true });
  • lib/engines.js:74await fs.writeFile(file, doc.text, { flag: 'wx', mode: 0o600 });
  • lib/policy.js:55await fs.writeFile(tmp, JSON.stringify(policy, null, 2) + '\n', { flag: 'wx', mode: 0o600, signal });
  • 看写入位置。写自己的配置目录是正常的，写系统目录不是。
• 发起网络请求network需留意5 处命中
  • lib/client.js:66async function action(body,signal){const response=await fetch('api/dsh-code-security/workbench',{method:'POST',credentia…
  • lib/workbench.js:193async fetch(request) {
  • lib/workbench.js:241ctx.inject?.(['connection'], (host) => host.connection.fetch.register({ path: SECURITY_WORKBENCH_ROUTE, methods: ['POST'…
  • lib/workbench.d.ts:27fetch(request: Request): Promise<Response>;
  • 看请求发往哪些域名。发往作者自己的后端是常见做法。
• 启动伪终端spawns-pty需留意4 处命中
  • rules/semgrep.yml:17- pattern: exec($CMD, ...)
  • rules/semgrep.yml:18- pattern: execSync($CMD, ...)
  • rules/semgrep.yml:19- pattern: $CP.exec($CMD, ...)
  • rules/semgrep.yml:20- pattern: $CP.execSync($CMD, ...)
  • 伪终端等于给了一个完整 shell。确认这是插件的核心功能。
• 动态执行代码dynamic-code需留意1 处命中
  • CHANGELOG.md:11- 新增 4 项回归测试：fsmonitor 不执行、clean 过滤器不执行且藏不掉 `eval(input)`、`--attr-source` 参数、`includeUntracked` 的 `ls-files` 路径；同步更新既有的 …
  • 看 eval / new Function 的入参来自哪里。常量字符串和用户输入是两回事。
• 读取文件reads-files一般11 处命中
  • lib/audit.js:55text = await fs.readFile(absolute, { encoding: 'utf8', signal });
  • lib/deps.js:166entries = await fs.readdir(absolute, { withFileTypes: true });
  • lib/deps.js:191const text = await fs.readFile(absolute, { encoding: 'utf8', signal: options.signal }), file = rel || path.basename(abso…
  • lib/engines.js:19for (const entry of (await fs.readdir(directory, { withFileTypes: true })).slice(0, 200)) {
  • 读文件是绝大多数插件的正常工作方式，本身不构成顾虑。
• 读取环境变量reads-env一般7 处命中
  • lib/engines.js:15const defaultRoot = process.platform === 'win32' ? process.env.LOCALAPPDATA : process.platform === 'darwin' ? path.join(…
  • lib/engines.js:16const root = cfg.osvDatabaseDir || process.env.OSV_SCALIBR_LOCAL_DB_CACHE_DIRECTORY || process.env.OSV_SCANNER_LOCAL_DB_…
  • lib/engines.js:138const databaseRoot = cfg.osvDatabaseDir || process.env.OSV_SCALIBR_LOCAL_DB_CACHE_DIRECTORY || process.env.OSV_SCANNER_L…
  • lib/workbench.js:23this.directory = options.directory ?? path.join(process.env.DSH_HOME || path.join(homedir(), '.dsh'), 'plugins', 'dsh-co…
  • 环境变量里常混有密钥。看它是否把整个 env 都读走。
• 动态加载模块dynamic-import一般1 处命中
  • lib/audit.d.ts:17findings: import("./scanner.js").Finding[];
  • 动态导入本身无害，但会让依赖关系难以静态审阅。

这份报告的口径

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

可信度

• 0没有可扣分的项