False positive on `no-undef` with mixed JS & TS codebase

[tchakabam]

What version of TypeScript are you using?

2.6.1

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

11.0.0

What code were you trying to parse?

export type SomeThing = {
    id: string;
}

What did you expect to happen?

No lint error

What happened?

False positive on no-undef eslint core rule.

It basically means we can not use type definitions without disabling that rule (globablly or locally)

This has already been discussed in #77

It really becomes tricky when the project mixes JS and TS code.

But we haven't found any viable solution yet, apart applying different configuration to the various file types.

It would be nice if the eslint core rules would be overridable by the Typescript parser usage somehow I guess.

BUT this is really only an issue for mixed JS/TS codebases, because:

let's keep in mind this is easily fixable if your code is Typescript only,
since you can boldly disable this core rule as the TS compiler should complain anyway about a
missing definition of something :)
It really only becomes an issue when you want to take care of JS code using the same lint config.

[tchakabam]

problem also appears on enumarations like here:

export enum Events {
    FOO: 'bar'
}

Error: 'FOO' is not defined

and Error: 'Events' is not defined

[StoneCypher]

Also happening here. Basically everything in this gets flagged no-undef:

type item = string;
type row  = item[];
type doc  = row[];



enum header_mode {

  none   = 'none',
  index  = 'index',
  number = 'number',
  first  = 'first'

}



enum quoting_circumstance {

  minimal        = 'minimal',
  always         = 'always',
  except_numbers = 'except_numbers',
  strict_nl      = 'strict_nl'

}



interface stringify_options {

  headers?                : row | false,
  quoter?                 : (s: string) => string,
  field_separator?        : string,
  row_separator?          : string,
  trailing_row_separator? : boolean

// , consumer: function
}



export { item, row, doc, header_mode, quoting_circumstance, stringify_options };

As so:

C:\Users\john\projects\csv_4180\src\ts\csv_types.ts
8:6 error 'header_mode' is not defined no-undef
10:3 error 'None' is not defined no-undef
11:3 error 'Index' is not defined no-undef
13:3 error 'First' is not defined no-undef
19:6 error 'quoting_circumstance' is not defined no-undef
21:3 error 'minimal' is not defined no-undef
22:3 error 'always' is not defined no-undef
23:3 error 'except_numbers' is not defined no-undef
24:3 error 'strict_nl' is not defined no-undef
30:11 error 'stringify_options' is not defined no-undef
32:3 error 'headers' is not defined no-undef
33:3 error 'quoter' is not defined no-undef
33:30 error 's' is not defined no-undef
34:3 error 'field_separator' is not defined no-undef
35:3 error 'row_separator' is not defined no-undef
36:3 error 'trailing_row_separator' is not defined no-undef
43:26 error 'header_mode' is not defined no-undef
43:39 error 'quoting_circumstance' is not defined no-undef
43:61 error 'stringify_options' is not defined no-undef

[tchakabam]

Let's keep in mind this is easily fixable if your code is Typescript only, since you can boldly disable this core rule as the TS compiler should complain anyway about a missing definition of something :)

It really only becomes an issue when you want to take care of JS code using the same lint config.

[StoneCypher]

unless you want the actual protection afforded by the rule, which is significant

[tchakabam]

hm, i must be missing something then :) what missing protection do you mean? when compiling actual typescript, the compiler will fail on accessing any symbol hat is "not defined" in the ecmascript sense :) so that protection from undefined symbols is basically ensured for any typescript code. the problem only occurs for code that is plain JavaScript (which the ts compiler also can process, but it will not apply the typescript compilation rules to it).

for example, in a proper typescript file, if you try to assign a variable like this: foobar = 4 and the symbol doesn't exist in any way in the respective scope (and which would also lead eslint to flag it with a no-undef error), then the Typescript compiler will also fail anyway. It will just look like this in your output:

TS2304: Cannot find name 'foobar'.

... but maybe i am just completely missing what you're trying to say or confused about something?

[corbinu]

@StoneCypher You can run eslint once against all .js files in your project and then again with the TS parser against all .ts files

Also the TS compiler also catches undefined in JS files not just TS files

[gpoitch]

Running this eslint config against a mixed codebase with success:

module.exports = {
  rules: {
    // ... your js rules ...
  },

  overrides: {
    files: ['**/*.ts'],
    parser: 'typescript-eslint-parser',
    rules: {
      'no-undef': 'off'
    }
  }
}

[tchakabam]

that's a very nice workaround 👍