Incorrect formating with semicolon-less code

[derekchiang]

Consider this piece of code:

var whatever = require('whatever')
function() {...}

Using JsFormat, this code will be formatted to:

var whatever = require('whatever')
    function() {...}  // indentation at the beginning of the line

Specifically, it seems like whenever there is a bracket following by a function, JsFormat would indent the function definition (incorrectly).

[jdavisclark]

This should definitely be handled better. I will open a bug upstream with jsbeautifier.

Not to start a semicolon vs no semicolon debate, but it does behave as expected if the require statement is terminated with a semicolon.

[jdavisclark]

After having a look through the einars/js-beautify issue tracker, it looks like this is part of a larger issue. Based on a cursory glance, I do not see this being fixed any time soon. So, I hate to tell you to use semicolons, as it's perfectly valid as is, but thats the workaround for the time being.

Relevant Issues: beautifier/js-beautify#56 , beautifier/js-beautify#200

[bitwiseman]

You should still file an issue. We'll evaluate and fix issues on a point-wise basis.

[bitwiseman]

This is fixed now. 😄