Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce npm cli (npm) dependency count #64

Open
AbhiPrasad opened this issue Jul 2, 2024 · 4 comments
Open

Reduce npm cli (npm) dependency count #64

AbhiPrasad opened this issue Jul 2, 2024 · 4 comments
Labels
rejected Changes to this package were rejected by the repo maintainer. Maybe try again later

Comments

@AbhiPrasad
Copy link

AbhiPrasad commented Jul 2, 2024

https://www.npmjs.com/package/npm

https://npmgraph.js.org/?q=npm

It's a lot. At current time of creating the issue it's 55 MB / 185 deps.

I think there is opportunity for us to slowly go in and modernize everything.

For example, https://github.com/npm/cacache has a bunch of stale dependabot PRs that would help reduce the dep count.

@AbhiPrasad AbhiPrasad changed the title Reduce npm Reduce npm cli (npm) dependency count Jul 2, 2024
@AbhiPrasad
Copy link
Author

first change I'm looking at is with cacache, specifically eliminating p-map to prevent it from bringing in aggregate-error. This was driven by my desire to improve semantic-release as well: npm/cacache#281

@AbhiPrasad
Copy link
Author

AbhiPrasad commented Jul 2, 2024

Well the cacache change was shot down unfortunately! npm/cacache#281 (comment)

It seems like npm cli changes will be harder than expected! It might not be feasible to switch glob for example as well.

@43081j
Copy link
Collaborator

43081j commented Jul 3, 2024

Bumping the engines version would possibly let you get rid of aggregate error at least

I would ask them how likely it is that kind of change would be accepted. And what they intend to do about an esm only dependency

@SuperchupuDev
Copy link

SuperchupuDev commented Jul 5, 2024

Removing util-deprecate from postcss-selector-parser shouldn't hard to do and it would be a small win for npm's dependency count https://npmgraph.js.org/?q=npm#select=exact%3Autil-deprecate%401.0.2
(relevant: postcss/postcss-selector-parser#210)
image

@Fuzzyma Fuzzyma added the rejected Changes to this package were rejected by the repo maintainer. Maybe try again later label Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rejected Changes to this package were rejected by the repo maintainer. Maybe try again later
Projects
None yet
Development

No branches or pull requests

4 participants