feat: strip literals

src/index.js

@@ -4,6 +4,7 @@ const visitors = [
   require('./visitors/findFunctionNames'),
   require('./visitors/extractObjectProperties'),
   require('./visitors/propTypes'),
+  require('./visitors/stripLiterals'),
   require('./visitors/combineArguments'),
   require('./visitors/createConditionalExpression'),
 ];

src/visitors/stripLiterals.js

@@ -0,0 +1,58 @@
+const t = require('@babel/types');
+const helpers = require('../utils/helpers');
+const _ = require('lodash');
+
+const visitor = {
+  CallExpression(path) {
+    const c = path.node.callee;
+    if (!t.isIdentifier(c) || !this.options.functionNames.includes(c.name)) {
+      return;
+    }
+
+    const [match, noMatch] = _.partition(
+      path.node.arguments,
+      item => t.isLogicalExpression(item) && helpers.isAllLogicalAndOperators(item),
+    );
+
+    if (match.length === 0) return;
+
+    const result = match
+      .map(helpers.flattenLogicalOperator)
+      .map(expression =>
+        expression.filter((item, index) => {
+          if (t.isBooleanLiteral(item, { value: true })) {
+            return false;
+          }
+          if (index !== expression.length - 1 && t.isStringLiteral(item) && item.value.length > 0) {
+            return false;
+          }
+          return true;
+        }),
+      )
+      .map(expression =>
+        expression.some(item => {
+          if (t.isBooleanLiteral(item, { value: false })) {
+            return true;
+          }
+          if (t.isStringLiteral(item) && item.value.length === 0) {
+            return true;
+          }
+          return false;
+        })
+          ? []
+          : expression,
+      )
+      .filter(expression => expression.length !== 0)
+      .map(expression => expression.reduce((prev, curr) => t.logicalExpression('&&', prev, curr)));
+
+    path.node.arguments = [...noMatch, ...result];
+  },
+};
+
+module.exports = (path, options) => {
+  if (!t.isProgram(path.node)) {
+    throw new Error('Node has to be a program node');
+  }
+
+  path.traverse(visitor, { options });
+};

test/fixtures/extract-object-properties/mixed-arguments/output.js

@@ -1 +1 @@
-clsx(foo && bar, true && 'baz');
+clsx('baz', foo && bar);

test/fixtures/extract-object-properties/multiple-objects/output.js

@@ -1 +1 @@
-clsx(true && 'foo', false && 'bar', 'hello' && '--foobar');
+clsx('foo', '--foobar');

test/fixtures/find-function-names/using-custom-library/output.js

@@ -1,3 +1,3 @@
 import myCustomFunction from 'my-custom-library';
 
-myCustomFunction(true && 'foo');
+myCustomFunction('foo');

test/fixtures/find-function-names/using-import/output.js

@@ -1,3 +1,3 @@
 import fooFunction from 'clsx';
 
-fooFunction(true && 'foo');
+fooFunction('foo');

test/fixtures/find-function-names/using-require/output.js

@@ -1,3 +1,3 @@
 const fooFunction = require('clsx');
 
-fooFunction(true && 'foo');
+fooFunction('foo');

test/fixtures/strip-literals/boolean-literal/code.js

@@ -0,0 +1,6 @@
+clsx({
+  [classes.root]: false,
+  [classes.cellHide]: true,
+  [classes.cellStacked]: options.responsive === 'stacked',
+  'datatables-noprint': !print,
+});

test/fixtures/strip-literals/boolean-literal/output.js

@@ -0,0 +1,5 @@
+clsx(
+  classes.cellHide,
+  options.responsive === 'stacked' && classes.cellStacked,
+  !print && 'datatables-noprint',
+);

test/fixtures/strip-literals/string-literal/code.js

@@ -0,0 +1,6 @@
+clsx({
+  [classes.root]: 'true',
+  [classes.cellHide]: '',
+  [classes.cellStacked]: options.responsive === 'stacked',
+  'datatables-noprint': !print,
+});

test/fixtures/strip-literals/string-literal/output.js

@@ -0,0 +1,6 @@
+clsx(
+  classes.root,
+
+  options.responsive === 'stacked' && classes.cellStacked,
+  !print && 'datatables-noprint',
+);