Skip to content

fix: 修复 pnpm dev 时特定 Node.js 版本 Electron 半安装导致的 Electron uninstall 启动失败 - #112

Open
MoYingJi wants to merge 1 commit into
SPlayer-Dev:devfrom
MoYingJi:pr/fix/dev
Open

fix: 修复 pnpm dev 时特定 Node.js 版本 Electron 半安装导致的 Electron uninstall 启动失败#112
MoYingJi wants to merge 1 commit into
SPlayer-Dev:devfrom
MoYingJi:pr/fix/dev

Conversation

@MoYingJi

Copy link
Copy Markdown
Collaborator

改动类型

  • 新功能(feat)
  • 缺陷修复(fix)
  • 重构 / 优化(不改变对外行为)
  • 文档(docs)
  • 其他(请在「改动说明」中注明)

是否包含破坏性变更

  • 是(请在「改动说明」中详细描述)

改动说明

Node.js 26.1.0+ 和 24.16.0+ 时,pnpm install Electron 会出现 “半安装” 状态,导致 pnpm dev 时报错 Electron uninstall

按照 Electron 推荐的修复方法,将 yauzl 升级到 ^3.3.1

Electron 已在最新版本修复此问题,但升级 Electron 可能引发一点问题我就没这么做。如果选择升级 Electron 版本,可以关闭此 PR。合并此 PR 后升级了 Electron 也可以将 overrides 里的 yauzl 删掉

测试情况

已在 Arch Linux,nodejs 26.5.0-1 测试通过

自查清单

  • 本 PR 只包含一个主要功能 / 修复,没有夹带无关改动
  • 已在本地完整测试通过;AI 生成的代码同样自行测试并审阅过,未做未经验证的提交
  • 已运行 pnpm format,并确认 pnpm typecheckpnpm lint 通过
  • 改动涉及原生模块时已 pnpm build:native 验证;未手写 native/*/index.d.ts
  • 已向 dev 分支提交

Copilot AI review requested due to automatic review settings July 28, 2026 14:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

此 PR 通过覆盖 yauzl 版本,修复特定 Node.js 版本下 Electron 安装不完整导致的开发启动失败。

变更:

  • yauzl 强制升级至兼容版本 ^3.3.1
  • 同步更新 pnpm 锁文件及传递依赖。

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
pnpm-workspace.yaml 添加 yauzl 版本覆盖配置。
pnpm-lock.yaml 锁定 yauzl@3.4.0 并更新依赖快照。
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

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