Auto-indent changes multiline strings

[trbrc]

The auto-indent feature adds indentation to the contents of multiline strings.

export const myTemplateString = `
Template string content
`;

export const myEscapedString = "\
Escaped string content\
";

$ rollup -f iife -n strings strings.js

(function (exports) { 'use strict';

    const myTemplateString = `
    Template string content
    `;

    const myEscapedString = "\
    Escaped string content\
    ";

    exports.myTemplateString = myTemplateString;
    exports.myEscapedString = myEscapedString;

})((this.strings = {}));

A workaround would be to use the indent: false option, but as far as I can tell that's not possible with the CLI. Since the defaults are indent: true, rollup is currently subtly breaking such code out of the box. A better solution would be to make the auto-indent feature treat these cases as an exception.

I suggest that the output should be:

(function (exports) { 'use strict';

    const myTemplateString = `
Template string content
`;

    const myEscapedString = "\
Escaped string content\
";

    exports.myTemplateString = myTemplateString;
    exports.myEscapedString = myEscapedString;

})((this.strings = {}));

[Victorystick]

That's definitely a bug. Thanks for reporting it. I'm not sure how to best fix the general case of indented template strings, but we should make sure to have a CLI option, such as --no-indent.

[Rich-Harris]

magic-string does allow you to pass an exclude option to magicString.indent(...), for exactly this reason. I just had a quick look now and it looks like there'll be a bit of an art to rigging it up, but it's definitely solvable. We should expose that CLI option anyway though

[Rich-Harris]

Fixed by 4fcbba6, which I've rolled into #163 – will release as 0.18.0 as soon as I've tied down one loose end