update to 2.0.0

- shows install progress
- option to prefer npm, else defaults to yarn

Author: Trevor Brindle

README.md

@@ -15,21 +15,21 @@ Worse, sometimes you have to build a native binding like node-sass or couchbase,
 - battery
 - patience
 
-With install subsets, you can cleverly avoid some devDependencies in certain contexts. For example, on a CI server where you are simply building an artifact, you do not need to install dependencies like eslint, karma, or other testing frameworks. With a busy or sharedserver, you can shave precious time off of each build this way. 
+You can cleverly avoid some devDependencies in certain contexts. For example, on a CI server where you are simply building an artifact, you do not need to install dependencies like eslint, karma, or other testing frameworks. With a busy or shared server, you can shave precious time off of each build this way. 
 
 ## Installation
 
-    `npm install -g install-subset`
+`npm install -g install-subset`
 
 ## Usage
 
 Add something like to your package.json:
 ```
-"installSubsets": {
+"subsets": {
 	"build": {
 		"whitelist": [
-			"dotenv",
-			"prettier"
+			"babel-cli",
+            "dotenv"
 		]
 	},
 	"test": {
@@ -42,15 +42,14 @@ Add something like to your package.json:
 }
 ```
 
-then call:
-`install-subset install build`
+Then call `subset install test`
 
 ## Info
 
 - 'build' and 'test' are subset names
 - 'whitelist' and 'blacklist' are methods of install
-- 'whitelist' allows that subset of your devDependencies to install
-- 'blacklist' disallows that subset of your devDependencies to install
+    - 'whitelist' allows that subset of your devDependencies to install
+    - 'blacklist' disallows that subset of your devDependencies to install
 - package.json will be temporarily changed, and restored after install
 
 

index.js

@@ -3,53 +3,59 @@
 
 var fs = require("fs");
 var _ = require("lodash");
+var shelljs = require("shelljs");
 var cli = require("commander");
-var originalPackage = require(process.cwd() + "/package.json");
-var tempPackage = _.clone(originalPackage);
-var exec = require("child_process").exec;
+var cwd = process.cwd();
+var packageJson = require(cwd + "/package.json");
+var tempPackage = _.clone(packageJson);
+var spawn = require("child_process").spawn;
 
-var run = function(cmd, cb) {
-  var child = exec(cmd, function(error, stdout, stderr) {
-    if (stderr !== null) {
-      console.log("" + stderr);
-    }
-    if (stdout !== null) {
-      console.log("" + stdout);
-    }
-    if (error !== null) {
-      console.log("" + error);
-    }
-    if (cb) {
-      cb();
-    }
-  });
-};
-
-// only works for dev dependencies
 cli
   .command("install [input_string]")
-  .description("Check for word or phrase in translation memory")
-  .action(function(input_string) {
+  .alias("i")
+  .option("-c, --clean", "remove node_modules first")
+  .option("--npm", "use npm to install")
+  .description("install a given subset defined in package.json")
+  .action(function(input_string, options) {
     if (input_string) {
-      if (tempPackage.installSubsets) {
-        if (tempPackage.installSubsets[input_string]) {
-          const subset = tempPackage.installSubsets[input_string];
+      if (packageJson.subsets) {
+        if (packageJson.subsets[input_string]) {
+          const subset = packageJson.subsets[input_string];
           let devDependencies;
 
           if (subset.whitelist) {
-            devDependencies = _.pick(tempPackage.devDependencies, subset.whitelist);
+            devDependencies = _.pick(packageJson.devDependencies, subset.whitelist);
           } else if (subset.blacklist) {
-            devDependencies = _.omit(tempPackage.devDependencies, subset.blacklist);
+            devDependencies = _.omit(packageJson.devDependencies, subset.blacklist);
           } else {
             throw "No valid subset actions found";
           }
 
           tempPackage.devDependencies = devDependencies;
 
-          fs.writeFile(process.cwd() + "/package.json", JSON.stringify(tempPackage, null, "  "), function(err) {
+          if (options.clean) {
+            shelljs.rm("-rf", cwd + "/yarn.lock");
+            shelljs.rm("-rf", cwd + "/package-lock.json ");
+            shelljs.rm("-rf", cwd + "/node_modules");
+          }
+
+          fs.writeFile(cwd + "/package.json", JSON.stringify(tempPackage, null, "  "), function(err) {
             if (err) throw err;
-            run("type yarn 2>/dev/null && yarn || npm set depth=0; npm install --silent", function() {
-              fs.writeFile(process.cwd() + "/package.json", JSON.stringify(originalPackage, null, "  "), function(err) {
+
+            if (!options.npm && shelljs.which('yarn')){
+              var installer = spawn("yarn", [], { stdio: "inherit" });
+            } else {
+              var installer = spawn("npm", ['install'], { stdio: "inherit" });
+            }
+
+            installer.on("close", function(code) {
+              if (code !== 0) {
+                throw "Error code " + code;
+              } else {
+                installer.unref();
+                console.log("Installation of subset " + input_string + " successful");
+              }
+              fs.writeFile(cwd + "/package.json", JSON.stringify(packageJson, null, "  "), function(err) {
                 if (err) throw err;
               });
             });
@@ -71,11 +77,10 @@ cli.parse(process.argv);
 
 if (cli.args.length === 0) cli.help();
 
-process.on("unhandledRejection", (reason, promise) => {
-  console.log("Unhandled Rejection at:", promise, "reason:", reason);
-});
-
 process.on("uncaughtException", err => {
   console.log("ERROR: " + err);
+  fs.writeFile(cwd + "/package.json", JSON.stringify(packageJson, null, "  "), function(err) {
+    if (err) throw err;
+  });
   process.exit();
 });

package.json

@@ -1,40 +1,28 @@
 {
   "name": "install-subset",
-  "version": "1.0.0",
+  "version": "2.0.0",
   "description": "Install a subset of npm dependencies based on given contexts",
   "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   },
-  "bin": "index.js",
+  "bin": {
+    "subset": "index.js"
+  },
   "keywords": [
     "npm",
     "install",
     "subset",
     "dependency"
   ],
   "author": "[email protected]",
-  "license": "ISC",
-  "babel": {
-    "presets": [
-      [
-        "env",
-        {
-          "targets": {
-            "node": 4.3
-          }
-        }
-      ]
-    ],
-    "plugins": [
-      "transform-runtime"
-    ]
-  },
+  "license": "MIT",
   "devDependencies": {
     "prettier": "^1.3.1"
   },
   "dependencies": {
     "commander": "^2.9.0",
-    "lodash": "^4.17.4"
+    "lodash": "^4.17.4",
+    "shelljs": "^0.7.7"
   }
 }