Svelte should throw a compile time error when illegal characters are used in computed names

[lnryan]

Computed values are transformed into functions so random-api becomes random-api(){} which is not allowed.

        computed:{
            "random-api":(pw_min,pw_max,passwords)=>{
                return `apiurlsomethingsomething`
            }
        }

The compiler should suggest an alternative like random_api, which is admittedly uglier 🤷‍♀️, but completely valid, and not svelte's fault obvi

[eh-dub]

@arxpoetica, @Rich-Harris threw this issue my way on gitter this morning.
I understand the high-level problem and will muddle through the code to figure out how to go about implementing a fix.

[eh-dub]

Submitted #1099
I'm open to all questions and suggestions.

[Rich-Harris]

Fix released in 1.52 — thanks