Missing undefined possible type with conditional `skip` in useSuspenseQuery

[simPod]

Issue Description

const { data } = useSuspenseQuery(
    MyDocument,
    var === undefined
      ? { skip: true }
      : { variables: { x: var.x } },
  );

type of data is wrongly inferred to not be undefined. But the type should be T|undefined since skip is possible.

Link to Reproduction

do we have a TS template?

Reproduction Steps

No response

@apollo/client version

3.14.0

[phryneas]

That is a problem in Apollo Client 3 (see Playground) but not as much in Apollo Client 4 see AC4 playground

In Apollo Client 4, this code will not be accepted because your skip case is missing variables - you can only use this in a typesafe way when using skipToken instead of the skip option (which, by the way, we also recommend for AC3):

That said, I think you did catch an edge case here - if all variables are optional we run into the same problem as in AC3.

So thanks for reporting it - but I hope in the meantime skipToken can already work for you!