refactor: improve code formatting and consistency in GraphQL interfaces

Author: nookyo

actions/cla-assistant/dist/index.js

@@ -3,40 +3,39 @@ import { context } from '@actions/github'
 import type { ICommittersDetails } from './interfaces.js'
 
 interface IGraphQlUser {
-    id: string
-    databaseId: number
-    login: string
+  id: string
+  databaseId: number
+  login: string
 }
 
 interface IGraphQlCommit {
-    author: { email: string; name: string; user: IGraphQlUser | null }
-    committer: { name: string; user: IGraphQlUser | null }
+  author: { email: string; name: string; user: IGraphQlUser | null }
+  committer: { name: string; user: IGraphQlUser | null }
 }
 
 interface IGraphQlEdge {
-    node: { commit: IGraphQlCommit }
-    cursor: string
+  node: { commit: IGraphQlCommit }
+  cursor: string
 }
 
 interface IGraphQlResponse {
-    repository: {
-        pullRequest: {
-            commits: {
-                totalCount: number
-                edges: IGraphQlEdge[]
-                pageInfo: { endCursor: string; hasNextPage: boolean }
-            }
-        }
+  repository: {
+    pullRequest: {
+      commits: {
+        totalCount: number
+        edges: IGraphQlEdge[]
+        pageInfo: { endCursor: string; hasNextPage: boolean }
+      }
     }
+  }
 }
 
-
-
 export default async function getCommitters(): Promise<ICommittersDetails[]> {
-    try {
-        const committers: ICommittersDetails[] = []
-        let filteredCommitters: ICommittersDetails[] = []
-        const response = await octokit.graphql<IGraphQlResponse>(`
+  try {
+    const committers: ICommittersDetails[] = []
+    let filteredCommitters: ICommittersDetails[] = []
+    const response = await octokit.graphql<IGraphQlResponse>(
+      `
         query($owner:String! $name:String! $number:Int! $cursor:String!){
             repository(owner: $owner, name: $name) {
             pullRequest(number: $number) {
@@ -73,33 +72,44 @@ export default async function getCommitters(): Promise<ICommittersDetails[]> {
                 }
             }
         }
-    }`.replace(/ /g, ''), {
-            owner: context.repo.owner,
-            name: context.repo.repo,
-            number: context.issue.number,
-            cursor: ''
-        })
-        response.repository.pullRequest.commits.edges.forEach((edge: IGraphQlEdge) => {
-            const committer = extractUserFromCommit(edge.node.commit)
-            const user = {
-                name: committer.login || committer.name,
-                id: committer.databaseId || '',
-                pullRequestNo: context.issue.number
-            }
-            if (committers.length === 0 || committers.map((c) => {
-                return c.name
-            }).indexOf(user.name) < 0) {
-                committers.push(user)
-            }
-        })
-        filteredCommitters = committers.filter((committer) => {
-            return committer.id !== 41898282
-        })
-        return filteredCommitters
-
-    } catch (e) {
-        throw new Error(`graphql call to get the committers details failed: ${e}`)
-    }
-
+    }`.replace(/ /g, ''),
+      {
+        owner: context.repo.owner,
+        name: context.repo.repo,
+        number: context.issue.number,
+        cursor: ''
+      }
+    )
+    response.repository.pullRequest.commits.edges.forEach(
+      (edge: IGraphQlEdge) => {
+        const committer = extractUserFromCommit(edge.node.commit)
+        const user = {
+          name: committer.login || committer.name,
+          id: committer.databaseId || '',
+          pullRequestNo: context.issue.number
+        }
+        if (
+          committers.length === 0 ||
+          committers
+            .map(c => {
+              return c.name
+            })
+            .indexOf(user.name) < 0
+        ) {
+          committers.push(user)
+        }
+      }
+    )
+    filteredCommitters = committers.filter(committer => {
+      return committer.id !== 41898282
+    })
+    return filteredCommitters
+  } catch (e) {
+    throw new Error(`graphql call to get the committers details failed: ${e}`)
+  }
 }
-const extractUserFromCommit = (commit) => commit.author.user || commit.committer.user || commit.author || commit.committer
+const extractUserFromCommit = commit =>
+  commit.author.user ||
+  commit.committer.user ||
+  commit.author ||
+  commit.committer