Skip to content

fix(deps): install mpegts.js from github#520

Merged
xrgzs merged 3 commits into
OpenListTeam:mainfrom
xrgzs:fix/pnpm
May 16, 2026
Merged

fix(deps): install mpegts.js from github#520
xrgzs merged 3 commits into
OpenListTeam:mainfrom
xrgzs:fix/pnpm

Conversation

@xrgzs

@xrgzs xrgzs commented May 16, 2026

Copy link
Copy Markdown
Member

Description / 描述

通过指定 mpegts.js 的版本为 GitHub Commit 升级依赖,移除 webworkify-webpack 子依赖,修复 mpegts.js 一些已知问题

与 1.8.0 版本的 diff:xqq/mpegts.js@v1.8.0...c91ab33

Motivation and Context / 背景

  • mpegts.js 依赖修改版的xqq/webworkify-webpack,导致PNPM 11报错 ERR_PNPM_EXOTIC_SUBDEP
// mpegts.js/package.json
"dependencies": {
  "es6-promise": "^4.2.5",
  "webworkify-webpack": "xqq/webworkify-webpack"
},

报错日志:

Progress: resolved 1, reused 0, downloaded 0, added 0
Progress: resolved 73, reused 0, downloaded 0, added 0
Progress: resolved 75, reused 0, downloaded 0, added 0
Progress: resolved 256, reused 0, downloaded 0, added 0
Progress: resolved 413, reused 0, downloaded 0, added 0
Packages are hard linked from the content-addressable store to the virtual store.
  Content-addressable store is at: /runner/cache/others/pnpm/store/v11
  Virtual store is at:             node_modules/.pnpm
[WARN] The git-hosted package fetched from "https://codeload.github.com/xqq/webworkify-webpack/tar.gz/24d1e719b4a6cac37a518b2bb10fe124527ef4ef" has to be built but the build scripts were ignored.
[ERR_PNPM_EXOTIC_SUBDEP] Exotic dependency "webworkify-webpack" (resolved via git-repository) is not allowed in subdependencies when blockExoticSubdeps is enabled

This error happened while installing the dependencies of mpegts.js@1.8.0
  • mpegts.js 仓库早已修复上述问题 xqq/mpegts.js@15d0669 (直接将改仓库复制到项目里面,因为原项目看似不再维护),但未发新版 npm 包

xqq/mpegts.js#260 (comment)

How Has This Been Tested? / 测试

  • 直接修改 mpegts.js 为原仓库GitHub Commit,pnpm install成功,vite build构建失败
  • 无法使用 pnpm patch 给 mpegts.js 增加 prepare hook,只能通过 fork 仓库实现
  • fork 仓库后打包正常
  • 测试播放 m2ts 文件正常(之前无法播放m2ts),不支持 seek 是已知问题不用管

Checklist / 检查清单

  • I have read the CONTRIBUTING document.
    我已阅读 CONTRIBUTING 文档。
  • I have formatted my code with go fmt or prettier.
    我已使用 go fmtprettier 格式化提交的代码。
  • I have added appropriate labels to this PR (or mentioned needed labels in the description if lacking permissions).
    我已为此 PR 添加了适当的标签(如无权限或需要的标签不存在,请在描述中说明,管理员将后续处理)。
  • I have requested review from relevant code authors using the "Request review" feature when applicable.
    我已在适当情况下使用"Request review"功能请求相关代码作者进行审查。
  • I have updated the repository accordingly (If it’s needed).
    我已相应更新了相关仓库(若适用)。

…fda7fcad9836c628eb328b9

Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
@xrgzs xrgzs changed the title fix(deps): update mpegts.js to github:xqq/mpegts.js#c91ab33a98f63a5b2… fix(deps): install mpegts.js from github May 16, 2026
xrgzs added 2 commits May 16, 2026 17:35
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
Signed-off-by: MadDogOwner <xiaoran@xrgzs.top>
@xrgzs xrgzs added bug Something isn't working enhancement New feature or request dependencies Pull requests that update a dependency file labels May 16, 2026
@xrgzs xrgzs marked this pull request as ready for review May 16, 2026 10:13
@xrgzs xrgzs requested a review from j2rong4cn May 16, 2026 10:13

@j2rong4cn j2rong4cn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

看起来没问题

@xrgzs xrgzs merged commit 58340d3 into OpenListTeam:main May 16, 2026
1 check passed
@xrgzs xrgzs deleted the fix/pnpm branch May 16, 2026 10:51
@j2rong4cn

Copy link
Copy Markdown
Member

@xrgzs 可以把hope-ui也替换一下吗,

@xrgzs

xrgzs commented May 17, 2026

Copy link
Copy Markdown
Member Author

@xrgzs 可以把hope-ui也替换一下吗,

@j2rong4cn 看看能不能从 https://github.com/PianoRhythm/hope-ui 迁移点代码过来

@j2rong4cn

Copy link
Copy Markdown
Member

@xrgzs 可以把hope-ui也替换一下吗,

@j2rong4cn 看看能不能从 https://github.com/PianoRhythm/hope-ui 迁移点代码过来

他这个有什么特别的吗。我主要想解决编辑器报错的问题

@j2rong4cn

Copy link
Copy Markdown
Member
image 替换后 {B1B09421-418A-425F-A6B5-B5B9D9262057}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants