declare global causing parse errors

[milesj]

What version of TypeScript are you using?

3.0

What version of typescript-eslint-parser are you using?

21.0.1

What code were you trying to parse?

declare global {
  namespace NodeJS {
    interface Process {
      beemo: {
        context: any;
        tool: BeemoTool;
      };
    }
  }
}

What did you expect to happen?

It parse.

What happened?

It fails with this error.

/Users/Miles/Sites/beemo/packages/core/src/index.ts
  17:15  error  Parse errors in imported module './types': Cannot read property '0' of undefined (undefined:undefined)  import/export

This has happened on 2 projects now, both because of declare global.

[milesj]

Probably more related to export * coupled with declare global.

Here's the index file: https://github.com/milesj/beemo/blob/master/packages/core/src/index.ts#L17

And types file: https://github.com/milesj/beemo/blob/master/packages/core/src/types.ts#L62

[armano2]

@milesj this issue is in https://github.com/benmosher/eslint-plugin-import

this plugin is not supporting typescript https://github.com/benmosher/eslint-plugin-import/issues?q=is%3Aopen+is%3Aissue+label%3Atypescript

can you report this issue there?

[milesj]

This was working before a recent parser upgrade though. Do you know exactly what changed that may have caused this?

[armano2]

@milesj hmm i did some testing with plain eslint configuration, without your wrappers, on core package from your repository. And i was not able to reproduce this issue.

{
  "parser": "typescript-eslint-parser",
  "parserOptions": {
    "ecmaVersion": 8,
    "sourceType": "module",
    "jsx": true,
    "useJSXTextNode": true
  },
  "env": {
      "browser": true,
      "node": true
  },
  "settings": {
    "import/resolver": {
      "node": {
        "extensions": [
          ".js",
          ".jsx",
          ".ts",
          ".tsx"
        ]
      }
    }
  },
  "extends": [
    "eslint:recommended",
    "plugin:import/errors",
    "plugin:import/warnings"
  ],
  "plugins": [
    "import",
    "typescript"
  ],
  "rules": {
    "no-unused-vars": "off",
    "no-undef": "off",
    "import/export": "error"
  }
}

$ eslint --ext .ts src/
Done in 1.43s.

[milesj]

This is the eslint config I'm using: https://github.com/milesj/build-tool-config/blob/master/packages/config/configs/eslint.js

Nothing atm stands out, except for.

'import/parsers': {
      'typescript-eslint-parser': ['.ts', '.tsx'],
},

[armano2]

yea i found it, and looks like its issue in AST after all xd

[armano2]

TypeError: Cannot read property '0' of undefined
    at isGlobalAugmentation (******\typescript-eslint-parser\analyze-scope.js:32:26)
    at Referencer.TSModuleDeclaration (******\typescript-eslint-parser\analyze-scope.js:578:13)
    at Referencer.Visitor.visit (******\typescript-eslint-parser\node_modules\esrecurse\esrecurse.js:104:34)
    at Referencer.Visitor.visitChildren (******\typescript-eslint-parser\node_modules\esrecurse\esrecurse.js:83:38)
    at Referencer.Program (******\typescript-eslint-parser\node_modules\eslint-scope\lib\referencer.js:423:14)
    at Referencer.Visitor.visit (******\typescript-eslint-parser\node_modules\esrecurse\esrecurse.js:104:34)
    at module.exports (******\typescript-eslint-parser\analyze-scope.js:685:16)
    at Object.parseForESLint (******\typescript-eslint-parser\parser.js:74:26)
    at Object.exports.parse (******\typescript-eslint-parser\parser.js:79:17)
    at parse (******\test\node_modules\eslint-module-utils\parse.js:36:17)

[milesj]

Oh nice! Luckily we found it.

[armano2]

this is related to JamesHenry/typescript-estree#27

[mysticatea]

The cause seems that eslint-plugin-import doesn't give implicit tokens option to the parser.