Extra quotation marks when completing shape fields

[slackhq]

Extra quotation marks when completing shape fields

[PranayAgarwal]

I remember discussing this with the HHVM team a little while ago. I'll try and pull up that thread to see what the status of the fix is, but right now an option is to not insert the first ' and just start typing the field name after [. Results will still show up, and it will automatically insert both ' characters after you hit enter.

[fredemmott]

Relevant code and som explanation of the problem at https://github.com/facebook/hhvm/blob/a8a3a07ee94f2304fba2f21d366be45382b98e38/hphp/hack/src/server/autocompleteService.ml#L608

Two ways come to mind to resolve this without a significant rewrite:

• lookahead here to make the pos_end include the next character if it's a single quote - i.e. replace 'f' with 'foo'
• lookahead in autocomplete_shape_key, and don't include a trailing quote in the replacement if the next character is a ' - i.e. replace 'f with 'foo, with the trailing ' untouched

The first option is probably best