Property assignments in constructor should show up in intellisense

[sandersn]

If you request completion at the /**/ below...

class C {
  constructor() {
    this.x = 'foo';
  }
}
let c = new C();
c./**/;

Expected behavior:

Completion list should contain x.

Actual behavior:

Completion list does not contain x.

Currently there is an error at this.x = 'foo', "Property 'x' does not exist in 'C'". That error should stay.

[DanielRosenwasser]

Is this a JS language service bug? We explicitly said TypeScript shouldn't recognize this back in October.

#5196 (comment)

Though I wouldn't be opposed to changing it.

[mhegazy]

We should understand the property assignments but still give you an error. This way js and TS work the same way, and we can have better quick fix support.

[RyanCavanaugh]

I think this is as fixed as it needs to be