standard confuses string with missing quotes and new class instance without brackets

[lukechilds]

If I have the code:

const onionoo = new Onionoo()

to create a new instance of the Onionoo class but leave off the brackets:

const onionoo = new Onionoo

standard says:

standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.

/Users/lukechilds/Dev/oss/onionoo-node-client/test/unit.js
  18:19  error  Missing '

It seems like maybe it assumes Onionoo is a string without quotes. It would be better if it would warn that there are no parenthesis on the class (or not warn at all, not sure of the pros/cons of leaving off parenthesis). I would assume this would be fairly easy to fix by checking if the new keyword is directly before the string.

[lukechilds]

Just realised standard uses eslint under the hood, should I be reporting this there?

[dcousens]

@lukechilds yup 👍

[lukechilds]

@dcousens according to ESLint this appears to be an issue with standard. It gets reported correctly as Missing '()' invoking a constructor. (new-parens) with vanilla ESLint.

eslint/eslint#7792

[dcousens]

@lukechilds actually, it appears to be an issue on your end 😛

var Onionoo
const onionoo = new Onionoo

console.log(onionoo)

standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  /home/daniel/test.js:2:17: Missing '()' invoking a constructor.

Do you have a test case we could try?

[lukechilds]

Ahhhh, found the culprit!

It's not doing it if I call standard directly but if I call snazzy it does with the exact same input.

$ standard
standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  /Users/lukechilds/Dev/oss/onionoo-node-client/test/unit.js:14:19: Missing '()' invoking a constructor.

$ snazzy
standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.

/Users/lukechilds/Dev/oss/onionoo-node-client/test/unit.js
  14:19  error  Missing '

✖ 1 problem

Should I open the issue on snazzy? 😆

[dcousens]

@lukechilds I think so, feel free to post back and we'll re-open if needed 👍