@putout/engine-loader 11.4.0 → 11.6.0

package/lib/check/index.js

@@ -0,0 +1,13 @@
+'use strict';
+
+const isString = (a) => typeof a === 'string';
+
+module.exports.check = (options) => {
+    if (!options || typeof options !== 'object')
+        throw Error('options should be an object!');
+};
+
+module.exports.checkRule = (rule) => {
+    if (!isString(rule))
+        throw Error(`☝️ Looks like plugin name type is not 'string', but: '${typeof rule}'`);
+};

package/lib/index.js

@@ -2,18 +2,23 @@
 
 const {nanomemoize} = require('nano-memoize');
 
-const isEnabled = require('./is-enabled');
-const {loadPlugin} = require('./load');
-const {createAsyncLoader} = require('./async-loader');
-const {parsePluginNames} = require('./parse-plugin-names');
-const parseProcessorNames = require('./parse-processor-names');
-const parseRules = require('./parse-rules');
-const validateRules = require('./validate-rules');
-const validatePlugin = require('./validate-plugin');
-const {mergeRules} = require('./merge-rules');
-const {loadPluginsAsync} = require('./load-plugins-async');
+const {loadPlugin} = require('./load/load');
+const {createAsyncLoader} = require('./load/async-loader');
+const {parsePluginNames} = require('./plugins/parse-plugin-names');
+const parseProcessorNames = require('./processors/parse-processor-names');
+const validatePlugin = require('./plugins/validate-plugin');
+const {loadPluginsAsync} = require('./plugins/load-plugins-async');
 
-const isString = (a) => typeof a === 'string';
+const {
+    parseRules,
+    validateRules,
+    isEnabled,
+    getLoadedRules,
+} = require('./rules');
+
+const {filterEnabledPlugins} = require('./plugins/filter-enabled-plugins');
+
+const {check, checkRule} = require('./check');
 
 module.exports.loadPluginsAsync = loadPluginsAsync;
 module.exports.loadProcessorsAsync = nanomemoize(async (options, load) => {
@@ -59,41 +64,12 @@ module.exports.loadPlugins = (options) => {
         items,
     });
     
-    const result = [];
-    
-    // Would be great to have ability to filter
-    // disabled plugins and prevent them from loading
-    // but we can't because of a way multi-rule plugins
-    // works. We can't determine count and names of all
-    // rules of a plugin before load.
-    for (const [name, plugin] of plugins) {
-        if (!isEnabled(name, cookedRules))
-            continue;
-        
-        result.push(mergeRules(
-            [name, plugin],
-            cookedRules,
-        ));
-    }
-    
-    return result;
+    return filterEnabledPlugins({
+        plugins,
+        cookedRules,
+    });
 };
 
-function getLoadedRules(rules) {
-    const loadedRules = [];
-    
-    for (const item of rules) {
-        const {rule} = item;
-        
-        if (rule.includes('/'))
-            continue;
-        
-        loadedRules.push(item);
-    }
-    
-    return loadedRules;
-}
-
 function splitRule(rule) {
     return [rule, 'putout'];
 }
@@ -141,13 +117,3 @@ function extendRules(rule, plugin) {
     
     return result;
 }
-
-function check(options) {
-    if (!options || typeof options !== 'object')
-        throw Error('options should be an object!');
-}
-
-function checkRule(rule) {
-    if (!isString(rule))
-        throw Error(`☝️ Looks like plugin name type is not 'string', but: '${typeof rule}'`);
-}

package/lib/{load.js → load/load.js}

@@ -1,9 +1,11 @@
 'use strict';
 
+const process = require('process');
 const {createRequire} = require('module');
 
 const tryCatch = require('try-catch');
 const once = require('once');
+const {assign} = Object;
 
 const bigFirst = (a) => `${a[0].toUpperCase()}${a.slice(1)}`;
 
@@ -13,7 +15,18 @@ const load = (type) => ({name, namespace}) => {
     if (!pluginPath)
         throw Error(`${bigFirst(type)} "${namespace}-${type}-${name}" could not be found!`);
     
-    return customRequire(pluginPath);
+    const [error, result] = tryCatch(customRequire, pluginPath);
+    
+    if (error?.code === 'ERR_REQUIRE_ESM')
+        assign(error, {
+            message: `☝️ Looks like '${name}' is ESM, use 'await putoutAsync()' instead`,
+            name,
+        });
+    
+    if (error)
+        throw error;
+    
+    return result;
 };
 
 module.exports.loadPlugin = load('plugin');

package/lib/plugins/filter-enabled-plugins.js

@@ -0,0 +1,24 @@
+'use strict';
+
+const {isEnabled, mergeRules} = require('../rules');
+
+// Would be great to have ability to filter
+// disabled plugins and prevent them from loading
+// but we can't because of a way multi-rule plugins
+// works. We can't determine count and names of all
+// rules of a plugin before load.
+module.exports.filterEnabledPlugins = ({plugins, cookedRules}) => {
+    const result = [];
+    
+    for (const [name, plugin] of plugins) {
+        if (!isEnabled(name, cookedRules))
+            continue;
+        
+        result.push(mergeRules(
+            [name, plugin],
+            cookedRules,
+        ));
+    }
+    
+    return result;
+};

package/lib/{load-plugins-async.js → plugins/load-plugins-async.js}

@@ -2,16 +2,21 @@
 
 const {basename} = require('path');
 
-const isEnabled = require('./is-enabled');
-const parseRules = require('./parse-rules');
-const {createAsyncLoader} = require('./async-loader');
+const {
+    parseRules,
+    validateRules,
+    isEnabled,
+    getLoadedRules,
+} = require('../rules');
+
+const {createAsyncLoader} = require('../load/async-loader');
 const {parsePluginNames} = require('./parse-plugin-names');
-const validateRules = require('./validate-rules');
 const validatePlugin = require('./validate-plugin');
-const {mergeRules} = require('./merge-rules');
+const {filterEnabledPlugins} = require('./filter-enabled-plugins');
+
+const {check, checkRule} = require('../check');
 
 const loadPluginAsync = createAsyncLoader('plugin');
-const isString = (a) => typeof a === 'string';
 
 module.exports.loadPluginsAsync = async (options) => {
     check(options);
@@ -33,24 +38,10 @@ module.exports.loadPluginsAsync = async (options) => {
         items,
     });
     
-    const result = [];
-    
-    // Would be great to have ability to filter
-    // disabled plugins and prevent them from loading
-    // but we can't because of a way multi-rule plugins
-    // works. We can't determine count and names of all
-    // rules of a plugin before load.
-    for (const [name, plugin] of plugins) {
-        if (!isEnabled(name, cookedRules))
-            continue;
-        
-        result.push(mergeRules(
-            [name, plugin],
-            cookedRules,
-        ));
-    }
-    
-    return result;
+    return filterEnabledPlugins({
+        plugins,
+        cookedRules,
+    });
 };
 
 function splitRule(rule) {
@@ -98,21 +89,6 @@ async function loadPlugins({items, loadedRules}) {
     return plugins;
 }
 
-function getLoadedRules(rules) {
-    const loadedRules = [];
-    
-    for (const item of rules) {
-        const {rule} = item;
-        
-        if (rule.includes('/'))
-            continue;
-        
-        loadedRules.push(item);
-    }
-    
-    return loadedRules;
-}
-
 function extendRules(rule, plugin) {
     const result = [];
     const entries = Object.entries(plugin);
@@ -124,16 +100,6 @@ function extendRules(rule, plugin) {
     return result;
 }
 
-function check(options) {
-    if (!options || typeof options !== 'object')
-        throw Error('options should be an object!');
-}
-
-function checkRule(rule) {
-    if (!isString(rule))
-        throw Error(`☝️ Looks like plugin name type is not 'string', but: '${typeof rule}'`);
-}
-
 function parseRuleName(rule) {
     if (rule.startsWith('import:')) {
         const shortName = basename(rule.replace('import:', ''));

package/lib/rules/get-loaded-rules.js

@@ -0,0 +1,16 @@
+'use strict';
+
+module.exports.getLoadedRules = (rules) => {
+    const loadedRules = [];
+    
+    for (const item of rules) {
+        const {rule} = item;
+        
+        if (rule.includes('/'))
+            continue;
+        
+        loadedRules.push(item);
+    }
+    
+    return loadedRules;
+};

package/lib/rules/index.js

@@ -0,0 +1,15 @@
+'use strict';
+
+const {parseRules} = require('./parse-rules');
+const {mergeRules} = require('./merge-rules');
+const {validateRules} = require('./validate-rules');
+const {isEnabled} = require('./is-enabled');
+const {getLoadedRules} = require('./get-loaded-rules');
+
+module.exports = {
+    parseRules,
+    mergeRules,
+    validateRules,
+    isEnabled,
+    getLoadedRules,
+};

package/lib/{is-enabled.js → rules/is-enabled.js}

@@ -1,6 +1,6 @@
 'use strict';
 
-module.exports = (name, rules) => {
+module.exports.isEnabled = (name, rules) => {
     for (const {rule, state} of rules) {
         if (rule === name)
             return state;

package/lib/{parse-rules.js → rules/parse-rules.js}

@@ -13,7 +13,7 @@ const stateOptionError = ({rule, value}) => Error(`☝️ ${rule}: state option
 const defaultOptions = () => Object.create(null);
 const parseState = (rule, value) => validateState(rule, value) && value === 'on' || value !== 'off';
 
-module.exports = (rules) => {
+module.exports.parseRules = (rules) => {
     rules = parseSubrules(rules);
     
     const result = [];

package/lib/{validate-rules.js → rules/validate-rules.js}

@@ -9,7 +9,7 @@ const parse = (rule) => {
     return rule;
 };
 
-module.exports = ({items, rules}) => {
+module.exports.validateRules = ({items, rules}) => {
     const ruleItems = Object.keys(rules);
     
     for (const rule of ruleItems) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@putout/engine-loader",
-  "version": "11.4.0",
+  "version": "11.6.0",
   "type": "commonjs",
   "author": "coderaiser <mnemonic.enemy@gmail.com> (https://github.com/coderaiser)",
   "description": "load plugins and prepare them to run",
@@ -41,6 +41,7 @@
   },
   "devDependencies": {
     "@putout/formatter-progress": "*",
+    "@putout/plugin-apply-nullish-coalescing": "*",
     "@putout/plugin-convert-commonjs-to-esm": "*",
     "@putout/plugin-remove-unused-variables": "*",
     "@putout/processor-javascript": "*",

package/lib/build-paths.js

@@ -1,20 +0,0 @@
-'use strict';
-
-const addNodeModules = (a) => `${a}/node_modules`;
-
-module.exports = (path) => {
-    const names = path
-        .split('/')
-        .slice(1);
-    
-    const result = [''];
-    
-    let current = '';
-    
-    for (const name of names) {
-        current += `/${name}`;
-        result.push(current);
-    }
-    
-    return result.map(addNodeModules);
-};