After v0.84 type-at-pos on flow-try doesn't work with object types

[eilvelia]

For example, with this code

declare function bar<T>(x: T, y: string | T): T;
const y = bar({ a: 1, b: 2 }, { c: 3 })

y.a

flow try

^ v0.83.0

^ v0.84.0

In the editor I see hints with any version, thus I think it is related to js-of-ocaml build.

I guess it is because re_string_match isn't implemented in JS and it fails on the property_key_quotes_needed function.

[goodmind]

/cc @mroch
Related #7698

[mroch]

a couple of thoughts, if property_key_quotes_needed is indeed the culprit:

1. making Str work in JS is way out of scope, better to do this without regexes
2. technically, property keys are IdentiferNames, which can contain unicode without having to be quoted. the IdStart and IdContinue unicode character sets are huge. the lexer can match them if we need to be pedantic about it, but I don't think it's worth the complexity; better to just unnecessarily quote some keys

[SamChou19815]

Fixed