Switch to alternative format for type imports

Author: danielrearden

.eslintrc.yml

@@ -52,7 +52,7 @@ rules:
   flowtype/sort-keys: off
   flowtype/spread-exact-type: off
   flowtype/type-id-match: [error, '^[A-Z]|^\$']
-  flowtype/type-import-style: [error, identifier, { ignoreTypeDefault: true }]
+  flowtype/type-import-style: [error, declaration]
   flowtype/use-flow-type: error
 
   # Bellow rules are disabled because coflicts with Prettier, see:
@@ -391,7 +391,7 @@ rules:
   no-class-assign: error
   no-const-assign: error
   no-dupe-class-members: error
-  no-duplicate-imports: error
+  no-duplicate-imports: off # Superseded by `import/no-duplicates`
   no-new-symbol: error
   no-restricted-exports: off
   no-restricted-imports: off

src/__tests__/http-test.js

@@ -8,13 +8,12 @@ import restify from 'restify';
 import supertest from 'supertest';
 import bodyParser from 'body-parser';
 
+import type { ASTVisitor, ValidationContext } from 'graphql';
 import sinon from 'sinon';
 import multer from 'multer';
 import { expect } from 'chai';
 import { describe, it } from 'mocha';
 import {
-  type ASTVisitor,
-  type ValidationContext,
   Source,
   GraphQLError,
   GraphQLString,

src/index.js

@@ -1,7 +1,19 @@
 // @flow strict
 
-import { type IncomingMessage, type ServerResponse } from 'http';
-
+import type { IncomingMessage, ServerResponse } from 'http';
+
+import type {
+  ASTVisitor,
+  DocumentNode,
+  ValidationRule,
+  ValidationContext,
+  ExecutionArgs,
+  ExecutionResult,
+  GraphQLError,
+  GraphQLSchema,
+  GraphQLFieldResolver,
+  GraphQLTypeResolver,
+} from 'graphql';
 import accepts from 'accepts';
 import httpError from 'http-errors';
 import {
@@ -13,24 +25,11 @@ import {
   validateSchema,
   getOperationAST,
   specifiedRules,
-  type ASTVisitor,
-  type DocumentNode,
-  type ValidationRule,
-  type ValidationContext,
-  type ExecutionArgs,
-  type ExecutionResult,
-  type GraphQLError,
-  type GraphQLSchema,
-  type GraphQLFieldResolver,
-  type GraphQLTypeResolver,
 } from 'graphql';
 
+import type { GraphiQLOptions, GraphiQLData } from './renderGraphiQL';
 import { parseBody } from './parseBody';
-import {
-  renderGraphiQL,
-  type GraphiQLOptions,
-  type GraphiQLData,
-} from './renderGraphiQL';
+import { renderGraphiQL } from './renderGraphiQL';
 
 type $Request = IncomingMessage;
 type $Response = ServerResponse & {| json?: (data: mixed) => void |};

src/parseBody.js

@@ -1,7 +1,8 @@
 // @flow strict
 
-import { type IncomingMessage } from 'http';
-import zlib, { type Inflate, type Gunzip } from 'zlib';
+import type { IncomingMessage } from 'http';
+import type { Inflate, Gunzip } from 'zlib';
+import zlib from 'zlib';
 import querystring from 'querystring';
 
 import getBody from 'raw-body';

src/renderGraphiQL.js

@@ -1,6 +1,6 @@
 // @flow strict
 
-import { type ExecutionResult } from 'graphql';
+import type { ExecutionResult } from 'graphql';
 
 export type GraphiQLData = {|
   query?: string | null,