globalThis property access errors are behind noImplicitAny, not noImplicitThis

[sandersn]

Now that globalThis is available, top-level this also has a more precise type, and issues the normal noImplicitAny element access errors for unknown properties:

if (this['unknown']) { ... } // error, `typeof globalThis` has no index signature
if (this.unknown) { ... } // error, `typeof globalThis` has no index signature
this.onmousemove = function () { /* ok, and this: Window */ }

We have a conflict: traditionally, "x has no index signature" errors are produced when noImplicitAny is on. Also traditionally, "this has type any" errors are produced when noImplicitThis is on.

Technically, top-level this no longer has type any, so it's technically correct to issue the errors from behind noImplicitAny. But people might expect to see them only when noImplicitThis is on instead.

Let's discuss at the next design meeting.

[sandersn]

No complaints so far. I think this is fine where it is.

[IgorKurkov]

Hi! Faced this issue Element implicitly has an 'any' type because type 'typeof globalThis' has no index signature.
Any updates?

[sandersn]

Can you explain more about your expectations and usage?

1. Is your usage the same as the bug's example?
2. Do you expect no error or a different error?
3. Do you have noImplicitAny: true but noImplicitThis: false? That's the only configuration that would no longer have an error if this bug were fixed.

[KevinGruber]

@sandersn I have exactly that problem
with the config noImplicitAny: true and noImplicitThis: false

It worked fine in lower versions (ts 4.2.*)
but since the upgrade to ts4.3 it is behaving like that

do you have any idea?

[sandersn]

@KevinGruber nothing comes to mind except maybe external changes to @types/node. Can you open a new issue with more detail about the error you get?

[nulldreams]

@KevinGruber I managed to remove this message with the property "noImplicitAny": false.

ts 4.3.5

[bennycode]

I am getting this error when bumping @types/node from 15.12.4 to 16.7.10.

I have the following defined in my code:

import {APIClient} from '../../APIClient';

declare global {
  module NodeJS {
    interface Global {
      client: APIClient;
    }
  }
}

global.client = new APIClient(APIClient.URL_DEMO, 'global-demo-api-key');
^----- here I am getting the error: TS7017: Element implicitly has an 'any' type because type 'typeof globalThis' has no index signature.

[mstaicu]

@bennycode I had the same issue with exactly the same use case as yours. Seems that the latest @types/node has the following type definition file content

declare var global: typeof globalThis;

Seems that at some point it had a different content

declare var global: NodeJS.Global & typeof globalThis;

What I ended up doing is this ( porting your changes ):

declare global {
  function client(): APIClient;
}

or

declare module globalThis {
  function client(): APIClient;
}