feature: @putout/engine-loader: add ability to disable all rules except couple enabled

Author: coderaiser

packages/engine-loader/lib/index.js

@@ -67,9 +67,13 @@ module.exports.loadPlugins = (options) => {
         loadedRules,
     });
 
+    const cookedNotEnabled = parseRules(rules, {
+        enable: false,
+    });
+    
     return filterEnabledPlugins({
         plugins,
-        cookedRules,
+        cookedRules: cookedNotEnabled,
     });
 };
 

packages/engine-loader/lib/rules/parse-rules.js

@@ -24,8 +24,9 @@ const parseState = (rule, value) => {
     return value;
 };
 
-module.exports.parseRules = (rules) => {
-    rules = parseSubrules(rules);
+module.exports.parseRules = (rules, {enable = true} = {}) => {
+    if (enable)
+        rules = parseSubrules(rules);
 
     const result = [];
     const plugin = null;

packages/engine-loader/test/load-plugins.js

@@ -243,7 +243,7 @@ test('putout: loader: no options.pluginNames', (t) => {
     t.end();
 });
 
-test('putout: loader: enable part of rule', (t) => {
+test('putout: loader: enable part of plugin', (t) => {
     const source = `const {run} = require('madrun');`;
 
     const {code} = putout(source, {
@@ -294,6 +294,31 @@ test('putout: loader: disabled part of rule', (t) => {
     t.end();
 });
 
+test('putout: loader: enable part of rule', (t) => {
+    const code = montag`
+        const {run} = require('madrun');
+        
+        module.exports = {
+            'lint': () => 'bin/putout.js .',
+            'fix:lint': () => run('lint', '--fix'),
+        };
+    `;
+    
+    const {places} = putout(code, {
+        fix: false,
+        rules: {
+            'madrun': 'off',
+            'madrun/add-madrun-to-lint': 'on',
+        },
+        plugins: ['madrun'],
+    });
+    
+    const expected = [];
+    
+    t.deepEqual(places, expected, 'should disable all but couple of rules in plugin');
+    t.end();
+});
+
 test('putout: loader: plugin is a function', (t) => {
     const [e] = tryCatch(putout, 'hello', {
         loadPlugins,

packages/engine-runner/lib/progress.spec.js

@@ -24,6 +24,7 @@ test('putout: runner: progress: start', async (t) => {
         putout(source, {
             progress,
             rules: {
+                'filesystem': 'off',
                 'filesystem/remove-files': ['on', {
                     names: ['coverage'],
                 }],
@@ -32,7 +33,7 @@ test('putout: runner: progress: start', async (t) => {
         }),
     ]);
 
-    t.equal(pluginsCount, 4);
+    t.equal(pluginsCount, 1);
     t.end();
 });
 
@@ -52,6 +53,7 @@ test('putout: runner: progress: end', async (t) => {
         putout(source, {
             progress,
             rules: {
+                'filesystem': 'off',
                 'filesystem/remove-files': ['on', {
                     names: ['coverage'],
                 }],
@@ -60,7 +62,7 @@ test('putout: runner: progress: end', async (t) => {
         }),
     ]);
 
-    t.equal(pluginsCount, 4);
+    t.equal(pluginsCount, 1);
     t.end();
 });
 
@@ -90,6 +92,6 @@ test('putout: runner: progress: push', async (t) => {
         }),
     ]);
 
-    t.equal(pluginsCount, 4);
+    t.equal(pluginsCount, 1);
     t.end();
 });

packages/operate/lib/replace-with/replace-with.spec.js

@@ -187,8 +187,12 @@ test('putout: operate: replaceWith: parentPath', (t) => {
     const {code} = putout(source, {
         rules: {
             'putout': 'off',
+            'putout/apply-create-test': 'on',
             'putout/apply-fixture-name-to-message': 'on',
+            'putout/declare': 'on',
+            'putout/convert-dirname-to-url': 'on',
             'putout/move-require-on-top-level': 'on',
+            'putout/replace-test-message': 'on',
         },
         plugins: ['putout'],
     });