Simplify types for requiring or not requiring variables

Thinkmill · Apr 22, 2022 · d002ea4 · d002ea4
1 parent 0e3e2f5
commit d002ea4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/.changeset/little-dodos-run.md b/.changeset/little-dodos-run.md
@@ -0,0 +1,5 @@
+---
+"@ts-gql/apollo": patch
+---
+
+Simplified types for requiring or not requiring variables
diff --git a/packages/apollo/index.d.ts b/packages/apollo/index.d.ts
@@ -36,19 +36,11 @@ type FetchResult<
   context?: C;
 };
 
-export type RequiredKeys<T> = {
-  [K in keyof T]: {} extends Pick<T, K> ? never : K;
-} extends { [_ in keyof T]: infer U }
-  ? {} extends U
-    ? never
-    : U
-  : never;
-
 export type HasRequiredVariables<
   TTypedDocumentNode extends TypedDocumentNode<BaseOperations>,
   RequiredResult,
   OptionalResult
-> = RequiredKeys<OperationVariables<TTypedDocumentNode>> extends never
+> = {} extends OperationVariables<TTypedDocumentNode>
   ? OptionalResult
   : RequiredResult;