Merge pull request #14 from microfeedback/core-2

Upgrade to microfeedback-core 2

Author: sloria

app.json

@@ -11,6 +11,22 @@
       "description": "Comma-separated list of GitHub repos that may be posted to. If '*', allow posting to any repo.",
       "required": false,
       "value": "*"
+    },
+    "AKISMET_API_KEY": {
+      "description": "Akismet API key to use for spam detection",
+      "required": false
+    },
+    "ENABLE_AKISMET": {
+      "description": "Whether to use the Akismet API for syntax checking. Set to 'false' if AKISMET_API_KEY is set but you don't want spam checking.",
+      "required": false
+    },
+    "PERSPECTIVE_API_KEY": {
+      "description": "Perspective API key to use for toxicity scoring",
+      "required": false
+    },
+    "ENABLE_PERSPECTIVE": {
+      "description": "Whether to use the Perspective API for toxicity scoring. Set to 'false' if PERSPECTIVE_API_KEY is set but you don't want toxicity scoring.",
+      "required": false
     }
   },
   "buildpacks": [

index.js

@@ -61,6 +61,12 @@ const issueTemplate = `
 {{&osTable}}
 {{/osTable}}
 
+{{#perspectiveTable}}
+### Perspective API
+
+{{&perspectiveTable}}
+{{/perspectiveTable}}
+
 {{#extraTable}}
 ### Extra information
 
@@ -75,7 +81,7 @@ Reported via *[{{pkg.name}}]({{&pkg.repository}}) v{{pkg.version}}*.
 `;
 mustache.parse(issueTemplate);
 
-const makeIssue = ({body, extra, screenshotURL}, req) => {
+const makeIssue = ({body, extra, perspective, screenshotURL}, req) => {
   let suffix = '';
   if (req && req.headers.referer) {
     suffix = ` on ${req.headers.referer}`;
@@ -106,10 +112,15 @@ const makeIssue = ({body, extra, screenshotURL}, req) => {
     const osEntries = Object.entries(userAgent.os).filter(e => e[1]);
     view.osTable = makeTable(['Key', 'Value'], osEntries, false);
   }
+  // Format perspective information as table
+  if (perspective) {
+    view.perspectiveTable = makeTable(['Key', 'Value'], Object.entries(perspective));
+  }
   // Format extra information as table
   if (extra) {
     view.extraTable = makeTable(['Key', 'Value'], Object.entries(extra));
   }
+  // TODO: Add spam label if akismet.spam is true
   return {title, body: mustache.render(issueTemplate, view)};
 };
 
@@ -120,7 +131,7 @@ function getAllowedRepos() {
   return process.env.ALLOWED_REPOS.split(',').map(each => each.trim());
 }
 
-const GitHubBackend = async (input, req) => {
+const GitHubBackend = async ({input, perspective, akismet}, req) => {
   // Match /<username>/<repo>/ in the URL
   // TODO: Allow base64-encoded repo URL
   const {pathname} = url.parse(req.url);
@@ -131,14 +142,21 @@ const GitHubBackend = async (input, req) => {
     throw createError(400, `Repo "${repo}" not allowed.`);
   }
   const issueURL = `https://api.github.com/repos/${repo}/issues`;
+  const {body, extra, screenshotURL} = input;
   try {
     const {data} = await axios({
       method: 'POST',
       url: issueURL,
       params: {
         access_token: GH_TOKEN,
       },
-      data: makeIssue(input, req),
+      data: makeIssue({
+        body,
+        extra,
+        screenshotURL,
+        perspective,
+        akismet,
+      }, req),
     });
     return data;
   } catch (err) {