removed redundant logic and moved comments to outside for posterity

Author: swittk

src/GraphQL/loaders/parseClassQueries.js

@@ -107,8 +107,13 @@ const load = function (parseGraphQLSchema, parseClass, parseClassConfig: ?ParseG
           const { keys, include } = extractKeysAndInclude(
             selectedFields
               .filter(field => field.startsWith('edges.node.'))
-              .map(field => field.replace('edges.node.', ''))
-              .map(field => field.replace(/\.edges\.node/g, ''))
+              // GraphQL relation connections expose data under `edges.node.*`. Those
+              // segments do not correspond to actual Parse fields, so strip them to
+              // ensure the root relation key remains in the keys list (e.g. convert
+              // `users.edges.node.username` -> `users.username`). This preserves the
+              // synthetic relation placeholders that Parse injects while still
+              // respecting field projections.
+              .map(field => field.replace('edges.node.', '').replace(/\.edges\.node/g, ''))
               .filter(field => field.indexOf('edges.node') < 0)
           );
           const parseOrder = order && order.join(',');

src/GraphQL/loaders/parseClassTypes.js

@@ -351,11 +351,11 @@ const load = (parseGraphQLSchema, parseClass, parseClassConfig: ?ParseGraphQLCla
     ...defaultGraphQLTypes.PARSE_OBJECT_FIELDS,
     ...(className === '_User'
       ? {
-          authDataResponse: {
-            description: `auth provider response when triggered on signUp/logIn.`,
-            type: defaultGraphQLTypes.OBJECT,
-          },
-        }
+        authDataResponse: {
+          description: `auth provider response when triggered on signUp/logIn.`,
+          type: defaultGraphQLTypes.OBJECT,
+        },
+      }
       : {}),
   };
   const outputFields = () => {
@@ -386,8 +386,13 @@ const load = (parseGraphQLSchema, parseClass, parseClassConfig: ?ParseGraphQLCla
                 const { keys, include } = extractKeysAndInclude(
                   selectedFields
                     .filter(field => field.startsWith('edges.node.'))
-                    .map(field => field.replace('edges.node.', ''))
-                    .map(field => field.replace(/\.edges\.node/g, ''))
+                    // GraphQL relation connections expose data under `edges.node.*`. Those
+                    // segments do not correspond to actual Parse fields, so strip them to
+                    // ensure the root relation key remains in the keys list (e.g. convert
+                    // `users.edges.node.username` -> `users.username`). This preserves the
+                    // synthetic relation placeholders that Parse injects while still
+                    // respecting field projections.
+                    .map(field => field.replace('edges.node.', '').replace(/\.edges\.node/g, ''))
                     .filter(field => field.indexOf('edges.node') < 0)
                 );
                 const parseOrder = order && order.join(',');

src/GraphQL/parseGraphQLUtils.js

@@ -22,13 +22,7 @@ export function toGraphQLError(error) {
 export const extractKeysAndInclude = selectedFields => {
   selectedFields = selectedFields
     .filter(field => !field.includes('__typename'))
-    // GraphQL relation connections expose data under `edges.node.*`. Those
-    // segments do not correspond to actual Parse fields, so strip them to
-    // ensure the root relation key remains in the keys list (e.g. convert
-    // `users.edges.node.username` -> `users.username`). This preserves the
-    // synthetic relation placeholders that Parse injects while still
-    // respecting field projections.
-    .map(field => field.replace(/\.edges\.node/g, ''));
+
   // Handles "id" field for both current and included objects
   selectedFields = selectedFields.map(field => {
     if (field === 'id') { return 'objectId'; }