Skip to content

fix(dsh-plugin): bump dev cordis 4.0.4 -> 4.0.5 (pnpm lock only) - #3

Merged
KyoMio merged 1 commit into
mainfrom
fix/dsh-plugin-cordis-eresolve
Oct 28, 2026
Merged

KyoMio merged 1 commit into
mainfrom
fix/dsh-plugin-cordis-eresolve

Conversation

@kyomarshal

Copy link
Copy Markdown
Collaborator

改动(只动 dsh-plugin/)

  • package.json:devDependencies @deepseek-ai/cordis 4.0.4 → 4.0.5(npm latest)。peerDependencies 仍为 *,不排除实际安装版本;dsh 0.1.7-rc.2 各 dev 包要求的 cordis ~4.0.4 仍满足。
  • pnpm-lock.yaml:用 pnpm 11.7.0 --lockfile-only 重新解析。没有新增 package-lock.json。

包管理器说明

这个子目录用 pnpm,依据有三条:packageManager: pnpm@11.7.0;仓库里只有 pnpm-lock.yaml 和 pnpm-workspace.yaml;docs/dsh-plugin.md 和 dsh-plugin/README.md 写的安装步骤都是 pnpm install。npm install / npm ci 不在支持范围内。

原来报的 ERESOLVE 是用 npm 安装时出现的:cordis-plugin-include@1.0.10 要求 peer cordis ~4.0.5,而 devDependencies 锁的是 4.0.4,npm 的严格 peer 解析会直接报错。改动前 pnpm install --frozen-lockfile 本来就能成功(这个 peer 不匹配在 pnpm 下只是警告),所以这次改动的作用是把 dev 依赖对齐到满足 peer 要求的版本,不是修复 pnpm 安装失败。pnpm peers check 剩下的缺失 peer(dsh-workspace 等)改动前后完全一样,都是原来就有的。

验证

在断网 bwrap 沙箱里跑(--unshare-net,不挂 /home,env -i,不带凭据)。store 和元数据缓存事先在沙箱外用 --ignore-scripts 下载好。

Node pnpm install --frozen-lockfile --offline build verify node --test test
22.23.4 ✅(通过 supply-chain 校验) ✅ ✅ 112/112
24.22.1 ✅(通过 supply-chain 校验) ✅ ✅ 112/112

npm run build 的产物和已入库的 lib/ 完全一致,所以 lib 不用更新。

cordis-plugin-include@1.0.10 (cordis@4.0.4's optional peer ~1.0.9, also
pulled in by dsh-agent-preset-registry) peers on cordis ~4.0.5. npm's strict
peer resolver turns that into ERESOLVE; pnpm only warned. 4.0.5 still
satisfies every dsh 0.1.7-rc.2 dev package's cordis ~4.0.4 peer, and the
plugin's own peer range stays '*'.

dsh-plugin is a pnpm package (packageManager pnpm@11.7.0, pnpm-lock.yaml,
docs say pnpm install); only pnpm-lock.yaml is re-resolved, no npm lock is
added. lib/ build output unchanged.
@KyoMio
KyoMio merged commit 8d45e33 into main Oct 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants