Inquirer.js
Inquirer.js copied to clipboard
SBoudrias
Consider making `@types/*` dev dependencies
It would be great if the @types/* were not direct dependencies but dev dependencies as to avoid the pit fall of having the types for node 20, but running under node 18.
This cannot work today because Inquirer code accesses the node types. So it cannot ship without the @types/node as the imported types will be missing. Specifically this caused a bug in the past, see https://github.com/SBoudrias/Inquirer.js/issues/1241
I'm open to review this or change the code somewhat to get to a better result. But I'll need documentation and reference from you supporting how this is possible. We can work together on that if you're willing to.
it looks like you only ship javascript in your package. so the types package is not directly needed. i would advise using this as a devDependency https://stackoverflow.com/questions/18875674/whats-the-difference-between-dependencies-devdependencies-and-peerdependencie
That is not the case, the types are shipped. See https://www.npmjs.com/package/@inquirer/core?activeTab=code under dist/esm/types/ for one example.
then a peerDependency i think might be more appropriate, with the minimum supported version of node
This also add quite a lot of size to install the package.
2Mb from @types/mute-stream and 2Mb from @types/node.
npx howfat @inquirer/core
@inquirer/[email protected] (28 deps, 2.57mb, 385 files, ©MIT)
├── @inquirer/[email protected] (4.54kb, 7 files, ©MIT)
├─┬ @types/[email protected] (2 deps, 2.01mb, 105 files, ©MIT)
│ ╰── @types/[email protected] (🔗, 1 dep, 2mb, 100 files, ©MIT)
├─┬ @types/[email protected] (1 dep, 2mb, 100 files, ©MIT)
│ ╰── [email protected] (71.34kb, 35 files, ©MIT)
├── @types/[email protected] (3kb, 4 files, ©MIT)
├─┬ [email protected] (1 dep, 132.26kb, 51 files, ©MIT)
│ ╰── [email protected] (116.26kb, 46 files, ©(MIT OR CC0-1.0))
├─┬ [email protected] (5 deps, 95.08kb, 33 files, ©MIT)
│ ├─┬ [email protected] (2 deps, 49.67kb, 16 files, ©MIT)
│ │ ╰─┬ [email protected] (1 dep, 33.09kb, 11 files, ©MIT)
│ │ ╰── [email protected] (6.54kb, 4 files, ©MIT)
│ ╰─┬ [email protected] (1 dep, 11.19kb, 10 files, ©MIT)
│ ╰── [email protected] (4.32kb, 5 files, ©MIT)
├── [email protected] (31.36kb, 6 files, ©MIT)
├── [email protected] (4.66kb, 5 files, ©ISC)
├─┬ [email protected] (1 dep, 14.42kb, 9 files, ©MIT)
│ ╰── [email protected] (2.63kb, 4 files, ©MIT)
├── [email protected] (6.28kb, 4 files, ©ISC)
├── [email protected] (75.16kb, 29 files, ©ISC)
├─┬ [email protected] (1 dep, 9.41kb, 10 files, ©MIT)
│ ╰── [email protected] (5.48kb, 5 files, ©MIT)
╰─┬ [email protected] (8 deps, 125.4kb, 48 files, ©MIT)
├─┬ [email protected] (2 deps, 49.67kb, 16 files, ©MIT)
│ ╰─┬ [email protected] (1 dep, 33.09kb, 11 files, ©MIT)
│ ╰── [email protected] (6.54kb, 4 files, ©MIT)
├─┬ [email protected] (4 deps, 66.45kb, 28 files, ©MIT)
│ ├── [email protected] (47.12kb, 8 files, ©MIT)
│ ├── [email protected] (4.88kb, 5 files, ©MIT)
│ ╰── [email protected] (🔗, 1 dep, 9.41kb, 10 files, ©MIT)
╰── [email protected] (🔗, 1 dep, 9.41kb, 10 files, ©MIT)
