Address comments on flow type imports (#312)

src/plugins/flow.js

@@ -1234,7 +1234,10 @@ export default function (instance) {
         specifier.local = specifier.imported.__clone();
       }
 
-      if (node.importKind !== "value" && specifier.importKind !== null) {
+      if (
+        (node.importKind === "type" || node.importKind === "typeof") &&
+        (specifier.importKind === "type" || specifier.importKind === "typeof")
+      ) {
         this.raise(firstIdentLoc, "`The `type` and `typeof` keywords on named imports can only be used on regular `import` statements. It cannot be used with `import type` or `import typeof` statements`");
       }
 

test/fixtures/flow/type-imports/invalid-import-type-shorthand-2/actual.js

@@ -0,0 +1 @@
+import type {type t} from "foo";

test/fixtures/flow/type-imports/invalid-import-type-shorthand-2/options.json

@@ -0,0 +1,3 @@
+{
+  "throws": "`The `type` and `typeof` keywords on named imports can only be used on regular `import` statements. It cannot be used with `import type` or `import typeof` statements` (1:13)"
+}

test/fixtures/flow/type-imports/invalid-import-type-shorthand/actual.js

@@ -1,2 +1 @@
 import typeof {typeof t} from "foo";
-import type {type t} from "foo";