Support for checking `require.resolve()` · import-js/eslint-plugin-import#585

(2 comments) (4 reactions) (0 assignees)JavaScript (4,946 stars) (1,540 forks)batch import

acceptedenhancementhelp wanted

説明

Sometimes I use require.resolve() just to resolve paths to other resources than javascript modules.

Okay, this is not a regular import, but it would be nice if this plugin could also check these paths.

Maybe it could be a new setting like 'import/check-require-resolve': true, whereas various rules related to CommonJS imports also apply here.

技術スタック: javascriptnodejs
領域: tooling
Issue 種別: feature
難度: 3
推定時間: half day
活動状況: fresh
明確さ: clear
前提条件: JavaScriptNode.js require.resolve()ESLint plugin developmenteslint plugin import structure
初心者向け度: 60
調査方針: Examine how the plugin currently handles `require()` calls, particularly in rules like `no unresolved` and `no extraneous dependencies`. Identify where `require.resolve` is used as an expression and consider extending the AST visitor to detect `CallExpression` with callee `require.resolve`. Introduce a configuration option, e.g., `'import/check require resolve': true`, and apply the same import validation logic to the arguments of `require.resolve()`.