feat(ncu-ci): check for request-ci label (#806)

When running `ncu-ci` without the `--certify-safe` CLI flag,
if something was pushed to a PR since the last approving review, check
if the last time the `request-ci` label was added, it was done by a
Collaborator and after the last push event on the PR.

Co-authored-by: Mathis Wiehl <[email protected]>

Author: aduh95

lib/ci/run_ci.js

@@ -27,7 +27,7 @@ export class RunPRJob {
     this.certifySafe =
       certifySafe ||
       Promise.all([this.prData.getReviews(), this.prData.getPR()]).then(() =>
-        new PRChecker(cli, this.prData, request, {}).checkCommitsAfterReview()
+        new PRChecker(cli, this.prData, request, {}).checkCommitsAfterReviewOrLabel()
       );
   }
 

lib/pr_checker.js

@@ -523,6 +523,32 @@ export default class PRChecker {
     return true;
   }
 
+  async checkCommitsAfterReviewOrLabel() {
+    if (this.checkCommitsAfterReview()) return true;
+
+    await Promise.all([this.data.getLabeledEvents(), this.data.getCollaborators()]);
+
+    const {
+      cli, data, pr
+    } = this;
+
+    const { updatedAt } = pr.timelineItems;
+    const requestCiLabels = data.labeledEvents.findLast(
+      ({ createdAt, label: { name } }) => name === 'request-ci' && createdAt > updatedAt
+    );
+    if (requestCiLabels == null) return false;
+
+    const { actor: { login } } = requestCiLabels;
+    const collaborators = Array.from(data.collaborators.values(),
+      (c) => c.login.toLowerCase());
+    if (collaborators.includes(login.toLowerCase())) {
+      cli.info('request-ci label was added by a Collaborator after the last push event.');
+      return true;
+    }
+
+    return false;
+  }
+
   checkCommitsAfterReview() {
     const {
       commits, reviews, cli, argv

lib/pr_data.js

@@ -5,6 +5,7 @@ import {
 } from './user_status.js';
 
 // lib/queries/*.gql file names
+const LABELED_EVENTS_QUERY = 'PRLabeledEvents';
 const PR_QUERY = 'PR';
 const REVIEWS_QUERY = 'Reviews';
 const COMMENTS_QUERY = 'PRComments';
@@ -33,6 +34,7 @@ export default class PRData {
     this.comments = [];
     this.commits = [];
     this.reviewers = [];
+    this.labeledEvents = [];
   }
 
   getThread() {
@@ -90,6 +92,14 @@ export default class PRData {
     ]);
   }
 
+  async getLabeledEvents() {
+    const { prid, owner, repo, cli, request, prStr } = this;
+    const vars = { prid, owner, repo };
+    cli.updateSpinner(`Getting labels from ${prStr}`);
+    this.labeledEvents = (await request.gql(LABELED_EVENTS_QUERY, vars))
+      .repository.pullRequest.timelineItems.nodes;
+  }
+
   async getComments() {
     const { prid, owner, repo, cli, request, prStr } = this;
     const vars = { prid, owner, repo };

lib/queries/PRLabeledEvents.gql

@@ -0,0 +1,19 @@
+query PRLabeledEvents($prid: Int!, $owner: String!, $repo: String!, $after: String) {
+  repository(owner: $owner, name: $repo) {
+    pullRequest(number: $prid) {
+      timelineItems(itemTypes: LABELED_EVENT, after: $after, last: 100) {
+        nodes { 
+          ... on LabeledEvent {
+            actor {
+              login
+            }
+            label {
+              name
+            }
+            createdAt
+          }
+        }
+      }
+    }
+  }
+}

test/fixtures/data.js

@@ -134,3 +134,12 @@ for (const subdir of readdirSync(path('./jenkins'))) {
       readJSON(`./jenkins/${subdir}/${item}`);
   }
 };
+
+export const labeledEvents = {};
+
+for (const item of readdirSync(path('./labeled_events'))) {
+  if (!item.endsWith('.json')) {
+    continue;
+  }
+  labeledEvents[basename(item, '.json')] = readJSON(`./labeled_events/${item}`);
+}

test/fixtures/first_timer_pr.json

@@ -22,6 +22,7 @@
       }
     ]
   },
+  "timelineItems": { "updatedAt": "2017-10-24T11:13:43Z" },
   "title": "test: awesome changes",
   "baseRefName": "main",
   "headRefName": "awesome-changes"

test/fixtures/labeled_events/no-request-ci.json

@@ -0,0 +1,12 @@
+[
+  {
+    "actor": { "login": "nodejs-github-bot" },
+    "label": { "name": "doc" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  },
+  {
+    "actor": { "login": "nodejs-github-bot" },
+    "label": { "name": "test_runner" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  }
+]

test/fixtures/labeled_events/old-request-ci-collaborator.json

@@ -0,0 +1,12 @@
+[
+  {
+    "actor": { "login": "nodejs-github-bot" },
+    "label": { "name": "doc" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  },
+  {
+    "actor": { "login": "foo" },
+    "label": { "name": "request-ci" },
+    "createdAt": "1999-10-24T11:13:43Z"
+  }
+]

test/fixtures/labeled_events/recent-request-ci-collaborator.json

@@ -0,0 +1,12 @@
+[
+  {
+    "actor": { "login": "nodejs-github-bot" },
+    "label": { "name": "doc" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  },
+  {
+    "actor": { "login": "foo" },
+    "label": { "name": "request-ci" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  }
+]

test/fixtures/labeled_events/recent-request-ci-non-collaborator.json

@@ -0,0 +1,12 @@
+[
+  {
+    "actor": { "login": "nodejs-github-bot" },
+    "label": { "name": "doc" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  },
+  {
+    "actor": { "login": "random-person" },
+    "label": { "name": "request-ci" },
+    "createdAt": "2024-05-13T15:57:10Z"
+  }
+]