chore: replace 'npm install' with 'npm ci --omit=dev' for less update issues and improved performance

Author: KristjanESPERANTO

CONTRIBUTING.md

@@ -13,6 +13,7 @@ Please note that this project is released with a [Contributor Code of Conduct](C
 
 ## Developer commands
 
+- `npm install` - Install all dependencies.
 - `npm run download_modules` - Download module list manually.
 - `npm run lint` - Run linting and formatter checks.
 - `npm run lint:fix` - Fix linting and formatter issues.

README.md

@@ -29,10 +29,10 @@ bash -c "$(curl -s https://raw.githubusercontent.com/Jopyth/MMM-Remote-Control/m
 - (1) Clone this repository in your `modules` folder, and install dependencies:
 
 ```bash
-cd ~/MagicMirror/modules # adapt directory if you are using a different one
+cd ~/MagicMirror/modules
 git clone https://github.com/Jopyth/MMM-Remote-Control
 cd MMM-Remote-Control
-npm install
+npm ci --omit=dev
 ```
 
 - (2) Add the module to your `config/config.js` file, if you add a `position`, it will display the URL to the remote on the mirror.
@@ -82,9 +82,9 @@ Note: If your user does not have `sudo` rights, the shutdown does not work (it _
 Update this module by navigating into its folder on the command line and using `git pull`:
 
 ```bash
-cd ~/MagicMirror/modules/MMM-Remote-Control # adapt directory if you are using a different one
+cd ~/MagicMirror/modules/MMM-Remote-Control
 git pull
-npm install # install (new) dependencies
+npm ci --omit=dev
 ```
 
 Alternatively you can run the `installer.sh` script again:

installer.sh

@@ -89,7 +89,7 @@ if [ -d "$MM_HOME/modules/$MODULE_NAME" ] ; then
         echo ""
         echo "Checking for new dependencies to install..."
         echo ""
-        npm install
+        npm ci --omit=dev
         if [ $? -ne 0 ]; then
             echo "Failed to install new dependencies."
             exit 1;
@@ -131,7 +131,7 @@ else
             echo ""
             echo "Installing dependencies..."
             echo ""
-            npm install
+            npm ci --omit=dev
             if [ $? -ne 0 ]; then
                 echo "Failed to install dependencies."
                 exit 1;

node_helper.js

@@ -940,7 +940,7 @@ module.exports = NodeHelper.create({
         self.sendResponse(res, error);
       } else {
         const workDir = path.resolve(`${__dirname}/../${path.basename(url)}`);
-        exec("npm install", {cwd: workDir, timeout: 120000}, (error, stdout, stderr) => {
+        exec("npm ci --omit=dev", {cwd: workDir, timeout: 120000}, (error, stdout, stderr) => {
           if (error) {
             Log.error("[MMM-Remote-Control]", error);
             self.sendResponse(res, error, {stdout, stderr, ...data});
@@ -986,7 +986,7 @@ module.exports = NodeHelper.create({
           return;
         }
         if (result.summary.changes) {
-          exec("npm install", {cwd: path, timeout: 120000}, (error, stdout, stderr) => {
+          exec("npm ci --omit=dev", {cwd: path, timeout: 120000}, (error, stdout, stderr) => {
             if (error) {
               Log.error("[MMM-Remote-Control]", error);
               self.sendResponse(res, error, {stdout, stderr});