Completions should be aware of string literals

[minestarks]

Completions have buggy behavior in the following cases:

Should show completions, but are suppressed due to naive check for line comments:

let x = "// comment" + |

Should NOT show completions, but do today:

let x = "foo.|"

See #1999 for some context