wire adapter typescript def is incorrect about lwc HOT 7 CLOSED

@caridy You mentioned there were other gaps in engine.d.ts. Can we merge this with that issue or repurpose this to cover it all?

from lwc.

The idea is to eliminate engine.d.ts as a manual file, and produce one as part of the build process. @rsalvador and I decided to keep this around because it was easier, but right after the next cut, we should remove that.

The concern is mostly that a generated file will contain everything, while devs will only have access to some things, other are meant to be used for other tools (e.g.: Aura), and we haven't found an easy way to have that split in the types as well.

from lwc.

For the lwc-language-service we will also need an updated engine.d.ts for 214

from lwc.

Here I was adding a missing method: #60
After that I did look a little into how to generate the engine.d.ts automatically but it did look pretty complicated to generate the equivalent of engine.d.ts

from lwc.

Is this still an issue @rsalvador @kevinv11n ?

from lwc.

We can close this, the wire def was updated here: a091304

from lwc.

Config bag needs to be optional. I'll issue a PR shortly.

from lwc.