Improve GitHub API requests efficiency (#19)

* Improve GitHub API requests efficiency

* Split repositories into individual fetches

Author: Aliamondo

src/helpers/dataFetcher.ts

@@ -414,33 +414,38 @@ async function fetchPullRequests({
 
   if (!orgName || !teamName) return []
 
-  const allTeamUsers: string[] = await gql<GraphQL_UserResponse>(
-    getTeamUsersQuery({ orgName, teamName }),
-  )
-    .then((res: GraphQL_UserResponse) =>
-      res.organization.teams.nodes[0].members.nodes.map(
-        (user: GraphQL_User) => user.login,
-      ),
+  let teamUsers: string[]
+
+  if (savedMembers && savedMembers.length > 0) {
+    teamUsers = savedMembers
+  } else {
+    const allTeamUsers: string[] = await gql<GraphQL_UserResponse>(
+      getTeamUsersQuery({ orgName, teamName }),
     )
-    .catch((error: { response?: { status?: number } }) => {
-      if (error.response?.status === 401 || error.response?.status === 403) {
-        handleInvalidTokenError()
-        return []
-      } else throw error
-    })
+      .then((res: GraphQL_UserResponse) =>
+        res.organization.teams.nodes[0].members.nodes.map(
+          (user: GraphQL_User) => user.login,
+        ),
+      )
+      .catch((error: { response?: { status?: number } }) => {
+        if (error.response?.status === 401 || error.response?.status === 403) {
+          handleInvalidTokenError()
+          return []
+        } else throw error
+      })
 
-  if (!allTeamUsers.length) {
-    setProgress(100)
-    return []
-  }
+    if (!allTeamUsers.length) {
+      setProgress(100)
+      return []
+    }
 
-  const teamUsers =
-    savedMembers && savedMembers.length > 0 ? savedMembers : allTeamUsers
+    teamUsers = allTeamUsers
+  }
 
   setProgress(10)
 
   let progress = 10
-  const totalResources = teamUsers.length + (teamRepositories ? 1 : 0)
+  const totalResources = teamUsers.length + (teamRepositories?.length ?? 0)
 
   const pullRequestPromises = teamUsers.map(user =>
     gql<GraphQL_PullRequestsResponse>(
@@ -452,19 +457,21 @@ async function fetchPullRequests({
     }),
   )
 
-  teamRepositories?.length &&
+  teamRepositories?.forEach(repo =>
     pullRequestPromises.push(
       gql<GraphQL_PullRequestsResponse>(
         getPullRequestsByRepositoriesQuery({
-          repositories: teamRepositories,
+          repository: repo,
+          excludeAuthors: teamUsers,
           includeChecks,
         }),
       ).then((res: GraphQL_PullRequestsResponse) => {
         progress += 90 / totalResources
         setProgress(progress)
         return res
       }),
-    )
+    ),
+  )
 
   const rawPullRequestsData = await Promise.all(pullRequestPromises)
   const rawPullRequests = rawPullRequestsData

src/helpers/graphqlQueries.ts

@@ -386,18 +386,19 @@ function getPullRequestsByUserQuery({
 }
 
 type GetPullRequestsByRepositoriesQueryProps = {
-  repositories: string[]
+  repository: string
+  excludeAuthors?: string[]
   includeChecks?: boolean
 }
 function getPullRequestsByRepositoriesQuery({
-  repositories,
+  repository,
+  excludeAuthors = [],
   includeChecks = false,
 }: GetPullRequestsByRepositoriesQueryProps) {
+  const exclusions = excludeAuthors.map(login => `-author:${login}`).join(' ')
   const query = `{
     search(
-      query: "is:open type:pr ${repositories
-        .map(name => `repo:${name}`)
-        .join(' ')}"
+      query: "is:open type:pr repo:${repository} ${exclusions}"
       first: 100
       type: ISSUE
     ) {