[BUG] ESLint crashes with `import/no-unused-modules`: Cannot read property 'get' of undefined · import-js/eslint-plugin-import#1716

(1 评论) (0 反应) (0 负责人)JavaScript (4,946 star) (1,540 fork)batch import

bughelp wanted

描述

Eslint-plugin-import crashes running eslint with Cannot read property 'get' of undefined error.

The error is in lib/rules/no-unused-module.js on line 262:

exportAll.forEach((value, key) => { value.forEach(val => { **const currentExports = exportList.get(val);** const currentExport = currentExports.get(EXPORT_ALL_DECLARATION); currentExport.whereUsed.add(key); }); }); However, if I add condition:

if (currentExports) { const currentExport = currentExports.get(EXPORT_ALL_DECLARATION); currentExport.whereUsed.add(key); } the lint works and I can see all the lint errors .

For now, I disabled the rule in .eslintrc.js file, but would it be possible to create a fix for it, or can I do it as well?

Version: "eslint-plugin-import": "^2.20.0"

贡献者指南

技术栈: javascriptnodejs
领域: backenddeveloper experience
议题类型: bug
难度: 2
预计时间: under 1 hour
活动状态: stale
清晰度: clear
前置要求: Basic JavaScriptFamiliarity with ESLint plugin rules
新手友好度: 85
研究方向: The bug occurs in lib/rules/no unused module.js at line 262 where exportList.get(val) can return undefined. The fix is to add a null check before accessing currentExports. The issue already provides the exact code change. Look at the file to understand the context and ensure the fix doesn't break other cases.