@playlyfe/babel-plugin-flow-react-proptypes - npm Package Compare versions

Comparing version 24.0.1-patch.1 to 24.0.1-patch.2

lib/convertToPropTypes.js

@@ -178,2 +178,5 @@ 'use strict';
     var _properties = spreadShape.properties;
+    if (!_properties) {
+      return [];
+    }
 
@@ -180,0 +183,0 @@ // Unless or until the strange default behavior changes in flow (https://github.com/facebook/flow/issues/3214)

lib/index.js

@@ -477,3 +477,3 @@ 'use strict';
       },
-      "ClassExpression|ClassDeclaration": function ClassExpressionClassDeclaration(path) {
+      "ClassExpression|ClassDeclaration": function ClassExpressionClassDeclaration(path, state) {
         if (opts.noStatic && path.node.type === 'ClassExpression') return;
@@ -512,3 +512,3 @@
 
-          if (bodyNode && bodyNode.key.name === 'context' && bodyNode.typeAnnotation) {
+          if (bodyNode && bodyNode.key.name === 'context' && bodyNode.typeAnnotation && !state.opts.disableContextTypes) {
             var _annotation = bodyNode.typeAnnotation.typeAnnotation;
@@ -543,3 +543,3 @@ var context = getPropsForTypeAnnotation(_annotation);
         var thirdSuperParam = getContextTypeParam(path.node);
-        if (thirdSuperParam && thirdSuperParam.type === 'GenericTypeAnnotation') {
+        if (thirdSuperParam && thirdSuperParam.type === 'GenericTypeAnnotation' && !state.opts.disableContextTypes) {
           var _typeAliasName = thirdSuperParam.id.name;
@@ -633,10 +633,13 @@ if (_typeAliasName === 'Object') return;
               var imported = importedTypes[_spec.local.name];
-              if (!imported) continue;
-
-              if (_spec.local.name !== _spec.exported.name) {
-                // TODO: handle this properly
-                continue;
+              if (imported) {
+                if (_spec.local.name === _spec.exported.name) {
+                  addExportTypeDecl(path, (0, _util.getExportNameForType)(_spec.local.name), imported.accessNode);
+                }
+                // TODO: handle case when name not same
+              } else {
+                var internal = internalTypes[_spec.local.name];
+                if (internal) {
+                  exportInternalType(_spec.local.name, internal);
+                }
               }
-
-              addExportTypeDecl(path, (0, _util.getExportNameForType)(_spec.local.name), imported.accessNode);
             }
@@ -662,2 +665,3 @@ } catch (err) {
         var declarationObject = null;
+
         if (!node.declaration) return;
@@ -679,14 +683,17 @@ if (node.declaration.type === 'TypeAlias') {
         internalTypes[name] = propTypes;
+        exportInternalType(name, propTypes);
 
-        var propTypesAst = (0, _makePropTypesAst.makePropTypesAstForExport)(propTypes);
+        function exportInternalType(name, propTypes) {
+          var propTypesAst = (0, _makePropTypesAst.makePropTypesAstForExport)(propTypes);
 
-        // create a variable for reuse
-        var exportName = (0, _util.getExportNameForType)(name);
-        exportedTypes[name] = exportName;
-        var variableDeclarationAst = t.variableDeclaration('var', [t.variableDeclarator(t.identifier(exportName), wrapInDceCheck(propTypesAst))]);
-        path.insertBefore(variableDeclarationAst);
+          // create a variable for reuse
+          var exportName = (0, _util.getExportNameForType)(name);
+          exportedTypes[name] = exportName;
+          var variableDeclarationAst = t.variableDeclaration('var', [t.variableDeclarator(t.identifier(exportName), wrapInDceCheck(propTypesAst))]);
+          path.insertBefore(variableDeclarationAst);
 
-        if (!omitRuntimeTypeExport) {
-          if (path.node[SKIP]) return;
-          addExportTypeDecl(path, exportName);
+          if (!omitRuntimeTypeExport) {
+            if (path.node[SKIP]) return;
+            addExportTypeDecl(path, exportName);
+          }
         }
@@ -693,0 +700,0 @@ },

package.json

 {
   "name": "@playlyfe/babel-plugin-flow-react-proptypes",
-  "version": "24.0.1-patch.1",
+  "version": "24.0.1-patch.2",
   "description": "converts flow types to react proptypes",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

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

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

81806

increased by0.56%

Lines of code

1398

increased by0.79%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes