isolate async iterator next rejection within try block

to improve readability

Author: yaacovCR

src/execution/execute.ts

@@ -1064,15 +1064,16 @@ async function completeAsyncIteratorValue(
     try {
       // eslint-disable-next-line no-await-in-loop
       iteration = await iterator.next();
-      if (iteration.done) {
-        break;
-      }
     } catch (rawError) {
       const error = locatedError(rawError, fieldNodes, pathToArray(itemPath));
       completedResults.push(handleFieldError(error, itemType, errors));
       break;
     }
 
+    if (iteration.done) {
+      break;
+    }
+
     if (
       completeListItemValue(
         iteration.value,
@@ -1952,20 +1953,23 @@ async function executeStreamIteratorItem(
   asyncPayloadRecord: StreamRecord,
   itemPath: Path,
 ): Promise<IteratorResult<unknown>> {
-  let item;
+  let iteration;
   try {
-    const { value, done } = await iterator.next();
-    if (done) {
-      asyncPayloadRecord.setIsCompletedIterator();
-      return { done, value: undefined };
-    }
-    item = value;
+    iteration = await iterator.next();
   } catch (rawError) {
     const error = locatedError(rawError, fieldNodes, pathToArray(itemPath));
     const value = handleFieldError(error, itemType, asyncPayloadRecord.errors);
     // don't continue if iterator throws
     return { done: true, value };
   }
+
+  const { done, value } = iteration;
+
+  if (done) {
+    asyncPayloadRecord.setIsCompletedIterator();
+    return { done, value: undefined };
+  }
+
   return {
     done: false,
     value: completeValueCatchingErrors(
@@ -1974,7 +1978,7 @@ async function executeStreamIteratorItem(
       fieldNodes,
       info,
       itemPath,
-      item,
+      value,
       asyncPayloadRecord,
     ),
   };