multi-var statement gets mangled [regression in 0.22.1]

[lukeapage]

Unfortunately the code is closed source so I'll try and paint a picture of what is going on..

I'm using format iife and the rollup-plugin-babel all latest versions.

I have code like..

import mockFetch from '../../mocks/fetch';

var a, fetch;

describe("x", () => {
    beforeEach(() => {
        fetch = mockFetch();
        installClock();
    });

fetch is a global and is used as a global in mockFetch, so I guess rollup needs to rename the var fetch

but it becomes

    var a;
    fetch$1;
    describe("x", function () {
        beforeEach(function () {
            fetch$1 = mockFetch();
            installClock();
        });

which means that fetch$1 isn't defined and the code falls over.

This was working a couple of weeks ago so I'll try and figure out what version causes the issue.

[lukeapage]

ok thats simple, this occurs in 0.22.1 but not 0.22.0.

In 0.22.0 I get the following output

var a;
var fetch$1;

[kobezzza]

👍

index.js

import './a';
import './b';

a.js

const a = 1, b = 2;

console.log(a, b);

b.js

const a = 2, b = 3;

console.log(a, b);

Command:

rollup rollup/index.js --export umd --id foo --name foo

Output:

const a = 1;
const b = 2;
console.log(a, b);

a$1 = 2; // Error
b$1 = 3; // Error
console.log(a$1, b$1);

[lukeapage]

thanks @kobezzza for the repro-case

[Rich-Harris]

Thanks for flagging this up. Looks like the latest version of magic-string is to blame – looking into it

[Rich-Harris]

Fixed in 0.22.2