Ungraceful error: Already found a mapping for inlining short name

[SLaks]

goog.module('a');

const a = goog.require('a');
const a = goog.require('a');

Result:

java.lang.RuntimeException: INTERNAL COMPILER ERROR.
Please file a bug at http://go/jscompiler-bug with this stack trace, sponge link, and repro steps (a CL# and a build target).

Already found a mapping for inlining short name: a
  Node(CALL): input0:4:10
const a = goog.require('a');
  Parent(NAME a): input0:4:6
const a = goog.require('a');

    at com.google.common.base.Preconditions.checkState(Preconditions.java:523)
    at com.google.javascript.jscomp.ClosureRewriteModule.recordNameToInline(ClosureRewriteModule.java:1296)
    at com.google.javascript.jscomp.ClosureRewriteModule.updateGoogRequire(ClosureRewriteModule.java:860)

https://closure-compiler-debugger.appspot.com/#input0%3Dgoog.module('a')%253B%250A%250Aconst%2520a%2520%253D%2520goog.require('a')%253B%250Aconst%2520a%2520%253D%2520goog.require('a')%253B%26input1%26conformanceConfig%26externs%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26CLOSURE_PASS%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1

[MatrixFrog]

Er, shouldn't this be a syntax error (or at least an early error in CheckVars or something) to have two const as?

[SLaks]

Yes, but it should not be an INTERNAL COMPILER ERROR.

This might be caused by ClosureRewriteModule running before the real error.

[Dominator008]

@MatrixFrog I now think VariableReferenceCheck should run before the closure passes.