sem: re-work ImportNamespaceSpecifier comments

Instead of dropping comments of ImportNamespaceSpecifier
as implemented in #60, we just skip both, leading and
trailing comments now.

This avoids loosing a Comment node in UAST.

Signed-off-by: Alexander Bezzubov <[email protected]>

Author: bzz

driver/normalizer/normalizer.go

@@ -70,16 +70,6 @@ var Preprocessors = []Mapping{
 		Part("_", Obj{"extra": Any()}),
 		Part("_", Obj{}),
 	),
-	// FIXME(bzz): make sure such comments are mapped properly
-	Map(
-		Part("_", Obj{
-			uast.KeyType: String("ImportNamespaceSpecifier"),
-			"leadingComments": Any(),
-		}),
-		Part("_", Obj{
-			uast.KeyType: String("ImportNamespaceSpecifier"),
-		}),
-	),
 }
 
 // Normalizers is the main block of normalization rules to convert native AST to semantic UAST.
@@ -174,11 +164,14 @@ var Normalizers = []Mapping{
 				// namespace
 				{
 					"importKind": String("value"),
-					"specifiers": ArrWith(Var("names"), Obj{
+					"specifiers": ArrWith(Var("names"), Fields{
 						//uast.KeyType: Var("spec_type"),
-						uast.KeyType: String("ImportNamespaceSpecifier"),
-						uast.KeyPos:  Var("local_pos"),
-						"local":      Var("local"),
+						{Name: uast.KeyType, Op: String("ImportNamespaceSpecifier")},
+						{Name: uast.KeyPos, Op: Var("local_pos")},
+						{Name: "local", Op: Var("local")},
+						// FIXME(bzz): make sure such comments are linked properly
+						{Name: "leadingComments", Drop: true, Op: Any()},
+						{Name: "trailingComments", Drop: true, Op: Any()},
 					}),
 				},
 				// specific type

fixtures/59-jsx-flow.js.uast

@@ -125,6 +125,7 @@
                         col: 18,
                      },
                   },
+                  leadingComments: ~,
                   local: { '@type': "Identifier",
                      '@token': "React",
                      '@role': [Expression, Identifier, Import],