AST correction for ImportDeclarationSpecifier (Local is an Identifier instead of an Expression) (#238)

Author: adams85

src/Esprima/Ast/ImportDeclarationSpecifier.cs

@@ -2,12 +2,9 @@
 {
     public abstract class ImportDeclarationSpecifier : Declaration
     {
-        /// <summary>
-        /// Identifier | StringLiteral
-        /// </summary>
-        public readonly Expression Local;
+        public readonly Identifier Local;
 
-        protected ImportDeclarationSpecifier(Expression local, Nodes type) : base(type)
+        protected ImportDeclarationSpecifier(Identifier local, Nodes type) : base(type)
         {
             Local = local;
         }

src/Esprima/Ast/ImportSpecifier.cs

@@ -9,7 +9,7 @@ public sealed class ImportSpecifier : ImportDeclarationSpecifier
         /// </summary>
         public readonly Expression Imported;
 
-        public ImportSpecifier(Expression local, Expression imported) : base(local, Nodes.ImportSpecifier)
+        public ImportSpecifier(Identifier local, Expression imported) : base(local, Nodes.ImportSpecifier)
         {
             Imported = imported;
         }

src/Esprima/JavascriptParser.cs

@@ -4691,13 +4691,13 @@ private ImportSpecifier ParseImportSpecifier()
         {
             var node = CreateNode();
 
-            Expression local;
+            Identifier local;
             Expression imported;
 
             if (_lookahead.Type == TokenType.Identifier)
             {
-                imported = ParseVariableIdentifier(allowAwaitKeyword: true);
-                local = imported;
+                imported = local = ParseVariableIdentifier(allowAwaitKeyword: true);
+
                 if (MatchContextualKeyword("as"))
                 {
                     NextToken();
@@ -4710,7 +4710,6 @@ private ImportSpecifier ParseImportSpecifier()
                     ? ParseModuleSpecifier()
                     : ParseIdentifierName();
 
-                local = imported;
                 if (MatchContextualKeyword("as"))
                 {
                     NextToken();
@@ -4719,6 +4718,7 @@ private ImportSpecifier ParseImportSpecifier()
                 else
                 {
                     ThrowUnexpectedToken(NextToken());
+                    local = default!; // never executes, just keeps the compilery happy
                 }
             }