Parsing: strictly enforce the produced AST matches the spec and enforce most "error recovery" parsing errors #1852
Description
{ "rules": { "no-throw-literal": "error" } }throwJust that in a file somewhere
Expected Result
This should be a parse-error and should not be linted on.
Actual Result
Parser creates a ThrowStatement with argument set to null, which breaks rules downstream, see eslint/eslint#13143 for context
Additional Info
[Error - 1:41:32 PM] TypeError: Cannot read property 'type' of null Occurred while linting [fileName].ts:132 at Object.couldBeError (eslint/lib/rules/utils/ast-utils.js:1262:22) at ThrowStatement (eslint/lib/rules/no-throw-literal.js:38:31) at eslint/lib/linter/safe-emitter.js:45:58 at Array.forEach (<anonymous>) at Object.emit (eslint/lib/linter/safe-emitter.js:45:38) at NodeEventGenerator.applySelector (eslint/lib/linter/node-event-generator.js:254:26) at NodeEventGenerator.applySelectors (eslint/lib/linter/node-event-generator.js:283:22) at NodeEventGenerator.enterNode (eslint/lib/linter/node-event-generator.js:297:14) at CodePathAnalyzer.enterNode (eslint/lib/linter/code-path-analysis/code-path-analyzer.js:634:23) at eslint/lib/linter/linter.js:936:32 Versions
| package | version |
|---|---|
@typescript-eslint/parser | ^2.13.0 |
TypeScript | ^3.8.2 |
ESLint | 5.16.0 |
node | 12.16.0 |
npm | 6.13.7 |