Fix bad completion with keyword prefixes

[RblSb]

var innerText = 1;
in<tab> // innerText instead of ininnerText

Basically this should be a switch with a full KeywordKind, but i only added short keyword as most annoying while auto-completing stuff. Some macro to extract all enum fields to array can be used, but maybe this is enough.

I also wonder, do we need a simple do keyword completion? It prevents some do... fields from being autocompleted.

[kLabz]

Thanks; I cannot test today but will check early next week

I also saw the same behavior with anon object field completion just now, maybe the fix would be similar 🤔

[kLabz]

lgtm

Well.. it doesn't look great but other solutions wouldn't be better, and it does the job (note that repro isn't too straightforward)

I also wonder, do we need a simple do keyword completion? It prevents some do... fields from being autocompleted.

Probably not. Especially since do is likely not used much.. and in real world use cases you probably don't have it often as a suggestion until you add the o, making it even more useless.

On a side note, having var in that list would have been nice too, but it seems like the snippet is getting in the way anyway, making completion of things like variadic impossible to begin with in cases where the issue would happen 🤔

[RblSb]

Yeah, i was wondering if i can always replace zero size replace range to length of non-space chars before cursor when completion triggered, but it feels even more error prone. At least this prefix hack will be easier to find after.

Also checked if can find better solution at compiler side, but had no success for now. Probably compiler also cannot easily detect if we are typing download field, or reopen completion after do| and there is no space. There is a lot of hacks in grammar.mly, related to HaxeFoundation/haxe#8669