[Fix] no-webpack-loader-syntax/TypeScript: avoid crash on missing name

[leonardodino]

Tiny fix 😄

Every time I was manually adding a import on a typescript in vscode file it would trigger this rule, because the parser interprets the code as valid, passing an incomplete AST node down.

The test case seems a bit weird, but it's what naturally happens when typing on the top part of a file, and having eslint constantly running 😄

Logs from a usual session:

[Error] ESLint stack trace:
[Error] TypeError: Cannot read property 'indexOf' of undefined
Occurred while linting /redacted/file.tsx:1
    at reportIfNonStandard (/redacted/node_modules/eslint-plugin-import/lib/rules/no-webpack-loader-syntax.js:5:12)
    at handleImports (/redacted/node_modules/eslint-plugin-import/lib/rules/no-webpack-loader-syntax.js:24:9)
    at /redacted/node_modules/eslint/lib/linter/safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
    at Object.emit (/redacted/node_modules/eslint/lib/linter/safe-emitter.js:45:38)
    at NodeEventGenerator.applySelector (/redacted/node_modules/eslint/lib/linter/node-event-generator.js:254:26)
    at NodeEventGenerator.applySelectors (/redacted/node_modules/eslint/lib/linter/node-event-generator.js:283:22)
    at NodeEventGenerator.enterNode (/redacted/node_modules/eslint/lib/linter/node-event-generator.js:297:14)
    at CodePathAnalyzer.enterNode (/redacted/node_modules/eslint/lib/linter/code-path-analysis/code-path-analyzer.js:711:23)
    at /redacted/node_modules/eslint/lib/linter/linter.js:952:32

[coveralls]

Coverage decreased (-2.1%) to 95.774% when pulling 6797bff on leonardodino:avoid-crash-no-webpack-loader-syntax into 957092a on benmosher:master.

[coveralls]

Coverage remained the same at 97.904% when pulling dd4e416 on leonardodino:avoid-crash-no-webpack-loader-syntax into 957092a on benmosher:master.

[ljharb]

Thanks, this looks great! (assuming the test fails without the fix)

[JounQin]

@ljharb It seems this fix has not been released yet?

https://unpkg.com/browse/eslint-plugin-import@2.22.1/lib/rules/no-webpack-loader-syntax.js

Can you help to release it??

[ljharb]

That's correct, it hasn't been. Until yesterday, we've been blocked by #1986.

After I move through all the open PRs, I'll cut a release.