Exposing a GitHub API to users of Danger

changelog.md

@@ -2,6 +2,8 @@
 
 ### Master
 
+* Adds `github.api` more to come - @orta
+
 ### 0.17.0
 
 * [Enhancements to `danger.git.diffForFile()`](https://github.com/danger/danger-js/pull/223) - @namuol

dangerfile.ts

@@ -12,6 +12,8 @@ declare function schedule(promise: () => Promise<any | void>): void
 
 import * as fs from "fs"
 import * as child_process from "child_process"
+import fetch from "node-fetch"
+import { distanceInWords } from "date-fns"
 
 // For some reason we're getting type errors on this includes module?
 // Wonder if we could move to the includes function in ES2015?
@@ -46,27 +48,105 @@ const checkForRelease = (packageDiff) => {
 
 // This is a great candidate for being a Danger plugin.
 
-const checkForNewDependencies = (packageDiff) => {
+const checkForNewDependencies = async (packageDiff) => {
   if (packageDiff.dependencies) {
     if (packageDiff.dependencies.added.length) {
       const newDependencies = packageDiff.dependencies.added as string[]
       warn(`New dependencies added: ${sentence(newDependencies)}.`)
 
-      newDependencies.forEach(dep => {
+      for (const dep of newDependencies) {
         const output = child_process.execSync(`yarn why ${dep} --json`)
 
         // Comes as a series of little JSON messages
         const usefulJSONContents = output.toString().split(`{"type":"activityEnd","data":{"id":0}}`).pop() as string
         const asJSON = usefulJSONContents.split("}\n{").join("},{")
 
         const whyJSON = JSON.parse(`[${asJSON}]`) as any[]
-        const messages = whyJSON.filter(msg => typeof msg.data === "string")
-        markdown(`
-## ${dep}
-
-${messages.join("\n\n - ")}
-          `)
-      })
+        const messages = whyJSON.filter(msg => typeof msg.data === "string").map(m => m.data)
+        const yarnDetails = `
+<details>
+  <summary><code>yarn why ${dep}</code> output</summary>
+   <p><code><ul><li>${messages.join("</li><li>")}
+   </li></ul></code></p>
+</details>
+`
+        // Grab the NPM metadata
+        let npmMetadata = ""
+        const npmResponse = await fetch(`https://registry.npmjs.org/${dep}`)
+        if (npmResponse.ok) {
+          const tableDeets = [] as [{ name: string, message: string}]
+          const npm = await npmResponse.json()
+
+          if (npm.time && npm.time.created) {
+            const distance = distanceInWords(new Date(npm.time.created), new Date)
+            tableDeets.push ({ name: "Created", message: `${distance} ago`  })
+          }
+
+          if (npm.time && npm.time.modified) {
+            const distance = distanceInWords(new Date(npm.time.modified), new Date)
+            tableDeets.push ({ name: "Last Updated", message: `${distance} ago` })
+          }
+
+          if (npm.license) {
+            tableDeets.push({ name: "License", message: npm.license })
+          } else {
+            tableDeets.push({ name: "License", message: "<b>NO LICENSE FOUND</b>" })
+          }
+
+          if (npm.maintainers) {
+            tableDeets.push({ name: "Maintainers", message: npm.maintainers.length })
+          }
+
+          if (npm["dist-tags"] && npm["dist-tags"]["latest"]) {
+            const currentTag = npm["dist-tags"]["latest"]
+            const tag = npm.versions[currentTag]
+            tableDeets.push ({ name: "Releases", message: String(Object.keys(npm.versions).length) })
+            if (tag.dependencies) {
+              const deps = Object.keys(tag.dependencies)
+              const depLinks = deps.map(d => `<a href='http: //npmjs.com/package/${d}'>${d}</a>`)
+              tableDeets.push ({ name: "Direct Dependencies", message: sentence(depLinks) })
+            }
+          }
+
+          if (npm.keywords && npm.keywords.length) {
+            tableDeets.push({ name: "Keywords", message: sentence(npm.keywords) })
+          }
+
+          let readme = "This README is too long to show."
+          if (npm.readme && npm.readme.length < 10000) {
+            readme = `
+<details>
+<summary><code>README</code></summary>
+
+${npm.readme}
+
+</details>
+`
+          }
+
+          const homepage = npm.homepage ? npm.homepage : `http: //npmjs.com/package/${dep}`
+
+          npmMetadata = `
+<h2><a href="${homepage}">${dep}</a></h2>
+
+Author: ${npm.author && npm.author.name ? npm.author.name : "Unknown"}
+Description: ${npm.description}
+Repo: ${homepage}
+
+<table>
+  <thead><tr><th></th><th width="100%"></th></tr></thead>
+  ${tableDeets.map(deet => `<tr><td>${deet.name}</td><td>${deet.message}</td></tr>`).join("")}
+</table>
+
+${readme}
+`
+        } else {
+          const errorMessage = await npmResponse.text()
+          npmMetadata = `Error getting NPM details: ${errorMessage}`
+        }
+
+        markdown(`${npmMetadata} ${yarnDetails}`)
+      }
     }
   }
 }
@@ -103,7 +183,7 @@ const checkForTypesInDeps = (packageDiff) => {
 schedule(async () => {
   const packageDiff = await danger.git.JSONDiffForFile("package.json")
   checkForRelease(packageDiff)
-  checkForNewDependencies(packageDiff)
+  await checkForNewDependencies(packageDiff)
   checkForLockfileDiff(packageDiff)
   checkForTypesInDeps(packageDiff)
 })

package.json

@@ -85,6 +85,7 @@
     "babel-plugin-transform-regenerator": "^6.22.0",
     "babel-preset-es2015": "^6.24.0",
     "babel-preset-stage-3": "^6.22.0",
+    "date-fns": "^1.28.3",
     "husky": "^0.13.3",
     "in-publish": "^2.0.0",
     "jest": "^19.0.2",
@@ -102,6 +103,7 @@
     "chalk": "^1.1.1",
     "commander": "^2.9.0",
     "debug": "^2.6.0",
+    "github": "^9.2.0",
     "jest-config": "^19.0.2",
     "jest-environment-node": "^19.0.2",
     "jest-runtime": "^19.0.2",

source/danger.d.ts

@@ -223,6 +223,8 @@ export interface GitDSL {
 
 /** The GitHub metadata for your PR */
 export interface GitHubDSL {
+  /** An authenticated API so you can extend danger's behavior. An instance of the "github" npm module. */
+  api: GitHub,
   /** The issue metadata for a code review session */
   issue: GitHubIssue,
   /** The PR metadata for a code review session */

source/dsl/GitHubDSL.ts

@@ -1,9 +1,12 @@
 import { GitCommit } from "./Commit"
+import * as GitHub from "github"
 
 // This is `danger.github`
 
 /** The GitHub metadata for your PR */
 export interface GitHubDSL {
+  /** An authenticated API so you can extend danger's behavior. An instance of the "github" npm module. */
+  api: GitHub,
   /** The issue metadata for a code review session */
   issue: GitHubIssue,
   /** The PR metadata for a code review session */

source/platforms/GitHub.ts

@@ -75,8 +75,10 @@ export class GitHub {
     const commits = await this.api.getPullRequestCommits()
     const reviews = await this.api.getReviews()
     const requested_reviewers = await this.api.getReviewerRequests()
+    const externalAPI = this.api.getExternalAPI()
 
     return {
+      api: externalAPI,
       issue,
       pr,
       commits,

source/platforms/_tests/_github.test.ts

@@ -34,6 +34,10 @@ jest.mock("../github/GitHubAPI", () => {
       const fixtures = await requestWithFixturedJSON("requested_reviewers.json")
       return await fixtures()
     }
+
+    getExternalAPI() {
+      return {}
+    }
   }
 
   return { GitHubAPI }

source/platforms/github/GitHubAPI.ts

@@ -4,6 +4,7 @@ import { GitHubPRDSL, GitHubUser} from "../../dsl/GitHubDSL"
 import * as find from "lodash.find"
 
 import * as node_fetch from "node-fetch"
+import * as GitHubNodeAPI from "github"
 
 // The Handle the API specific parts of the github
 
@@ -25,6 +26,26 @@ export class GitHubAPI {
     this.additionalHeaders = {}
   }
 
+  /**
+   * Bit weird, yes, but we want something that can be exposed to an end-user.
+   * I wouldn't have a problem with moving this to use this API under the hood
+   * but for now that's just a refactor someone can try.
+   */
+  getExternalAPI(): GitHubNodeAPI {
+    const baseUrl = process.env["DANGER_GITHUB_API_BASE_URL"] || null
+    const api = new GitHubNodeAPI({
+      host: baseUrl,
+      headers: {
+        ...this.additionalHeaders
+      }
+    })
+
+    if (this.token) {
+      api.authenticate({ type: "token", token: this.token })
+    }
+    return api
+  }
+
   /**
    * Grabs the contents of an individual file on GitHub
    *
@@ -166,7 +187,6 @@ export class GitHubAPI {
 
     return res.ok ? res.json() : { labels: [] }
   }
-
   // API Helpers
 
   private api(path: string, headers: any = {}, body: any = {}, method: string) {