Missing/incomplete mutate function variables types

Hi, I started using graphql-let yesterday as a replacement for GraphQL Code Generator. Today I noticed a weird issue with types not being as strong as I’d like.

Maybe I remember wrong, but while using GraphQL Code Generator, I think I had variables of the mutate function typed and VSCode provided autocomplete when writing them. Here is an example of what I mean:

import { useChangeTickerMutation } from './tickersMutations.graphql'

const Example: FC = () => {
  const [changeTicker] = useChangeTickerMutation({ variables: { } })
  //                                                           ^ autocomplete/typecheck works here

  const sendUpdate = () => changeTicker({ variables: { } })
  //                                                  ^ autocomplete/typecheck doesn't work here (`variables` are `any`)
  ...
}

I tracked this to the tickersMutations.graphql.d.ts file:

export declare function useChangeTickerMutation(baseOptions?: Apollo.MutationHookOptions<ChangeTickerMutation, ChangeTickerMutationVariables>): Apollo.MutationTuple<ChangeTickerMutation, any>;

I believe there should be Apollo.MutationTuple<ChangeTickerMutation, ChangeTickerMutationVariables> in the end.

[piglovesyou]

@rtrembecky Great, thanks for taking your time. I’ll publish v0.18.1 soon.

[rtrembecky]

I took a bit of a dive into your codebase, it seems there are no tests/examples for mutations. Luckily, I think I found the same problems for queries. E.g. in https://github.com/piglovesyou/graphql-let/blob/main/src/__snapshots__/gen.test.ts.snap#L58-L59:

export declare function useViewerQuery(baseOptions?: Apollo.QueryHookOptions<ViewerQuery, ViewerQueryVariables>): Apollo.QueryResult<ViewerQuery, any>;
export declare function useViewerLazyQuery(baseOptions?: Apollo.LazyQueryHookOptions<ViewerQuery, ViewerQueryVariables>): Apollo.QueryTuple<ViewerQuery, any>;

Both of these should have ViewerQueryVariables instead of any as a second generic type argument to Apollo.QueryResult/QueryTuple respectively.