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,