hermes-parser 0.36.1 → 0.37.0

package/dist/transform/astNodeMutationHelpers.js.flow

@@ -25,7 +25,7 @@ function getParentKey(
   target: ESNode,
   parent: ESNode,
   visitorKeys?: ?VisitorKeysType,
-): $ReadOnly<
+): Readonly<
   | {
       type: 'single',
       node: ESNode,
@@ -52,10 +52,16 @@ function getParentKey(
       if (parent[key] === target) {
         return {type: 'single', node: parent, key};
       }
-    } else if (Array.isArray(parent[key])) {
+    } else if (
+      Array.isArray(
+        // $FlowExpectedError[prop-missing]
+        parent[key],
+      )
+    ) {
       for (let i = 0; i < parent[key].length; i += 1) {
         // $FlowExpectedError[invalid-tuple-index]
         const current = parent[key][i];
+        // $FlowFixMe[invalid-compare]
         if (current === target) {
           return {type: 'array', node: parent, key, targetIndex: i};
         }
@@ -75,7 +81,7 @@ function getParentKey(
 export function replaceNodeOnParent(
   originalNode: ESNode,
   originalNodeParent: ESNode,
-  nodeToReplaceWith: ESNode | $ReadOnlyArray<ESNode>,
+  nodeToReplaceWith: ESNode | ReadonlyArray<ESNode>,
   visitorKeys?: ?VisitorKeysType,
 ): void {
   const replacementParent = getParentKey(
@@ -173,7 +179,7 @@ export function updateAllParentPointers(
  *
  * This will only create a new object if the overrides actually result in a change.
  */
-export function nodeWith<T: ESNode>(
+export function nodeWith<T extends ESNode>(
   node: T,
   overrideProps: Partial<T>,
   visitorKeys?: ?VisitorKeysType,
@@ -210,7 +216,7 @@ export function nodeWith<T: ESNode>(
 /**
  * Shallow clones node, providing a new reference for an existing node.
  */
-export function shallowCloneNode<T: ESNode>(
+export function shallowCloneNode<T extends ESNode>(
   node: T,
   visitorKeys?: ?VisitorKeysType,
 ): T {
@@ -226,7 +232,7 @@ export function shallowCloneNode<T: ESNode>(
 /**
  * Deeply clones node and its entire tree.
  */
-export function deepCloneNode<T: ESNode>(
+export function deepCloneNode<T extends ESNode>(
   node: T,
   visitorKeys?: ?VisitorKeysType,
 ): T {

package/dist/traverse/SimpleTraverser.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
 'use strict';
@@ -80,7 +80,9 @@ class SimpleTraverser {
     const keys = (0, _getVisitorKeys.getVisitorKeys)(node, options.visitorKeys);
 
     for (const key of keys) {
-      const child = node[key];
+      const lookupKey = key;
+      const childAny = node[lookupKey];
+      const child = childAny;
 
       if (Array.isArray(child)) {
         for (let j = 0; j < child.length; ++j) {

package/dist/traverse/SimpleTraverser.js.flow

@@ -16,7 +16,7 @@ import type {ESNode} from 'hermes-estree';
 import {getVisitorKeys, isNode} from './getVisitorKeys';
 
 export type TraverserCallback = (node: ESNode, parent: ?ESNode) => void;
-export type TraverserOptions = $ReadOnly<{
+export type TraverserOptions = Readonly<{
   /** The callback function which is called on entering each node. */
   enter: TraverserCallback,
   /** The callback function which is called on leaving each node. */
@@ -83,9 +83,9 @@ export class SimpleTraverser {
 
     const keys = getVisitorKeys(node, options.visitorKeys);
     for (const key of keys) {
-      const child: ESNode | $ReadOnlyArray<ESNode> = (node[
-        (key: $FlowFixMe)
-      ]: $FlowFixMe);
+      const lookupKey: $FlowFixMe = key;
+      const childAny: $FlowFixMe = node[lookupKey];
+      const child: ESNode | ReadonlyArray<ESNode> = childAny;
 
       if (Array.isArray(child)) {
         for (let j = 0; j < child.length; ++j) {

package/dist/traverse/getVisitorKeys.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @noformat
  */
 'use strict';
@@ -20,12 +20,14 @@ var _ESTreeVisitorKeys = _interopRequireDefault(require("../generated/ESTreeVisi
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
 function isNode(thing)
-/*: implies thing is {+[string]: mixed} */
+/*: implies thing is {readonly [string]: unknown} */
 {
   return typeof thing === 'object' && thing != null && typeof thing.type === 'string';
 }
 
-function getVisitorKeys(node, visitorKeys) {
+function getVisitorKeys(node, visitorKeys)
+/*: ReadonlyArray<keyof T> */
+{
   const keys = (visitorKeys != null ? visitorKeys : _ESTreeVisitorKeys.default)[node.type];
 
   if (keys == null) {

package/dist/traverse/getVisitorKeys.js.flow

@@ -15,17 +15,19 @@ import type {VisitorKeys as VisitorKeysType} from '../generated/ESTreeVisitorKey
 
 import FlowVisitorKeys from '../generated/ESTreeVisitorKeys';
 
-export function isNode(thing: mixed) /*: implies thing is {+[string]: mixed} */ {
+export function isNode(
+  thing: unknown,
+) /*: implies thing is {readonly [string]: unknown} */ {
   return (
     typeof thing === 'object' && thing != null && typeof thing.type === 'string'
   );
 }
 
 export type {VisitorKeysType};
-export function getVisitorKeys<T: ESNode>(
+export function getVisitorKeys<T extends ESNode>(
   node: T,
   visitorKeys?: ?VisitorKeysType,
-): $ReadOnlyArray<$Keys<T>> {
+) /*: ReadonlyArray<keyof T> */ {
   const keys = (visitorKeys ?? FlowVisitorKeys)[node.type];
   if (keys == null) {
     throw new Error(`No visitor keys found for node type "${node.type}".`);

package/dist/utils/Builders.js

@@ -4,10 +4,10 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
-'use strict';
+'use strict'; // Rely on the mapper to fix up parent relationships.
 
 Object.defineProperty(exports, "__esModule", {
   value: true
@@ -26,7 +26,6 @@ exports.stringLiteral = stringLiteral;
 exports.throwStatement = throwStatement;
 exports.typeofExpression = typeofExpression;
 exports.variableDeclaration = variableDeclaration;
-// Rely on the mapper to fix up parent relationships.
 const EMPTY_PARENT = null;
 exports.EMPTY_PARENT = EMPTY_PARENT;
 

package/dist/utils/Builders.js.flow

@@ -103,7 +103,7 @@ export function nullLiteral(info?: Etc): NullLiteral {
   };
 }
 
-export function conjunction(tests: $ReadOnlyArray<Expression>): Expression {
+export function conjunction(tests: ReadonlyArray<Expression>): Expression {
   if (tests.length === 0) {
     throw new Error('Must have at least one test.');
   }
@@ -116,7 +116,7 @@ export function conjunction(tests: $ReadOnlyArray<Expression>): Expression {
   }));
 }
 
-export function disjunction(tests: $ReadOnlyArray<Expression>): Expression {
+export function disjunction(tests: ReadonlyArray<Expression>): Expression {
   if (tests.length === 0) {
     throw new Error('Must have at least one test.');
   }
@@ -153,7 +153,7 @@ export function variableDeclaration(
 
 export function callExpression(
   callee: Expression | Super,
-  args: $ReadOnlyArray<Expression | SpreadElement>,
+  args: ReadonlyArray<Expression | SpreadElement>,
   info?: Etc,
 ): CallExpression {
   return {
@@ -175,9 +175,9 @@ export function throwStatement(arg: Expression, info?: Etc): ThrowStatement {
 }
 
 export function iife(
-  statements: $ReadOnlyArray<Statement>,
-  params: $ReadOnlyArray<BindingName> = [],
-  args: $ReadOnlyArray<Expression> = [],
+  statements: ReadonlyArray<Statement>,
+  params: ReadonlyArray<BindingName> = [],
+  args: ReadonlyArray<Expression> = [],
 ): CallExpression {
   const callee: ArrowFunctionExpression = {
     type: 'ArrowFunctionExpression',

package/dist/utils/GenID.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
 'use strict';

package/dist/utils/GenID.js.flow

@@ -14,8 +14,8 @@ const genPrefix = '$$gen$';
 
 export default class GenID {
   genN: number = 0;
-  +used: Set<string> = new Set();
-  +prefix: string;
+  readonly used: Set<string> = new Set();
+  readonly prefix: string;
 
   constructor(uniqueTransformPrefix: string) {
     this.prefix = `${genPrefix}${uniqueTransformPrefix}`;

package/dist/utils/createSyntaxError.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
 'use strict';

package/dist/utils/isReservedWord.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
 'use strict';

package/dist/utils/mutateESTreeASTForPrettier.js

@@ -4,7 +4,7 @@
  * This source code is licensed under the MIT license found in the
  * LICENSE file in the root directory of this source tree.
  *
- * 
+ *  strict
  * @format
  */
 'use strict';

package/dist/utils/mutateESTreeASTForPrettier.js.flow

@@ -17,7 +17,7 @@ import {SimpleTransform} from '../transform/SimpleTransform';
 // https://github.com/prettier/prettier/blob/d962466a828f8ef51435e3e8840178d90b7ec6cd/src/language-js/parse/postprocess/index.js#L161-L182
 function transformChainExpression(
   node: ESNode,
-  comments: ?$ReadOnlyArray<Comment>,
+  comments: ?ReadonlyArray<Comment>,
 ): ESNode {
   if (comments != null) {
     // $FlowExpectedError[prop-missing]

package/package.json

@@ -1,15 +1,16 @@
 {
   "name": "hermes-parser",
-  "version": "0.36.1",
+  "version": "0.37.0",
   "description": "A JavaScript parser built from the Hermes engine",
   "main": "dist/index.js",
   "license": "MIT",
   "repository": {
     "type": "git",
-    "url": "git@github.com:facebook/hermes.git"
+    "url": "git+https://github.com/facebook/hermes.git",
+    "directory": "tools/hermes-parser/js/hermes-parser"
   },
   "dependencies": {
-    "hermes-estree": "0.36.1"
+    "hermes-estree": "0.37.0"
   },
   "devDependencies": {
     "@babel/parser": "7.7.4",