[BUG] Please fix the wrong usage of module-alias inside zeus cli #111
fhfuih
started this conversation in
Developer Feedback
Replies: 2 comments
-
Thank you for your feedback, the relevant colleagues are already in the process of fixing. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Thanks. Just for your information, when installing Zeus-cli locally inside the project (-D) and using `pnpm exec zeus…` (essentially finding and running executables from local node_modules/bin), the error message is different. It complains to have failed to require a zeppos-..-utils file. But the fix is the same.
在 2023年3月14日,上午10:27,Myoung XUE - Zepp Health ***@***.***> 写道:
Thank you for your feedback, the relevant colleagues are already in the process of fixing.
—
Reply to this email directly, view it on GitHub<#111 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACVTZ6NJPKEOWRWJUYDD5A3W37JPZANCNFSM6AAAAAAVX3Z6RI>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I noticed that zeus cli uses a package called
module-alias
, and the usage is problematic. This causes the zeus cli to abort when installed bypnpm
Bug
pnpm
in node.js environmentpnpm add -g @zeppos/zeus-cli
Why it's
zeus-cil
's faultAs stated on
module-alias
's official readme regarding using this package as a dependency within another npm package, package developers should always specify thepackage.json
directory instead of usingregister()
to auto-detect.In pnpm (and presumably more future package managers), global node_modules are organized differently on the file system. This breaks some assumptions of the global package's file structures for npm users.
Temporary fix
In
bin/main.js
, change the following line (3)to
Looking forward to an official update!
Beta Was this translation helpful? Give feedback.
All reactions