@types deps should be listed under devDependencies

[lionello]

http-proxy-middleware/package.json

Line 84 in 7c86a2c

"@types/http-proxy": "^1.17.5",

Listing @types/http-proxy under the "dependencies" pulls in other types as runtime dependencies as well. These should only be needed during development.

This issue was introduced in #369

[chimurai]

http-proxy-middleware types are generated and shipped with the package as well:

http-proxy-middleware/package.json

Line 6 in 7c86a2c

"types": "dist/index.d.ts",

Removing @types/http-proxy from dependencies, will lead to a broken developer experience.
target is a transitive property from http-proxy (@types/http-proxy), which will be missing in the code completion.

Below you'll see target is missing:

[chimurai]

Feel free to reopen if you have suggestions to remove @types/http-proxy without breaking developer experience

[lionello]

Ah, yeah you're right. Only by listing the types under "dependencies" do they actually get pulled in automatically in the consuming project. TIL!
https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html#dependencies