Dependency graph improvements (via npmgraph.js.org)
- 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
Assessment
This issue has not been assessed yet.