imported function call without leading semicolon

Repro link

import {fun} from './foo'
let arr = []
fun()

…becomes…

"use strict";var _foo = require('./foo');
let arr = []
(0, _foo.fun)()

…which evaluates to TypeError: [] is not a function

[alangpierce]

(Oops, typo, it should be “just do the regular replacement”.)

I think it’s referring to cases like this:

export let x = 1;
x = 2;

Without being careful, we might transform the second line to (0, exports.x) = 2;, which isn’t a valid assignment. It’s from https://github.com/alangpierce/sucrase/pull/180 , where I switched the export approach to use variable replacement and changed the replacement strategy from “do the (0, f) thing everywhere for simplicity” to a more targeted approach.

If you’re reworking the code, not super-important to keep that line, I think it made more sense in the original context.

[aleclarson]

What does this comment mean?

In some cases, like export assignees, we must NOT turn the identifier into a normal expression, so we need to just to the regular replacement.