Introduce Github V4 Api For Chronological Tags (#31)

* Introduce Github V4 Api For Chronological Tags

* Remove Consoles

.eslintrc

@@ -4,6 +4,7 @@
     "jest": true
   },
   "rules": {
-    "max-len": ["error", 150]
+    "max-len": ["error", 150],
+    "no-debugger": 0
   }
 }

config/default.json

@@ -1,9 +1,11 @@
 {
   "regexp": null,
   "github": {
+    "apiVersion": "v3",
     "timeout": 0,
     "token": "<REPLACE_ME>",
     "host": "https://github.com/api/v3",
+    "v4Host": "https://api.github.com/graphql",
     "domain": null
   },
   "slack": {

package.json

@@ -10,6 +10,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
+    "@octokit/graphql": "^2.0.1",
     "@octokit/rest": "^15.8.1",
     "@slack/client": "^4.3.1",
     "common-tags": "^1.8.0",
@@ -18,7 +19,8 @@
     "lodash": "^4.17.10",
     "nconf": "^0.10.0",
     "node-fetch": "^2.1.2",
-    "pino": "^4.17.3"
+    "pino": "^4.17.3",
+    "prettier": "^1.16.4"
   },
   "devDependencies": {
     "eslint": "^4.19.1",

src/connectors/GithubConnector.js → src/connectors/GithubConnector/GithubConnectorV3.js

@@ -1,5 +1,5 @@
 const Github = require('@octokit/rest');
-const config = require('../config');
+const config = require('../../config');
 
 const {
   domain, host, timeout, token,

src/connectors/GithubConnector/GithubConnectorV4.js

@@ -0,0 +1,12 @@
+const config = require('../../config');
+
+const { domain, v4Host, token } = config.get('github');
+
+const GithubGraphQL = require('@octokit/graphql').defaults({
+  baseUrl: `${domain}/api` || v4Host,
+  headers: {
+    authorization: `bearer ${token}`,
+  },
+});
+
+module.exports = GithubGraphQL;

src/connectors/GithubConnector/__mocks__/GithubConnectorV4.js

@@ -0,0 +1 @@
+module.exports = () => () => jest.fn();

src/connectors/GithubConnector/__tests__/GithubConnector.test.js

@@ -0,0 +1,48 @@
+const GithubConnector = require('..');
+
+describe('GithubConnector', () => {
+  it('should initialize with the v3 api by default', () => {
+    const github = GithubConnector();
+
+    expect(Object.getOwnPropertyNames(github)).toMatchInlineSnapshot(`
+Array [
+  "hook",
+  "plugin",
+  "request",
+  "authenticate",
+  "activity",
+  "apps",
+  "authorization",
+  "checks",
+  "enterprise",
+  "gists",
+  "gitdata",
+  "integrations",
+  "issues",
+  "migrations",
+  "misc",
+  "orgs",
+  "projects",
+  "pullRequests",
+  "reactions",
+  "repos",
+  "search",
+  "users",
+  "getFirstPage",
+  "getLastPage",
+  "getNextPage",
+  "getPreviousPage",
+  "hasFirstPage",
+  "hasLastPage",
+  "hasNextPage",
+  "hasPreviousPage",
+]
+`);
+  });
+
+  it('should override the initial github v3 service with v4', () => {
+    const github = GithubConnector(true);
+
+    expect(github.name).toEqual('newApi');
+  });
+});

src/connectors/GithubConnector/index.js

@@ -0,0 +1,15 @@
+const v3API = require('./GithubConnectorV3');
+const v4API = require('./GithubConnectorV4');
+const { apiVersion } = require('../../config').get('github');
+
+module.exports = (overrideV4) => {
+  const isV4 = apiVersion.includes('v4');
+
+  let GithubConnector = v3API;
+
+  if (isV4 || overrideV4) {
+    GithubConnector = v4API;
+  }
+
+  return GithubConnector;
+};

src/handlers/getPullRequestHandler.js

@@ -1,5 +1,6 @@
-const { Github: github } = require('../connectors');
+const { Github } = require('../connectors');
 
+const github = Github();
 module.exports = async function getPullRequest(owner, repo, number) {
   let pr = {};
   try {

src/handlers/getTagDiffHandler.js

@@ -1,4 +1,6 @@
-const { Github: github } = require('../connectors');
+const { Github } = require('../connectors');
+
+const github = Github();
 
 module.exports = async function getTagDiff(owner, repo, head, base) {
   let tagDiff = {};

src/handlers/getTagsHandler.js

@@ -1,13 +1,41 @@
-const { Github: github } = require('../connectors');
-const logger = require('../logger');
+const idx = require('idx');
+
+const { Github } = require('../connectors');
+
+const github = Github(true);
 
 module.exports = async function getTags(owner, repo) {
-  let tags = [];
+  let repository = [];
   try {
-    tags = await github.repos.getTags({ owner, repo });
+    repository = await github({
+      query: `query GetTags($owner: String!, $repo: String!) {
+        repository(owner: $owner, name: $repo) {
+          refs(
+            refPrefix: "refs/tags/"
+            first: 100
+            orderBy: { field: TAG_COMMIT_DATE, direction: DESC }
+          ) {
+            edges {
+              node {
+                name
+              }
+            }
+          }
+        }
+      }`,
+      owner,
+      repo,
+    });
   } catch (e) {
     throw e;
   }
 
-  return tags.data;
+  const { repository: rep } = repository;
+  const response = idx(rep, _ => _.refs.edges) || [];
+
+  // We need to put the tags back into their original format so that we can be backwards compatible with
+  // the expected v3 response.
+  const tags = response.reduce((acc, edge) => [...acc, { name: idx(edge, _ => _.node.name) || '' }], []);
+
+  return tags;
 };

src/handlers/searchIssuesByCommitHandler.js

@@ -1,4 +1,6 @@
-const { Github: github } = require('../connectors');
+const { Github } = require('../connectors');
+
+const github = Github();
 
 // query is required
 module.exports = async function searchIssuesByCommit(query) {