fix: Remove explicit return types from sdk methods

.changeset/shiny-dogs-press.md

@@ -0,0 +1,16 @@
+---
+'@graphql-codegen/typescript-graphql-request': major
+'@graphql-codegen/website': patch
+---
+
+Do not specify return types of sdk methods explicitly.
+
+Previously, the return type of all SDK methods was declared manually. This can
+cause the return type to become outdated when graphql-request gets updated.
+This already happened, as graphql-request removed the errors attribute from the
+return type (see https://github.com/prisma-labs/graphql-request/issues/174).
+Also, data was marked as possibly undefined which is wrong. data always exists
+on a successful response (failed requests throw a ClientError).
+
+A sideeffect of this change is that the extensionsType config became obsolete
+and has been removed.

packages/plugins/typescript/graphql-request/src/config.ts

@@ -22,17 +22,4 @@ export interface RawGraphQLRequestPluginConfig extends RawClientSideBasePluginCo
    * ```
    */
   rawRequest?: boolean;
-
-  /**
-   * @description Allows you to override the type for extensions when `rawRequest` is enabled.
-   * @default any
-   *
-   * @exampleMarkdown
-   * ```yml
-   * config:
-   *   rawRequest: true
-   *   extensionsType: unknown
-   * ```
-   */
-  extensionsType?: string;
 }

packages/plugins/typescript/graphql-request/src/visitor.ts

@@ -12,7 +12,6 @@ import { RawGraphQLRequestPluginConfig } from './config';
 
 export interface GraphQLRequestPluginConfig extends ClientSideBasePluginConfig {
   rawRequest: boolean;
-  extensionsType: string;
 }
 
 const additionalExportedTypes = `
@@ -34,7 +33,6 @@ export class GraphQLRequestVisitor extends ClientSideBaseVisitor<
   constructor(schema: GraphQLSchema, fragments: LoadedFragment[], rawConfig: RawGraphQLRequestPluginConfig) {
     super(schema, fragments, rawConfig, {
       rawRequest: getConfigValue(rawConfig.rawRequest, false),
-      extensionsType: getConfigValue(rawConfig.extensionsType, 'any'),
     });
 
     autoBind(this);
@@ -111,19 +109,15 @@ export class GraphQLRequestVisitor extends ClientSideBaseVisitor<
           }
           return `${operationName}(variables${optionalVariables ? '?' : ''}: ${
             o.operationVariablesTypes
-          }, requestHeaders?: Dom.RequestInit["headers"]): Promise<{ data?: ${
-            o.operationResultType
-          } | undefined; extensions?: ${
-            this.config.extensionsType
-          }; headers: Dom.Headers; status: number; errors?: GraphQLError[] | undefined; }> {
+          }, requestHeaders?: Dom.RequestInit["headers"]) {
     return withWrapper((wrappedRequestHeaders) => client.rawRequest<${
       o.operationResultType
     }>(${docArg}, variables, {...requestHeaders, ...wrappedRequestHeaders}), '${operationName}');
 }`;
         } else {
           return `${operationName}(variables${optionalVariables ? '?' : ''}: ${
             o.operationVariablesTypes
-          }, requestHeaders?: Dom.RequestInit["headers"]): Promise<${o.operationResultType}> {
+          }, requestHeaders?: Dom.RequestInit["headers"]) {
   return withWrapper((wrappedRequestHeaders) => client.request<${
     o.operationResultType
   }>(${docVarName}, variables, {...requestHeaders, ...wrappedRequestHeaders}), '${operationName}');