@heroku/skynet 1.6.12 → 1.6.14

package/README.md

@@ -105,7 +105,8 @@ $ npm test
 $ npm run release
 ```
 
-* Verify that new package version is created on [npm website](https://www.npmjs.com/package/@heroku/skynet).
+* Command 'npm run release' will change the version number in package.json file and commit. but the pull request will fail due to the enabled git pull process. A follow-up manual pull request is needed to check in the new version number in Git repository.    
+* Verify that new package version is created on [npm website](https://www.npmjs.com/package/@heroku/skynet), or directly run command "npm view @heroku/skynet"
 * Note: You will get an error at the end of `npm run release`. Np attempts to commit to master branch the updated package version in the package.json. But the commit will fail because our master branch is protected. You can ignore the error message. 
 * Submit a new PR to update the version in `package.json` and `package-lock.json`. Here is one [PR example](https://github.com/heroku/heroku-skynet-cli/pull/74).
 * Test the newly released @heroku/skynet package

package/commands/categories/add.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/commands/categories/remove.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/commands/categories.js

@@ -1,7 +1,7 @@
-let cli = require('heroku-cli-util')
-let command = require('../lib/command')
-let sudo = require('../lib/sudo')
-let SkynetAPI = require('../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../lib/command')
+const sudo = require('../lib/sudo')
+const SkynetAPI = require('../lib/skynet')
 
 async function run (context) {
   sudo()

package/commands/deprovision.js

@@ -1,16 +1,16 @@
-let cli = require('heroku-cli-util')
-let command = require('../lib/command')
-let sudo = require('../lib/sudo')
-let SkynetAPI = require('../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../lib/command')
+const sudo = require('../lib/sudo')
+const SkynetAPI = require('../lib/skynet')
 
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let user = context.flags.user || process.env.HEROKU_USER
-  let notes = context.flags.notes
-  let category = context.flags.category
-  let force = context.flags.bypass
-  let notify = !context.flags['no-notify']
+  const user = context.flags.user || process.env.HEROKU_USER
+  const notes = context.flags.notes
+  const category = context.flags.category
+  const force = context.flags.bypass
+  const notify = !context.flags['no-notify']
 
   if (!user) {
     throw new Error('Required flag: --user USER')

package/commands/suspend/app-owner.js

@@ -1,14 +1,14 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let sudo = require('../../lib/sudo')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const sudo = require('../../lib/sudo')
+const SkynetAPI = require('../../lib/skynet')
 
 function readlines (file) {
   return new Promise(function (resolve, reject) {
-    let split = require('split')
-    let fs = require('fs')
+    const split = require('split')
+    const fs = require('fs')
 
-    let users = []
+    const users = []
 
     fs.createReadStream(file).pipe(split())
       .on('data', function (line) {
@@ -29,19 +29,19 @@ function readlines (file) {
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let app = context.flags.app
-  let file = context.flags.infile
-  let notes = context.flags.notes
-  let category = context.flags.category
-  let force = context.flags.bypass
+  const app = context.flags.app
+  const file = context.flags.infile
+  const notes = context.flags.notes
+  const category = context.flags.category
+  const force = context.flags.bypass
 
   if (app && file) {
     throw new Error('Either --app or --infile must be passed, but not both')
   }
 
   if (file) {
-    let apps = await readlines(file)
-    let response = await skynet.bulkSuspendAppOwner(apps.join(), notes, category)
+    const apps = await readlines(file)
+    const response = await skynet.bulkSuspendAppOwner(apps.join(), notes, category)
     cli.log(response)
   } else {
     if (!app) {

package/commands/suspend/apps.js

@@ -1,7 +1,7 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let sudo = require('../../lib/sudo')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const sudo = require('../../lib/sudo')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   sudo()

package/commands/suspend/user.js

@@ -1,14 +1,14 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let sudo = require('../../lib/sudo')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const sudo = require('../../lib/sudo')
+const SkynetAPI = require('../../lib/skynet')
 
 function readlines (file) {
   return new Promise(function (resolve, reject) {
-    let split = require('split')
-    let fs = require('fs')
+    const split = require('split')
+    const fs = require('fs')
 
-    let users = []
+    const users = []
 
     fs.createReadStream(file).pipe(split())
       .on('data', function (line) {
@@ -29,20 +29,20 @@ function readlines (file) {
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let user = context.flags.user || process.env.HEROKU_USER
-  let file = context.flags.infile
-  let notes = context.flags.notes
-  let category = context.flags.category
-  let force = context.flags.bypass
-  let notify = !context.flags['no-notify']
+  const user = context.flags.user || process.env.HEROKU_USER
+  const file = context.flags.infile
+  const notes = context.flags.notes
+  const category = context.flags.category
+  const force = context.flags.bypass
+  const notify = !context.flags['no-notify']
 
   if (user && file) {
     throw new Error('Either --user USER or --infile must be passed, but not both')
   }
 
   if (file) {
-    let users = await readlines(file)
-    let response = await skynet.bulkSuspendUsers(users.join(), notes, category, notify, force)
+    const users = await readlines(file)
+    const response = await skynet.bulkSuspendUsers(users.join(), notes, category, notify, force)
     cli.log(response)
   } else {
     if (!user) {

package/commands/unsuspend/apps.js

@@ -1,14 +1,14 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let sudo = require('../../lib/sudo')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const sudo = require('../../lib/sudo')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let app = context.flags.app
-  let category = context.flags.category
-  let notes = context.flags.notes
+  const app = context.flags.app
+  const category = context.flags.category
+  const notes = context.flags.notes
 
   if (!app) {
     throw new Error('Required flag: --app APP')
@@ -26,8 +26,8 @@ module.exports = {
     $ heroku skynet:unsuspend:app -a test-app -n "helpful unsuspend message" -c "ddos"`,
   flags: [
     { name: 'app', char: 'a', description: 'app to unsuspend', hasValue: true, required: true },
-    { name: 'category', char: 'c', description: 'unsuspension category', hasValue: true, required: false },
-    { name: 'notes', char: 'n', description: 'unsuspend notes', hasValue: true, required: false }
+    { name: 'category', char: 'c', description: 'unsuspension category', hasValue: true, required: true },
+    { name: 'notes', char: 'n', description: 'unsuspend notes', hasValue: true, required: true }
   ],
   run: command(run)
 }

package/commands/unsuspend/user.js

@@ -1,14 +1,14 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let sudo = require('../../lib/sudo')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const sudo = require('../../lib/sudo')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let user = context.flags.user || process.env.HEROKU_USER
-  let category = context.flags.category
-  let notes = context.flags.notes
+  const user = context.flags.user || process.env.HEROKU_USER
+  const category = context.flags.category
+  const notes = context.flags.notes
 
   if (!user) {
     throw new Error('Required flag: --user USER')
@@ -26,8 +26,8 @@ module.exports = {
     $ heroku skynet:unsuspend:user -u foo@bar.com -n "helpful unsuspend message" -c "ddos"`,
   flags: [
     { name: 'user', char: 'u', description: 'user to unsuspend', hasValue: true },
-    { name: 'category', char: 'c', description: 'unsuspension category', hasValue: true, required: false },
-    { name: 'notes', char: 'n', description: 'unsuspend notes', hasValue: true, required: false }
+    { name: 'category', char: 'c', description: 'unsuspension category', hasValue: true, required: true },
+    { name: 'notes', char: 'n', description: 'unsuspend notes', hasValue: true, required: true }
   ],
   run: command(run)
 }

package/commands/userpass/add.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/commands/userpass/remove.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/commands/whitelist/add.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/commands/whitelist/remove.js

@@ -1,6 +1,6 @@
-let cli = require('heroku-cli-util')
-let command = require('../../lib/command')
-let SkynetAPI = require('../../lib/skynet')
+const cli = require('heroku-cli-util')
+const command = require('../../lib/command')
+const SkynetAPI = require('../../lib/skynet')
 
 async function run (context) {
   const skynet = new SkynetAPI(context.auth.password)

package/lib/command.js

@@ -1,4 +1,4 @@
-let cli = require('heroku-cli-util')
+const cli = require('heroku-cli-util')
 
 module.exports = fn =>
   cli.command((context, client) =>

package/lib/skynet.js

@@ -24,7 +24,7 @@ module.exports = class SkynetAPI {
   }
 
   categories () {
-    return this.request(`/categories`, {
+    return this.request('/categories', {
       method: 'GET'
     })
   }
@@ -35,7 +35,7 @@ module.exports = class SkynetAPI {
       notes: notes
     }
 
-    return this.request(`/whitelist`, {
+    return this.request('/whitelist', {
       method: 'POST',
       body: body,
       form: true
@@ -56,7 +56,7 @@ module.exports = class SkynetAPI {
       flag: flag
     }
 
-    return this.request(`/userpass/remove`, {
+    return this.request('/userpass/remove', {
       method: 'POST',
       body: body,
       form: true
@@ -69,7 +69,7 @@ module.exports = class SkynetAPI {
       flag: flag
     }
 
-    return this.request(`/userpass/add`, {
+    return this.request('/userpass/add', {
       method: 'POST',
       body: body,
       form: true
@@ -126,7 +126,7 @@ module.exports = class SkynetAPI {
       notify: notify
     }
 
-    return this.request(`/suspend/user`, {
+    return this.request('/suspend/user', {
       method: 'POST',
       body: body,
       form: true
@@ -152,7 +152,7 @@ module.exports = class SkynetAPI {
       notify: notify
     }
 
-    return this.request(`/suspend/user`, {
+    return this.request('/suspend/user', {
       method: 'POST',
       body: body,
       form: true
@@ -168,7 +168,7 @@ module.exports = class SkynetAPI {
       bulk: 'true'
     }
 
-    return this.request(`/suspend/app-owner`, {
+    return this.request('/suspend/app-owner', {
       method: 'POST',
       body: body,
       form: true
@@ -185,7 +185,7 @@ module.exports = class SkynetAPI {
       notify: notify
     }
 
-    return this.request(`/deprovision`, {
+    return this.request('/deprovision', {
       method: 'POST',
       body: body,
       form: true
@@ -198,7 +198,7 @@ module.exports = class SkynetAPI {
       description: description
     }
 
-    return this.request(`/categories`, {
+    return this.request('/categories', {
       method: 'POST',
       body: body,
       json: true

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heroku/skynet",
-  "version": "1.6.12",
+  "version": "1.6.14",
   "description": "use Skynet from Heroku CLI",
   "main": "index.js",
   "author": "Bob Argenbright @byt3smith",
@@ -22,11 +22,13 @@
     "split": "^1.0.0"
   },
   "devDependencies": {
+    "acorn": ">=6.4.1",
+    "minimist": ">=1.2.2",
     "mocha": "^5.2.0",
     "mockdate": "^2.0.1",
-    "nock": "~9.0.13",
+    "nock": "^12.0.3",
     "np": "^3.1.0",
-    "standard": "^12.0.1",
+    "standard": "^14.3.3",
     "unexpected": "^10.29.0"
   },
   "scripts": {