reducing the size of dependencies

chalk -> picocolors cosmiconfig -> lilconfig markdown-it -> marked parse-json -> secure-json-parse
hcodes · Oct 26, 2021 · 7839f1a · 7839f1a
1 parent 48cd6a2
commit 7839f1a
Show file tree

Hide file tree

Showing 10 changed files with 145 additions and 286 deletions.
diff --git a/README.md b/README.md
@@ -131,7 +131,7 @@ Ignore Internet addresses, email addresses and filenames.
 Max count of requests in parallel.<br/>
 Default: `2`.
 
-#### `--no-colors`
+#### `--no-color`
 Clean output without colors.
 
 #### `--only-errors`

diff --git a/README.ru.md b/README.ru.md
@@ -123,7 +123,7 @@ JSON-файл собственного словаря.
 Одновременное количество запросов к API Yandex.Speller.<br/>
 По умолчанию: `2`.
 
-#### `--no-colors`
+#### `--no-color`
 Консольный вывод без цвета.
 
 #### `--only-errors`

diff --git a/lib/cli/index.js b/lib/cli/index.js
@@ -1,7 +1,6 @@
 'use strict';
 
 const async = require('async');
-const chalk = require('chalk');
 const program = require('commander');
 
 const { cliActionInit } = require('./actions');
@@ -17,9 +16,6 @@ setCliOptions(defaultConfig);
 program.parse(process.argv);
 
 setDebugMode(program.debug);
-if (!program.colors) {
-    chalk.level = 0;
-}
 
 const mergedOptions = getMergedOptions(program.config);
 

diff --git a/lib/cli/options.js b/lib/cli/options.js
@@ -30,7 +30,7 @@ function setCliOptions(defaultConfig) {
         .option('-e, --file-extensions <value>', 'set file extensions to search for files in a folder. Example: ".md,.html"', splitByCommas)
         .option('--init', 'save default config ".yaspellerrc" in current directory')
         .option('--dictionary <file>', 'json file for own dictionary', value => splitTrim(value, ':'))
-        .option('--no-colors', 'clean output without colors')
+        .option('--no-color', 'clean output without colors')
         .option('--report <type>', 'generate a report: console, text, html or json. Default: "console"', splitByCommas)
         .option('--ignore-tags <tags>', `ignore tags. Default: "${defaultConfig.ignoreTags.join(',')}"`, splitByCommas)
         .option('--ignore-text <regexp>', 'ignore text using RegExp')

diff --git a/lib/config/index.js b/lib/config/index.js
@@ -1,8 +1,8 @@
 'use strict';
 
-const cosmiconfig = require('cosmiconfig').cosmiconfigSync;
+const lilconfig = require('lilconfig').lilconfigSync;
 const stripJsonComments = require('strip-json-comments');
-const parseJson = require('parse-json');
+const sjson = require('secure-json-parse');
 const path = require('path');
 
 const exitCodes = require('../exit-codes');
@@ -12,7 +12,7 @@ const defaultConfig = require('../../.yaspellerrc.default.json');
 
 function loadJson(filepath, content) {
     try {
-        return parseJson(stripJsonComments(content));
+        return sjson.parse(stripJsonComments(content));
     } catch (err) {
         err.message = `JSON Error in ${filepath}:\n${err.message}`;
         throw err;
@@ -30,7 +30,7 @@ function getType(value) {
  * @returns {Object}
  */
 function getConfig(file) {
-    const explorer = cosmiconfig('yaspeller', {
+    const explorer = lilconfig('yaspeller', {
         loaders: {
             '.json': loadJson,
             noExt: loadJson

diff --git a/lib/helpers/console.js b/lib/helpers/console.js
@@ -1,5 +1,5 @@
 
-const chalk = require('chalk');
+const pico = require('picocolors');
 
 const { jsonStringify } = require('./string');
 const { isDebugMode } = require('./debug');
@@ -9,19 +9,19 @@ function consoleLog(data) {
 }
 
 function consoleInfo(data) {
-    console.info(chalk.cyan(data));
+    console.info(pico.cyan(data));
 }
 
 function consoleWarn(data) {
-    console.warn(chalk.yellow(data));
+    console.warn(pico.yellow(data));
 }
 
 function consoleError(data) {
-    console.error(chalk.red(data));
+    console.error(pico.red(data));
 }
 
 function consoleOk(data) {
-    console.log(chalk.green(data));
+    console.log(pico.green(data));
 }
 
 /**
@@ -31,12 +31,12 @@ function consoleOk(data) {
  */
 function consoleDebug(text) {
     if (isDebugMode()) {
-        const prefix = chalk.cyan('[DEBUG]');
+        const prefix = pico.cyan('[DEBUG]');
         if (typeof text === 'object' && text) {
             console.log(prefix);
 
             Object.keys(text).forEach(key => {
-                console.log(chalk.cyan('    ' + key + ': ') + jsonStringify(text[key]));
+                console.log(pico.cyan('    ' + key + ': ') + jsonStringify(text[key]));
             });
         } else {
             console.log(prefix + ' ' + text);

diff --git a/lib/reports/console.js b/lib/reports/console.js
@@ -1,6 +1,6 @@
 'use strict';
 
-const chalk = require('chalk');
+const pico = require('picocolors');
 const program = require('commander');
 
 const yaspeller = require('../yaspeller');
@@ -36,20 +36,20 @@ function getTyposByCode(code, data) {
         const pos = el.position;
 
         if (pos.length) {
-            comment.push(chalk.cyan(pos[0].line + ':' + pos[0].column));
+            comment.push(pico.cyan(pos[0].line + ':' + pos[0].column));
         }
 
         if (el.count > 1) {
-            comment.push(chalk.cyan('count: ' + el.count));
+            comment.push(pico.cyan('count: ' + el.count));
         }
 
         if (hasEngRusLetters(el.word)) {
-            comment.push(chalk.red('en: ' + replaceRusLettersWithAsterisk(el.word)));
-            comment.push(chalk.green('ru: ' + replaceEngLettersWithAsterisk(el.word)));
+            comment.push(pico.red('en: ' + replaceRusLettersWithAsterisk(el.word)));
+            comment.push(pico.green('ru: ' + replaceEngLettersWithAsterisk(el.word)));
         }
 
         if (el.suggest) {
-            comment.push(chalk.cyan('suggest: ' + el.suggest.join(', ')));
+            comment.push(pico.cyan('suggest: ' + el.suggest.join(', ')));
         }
 
         typos.push(num + '. ' + el.word + (comment.length ? ' (' + comment.join(', ') + ')' : ''));
@@ -70,32 +70,32 @@ module.exports = {
             consoleError(data);
         } else {
             if (hasManyErrors(data.data)) {
-                errors.push(chalk.red('Too many errors\n'));
+                errors.push(pico.red('Too many errors\n'));
             }
 
             yaspeller.errors.forEach(function(el) {
                 const typos = getTyposByCode(el.code, data.data);
                 if (typos.length) {
-                    errors.push(chalk.red(el.title + ': ' +
+                    errors.push(pico.red(el.title + ': ' +
                         typos.length + '\n') +
                         typos.join('\n') + '\n');
                 }
             });
 
-            const time = data.time ? ' ' + chalk.magenta(data.time + ' ms') : '';
-            const separator = chalk.red('-----\n');
+            const time = data.time ? ' ' + pico.magenta(data.time + ' ms') : '';
+            const separator = pico.red('-----\n');
 
             let res = data.resource;
 
             if (isUrl(res)) {
-                res = chalk.underline(res);
+                res = pico.underline(res);
             }
 
             if (errors.length) {
-                console.error(chalk.red(errorSymbol) +  ' ' + res + time + '\n' +
+                console.error(pico.red(errorSymbol) +  ' ' + res + time + '\n' +
                     separator + errors.join('\n') + separator);
             } else {
-                console.log(chalk.green(okSymbol) + ' ' + res + time);
+                console.log(pico.green(okSymbol) + ' ' + res + time);
             }
         }
     },

diff --git a/lib/yaspeller.js b/lib/yaspeller.js
@@ -8,8 +8,7 @@ const fetch = require('node-fetch');
 const pth = require('path');
 const xml2js = require('xml2js');
 const yaspellerApi = require('yandex-speller');
-const MarkdownIt = require('markdown-it');
-const md = new MarkdownIt({ html: true });
+const marked = require('marked');
 
 const eyo = require('./plugins/eyo');
 const { getFormat, getApiFormat } = require('./helpers/format');
@@ -63,7 +62,7 @@ function checkText(originalText, callback, settings) {
 
     if (format === 'html' || format === 'markdown') {
         if (format === 'markdown') {
-            text = md.render(text);
+            text = marked(text);
         }
 
         if (apiSettings.ignoreTags) {