chore: refactor commands

This is the first phase of refactoring the internal structure of the npm
commands to set us up for future changes.  This iteration changes the
function signature of `exec` for all the commands to be a async (no more
callbacks), and also groups all the commands into their own
subdirectory.

It also removes the Proxy `npm.commands` object, in favor of an
`npm.cmd` and `npm.exec` function that breaks up the two things that
proxy was doing. Namely, getting to the attributes of a given command
(`npm.cmd` now does this), and actually running the command `npm.exec`
does this.

PR-URL: https://github.com/npm/cli/pull/3959
Credit: @wraithgar
Close: #3959
Reviewed-by: @lukekarrys

1 files changed, 0 insertions, 29 deletions

diff --git a/lib/workspaces/arborist-cmd.js b/lib/workspaces/arborist-cmd.js
deleted file mode 100644
index a75b351be..000000000
--- a/lib/workspaces/arborist-cmd.js
+++ /dev/null
@@ -1,29 +0,0 @@
-// This is the base for all commands whose execWorkspaces just gets
-// a list of workspace names and passes it on to new Arborist() to
-// be able to run a filtered Arborist.reify() at some point.
-
-const BaseCommand = require('../base-command.js')
-class ArboristCmd extends BaseCommand {
-  get isArboristCmd () {
-    return true
-  }
-
-  /* istanbul ignore next - see test/lib/load-all-commands.js */
-  static get params () {
-    return [
-      'workspace',
-      'workspaces',
-      'include-workspace-root',
-    ]
-  }
-
-  execWorkspaces (args, filters, cb) {
-    this.setWorkspaces(filters, true)
-      .then(() => {
-        this.exec(args, cb)
-      })
-      .catch(er => cb(er))
-  }
-}
-
-module.exports = ArboristCmd