List Co-Committers as Learning Path Contributors (InnerSourceCommons#380)

rrrutledge · web-flow · commit 0c4a96ffb0f4 · 2021-03-10T16:52:54.000-08:00
* When someone leaves a commit suggestion that is merged, they show up as a 'co-author' on that commit * GitHub exposes this information in their graph API (https://bit.ly/3cdyYQQ)
diff --git a/scripts/get_contributors.js b/scripts/get_contributors.js
@@ -8,19 +8,21 @@ const graphqlWithAuth = graphql.defaults({
 })
 
 module.exports = async function (filepath) {
-  const contributors = await graphqlWithAuth(
+  const { repository: { object: { history }}} = await graphqlWithAuth(
     `{
       repository(owner: "InnerSourceCommons", name: "InnerSourceLearningPath") {
         object(expression: "master") {
           ... on Commit {
             history(first: 100, path: "${filepath}") {
               totalCount
               nodes {
-                author {
-                  name
-                  user {
+                authors(first: 100) {
+                  nodes {
                     name
-                    url
+                    user {
+                      name
+                      url
+                    }
                   }
                 }
               }
@@ -31,18 +33,22 @@ module.exports = async function (filepath) {
     }`
   )
 
-  const history = contributors.repository.object.history
-
   if (history.totalCount > 100) {
     throw Error('This script needs updating to handle >100 commits')
   }
 
-  return Object.values(history.nodes.reduce((acc, { author }) => {
-    const name = (author.user && author.user.name) || author.name
-    acc[name] = {
-      name,
-      url: author.user && author.user.url
-    }
-    return acc
-  }, {}))
-}
+  return Object.values(
+    history.nodes.flatMap(({ authors }) => {
+      return authors.nodes.map(author => {
+        return {
+          name: (author.user && author.user.name) || author.name,
+          url: author.user && author.user.url
+        }
+      })
+    }).reduce((accumulator, user) => {
+      // Dedupe users
+      accumulator[user.name] = user
+      return accumulator
+    }, {})
+  )
+}
