detect CI links in reviews

Author: joyeecheung

bin/metadata.js

@@ -55,7 +55,7 @@ async function main(prid, owner, repo) {
   const metadata = new MetadataGenerator(repo, pr, reviewers).getMetadata();
   logger.info({ raw: metadata }, 'Generated metadata:');
 
-  const checker = new PRChecker(pr, reviewers, comments);
+  const checker = new PRChecker(pr, reviewers, comments, reviews);
   checker.checkReviewers();
   checker.checkReviews();
   checker.checkPRWait();

lib/ci.js

@@ -24,19 +24,23 @@ const CI_TYPES = new Map([
 ]);
 
 class CIParser {
-  constructor(comments) {
-    this.comments = comments;
+  /**
+   * @param {{bodyText: string, publishedAt: string}[]} thread
+   */
+  constructor(thread) {
+    this.thread = thread;
   }
 
+  /**
+   * @returns {Map<string, {link: string, date: string}>} 
+   */
   parse() {
-    const comments = this.comments;
-    /**
-     * @type {Map<string, {link: string, date: string}>}
-     */
+    const thread = this.thread;
     const result = new Map();
-    for (const c of comments) {
-      if (!c.bodyText.includes(CI_DOMAIN)) continue;
-      const cis = this.parseText(c.bodyText);
+    for (const c of thread) {
+      const text = c.bodyText;
+      if (!text.includes(CI_DOMAIN)) continue;
+      const cis = this.parseText(text);
       for (const ci of cis) {
         const entry = result.get(ci.type);
         if (!entry || entry.date < c.publishedAt) {

lib/pr_checker.js

@@ -17,10 +17,11 @@ const CI_TYPES = CIParser.TYPES;
 const { FULL } = CIParser.constants;
 
 class PRChecker {
-  constructor(pr, reviewers, comments) {
+  constructor(pr, reviewers, comments, reviews) {
     this.reviewers = reviewers;
     this.pr = pr;
     this.comments = comments;
+    this.reviews = reviews;
   }
 
   checkReviews() {
@@ -110,11 +111,13 @@ class PRChecker {
   // TODO: not all PR requires CI...labels?
   checkCI() {
     const comments = this.comments;
+    const reviews = this.reviews;
     const prNode = {
       publishedAt: this.pr.createdAt,
       bodyText: this.pr.bodyText
     };
-    const ciMap = new CIParser(comments.concat([prNode])).parse();
+    const thread = comments.concat([prNode]).concat(reviews);
+    const ciMap = new CIParser(thread).parse();
     if (!ciMap.size) {
       logger.warn('No CI runs detected');
     } else if (!ciMap.get(FULL)) {

queries/Reviews.gql

@@ -8,6 +8,7 @@ query Reviews($prid: Int!, $owner: String!, $repo: String!, $after: String) {
           endCursor
         }
         nodes {
+          bodyText
           state
           author {
             login