babel-plugin-transform-react-remove-prop-types - npm Package Compare versions

Comparing version 0.4.19 to 0.4.20

lib/index.js

@@ -97,3 +97,3 @@ "use strict";
 
-  if (!memberExpression) {
+  if (!memberExpression || memberExpression.type !== 'MemberExpression') {
     // This case is currently unhandled by this plugin.
@@ -164,5 +164,25 @@ return null;
           }),
-          wrapTemplate: template("\n              LEFT = process.env.NODE_ENV !== \"production\" ? RIGHT : {}\n            ", {
-            placeholderPattern: /^(LEFT|RIGHT)$/
-          }),
+          wrapTemplate: function wrapTemplate(_ref) {
+            var LEFT = _ref.LEFT,
+                RIGHT = _ref.RIGHT;
+            var options = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : {};
+            var _options$as = options.as,
+                as = _options$as === void 0 ? 'assignmentExpression' : _options$as;
+            var right = template.expression("\n                process.env.NODE_ENV !== \"production\" ? RIGHT : {}\n              ", {
+              placeholderPattern: /^(LEFT|RIGHT)$/
+            })({
+              RIGHT: RIGHT
+            });
+
+            switch (as) {
+              case 'variableDeclarator':
+                return types.variableDeclarator(LEFT, right);
+
+              case 'assignmentExpression':
+                return types.assignmentExpression('=', LEFT, right);
+
+              default:
+                throw new Error("unrecognized template type ".concat(as));
+            }
+          },
           mode: state.opts.mode || 'remove',
@@ -169,0 +189,0 @@ ignoreFilenames: ignoreFilenames,

lib/remove.js

@@ -112,2 +112,4 @@ "use strict";
             RIGHT: path.node.init
+          }, {
+            as: 'variableDeclarator'
           }));
@@ -114,0 +116,0 @@ }

package.json

 {
   "name": "babel-plugin-transform-react-remove-prop-types",
-  "version": "0.4.19",
+  "version": "0.4.20",
   "description": "Remove unnecessary React propTypes from the production build",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

src/index.js

@@ -81,3 +81,3 @@ /* eslint-disable global-require, import/no-dynamic-require */
 
-  if (!memberExpression) {
+  if (!memberExpression || memberExpression.type !== 'MemberExpression') {
     // This case is currently unhandled by this plugin.
@@ -156,8 +156,19 @@ return null
           ),
-          wrapTemplate: template(
-            `
-              LEFT = process.env.NODE_ENV !== "production" ? RIGHT : {}
-            `,
-            { placeholderPattern: /^(LEFT|RIGHT)$/ }
-          ),
+          wrapTemplate: ({ LEFT, RIGHT }, options = {}) => {
+            const { as = 'assignmentExpression' } = options
+            const right = template.expression(
+              `
+                process.env.NODE_ENV !== "production" ? RIGHT : {}
+              `,
+              { placeholderPattern: /^(LEFT|RIGHT)$/ }
+            )({ RIGHT })
+            switch (as) {
+              case 'variableDeclarator':
+                return types.variableDeclarator(LEFT, right)
+              case 'assignmentExpression':
+                return types.assignmentExpression('=', LEFT, right)
+              default:
+                throw new Error(`unrecognized template type ${as}`)
+            }
+          },
           mode: state.opts.mode || 'remove',
@@ -164,0 +175,0 @@ ignoreFilenames,

src/remove.js

@@ -116,6 +116,9 @@ /* eslint-disable no-param-reassign */
           path.replaceWith(
-            wrapTemplate({
-              LEFT: path.node.id,
-              RIGHT: path.node.init,
-            })
+            wrapTemplate(
+              {
+                LEFT: path.node.id,
+                RIGHT: path.node.init,
+              },
+              { as: 'variableDeclarator' }
+            )
           )
@@ -122,0 +125,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

47031

increased by3.3%

Lines of code

1028

increased by3.32%

No dependency changes