[Bug]: plugin-transform-modules-systemjs transformation is unsafe

💻

• Would you like to work on a fix?

How are you using Babel?

Programmatic API (babel.transform, babel.parse)

Input code

REPL.

import { x } from './x.js';

if (true) {
	const x = 1;
	console.log(x);
}

new (class extends x {})();

Configuration file name

No response

Configuration

Using @babel/plugin-transform-modules-systemjs, nothing else

Current and expected behavior

I would expect the x = 1 to declare a local constant, as it does in the original code. Instead, it assigns to a new module-level variable (which in this case has the same name as an import, causing an error).

Environment

  System:
    OS: macOS 10.15.7
  Binaries:
    Node: 16.5.0 - ~/.nvm/versions/node/v16.5.0/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 7.19.1 - ~/.nvm/versions/node/v16.5.0/bin/npm
  npmPackages:
    @babel/core: ^7.15.8 => 7.15.8 
    @babel/plugin-proposal-dynamic-import: ^7.14.5 => 7.14.5 
    @babel/plugin-transform-modules-systemjs: ^7.15.4 => 7.15.4 

Possible solution

No response

Additional context

cc @guybedford 😃

[The-x-Theorist]

I’d like to work on this issue?

[The-x-Theorist]

Can you keep the name for var declarations, and only rename const/let ?

Yeah, I can do that.