mergeScripts should insert semicolons between <script> tags

[jordanbtucker]

With the following HTML and default options:

<script>document.write('Hello, ')</script><script>document.write('World!')</script>

It should produce:

<script>document.write('Hello, ');document.write('World!');</script>

But instead you get the following error:

Error: SyntaxError: Unexpected token: name (document)

So it appears that it is concatenating the two <script> tags as this (without a semicolon in between):

<script>document.write('Hello, ')document.write('World!')</script>

If you put the semicolon there yourself, mergeScripts does something even stranger. It replaces it with a comma:

<script>document.write('Hello, ');</script><script>document.write('World!')</script>

becomes

<script>document.write('Hello, '),document.write('World!');</script>

The comma is valid, but it does seem an odd choice over a semicolon.

[maltsev]

Thanks for the bug report! I fixed it. I'll release a new version today.

[maltsev]

If you put the semicolon there yourself, mergeScripts does something even stranger. It replaces it with a comma:

Actually, that is a result of minifyJs optimization. If you disable it ; shouldn't be replaced with , in that case.

[maltsev]

Released: 0.1.10.