feature: @putout/plugin-putout: apply-vars: add support of CommonJS

Author: coderaiser

packages/plugin-putout/lib/apply-vars/fixture/no-body-fix.js

@@ -0,0 +1,11 @@
+module.exports.match = () => ({
+    '(__a) => typeof __a === "__b"': (vars, path) => !path.node.returnType,
+});
+
+module.exports.replace = () => ({
+    '(__a) => typeof __a === "__b"': (vars, path) => !path.node.returnType,
+});
+
+const a = {
+    'hello': ({}, path) => !path.node.returnType,
+};

packages/plugin-putout/lib/apply-vars/fixture/no-body.js

@@ -0,0 +1,12 @@
+module.exports.match = () => ({
+    '(__a) => typeof __a === "__b"': ({}, path) => !path.node.returnType,
+});
+
+module.exports.replace = () => ({
+    '(__a) => typeof __a === "__b"': ({}, path) => !path.node.returnType,
+});
+
+
+const a = {
+    'hello': ({}, path) => !path.node.returnType,
+}

packages/plugin-putout/lib/apply-vars/index.js

@@ -1,7 +1,23 @@
 'use strict';
 
-module.exports.report = () => `Use 'var' instead of '{}'`;
+const {operator} = require('putout');
+const {compareAny} = operator;
+
+module.exports.report = () => `Use 'vars' instead of '{}'`;
+
+module.exports.match = () => ({
+    '({}, path) => __a': isInsideReplacer,
+});
 
 module.exports.replace = () => ({
-    '({}, path) => __body': '(vars, path) => __body',
+    '({}, path) => __a': '(vars, path) => __a',
 });
+
+const isInsideReplacer = (vars, path) => {
+    return compareAny(path, [
+        'module.exports.match = __',
+        'module.exports.replace = __',
+        'export const match = __',
+        'export const replace = __',
+    ]);
+};

packages/plugin-putout/lib/apply-vars/index.spec.js

@@ -5,16 +5,21 @@ const plugin = require('.');
 
 const test = createTest(__dirname, {
     plugins: [
-        ['apply-vars', plugin],
+        ['putout/apply-vars', plugin],
     ],
 });
 
-test('putout: apply-vars: report', (t) => {
-    t.report('apply-vars', `Use 'var' instead of '{}'`);
+test('plugin-putout: apply-vars: report', (t) => {
+    t.report('apply-vars', `Use 'vars' instead of '{}'`);
     t.end();
 });
 
-test('putout: apply-vars: transform', (t) => {
+test('plugin-putout: apply-vars: transform', (t) => {
     t.transform('apply-vars');
     t.end();
 });
+
+test('plugin-putout: apply-vars: transform: no-body', (t) => {
+    t.transform('no-body');
+    t.end();
+});

packages/plugin-putout/test/apply-vars.js

@@ -0,0 +1,20 @@
+'use strict';
+
+const {createTest} = require('@putout/test');
+const plugin = require('..');
+
+const test = createTest(__dirname, {
+    plugins: [
+        ['apply-vars', plugin],
+    ],
+});
+
+test('putout: apply-vars: report', (t) => {
+    t.report('apply-vars', `Use 'vars' instead of '{}'`);
+    t.end();
+});
+
+test('putout: apply-vars: transform', (t) => {
+    t.transform('apply-vars');
+    t.end();
+});

packages/plugin-typescript/lib/apply-type-guards/index.js

@@ -16,7 +16,7 @@ const create = template('(__a): __a is __c => typeof __a === "__b"', {
 module.exports.report = () => `Use 'type guards'`;
 
 module.exports.match = () => ({
-    '(__a) => typeof __a === "__b"': ({}, path) => !path.node.returnType,
+    '(__a) => typeof __a === "__b"': (vars, path) => !path.node.returnType,
 });
 
 module.exports.replace = () => ({