Trying to support non cli

.editorconfig

@@ -0,0 +1,5 @@
+[*]
+indent_size = 2
+indent_style = space
+tab_width = 2
+insert_final_newline = true

.eslintrc.js

@@ -0,0 +1,24 @@
+module.exports = {
+  extends: ['eslint:recommended', 'prettier'], // extending recommended config and config derived from eslint-config-prettier
+  plugins: ['prettier'], // extending recommended config and config derived from eslint-config-prettier
+  env: {
+    browser: true,
+    node: true,
+    amd: true,
+    es6: true,
+    commonjs: true,
+    jest: true,
+  },
+  parser: 'babel-eslint',
+  rules: {
+    'prettier/prettier': [
+      // customizing prettier rules (unfortunately not many of them are customizable)
+      'error',
+      {
+        singleQuote: true,
+        tabWidth: 2,
+      },
+    ],
+    eqeqeq: ['error', 'always'], // adding some custom ESLint rules
+  },
+};

.npmignore

@@ -3,3 +3,6 @@ ISSUE_TEMPLATE.md
 PULL_REQUEST_TEMPLATE.md
 yarn.lock
 package-lock.json
+.editorconfig
+.eslintrc.js
+tests

PULL_REQUEST_TEMPLATE.md

@@ -5,3 +5,8 @@ Fixes #
   -
   -
   -
+
+## Checklist
+make sure you have run the following before requesting a review
+* npm run lint
+* npm run typescript

README.md

@@ -12,7 +12,7 @@ $ npm install add-dependencies [-g]
 
 ### Usage
 
-Run:
+Run (for `nodejs` see Example):
 
 ```sh
 $ add-dependencies [package_file] <dependencies> [target] [--no-overwrite]
@@ -35,7 +35,7 @@ If no `package_file` argument passed, the script searches for a `package.json` f
 
 Use `--no-overwrite` flag to prevent already existing packages in `package.json` from being overwritten.
 
-Example:
+Example: 
 
 ```sh
 $ add-dependencies /home/user/project/package.json [email protected] [email protected] redux eslint --dev
@@ -46,3 +46,24 @@ or with `npx`:
 ```sh
 $ npx add-dependencies /home/user/project/package.json [email protected] [email protected] redux eslint --dev
 ```
+
+or via nodejs
+
+```js
+const npmAdd = require('add-dependencies');
+//OR
+// Note: to use import you must have `"type": "module"` in your projects package.json
+import npmAdd from 'add-dependencies';
+
+const dependencies = [
+    'package1',
+    'package2',
+    'package3',
+];
+const target = npmAdd.CONSTANTS.DEPENDENCIES;
+const overwrite = false;
+const packageFilePath = 'package.json';
+new npmAdd(dependencies, target, overwrite, packageFilePath)
+    .run()
+    .then(() => console.log('completed'));
+```

cli-index.js

@@ -0,0 +1,15 @@
+#!/usr/bin/env node
+
+const AddDependencies = require('./lib/AddDependencies');
+
+console.log(
+  '\x1b[33m%s\x1b[0m',
+  'This script adds dependencies (latest or specified versions) to the package.json file skipping the installation process.'
+);
+
+const app = new AddDependencies();
+
+app.run().catch((error) => {
+  console.error('\x1b[31m%s\x1b[0m', error);
+  process.exit(1);
+});

index.js

@@ -1,11 +1,3 @@
 #!/usr/bin/env node
-const AddDependencies = require('./lib/AddDependencies');
 
-console.log('\x1b[33m%s\x1b[0m', 'This script adds dependencies (latest or specified versions) to the package.json file skipping the installation process.');
-
-const app = new AddDependencies();
-
-app.addDependencies().then(app.saveToPackage.bind(app)).catch((error) => {
-  console.error('\x1b[31m%s\x1b[0m', error);
-  process.exit(1);
-});
+module.exports = require('./lib/AddDependencies');

index.mjs

@@ -0,0 +1,5 @@
+#!/usr/bin/env node
+
+import AddDependencies from './lib/AddDependencies.js';
+
+export default AddDependencies;

lib/AddDependencies.js

@@ -2,13 +2,18 @@ const npmRun = require('npm-run');
 const semver = require('semver');
 const Files = require('./Files');
 
-class AddDependencies {
-  constructor() {
+module.exports = class AddDependencies {
+  constructor(
+    dependencies = [],
+    target = 'dependencies',
+    overwrite = true,
+    packageFilePath = './package.json'
+  ) {
     this.result = {};
-    this.dependencies = [];
-    this.target = 'dependencies';
-    this.overwrite = true;
-    this.packageFilePath = './package.json';
+    this.dependencies = dependencies;
+    this.target = target;
+    this.overwrite = overwrite;
+    this.packageFilePath = packageFilePath;
   }
 
   addDependencies() {
@@ -38,7 +43,7 @@ class AddDependencies {
             break;
           }
           default: {
-            if (/package\.json/.test(val)) {
+            if (val.includes('package.json')) {
               this.packageFilePath = val;
             } else if (!/^-/.test(val)) {
               this.dependencies.push(val);
@@ -59,9 +64,21 @@ class AddDependencies {
     return Promise.all(this.dependencies.map((dep) => this.runNpmShow(dep)));
   }
 
+  run() {
+    return this.addDependencies().then(() => this.saveToPackage());
+  }
+
   runNpmShow(dep) {
+    if (!dep) {
+      console.error(
+        '\x1b[31m%s\x1b[0m',
+        `No Dependency Provided to runNpmShow()`
+      );
+      return new Promise((resolve) => resolve());
+    }
     const depSplit = dep.split('@');
-    const [depName, depVersion] = dep.charAt(0) !== '@' ? depSplit : [`@${depSplit[1]}`, depSplit[2]];
+    const [depName, depVersion] =
+    dep.charAt(0) !== '@' ? depSplit : [`@${depSplit[1]}`, depSplit[2]];
 
     if (depVersion) {
       const depRange = semver.validRange(depVersion) || '';
@@ -80,22 +97,37 @@ class AddDependencies {
                     this.result[depName] = `${depVersion}`;
                     break;
                   }
-                } else if (operators.length > 1 && specifiedVersions.length > 1) {
-                  if (semver.cmp(version, operators[0], specifiedVersions[0]) && semver.cmp(version, operators[1], specifiedVersions[1])) {
+                } else if (
+                  operators.length > 1 &&
+                  specifiedVersions.length > 1
+                ) {
+                  if (
+                    semver.cmp(version, operators[0], specifiedVersions[0]) &&
+                    semver.cmp(version, operators[1], specifiedVersions[1])
+                  ) {
                     this.result[depName] = `${depVersion}`;
                     break;
                   }
                 }
               }
+              // eslint-disable-next-line
             } catch (e) {}
 
             if (undefined === this.result[depName]) {
-              console.error('\x1b[31m%s\x1b[0m', `Could not obtain the specified version for: ${depName}. Skip.`);
+              console.error(
+                '\x1b[31m%s\x1b[0m',
+                `Could not obtain the specified version for: ${depName}. Skip.`
+              );
             } else {
-              console.log(`Processed: ${depName}, specified version: ${depVersion}`);
+              console.log(
+                `Processed: ${depName}, specified version: ${depVersion}`
+              );
             }
           } else {
-            console.error('\x1b[31m%s\x1b[0m', `Could not fetch version info for: ${depName}. Skip.`);
+            console.error(
+              '\x1b[31m%s\x1b[0m',
+              `Could not fetch version info for: ${depName}. Skip.`
+            );
           }
 
           return resolve();
@@ -109,14 +141,20 @@ class AddDependencies {
           const parsed = stdout.match(/latest: '(.*?)'/i);
 
           if (!parsed || undefined === parsed[1]) {
-            console.error('\x1b[31m%s\x1b[0m', `Could not obtain the latest version for: ${depName}. Skip.`);
+            console.error(
+              '\x1b[31m%s\x1b[0m',
+              `Could not obtain the latest version for: ${depName}. Skip.`
+            );
           } else {
             this.result[depName] = `^${parsed[1]}`;
 
             console.log(`Processed: ${depName}, latest version: ${parsed[1]}`);
           }
         } else {
-          console.error('\x1b[31m%s\x1b[0m', `Could not fetch version info for: ${depName}. Skip.`);
+          console.error(
+            '\x1b[31m%s\x1b[0m',
+            `Could not fetch version info for: ${depName}. Skip.`
+          );
         }
 
         return resolve();
@@ -125,39 +163,64 @@ class AddDependencies {
   }
 
   saveToPackage() {
-    Files.readFromFile(this.packageFilePath).then(async (data) => {
-      let json;
-
-      try {
-        json = JSON.parse(data);
-      } catch (e) {
-        console.error('\x1b[31m%s\x1b[0m', `Could not parse ${this.packageFilePath}. Stop.`);
-        process.exit(1);
-      }
-
-      this.result = this.overwrite
-        ? Object.assign(json[this.target] || {}, this.result)
-        : Object.assign(this.result, json[this.target] || {});
-
-      this.result = Object.keys(this.result).sort().reduce((res, key) => {
-        res[key] = this.result[key];
-
-        return res;
-      }, {});
-
-      json[this.target] = this.result;
+    return Files.readFromFile(this.packageFilePath)
+      .then(async (data) => {
+        let json;
+
+        try {
+          json = JSON.parse(data);
+        } catch (e) {
+          console.error(
+            '\x1b[31m%s\x1b[0m',
+            `Could not parse ${this.packageFilePath}. Stop.`
+          );
+          process.exit(1);
+        }
 
-      Files.writeToFile(this.packageFilePath, JSON.stringify(json, null, 2)).then(() => {
-        console.log('\x1b[32m%s\x1b[0m', 'Done.');
-      }).catch(() => {
-        console.error('\x1b[31m%s\x1b[0m', `Could not write to ${this.packageFilePath}. Stop.`);
+        this.result = this.overwrite ?
+          Object.assign(json[this.target] || {}, this.result) :
+          Object.assign(this.result, json[this.target] || {});
+
+        this.result = Object.keys(this.result)
+          .sort()
+          .reduce((res, key) => {
+            res[key] = this.result[key];
+
+            return res;
+          }, {});
+
+        json[this.target] = this.result;
+
+        return Files.writeToFile(
+            this.packageFilePath,
+            JSON.stringify(json, null, 2)
+          )
+          .then(() => {
+            console.log('\x1b[32m%s\x1b[0m', 'Done.');
+          })
+          .catch(() => {
+            console.error(
+              '\x1b[31m%s\x1b[0m',
+              `Could not write to ${this.packageFilePath}. Stop.`
+            );
+            process.exit(1);
+          });
+      })
+      .catch(() => {
+        console.error(
+          '\x1b[31m%s\x1b[0m',
+          `Could not read from ${this.packageFilePath}. Stop.`
+        );
         process.exit(1);
       });
-    }).catch(() => {
-      console.error('\x1b[31m%s\x1b[0m', `Could not read from ${this.packageFilePath}. Stop.`);
-      process.exit(1);
-    });
   }
-}
 
-module.exports = AddDependencies;
+  static get CONSTANTS() {
+    return {
+      DEPENDENCIES: 'dependencies',
+      DEV_DEPENDENCIES: 'devDependencies',
+      PEER_DEPENDENCIES: 'peerDependencies',
+      OPTIONAL_DEPENDENCIES: 'optionalDependencies',
+    };
+  }
+};

lib/Files.js

@@ -5,7 +5,7 @@ class Files {
     return new Promise((resolve, reject) => {
       fs.readFile(filePath, (err, data) => {
         if (err) {
-          return reject();
+          return reject(err);
         }
 
         return resolve(data.toString());
@@ -17,7 +17,7 @@ class Files {
     return new Promise((resolve, reject) => {
       fs.writeFile(filePath, fileContent, (err) => {
         if (err) {
-          return reject();
+          return reject(err);
         }
 
         return resolve();

package.json

@@ -11,7 +11,9 @@
   },
   "homepage": "https://github.com/arfeo/npm-add-dependencies#readme",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "jest --config ./tests/jest.config.js",
+    "lint": "eslint --fix --ext js lib/**.js index.js cli-index.js",
+    "typescript": "tsc"
   },
   "keywords": [
     "dependencies",
@@ -23,10 +25,23 @@
   "license": "MIT",
   "preferGlobal": true,
   "bin": {
-    "add-dependencies": "index.js"
+    "add-dependencies": "cli-index.js"
   },
   "dependencies": {
     "npm-run": "^5.0.1",
     "semver": "^6.3.0"
+  },
+  "devDependencies": {
+    "blueimp-md5": "^2.16.0",
+    "mock-argv": "^1.1.7",
+    "jest": "^26.0.1",
+    "eslint": "^7.3.0",
+    "babel-eslint": "^10.1.0",
+    "typescript": "^4.0.5",
+    "eslint-config-prettier": "^3.3.0",
+    "eslint-plugin-prettier": "^3.0.0",
+    "prettier": "^2.0.5",
+    "concat-stream": "^2.0.0",
+    "child_process": "^1.0.2"
   }
-}
+}