[Bug Report]：Windows 上 dsh plugin 的含空格路径参数会被拆断

[Pidreamleaves]

[Bug Report]：Windows 上 dsh plugin 的含空格路径参数会被拆断

我在用 Open Design 桌面版（把 DeepSeek Harness 作为设计引擎）时报错：它自动安装连接组件 @open-design/dsh-runtime 失败，报
DeepSeek Harness could not install the Open Design connection component.。查下来根因在 dsh ，所以提交这个 bug。

1. 环境

项	值
dsh	0.1.0-rc.6
系统	Windows 11，PowerShell 5.1
Node.js	v22.21.1
pnpm	任意近期版本，与版本无关

复现不需要 Open Design，本地任意一个含空格的包路径就能触发。

2. 复现步骤

# 把任意一个有效 .tgz 放到含空格的目录里
$dir = "$env:USERPROFILE\my plugins"
New-Item -ItemType Directory -Path $dir -Force | Out-Null
Copy-Item .\some-package-0.0.0.tgz "$dir\some-package-0.0.0.tgz" -Force

dsh plugin --profile test add "$dir\some-package-0.0.0.tgz"

Open Design 的场景等价于这条命令，它的连接组件默认路径带空格：

dsh plugin --profile open-design add "C:\Users\Pii\AppData\Local\Programs\Open Design\resources\open-design\agent-runtimes\deepseek-harness\open-design-dsh-runtime-0.1.0.tgz"

3. 实际行为 vs 期望行为

安装失败，每次必现。路径在空格处被拆成两段传给 pnpm。前半段（...\Programs\Open）被 pnpm 当成一个不存在的路径规格，报 ENOENT；后半段（Design\resources\...\x.tgz）变成多余参数，按相对路径从 profile 目录解析。实际报错：

ENOENT: no such file or directory, open 'C:\Users\Pii\.dsh\profiles\open-design\Design\resources\open-design\agent-runtimes\deepseek-harness\open-design-dsh-runtime-0.1.0.tgz'

Open Design 安装器对用户显示的是：

DeepSeek Harness could not install the Open Design connection component.
No agent selection was changed.

期望 pnpm 把完整路径当一个参数收下，安装成功。

4. 根因

apps/cli/src/plugin.ts 的 runPlugin（源码）在 Windows 上用 shell: true 转发 pnpm，参数不转义：

const result = spawnSync('pnpm', args.map(argument => anchorPathSpec(argument, process.cwd())), {
  cwd: dir,
  stdio: 'inherit',
  shell: process.platform === 'win32',
})

拆开看有三个细节：

• anchorPathSpec 只重写相对路径（.、..、file:、link:），绝对路径原样透传，它自己的注释就写着 "Absolute specs ... pass through untouched"。
• Node 的 shell: true 把命令和参数用空格拼成一行交给 cmd.exe，不做转义。Node 22 起对这个用法打 DEP0190 弃用警告。
• 仓库里其实已经记录过这个坑。scripts/verify-node-next-types.ts 的注释写 "shell:true, which space-joins args UNESCAPED (DEP0190) — a hazard"，那里改用 execFileSync(process.execPath, [...]) 绕开了，runPlugin 没跟上。

我在本机验证过：spawnSync(node, ['-e', …, 'C:/…/Open Design/…/x.tgz'], { shell: true })，cmd 在空格处就把参数拆了，报 'C:\Program' is not recognized。

5. 影响范围

触发条件是 Windows、本地路径规格（tarball、目录或 file:）、路径带空格，三个凑齐必现。从 registry 装包（dsh plugin add @scope/pkg）不受影响，包名没空格；macOS 和 Linux 上 shell 是 false，也不受影响。

受害的集成有 Open Design 桌面版，它自动装连接组件时路径在 %LOCALAPPDATA%\Programs\Open Design\...，必然撞上。

严重性算中等：报错清晰、不损坏数据，但确定性强，报错内容也不会让人联想到空格问题，还挡住了一个真实的三方集成。建议 P2。

6. 临时规避

先把 tarball 复制到无空格路径再 add：

New-Item -ItemType Directory -Path "$env:USERPROFILE\.dsh\od-runtime" -Force | Out-Null
Copy-Item "C:\Users\Pii\AppData\Local\Programs\Open Design\resources\open-design\agent-runtimes\deepseek-harness\open-design-dsh-runtime-0.1.0.tgz" `
          "$env:USERPROFILE\.dsh\od-runtime\open-design-dsh-runtime-0.1.0.tgz" -Force
dsh plugin --profile open-design add "$env:USERPROFILE\.dsh\od-runtime\open-design-dsh-runtime-0.1.0.tgz"

7. 修复方向

两个办法：

1. 找到 pnpm 的 JS 入口（.cmd 背后的 pnpm.cjs），用 spawnSync(process.execPath, [pnpmJs, ...args], { shell: false }) 启动。参数数组语义由 Node 保证，不经 shell 拼接，这也是 verify-node-next-types.ts 已经在用的办法。推荐这个。
2. 保留 shell: true，但对每个参数做 cmd 引号转义。能修，顺带消掉 DEP0190 警告。

8. 备注

行号和源码链接基于 master 当前代码（apps/cli/src/plugin.ts、scripts/verify-node-next-types.ts）。Open Design 只是发现场景，最小复现见 §2，不依赖它。

[Electricitysheep]

Windows 含空格路径被拆断——命令行参数解析的经典问题（未加引号/未转义空格）。

排查：dsh plugin 命令的路径参数加引号（"C:/path with space/..."），或检查调用方是否做了转义。Windows 安装/路径坑见第 3 章 + 第 12 章：https://github.com/Electricitysheep/dsh-handbook/blob/main/docs/03-profiles.md

[Pidreamleaves]

谢谢回复，不过这里调用方已经加了引号，问题不在这：

dsh plugin --profile test add "$env:USERPROFILE\my plugins\some-package-0.0.0.tgz"

PowerShell 会把引号内路径作为一个参数传给 dsh，这步没问题。拆断发生在 dsh 内部转发：apps/cli/src/plugin.ts 的 runPlugin 在 Windows 上 spawnSync('pnpm', args, { shell: true })，Node 用空格拼接参数且不转义（Node 22 起打 DEP0190 警告），调用方的引号到不了这一跳。

对照实验：同样语法、同一个 tarball 复制到无空格路径，dsh plugin add 就成功。引号一样，唯一变量是路径，可见问题在 dsh 转发而非调用方。

可以跑下上面的最小复现，几秒就能确认。修复应落在 runPlugin：改用 shell: false 执行 pnpm 的 JS 入口，或转发前做 cmd 转义。

[argszero]

分析完全正确——我把你的三点根因对着 master 源码逐一核实，全部成立，而且仓库自己的脚本早就记录了这个坑。补充几个关键事实 + 修复方向设计。

核实结果

1. anchorPathSpec 只重写相对路径：apps/cli/src/plugin.ts:108-111，正则 ^(?<prefix>(?:file|link):)?(?<path>\.{1,2}(?:[/\\].*)?)$ 只匹配 ./.. 开头，绝对路径原样返回——你的 tgz 绝对路径确实零处理直接进 argv。
2. shell: true 不转义：plugin.ts:129-133，spawnSync('pnpm', args..., { shell: process.platform === 'win32' })。shell:true 时 Node 把 command + args 用空格拼成一行交给 cmd.exe，不对参数做任何引号处理。
3. 仓库自己的文档：scripts/verify-node-next-types.ts:148 注释原文就是 "shell:true, which space-joins args UNESCAPED (DEP0190) — a hazard for the..."——这个坑在仓库内部已被识别，只是还没修到 runPlugin。
4. 为什么不能简单去掉 shell：注释里写了 CVE-2024-27980 加固——Node 在 Windows 上 shell: false 直接 spawn .cmd/.bat 会抛错（这正是当初加 shell:true 的原因）。所以修复必须保留 shell 但正确引号化，而不是去掉 shell。

修复方向（供维护者参考）

方案 A（最小改动）：runPlugin 里对每个参数做 cmd.exe 引号化后再交给 shell:true——含空白或 cmd 元字符（& | < > ^ ( ) % !）的参数用双引号包裹，内部双引号翻倍：

function quoteForCmd(argument: string): string {
  // cmd.exe 引号规则：含空白/元字符才加引号，内部 " 翻倍
  return /[\s&|<>^()%!"]/.test(argument)
    ? `"${argument.replace(/"/g, '""')}"`
    : argument
}
// spawnSync('pnpm', args.map(anchorPathSpec).map(quoteForCmd), { shell: true })

注意 windowsVerbatimArguments: true 在 shell: true 下是被忽略的，所以手动引号化是必须的，不能靠这个选项。

方案 B（npm 同款，更可控）：不用 shell:true，直接 spawnSync('cmd.exe', ['/d','/s','/c', pnpm ${args.map(quoteForCmd).join(' ')}], { windowsVerbatimArguments: true })——整条命令行自己拼、自己引号化，行为完全确定，还能消除 DEP0190 弃用警告。

建议：修复放 anchorPathSpec 之后独立加一个纯函数（如 quoteForCmd），方便单测；apps/cli/tests/ 目前没有任何 runPlugin/anchorPathSpec 测试，正好补一个 Windows 路径含空格（C:\Users\Pii\my plugins\x.tgz）的单元测试覆盖。

给用户的临时 workaround

在修复合入前，Open Design 这类调用方可以先把路径改成无空格副本（如 %TEMP%\od\open-design-dsh-runtime.tgz）或 8.3 短路径（dir /x 查 PROGRA~1 形式）。但这是绕过，真正的修复在 runPlugin 的引号化。

需要的话我可以把方案 A 的补丁（含单测）整理成完整草案贴上来，等上游开放 PR 通道直接用。

[Electricitysheep]

你说得对，我之前的排查方向偏了——调用方引号没问题是表象，根子在 runPlugin 的 spawnSync('pnpm', { shell: true })：Node 用空格拼接参数不转义，转发时丢引号（DEP0190 警告确实是信号）。对照实验（同语法无空格路径成功）很有说服力。

修复方向同意：shell: false 走 pnpm 的 JS 入口，或转发前转义。这个案例（Windows 转发层空格路径）会补进手册第 3 章 Windows 坑位。