Switch to TS syntax (#3090)

Author: saihaj

old_dts/utilities/astFromValue.d.ts

@@ -16,7 +16,7 @@ import { GraphQLInputType } from '../type/definition';
  * | Boolean       | Boolean              |
  * | String        | String / Enum Value  |
  * | Number        | Int / Float          |
- * | Mixed         | Enum Value           |
+ * | Unknown       | Enum Value           |
  * | null          | NullValue            |
  *
  */

old_dts/utilities/valueFromAST.d.ts

@@ -20,7 +20,7 @@ import { GraphQLInputType } from '../type/definition';
  * | Boolean              | Boolean       |
  * | String               | String        |
  * | Int / Float          | Number        |
- * | Enum Value           | Mixed         |
+ * | Enum Value           | Unknown       |
  * | NullValue            | null          |
  *
  */

src/__testUtils__/dedent.ts

@@ -27,8 +27,8 @@ export function dedentString(string: string): string {
  * str === "{\n  test\n}";
  */
 export function dedent(
-  strings: $ReadOnlyArray<string>,
-  ...values: $ReadOnlyArray<string>
+  strings: ReadonlyArray<string>,
+  ...values: ReadonlyArray<string>
 ): string {
   let str = strings[0];
 

src/__testUtils__/inspectStr.ts

@@ -1,7 +1,9 @@
+import type { Maybe } from '../jsutils/Maybe';
+
 /**
  * Special inspect function to produce readable string literal for error messages in tests
  */
-export function inspectStr(str: ?string): string {
+export function inspectStr(str: Maybe<string>): string {
   if (str == null) {
     return 'null';
   }

src/__tests__/starWarsData.ts

@@ -7,7 +7,6 @@ export type Character = {
   name: string;
   friends: Array<string>;
   appearsIn: Array<number>;
-  ...
 };
 
 export type Human = {

src/error/GraphQLError.ts

@@ -2,6 +2,7 @@
 // flowlint uninitialized-instance-property:off
 
 import { isObjectLike } from '../jsutils/isObjectLike';
+import type { Maybe } from '../jsutils/Maybe';
 
 import type { ASTNode } from '../language/ast';
 import type { Source } from '../language/source';
@@ -35,53 +36,53 @@ export class GraphQLError extends Error {
    *
    * Enumerable, and appears in the result of JSON.stringify().
    */
-  +locations: $ReadOnlyArray<SourceLocation> | void;
+  readonly locations: ReadonlyArray<SourceLocation> | void;
 
   /**
    * An array describing the JSON-path into the execution response which
    * corresponds to this error. Only included for errors during execution.
    *
    * Enumerable, and appears in the result of JSON.stringify().
    */
-  +path: $ReadOnlyArray<string | number> | void;
+  readonly path: ReadonlyArray<string | number> | void;
 
   /**
    * An array of GraphQL AST Nodes corresponding to this error.
    */
-  +nodes: $ReadOnlyArray<ASTNode> | void;
+  readonly nodes: ReadonlyArray<ASTNode> | void;
 
   /**
    * The source GraphQL document for the first location of this error.
    *
    * Note that if this Error represents more than one node, the source may not
    * represent nodes after the first node.
    */
-  +source: Source | void;
+  readonly source: Source | void;
 
   /**
    * An array of character offsets within the source GraphQL document
    * which correspond to this error.
    */
-  +positions: $ReadOnlyArray<number> | void;
+  readonly positions: ReadonlyArray<number> | void;
 
   /**
    * The original error thrown from a field resolver during execution.
    */
-  +originalError: ?Error;
+  readonly originalError: Maybe<Error>;
 
   /**
    * Extension fields to add to the formatted error.
    */
-  +extensions: { [key: string]: mixed; ... } | void;
+  readonly extensions: { [key: string]: unknown } | void;
 
   constructor(
     message: string,
-    nodes?: $ReadOnlyArray<ASTNode> | ASTNode | void | null,
-    source?: ?Source,
-    positions?: ?$ReadOnlyArray<number>,
-    path?: ?$ReadOnlyArray<string | number>,
-    originalError?: ?(Error & { +extensions?: mixed; ... }),
-    extensions?: ?{ [key: string]: mixed; ... },
+    nodes?: ReadonlyArray<ASTNode> | ASTNode | void | null,
+    source?: Maybe<Source>,
+    positions?: Maybe<ReadonlyArray<number>>,
+    path?: Maybe<ReadonlyArray<string | number>>,
+    originalError?: Maybe<Error & { readonly extensions?: unknown }>,
+    extensions?: Maybe<{ [key: string]: unknown }>,
   ) {
     super(message);
 

src/error/formatError.ts

@@ -29,22 +29,22 @@ export type GraphQLFormattedError = {
    * from occurrence to occurrence of the problem, except for purposes of
    * localization.
    */
-  +message: string;
+  readonly message: string;
   /**
    * If an error can be associated to a particular point in the requested
    * GraphQL document, it should contain a list of locations.
    */
-  +locations: $ReadOnlyArray<SourceLocation> | void;
+  readonly locations: ReadonlyArray<SourceLocation> | void;
   /**
    * If an error can be associated to a particular field in the GraphQL result,
    * it _must_ contain an entry with the key `path` that details the path of
    * the response field which experienced the error. This allows clients to
    * identify whether a null result is intentional or caused by a runtime error.
    */
-  +path: $ReadOnlyArray<string | number> | void;
+  readonly path: ReadonlyArray<string | number> | void;
   /**
    * Reserved for implementors to extend the protocol however they see fit,
    * and hence there are no additional restrictions on its contents.
    */
-  +extensions?: { [key: string]: mixed; ... };
+  readonly extensions?: { [key: string]: unknown };
 };

src/error/locatedError.ts

@@ -1,4 +1,5 @@
 import { inspect } from '../jsutils/inspect';
+import type { Maybe } from '../jsutils/Maybe';
 
 import type { ASTNode } from '../language/ast';
 
@@ -10,9 +11,9 @@ import { GraphQLError } from './GraphQLError';
  * document responsible for the original Error.
  */
 export function locatedError(
-  rawOriginalError: mixed,
-  nodes: ASTNode | $ReadOnlyArray<ASTNode> | void | null,
-  path?: ?$ReadOnlyArray<string | number>,
+  rawOriginalError: unknown,
+  nodes: ASTNode | ReadonlyArray<ASTNode> | void | null,
+  path?: Maybe<ReadonlyArray<string | number>>,
 ): GraphQLError {
   // Sometimes a non-error is thrown, wrap it as an Error instance to ensure a consistent Error interface.
   const originalError: Error | GraphQLError =

src/execution/__tests__/abstract-test.ts

@@ -20,7 +20,7 @@ import { executeSync, execute } from '../execute';
 async function executeQuery(args: {
   schema: GraphQLSchema;
   query: string;
-  rootValue?: mixed;
+  rootValue?: unknown;
 }) {
   const { schema, query, rootValue } = args;
   const document = parse(query);
@@ -534,7 +534,7 @@ describe('Execute: Handles execution of abstract types', () => {
       }
     `);
 
-    function expectError({ forTypeName }: { forTypeName: mixed }) {
+    function expectError({ forTypeName }: { forTypeName: unknown }) {
       const rootValue = { pet: { __typename: forTypeName } };
       const result = executeSync({ schema, document, rootValue });
       return {

src/execution/__tests__/lists-test.ts

@@ -8,7 +8,7 @@ import { buildSchema } from '../../utilities/buildASTSchema';
 import { execute, executeSync } from '../execute';
 
 describe('Execute: Accepts any iterable as list value', () => {
-  function complete(rootValue: mixed) {
+  function complete(rootValue: unknown) {
     return executeSync({
       schema: buildSchema('type Query { listField: [String] }'),
       document: parse('{ listField }'),
@@ -65,7 +65,7 @@ describe('Execute: Accepts any iterable as list value', () => {
 });
 
 describe('Execute: Handles list nullability', () => {
-  async function complete(args: { listField: mixed; as: string }) {
+  async function complete(args: { listField: unknown; as: string }) {
     const { listField, as } = args;
     const schema = buildSchema(`type Query { listField: ${as} }`);
     const document = parse('{ listField }');
@@ -85,11 +85,11 @@ describe('Execute: Handles list nullability', () => {
     }
     return result;
 
-    function executeQuery(listValue: mixed) {
+    function executeQuery(listValue: unknown) {
       return execute({ schema, document, rootValue: { listField: listValue } });
     }
 
-    function promisify(value: mixed): Promise<mixed> {
+    function promisify(value: unknown): Promise<unknown> {
       return value instanceof Error
         ? Promise.reject(value)
         : Promise.resolve(value);

src/execution/__tests__/nonnull-test.ts

@@ -106,7 +106,7 @@ const schema = buildSchema(`
 
 function executeQuery(
   query: string,
-  rootValue: mixed,
+  rootValue: unknown,
 ): ExecutionResult | Promise<ExecutionResult> {
   return execute({ schema, document: parse(query), rootValue });
 }
@@ -122,7 +122,7 @@ function patchData(data: ExecutionResult): ExecutionResult {
   return JSON.parse(patch(JSON.stringify(data)));
 }
 
-async function executeSyncAndAsync(query: string, rootValue: mixed) {
+async function executeSyncAndAsync(query: string, rootValue: unknown) {
   const syncResult = executeSync({ schema, document: parse(query), rootValue });
   const asyncResult = await execute({
     schema,

src/execution/__tests__/resolve-test.ts

@@ -95,7 +95,7 @@ describe('Execute: resolve function', () => {
       resolve: (source, args) => JSON.stringify([source, args]),
     });
 
-    function executeQuery(query: string, rootValue?: mixed) {
+    function executeQuery(query: string, rootValue?: unknown) {
       const document = parse(query);
       return executeSync({ schema, document, rootValue });
     }

src/execution/__tests__/variables-test.ts

@@ -119,7 +119,7 @@ const schema = new GraphQLSchema({ query: TestType });
 
 function executeQuery(
   query: string,
-  variableValues?: { [variable: string]: mixed; ... },
+  variableValues?: { [variable: string]: unknown },
 ) {
   const document = parse(query);
   return executeSync({ schema, document, variableValues });