Skip to content

docs(readme): add npm name collision warning - #8

Merged
Nicholas023 merged 1 commit into
mainfrom
docs/npm-name-warning
Oct 19, 2026
Merged

Nicholas023 merged 1 commit into
mainfrom
docs/npm-name-warning

Conversation

@Nicholas023

Copy link
Copy Markdown
Owner

背景

npm 上 dsh-code-review 属于另一个活跃项目(dsh-electron 生态的代码审查模式 agent preset),不是本插件。用户 npm install dsh-code-review 装到的不是本项目——踩坑记录 2026-10-18 条目已记录此决策。

改动

  • 安装章节开头加 ⚠️ 警告块:npm 同名包不是本插件,本插件不走 npm 分发
  • 步骤 1:从「npm 方式安装依赖」改为「从 GitHub Releases 下载」
  • npm install 注释从「发布安装:正式依赖」改为「仅安装本仓库声明的依赖,不会从 npm 装本插件」
  • 步骤 3:引擎获取改为「从 Release 下载编译产物」

验证

  • npm run check 语法检查通过(README 非 JS,不影响)
  • diff 确认仅修改安装章节,其余章节无变动

npm 上 dsh-code-review 属于另一个活跃项目(dsh-electron 生态),
npm install dsh-code-review 装到的不是本插件。
安装改为从 GitHub Release 获取,步骤 1/3 相应更新。
@Nicholas023
Nicholas023 merged commit 313be2c into main Oct 19, 2026
6 checks passed
@Nicholas023
Nicholas023 deleted the docs/npm-name-warning branch October 19, 2026 15:07
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.

1 participant