flow-parser - npm Package Compare versions

Comparing version 0.32.0 to 0.33.0

package.json

 {
   "name": "flow-parser",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "license": "BSD-3-Clause",
@@ -5,0 +5,0 @@ "description": "JavaScript parser written in OCaml. Produces SpiderMonkey AST",

test/custom_ast_types.js

@@ -72,1 +72,7 @@ var types = require("ast-types/lib/types");
     .field("exact", Boolean)
+
+def("MetaProperty")
+    .bases("Expression")
+    .build("meta", "property")
+    .field("meta", def("Identifier"))
+    .field("property", def("Identifier"));

test/esprima_test_runner.js

@@ -89,7 +89,2 @@ /**
   switch (esprima.type) {
-    case "SwitchStatement":
-      // Esprima doesn't support let statements so it doesn't include the
-      // lexical field
-      delete flow.lexical;
-      break;
     case 'TryStatement':
@@ -101,10 +96,5 @@ // The Mozilla spec changed over time. There used to be a "handlers"
       flow.handlers = flow.handler ? [ flow.handler ] : [];
+      flow.guardedHandlers = [];
       delete flow.handler;
       break;
-    case 'CatchClause':
-      // Esprima doesn't support the guard property for catch clauses
-      if (!esprima.hasOwnProperty("guard")) {
-        esprima.guard = null;
-      }
-      break;
     case 'Identifier':
@@ -259,12 +249,72 @@ if (esprima.optional === undefined) {
     case 'ArrowFunctionExpression':
-      if (Array.isArray(esprima.defaults)) {
+      // esprima-fb uses a "defaults" array that is parallel to the "params"
+      // array. Flow and modern esprima use AssignmentPatterns within the
+      // params array.
+      if (Array.isArray(esprima.defaults) && Array.isArray(esprima.params)) {
         for (var i = 0; i < esprima.defaults.length; i++) {
-          if (esprima.defaults[i] === undefined) {
-            esprima.defaults[i] = null;
+          var left = esprima.params[i];
+          var right = esprima.defaults[i];
+          if (right !== undefined && right !== null) {
+            var newParam = {
+              type: 'AssignmentPattern',
+              left: left,
+              right: right,
+            };
+            if (left.loc && right.loc) {
+              newParam.loc = {
+                start: {
+                  line: left.loc.start.line,
+                  column: left.loc.start.column,
+                },
+                end: {
+                  line: right.loc.end.line,
+                  column: right.loc.end.column,
+                },
+              };
+              if (left.loc.source) {
+                newParam.loc.source = left.loc.source;
+              }
+            }
+            if (left.range && right.range) {
+              newParam.range = [left.range[0], right.range[1]];
+            }
+            esprima.params[i] = newParam;
           }
         }
+        delete esprima.defaults;
       }
+
+      // esprima-fb uses a "rest" property. Flow and modern esprima use
+      // RestElement within the params array.
+      if ('rest' in esprima) {
+        if (esprima.rest !== undefined && esprima.rest !== null) {
+          var arg = esprima.rest;
+          var rest = {
+            type: 'RestElement',
+            argument: arg,
+          };
+          if (arg.loc) {
+            rest.loc = {
+              start: {
+                line: arg.loc.start.line,
+                column: arg.loc.start.column - 3, // HACKY!
+              },
+              end: arg.loc.end,
+            };
+            if (arg.loc.source) {
+              rest.loc.source = arg.loc.source;
+            }
+          }
+          if (arg.range) {
+            rest.range = [arg.range[0] - 3, arg.range[1]]; // HACKY!
+          }
+          esprima.params.push(rest);
+        }
+        delete esprima.rest;
+      }
+
       if (esprima.async === undefined) {
         esprima.async = false;
       }
+
       delete flow.predicate;
@@ -271,0 +321,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

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

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

670755

increased by0.51%

Worsened metrics

Lines of code

17262

decreased by-0.88%

Number of medium supply chain risk alerts

2

increased by100%

No dependency changes