add TODO

Author: yaacovCR

src/execution/execute.ts

@@ -139,6 +139,8 @@ const collectSubfields = memoize3(
  */
 export interface ValidatedExecutionArgs {
   schema: GraphQLSchema;
+  // TODO: consider deprecating if/when fragment arguments are officially supported
+  // and/or are exposed within GraphQLResolveInfo.
   fragmentDefinitions: ObjMap<FragmentDefinitionNode>;
   fragments: ObjMap<FragmentDetails>;
   rootValue: unknown;
@@ -499,6 +501,7 @@ export function validateExecutionArgs(
         }
         break;
       case Kind.FRAGMENT_DEFINITION: {
+        fragmentDefinitions[definition.name.value] = definition;
         let variableSignatures;
         if (definition.variableDefinitions) {
           variableSignatures = Object.create(null);
@@ -507,7 +510,6 @@ export function validateExecutionArgs(
             variableSignatures[signature.name] = signature;
           }
         }
-        fragmentDefinitions[definition.name.value] = definition;
         fragments[definition.name.value] = { definition, variableSignatures };
         break;
       }