@adamlui/minify.js 1.4.7 → 1.4.8

package/README.md

@@ -21,7 +21,7 @@
 
 <a href="https://www.npmjs.com/package/@adamlui/minify.js"><img height=31 src="https://img.shields.io/npm/dt/%40adamlui%2Fminify.js?logo=npm&color=af68ff&logoColor=white&labelColor=464646&style=for-the-badge"></a>
 <a href="#%EF%B8%8F-mit-license"><img height=31 src="https://img.shields.io/badge/License-MIT-orange.svg?logo=internetarchive&logoColor=white&labelColor=464646&style=for-the-badge"></a>
-<a href="https://github.com/adamlui/js-utils/releases/tag/minify.js-1.4.7"><img height=31 src="https://img.shields.io/badge/Latest_Build-1.4.7-44cc11.svg?logo=icinga&logoColor=white&labelColor=464646&style=for-the-badge"></a>
+<a href="https://github.com/adamlui/js-utils/releases/tag/minify.js-1.4.8"><img height=31 src="https://img.shields.io/badge/Latest_Build-1.4.8-44cc11.svg?logo=icinga&logoColor=white&labelColor=464646&style=for-the-badge"></a>
 <a href="https://www.npmjs.com/package/@adamlui/minify.js?activeTab=code"><img height=31 src="https://img.shields.io/npm/unpacked-size/%40adamlui%2Fminify.js?style=for-the-badge&logo=ebox&logoColor=white&labelColor=464646&color=blue"></a>
 <a href="https://sonarcloud.io/component_measures?metric=new_vulnerabilities&id=adamlui_js-utils:minify.js/minify.js"><img height=31 src="https://img.shields.io/badge/dynamic/json?url=https%3A%2F%2Fsonarcloud.io%2Fapi%2Fmeasures%2Fcomponent%3Fcomponent%3Dadamlui_js-utils%3Aminify.js%2Fminify.js%26metricKeys%3Dvulnerabilities&query=%24.component.measures.0.value&style=for-the-badge&logo=sonarcloud&logoColor=white&labelColor=464646&label=Vulnerabilities&color=gold"></a>
 
@@ -170,7 +170,7 @@ const srcCode = 'function add(first, second) { return first + second; }',
       minifyResult = minifyJS.minify(srcCode);
 
 console.log(minifyResult.error); // outputs runtime error, or `undefined` if no error
-console.log(minifyResult.code);  // outputs minified JS: function add(n,d){return n+d}
+console.log(minifyResult.code);  // outputs minified JS: 'function add(n,d){return n+d}'
 ```
 
 If a **file path** is passed, the file's code is loaded then minified, returning an object like above.
@@ -230,12 +230,12 @@ findJS() » Check returned array.
 
 Available parameters (and their default settings) are:
 
-Name         | Desciption                                              | Default value
--------------|---------------------------------------------------------|---------------
-`recursive`  | Recursively search for nested files if dir path passed. | `true`
-`verbose`    | Show logging in console/terminal.                       | `true`
-`dotFolders` | Include dotfolders in file search.                      | `false`
-`dotFiles`   | Include dotfiles in file search.                        | `false`
+Name         | Desciption                                                | Default value
+-------------|-----------------------------------------------------------|---------------
+`recursive`  | Recursively search for nested files in searchDir passed.  | `true`
+`verbose`    | Show logging in console/terminal.                         | `true`
+`dotFolders` | Include dotfolders in file search.                        | `false`
+`dotFiles`   | Include dotfiles in file search.                          | `false`
 
 <br>
 

package/minify.js

@@ -10,7 +10,12 @@ const fs = require('fs'),
 function findJS(searchDir, options = {}) {
 
     // Init options
-    const defaultOptions = { recursive: true, verbose: true, dotFolders: false, dotFiles: false };
+    const defaultOptions = {
+        recursive: true,   // recursively search for nested files in searchDir passed
+        verbose: true,     // enable logging
+        dotFolders: false, // include dotfolders in file search
+        dotFiles: false    // include dotfiles in file search
+    };
     options = { ...defaultOptions, ...options };
 
     // Validate searchDir
@@ -24,13 +29,13 @@ function findJS(searchDir, options = {}) {
     }
 
     // Validate options
-    for (const key of Object.keys(options)) {
+    for (const key in options) {
         if (!Object.prototype.hasOwnProperty.call(defaultOptions, key))
             if (key !== 'isRecursing') return console.error(
                 `findJS() » ERROR: \`${ key }\` is an invalid option.\n`
               + `findJS() » Valid options: [ ${ Object.keys(defaultOptions).join(', ') } ]`);
         else if (typeof options[key] !== 'boolean') return console.error(
-                `findJS() » ERROR: \`${ key }\` option can only be set to \`true\` or \`false\`.`);
+                `findJS() » ERROR: [${ key }] option can only be set to \`true\` or \`false\`.`);
     }
 
     // Search for unminified JS
@@ -45,15 +50,15 @@ function findJS(searchDir, options = {}) {
                     ...findJS(filePath, { ...options, isRecursing: true }));
         else if (/\.js(?<!\.min\.js)$/.test(file)
             && (options.dotFiles || !file.startsWith('.')))
-                jsFiles.push(filePath); // store eligible unminified JS file for minification
+                jsFiles.push(filePath); // store eligible unminified JS file for returning
     });
 
     // Log/return final result
     if (!options.isRecursing && options.verbose) console.info(
-        'findJS() » Search complete! ' + ( jsFiles.length === 0 ? 'No' : jsFiles.length )
-            + ` file${ jsFiles.length > 1 ? 's' : '' } found.`
+            'findJS() » Search complete! ' + ( jsFiles.length === 0 ? 'No' : jsFiles.length )
+                + ` file${ jsFiles.length > 1 ? 's' : '' } found.`
         + ( findJS.caller.name !== 'minify' && require.main !== module ?
-            '\nfindJS() » Check returned array.' : '' ));
+          '\nfindJS() » Check returned array.' : '' ));
     return options.isRecursing || jsFiles.length > 0 ? jsFiles : [];
 }
 
@@ -61,7 +66,12 @@ function minify(input, options = {}) {
 
     // Init options
     const defaultOptions = {
-        recursive: true, verbose: true, dotFolders: false, dotFiles: false, mangle: true };
+        recursive: true,   // recursively search for nested files if dir path passed
+        verbose: true,     // enable logging
+        dotFolders: false, // include dotfolders in file search
+        dotFiles: false,   // include dotfiles in file search
+        mangle: true       // shorten var names (typically to one character)
+    };
     options = { ...defaultOptions, ...options };
 
     // Validate input
@@ -69,12 +79,12 @@ function minify(input, options = {}) {
         'minify() » ERROR: 1st arg <input> must be a string.');
 
     // Validate options
-    for (const key of Object.keys(options)) {
+    for (const key in options) {
         if (!Object.prototype.hasOwnProperty.call(defaultOptions, key)) return console.error(
             `minify() » ERROR: \`${ key }\` is an invalid option.\n`
           + `minify() » Valid options: [ ${ Object.keys(defaultOptions).join(', ') } ]`);
         else if (typeof options[key] !== 'boolean') return console.error(
-            `minify() » ERROR: \`${ key }\` option can only be set to \`true\` or \`false\`.`);
+            `minify() » ERROR: [${ key }] option can only be set to \`true\` or \`false\`.`);
     }
 
     // Minify JS based on input
@@ -190,9 +200,10 @@ else { // run as CLI utility
             minifyData?.forEach(({ code, srcPath }) => {
                 const outputDir = path.join(
                     path.dirname(srcPath), // path of file to be minified
-                    /so?u?rce?$/.test(path.dirname(srcPath)) ? '../min' // + ../min/ if in *(src|source)/
-                        : outputArg.endsWith('.js') ? path.dirname(outputArg) // or path from file output arg
-                        : outputArg || 'min' // or path from folder output arg or min/ if no output arg passed
+                    /so?u?rce?$/.test(path.dirname(srcPath)) ? (
+                        '../' + ( outputArg || 'min' ) // + ../outputArg|min/ if in *(src|source)/
+                    ) : outputArg.endsWith('.js') ? path.dirname(outputArg) // or path from file output arg
+                      : outputArg || 'min' // or path from folder outputArg or min/ if no outputArg passed
                 );
                 const outputFilename = (
                     outputArg.endsWith('.js') && inputArg.endsWith('.js')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adamlui/minify.js",
-  "version": "1.4.7",
+  "version": "1.4.8",
   "description": "Recursively minify all JavaScript files",
   "author": {
     "name": "Adam Lui",
@@ -14,12 +14,17 @@
     "minifyjs": "minify.js",
     "minify-js": "minify.js"
   },
+  "directories": {
+    "lib": ".",
+    "doc": "./docs",
+    "test": "./utils/test"
+  },
   "scripts": {
     "test": "bash utils/test/minify-cli.test.sh",
     "bump:patch": "bash utils/bump.sh patch",
     "bump:minor": "bash utils/bump.sh minor",
     "bump:major": "bash utils/bump.sh major",
-    "prepublish": "npm test",
+    "prepublishOnly": "npm test",
     "publish:patch": "bash utils/bump.sh patch --publish",
     "publish:minor": "bash utils/bump.sh minor --publish",
     "publish:major": "bash utils/bump.sh major --publish"