expressjs / expressjs/express

Dependency graph improvements (via npmgraph.js.org)

Open
#6,647 4 comments 1 reaction 0 assignees View on GitHub
Dominant language
JavaScript
Stars
69.5k
Forks
25k
Avg merge
4d 20h
Merged PRs (30d)
9

Description

[Probably more of a tracking issue here than an actual issue in and of itself. I'm putting this up mostly because I'm curious if the `express` team has considered any of these concerns than out of any real expectation of action.]

See https://npmgraph.js.org/?q=express.

Under "Modules with Multiple Versions":

* [ ] `http-errors@2.0.0` is pinned to `statuses@2.0.1`, while the rest of the graph depends on `statuses@2.0.2`. Having the same clause appear in the `dependencies` section of the involved modules would avoid unnecessary duplication here.

Under "Suggested Replacements" (these come from the [e18e module-replacements project](https://github.com/es-tooling/module-replacements project)

* [ ] 6 of the suggestions - `es-define-property`, `es-errors`, `function-bind`, `gopd`, `has-symbols`, `hasown` - are part of the `get-intrinsics` module... all of which are suggested to be replaced with modern native equivalents. (This begs the question, "How much value is `get-intrinsics` really adding? attn @ljharb)
* [ ] `body-parser` - I'm skeptical of this one. module-replacements [suggests](https://github.com/es-tooling/module-replacements/blob/main/docs/modules/qs.md) replacing with inline code or `milliparsecs`, but I'm guessing `express` has more stringent requirements that make this non-actionable? Might be worth getting someone from e18e to comment on this. attn @43081j
* [ ] `inherits` - module-replacements suggests using native class syntax? Seems reasonable, given that ES class syntax has been widely available for a long time now.
* [ ] `qs` - This is pulled in by `body-parser` so may come for free if there's a viable alterantive to that library. module replacement [suggests](https://github.com/es-tooling/module-replacements/blob/main/docs/modules/qs.md) using native `URLSearchParams`, or other (mostly ESM-friendly) alternatives. Again might be worth input from e18e, attn @43081j

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.