Some dependencies are extraneous

[EvgenyOrekhov]

For example, the try-catch is used here:

putout/packages/engine-parser/lib/second-chance.js

Line 3 in 68d2707

const tryCatch = require('try-catch');

but it's declared in devDependencies, not dependencies:

putout/packages/engine-parser/package.json

Lines 48 to 68 in 68d2707

	"devDependencies": {
	"@cloudcmd/stub": "^3.0.0",
	"acorn": "^8.1.1",
	"acorn-jsx": "^5.0.2",
	"acorn-stage3": "^4.0.0",
	"c8": "^7.5.0",
	"eslint": "^8.0.1",
	"eslint-plugin-node": "^11.0.0",
	"eslint-plugin-putout": "^12.0.0",
	"espree": "^9.0.0",
	"esprima": "^4.0.1",
	"just-camel-case": "^4.0.2",
	"lerna": "^4.0.0",
	"madrun": "^8.0.1",
	"montag": "^1.0.0",
	"nodemon": "^2.0.1",
	"putout": "*",
	"supertape": "^6.0.0",
	"tenko": "^2.0.0",
	"try-catch": "^3.0.0"
	},

And if you use Yarn v3, it will give you the following error:

Error: Failed to load plugin 'putout' declared in '.eslintrc.json » eslint-config-hardcore » ./base.json': @putout/engine-parser tried to access try-catch, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Required package: try-catch
Required by: @putout/engine-parser@npm:4.9.0 (via /Users/evgeny/Downloads/next-ts/.yarn/cache/@putout-engine-parser-npm-4.9.0-40929d5a11-78a1d5c5c8.zip/node_modules/@putout/engine-parser/lib/)

Require stack:
- /Users/evgeny/Downloads/next-ts/.yarn/cache/@putout-engine-parser-npm-4.9.0-40929d5a11-78a1d5c5c8.zip/node_modules/@putout/engine-parser/lib/second-chance.js

The solution for this particular case is to move try-catch from devDependencies to dependencies.
But I suspect there might be similar problems in other Putout packages, so I would highly recommend enabling the import/no-extraneous-dependencies rule and checking all Putout packages.

Original issue: EvgenyOrekhov/eslint-config-hardcore#385.

[coderaiser]

Thank you! Fixed in @putout/engine-parser v4.9.1 🎉 , is it works for you?

[coderaiser]

But I suspect there might be similar problems in other Putout packages, so I would highly recommend enabling the import/no-extraneous-dependencies rule and checking all Putout packages.

Thanks :), just added.

[EvgenyOrekhov]

It works now! Thanks for a quick fix!