fix(ng-dev): filter out duplicate CheckRun results

When requesting the CheckRun and StatusContext objects for a pull request commit, it is possible that Github
will return multiple CheckRuns with the same context name but different results as they are additive rather
than replacing the value.  We now sort this original list by timestamp and then filter out the duplicates to
only consider the final status result.

Author: josephperrott

.github/local-actions/branch-manager/main.js

@@ -65,11 +65,13 @@ export const PR_SCHEMA = {
                         status: graphqlTypes.custom<CheckStatusState>(),
                         conclusion: graphqlTypes.custom<CheckConclusionState | null>(),
                         name: graphqlTypes.string,
+                        completedAt: graphqlTypes.string,
                       },
                       StatusContext: {
                         __typename: graphqlTypes.constant('StatusContext'),
                         state: graphqlTypes.custom<StatusState>(),
                         context: graphqlTypes.string,
+                        createdAt: graphqlTypes.string,
                       },
                     }),
                   ],
@@ -232,26 +234,35 @@ export function getStatusesForPullRequest(
     };
   }
 
-  const statuses = statusCheckRollup.contexts.nodes.map((context) => {
-    switch (context.__typename) {
-      case 'CheckRun':
-        return {
-          type: 'check' as const,
-          name: context.name,
-          status: normalizeGithubCheckState(context.conclusion, context.status),
-        };
-      case 'StatusContext':
-        return {
-          type: 'status' as const,
-          name: context.context,
-          status: normalizeGithubStatusState(context.state),
-        };
-    }
-  });
+  /** A map of all of the statuses indexed by their names. */
+  const statusMap = new Map<string, PullRequestStatusInfo['statuses'][number]>();
+
+  statusCheckRollup.contexts.nodes
+    .sort((a, b) => {
+      const aTimestamp = Date.parse(a.completedAt || a.createdAt || '0');
+      const bTimestamp = Date.parse(b.completedAt || b.createdAt || '0');
+      return aTimestamp - bTimestamp;
+    })
+    .forEach((context) => {
+      switch (context.__typename) {
+        case 'CheckRun':
+          statusMap.set(context.name, {
+            type: 'check' as const,
+            name: context.name,
+            status: normalizeGithubCheckState(context.conclusion, context.status),
+          });
+        case 'StatusContext':
+          statusMap.set(context.context!, {
+            type: 'status' as const,
+            name: context.context!,
+            status: normalizeGithubStatusState(context.state!),
+          });
+      }
+    });
 
   return {
     combinedStatus: normalizeGithubStatusState(statusCheckRollup.state),
-    statuses,
+    statuses: Array.from(statusMap.values()),
   };
 }