feat(save): remove all save-related functionality (#19)

Fixes: #17

BREAKING CHANGE: npx can no longer be used to save packages locally or globally. Use an actual package manager for that, instead.

Author: zkat

README.md

@@ -4,7 +4,7 @@
 
 ## SYNOPSIS
 
-`npx [--package|-p <package>] [--cache <path>] [--save-dev|-D] [--save-prod|-P] [--save-optional|-O] [--save-bundle|-B] [--save-exact|-E] [--global|-g] [--prefix|-C] [--userconfig <path>] [-c <string>] [--ignore-existing] [--version|-v] [--] <command>[@version] [command-arg]...`
+`npx [--package|-p <package>] [--cache <path>] [--userconfig <path>] [-c <string>] [--shell|-s <string>] [--shell-auto-fallback [shell]] [--ignore-existing] [--version|-v] [--] <command>[@version] [command-arg]...`
 
 ## INSTALL
 
@@ -26,10 +26,6 @@ If a version specifier is included, or if `--package` is used, npx will ignore t
 
 * `-g, --global` - install the package globally before execution.
 
-* `-D, --save-dev, -P, --save-prod, -O, --save-optional, -B, --save-bundle, -E, --save-exact` - install the package in the current npm project and save it to `package.json` following the same option conventions for this as `npm install` would.
-
-* `-C, --prefix` - The location to install global items. If used without `-g`, will force any installs to run in the specified folder. Defaults to whatever npm's default is.
-
 * `--userconfig` - path to the user configuration file to pass to npm. Defaults to whatever npm's current default is.
 
 * `-c <string>` - Execute `<string>` inside a shell. For unix, this will be `/bin/sh -c <string>`. For Windows, it will be `cmd.exe /d /s /c <string>`. Only the first item in `<string>` will be automatically used as `<command>`. Any others _must_ use `-p`.
@@ -60,14 +56,6 @@ $ cat package.json
 ...webpack not in "devDependencies"...
 ```
 
-### Execute binary and add it to package.json as a devDependency
-
-```
-$ npx -D webpack -- ...
-$ cat package.json
-...webpack added to "devDependencies"
-```
-
 ### Execute a full shell command using one npx call
 
 ```

index.js

@@ -71,18 +71,7 @@ function getCmdPath (command, spec, npmOpts) {
 }
 
 function getExistingPath (command, opts) {
-  if (
-    opts.saveProd ||
-    opts.saveDev ||
-    opts.saveOptional ||
-    opts.saveBundle ||
-    opts.saveExact ||
-    opts.global ||
-    opts.prefix ||
-    opts.cmdHadVersion ||
-    opts.packageRequested ||
-    opts.ignoreExisting
-  ) {
+  if (opts.cmdHadVersion || opts.packageRequested || opts.ignoreExisting) {
     return BB.resolve(false)
   } else {
     return which(command).catch({code: 'ENOENT'}, () => false)
@@ -103,17 +92,7 @@ function getNpmCache (opts) {
 
 function buildArgs (spec, prefix, opts) {
   const args = ['install', spec]
-  if (opts.saveProd) args.push('--save', '--save-prod')
-  if (opts.saveDev) args.push('--save-dev')
-  if (opts.saveOptional) args.push('--save-optional')
-  if (opts.saveBundle) args.push('--save-bundle')
-  if (opts.saveExact) args.push('--save-exact')
-  if (opts.global) args.push('--global')
-  if (opts.prefix) args.push('--prefix', opts.prefix)
-  if (args.length === 2) {
-    // No save opts passed in. Save it to the SUPERSEKRIT cache
-    args.push('--global', '--prefix', prefix)
-  }
+  args.push('--global', '--prefix', prefix)
   if (opts.cache) args.push('--cache', opts.cache)
   if (opts.userconfig) args.push('--userconfig', opts.userconfig)
   args.push('--loglevel', 'error')

parse-args.js

@@ -3,7 +3,7 @@
 const npa = require('npm-package-arg')
 const yargs = require('yargs')
 
-const usage = `$0 [--package|-p <package>] [--cache <path>] [--save-dev|-D] [--save-prod|-P] [--save-optional|-O] [--save-bundle|-B] [--save-exact|-E] [--global|-g] [--prefix|-C] [--userconfig <path>] [-c <string>] [--shell-auto-fallback [shell]] [--ignore-existing] [--version|-v] [--] <command>[@version] [command-arg]...`
+const usage = `$0 [--package|-p <package>] [--cache <path>] [--userconfig <path>] [-c <string>] [--shell <string>] [--shell-auto-fallback [shell]] [--ignore-existing] [--version|-v] [--] <command>[@version] [command-arg]...`
 
 module.exports = parseArgs
 function parseArgs () {
@@ -18,41 +18,6 @@ function parseArgs () {
     type: 'string',
     describe: 'location of the npm cache'
   })
-  .option('save-prod', {
-    alias: 'P',
-    type: 'boolean',
-    describe: 'add to project\'s dependencies'
-  })
-  .option('save-dev', {
-    alias: 'D',
-    type: 'boolean',
-    describe: 'add to project\'s devDependencies'
-  })
-  .option('save-optional', {
-    alias: 'O',
-    type: 'boolean',
-    describe: 'add to project\'s optionalDependencies'
-  })
-  .option('save-bundle', {
-    alias: 'B',
-    type: 'boolean',
-    describe: 'add to project\'s bundleDependencies'
-  })
-  .option('save-exact', {
-    alias: 'E',
-    type: 'boolean',
-    describe: 'save the exact specifier instead of a semver range'
-  })
-  .option('global', {
-    alias: 'g',
-    type: 'boolean',
-    describe: 'install the package globally'
-  })
-  .option('prefix', {
-    alias: 'C',
-    type: 'boolean',
-    describe: 'location to install global items, or where to run the install if not used with -g'
-  })
   .option('userconfig', {
     type: 'string',
     describe: 'path to user npmrc'