eslint 6.5.1 → 6.7.2

package/lib/rules/getter-return.js

@@ -18,8 +18,7 @@ const TARGET_NODE_TYPE = /^(?:Arrow)?FunctionExpression$/u;
 
 /**
  * Checks a given code path segment is reachable.
- *
- * @param {CodePathSegment} segment - A segment to check.
+ * @param {CodePathSegment} segment A segment to check.
  * @returns {boolean} `true` if the segment is reachable.
  */
 function isReachable(segment) {
@@ -30,8 +29,7 @@ function isReachable(segment) {
  * Gets a readable location.
  *
  * - FunctionExpression -> the function name or `function` keyword.
- *
- * @param {ASTNode} node - A function node to get.
+ * @param {ASTNode} node A function node to get.
  * @returns {ASTNode|Token} The node or the token of a location.
  */
 function getId(node) {
@@ -92,8 +90,7 @@ module.exports = {
          *
          * If the last code path segment is reachable, there are paths which are not
          * returned or thrown.
-         *
-         * @param {ASTNode} node - A node to check.
+         * @param {ASTNode} node A node to check.
          * @returns {void}
          */
         function checkLastSegment(node) {
@@ -113,7 +110,7 @@ module.exports = {
 
         /**
          * Checks whether a node means a getter function.
-         * @param {ASTNode} node - a node to check.
+         * @param {ASTNode} node a node to check.
          * @returns {boolean} if node means a getter, return true; else return false.
          */
         function isGetter(node) {

package/lib/rules/grouped-accessor-pairs.js

@@ -0,0 +1,224 @@
+/**
+ * @fileoverview Rule to require grouped accessor pairs in object literals and classes
+ * @author Milos Djermanovic
+ */
+
+"use strict";
+
+//------------------------------------------------------------------------------
+// Requirements
+//------------------------------------------------------------------------------
+
+const astUtils = require("./utils/ast-utils");
+
+//------------------------------------------------------------------------------
+// Typedefs
+//------------------------------------------------------------------------------
+
+/**
+ * Property name if it can be computed statically, otherwise the list of the tokens of the key node.
+ * @typedef {string|Token[]} Key
+ */
+
+/**
+ * Accessor nodes with the same key.
+ * @typedef {Object} AccessorData
+ * @property {Key} key Accessor's key
+ * @property {ASTNode[]} getters List of getter nodes.
+ * @property {ASTNode[]} setters List of setter nodes.
+ */
+
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+
+/**
+ * Checks whether or not the given lists represent the equal tokens in the same order.
+ * Tokens are compared by their properties, not by instance.
+ * @param {Token[]} left First list of tokens.
+ * @param {Token[]} right Second list of tokens.
+ * @returns {boolean} `true` if the lists have same tokens.
+ */
+function areEqualTokenLists(left, right) {
+    if (left.length !== right.length) {
+        return false;
+    }
+
+    for (let i = 0; i < left.length; i++) {
+        const leftToken = left[i],
+            rightToken = right[i];
+
+        if (leftToken.type !== rightToken.type || leftToken.value !== rightToken.value) {
+            return false;
+        }
+    }
+
+    return true;
+}
+
+/**
+ * Checks whether or not the given keys are equal.
+ * @param {Key} left First key.
+ * @param {Key} right Second key.
+ * @returns {boolean} `true` if the keys are equal.
+ */
+function areEqualKeys(left, right) {
+    if (typeof left === "string" && typeof right === "string") {
+
+        // Statically computed names.
+        return left === right;
+    }
+    if (Array.isArray(left) && Array.isArray(right)) {
+
+        // Token lists.
+        return areEqualTokenLists(left, right);
+    }
+
+    return false;
+}
+
+/**
+ * Checks whether or not a given node is of an accessor kind ('get' or 'set').
+ * @param {ASTNode} node A node to check.
+ * @returns {boolean} `true` if the node is of an accessor kind.
+ */
+function isAccessorKind(node) {
+    return node.kind === "get" || node.kind === "set";
+}
+
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+
+module.exports = {
+    meta: {
+        type: "suggestion",
+
+        docs: {
+            description: "require grouped accessor pairs in object literals and classes",
+            category: "Best Practices",
+            recommended: false,
+            url: "https://eslint.org/docs/rules/grouped-accessor-pairs"
+        },
+
+        schema: [
+            {
+                enum: ["anyOrder", "getBeforeSet", "setBeforeGet"]
+            }
+        ],
+
+        messages: {
+            notGrouped: "Accessor pair {{ formerName }} and {{ latterName }} should be grouped.",
+            invalidOrder: "Expected {{ latterName }} to be before {{ formerName }}."
+        }
+    },
+
+    create(context) {
+        const order = context.options[0] || "anyOrder";
+        const sourceCode = context.getSourceCode();
+
+        /**
+         * Reports the given accessor pair.
+         * @param {string} messageId messageId to report.
+         * @param {ASTNode} formerNode getter/setter node that is defined before `latterNode`.
+         * @param {ASTNode} latterNode getter/setter node that is defined after `formerNode`.
+         * @returns {void}
+         * @private
+         */
+        function report(messageId, formerNode, latterNode) {
+            context.report({
+                node: latterNode,
+                messageId,
+                loc: astUtils.getFunctionHeadLoc(latterNode.value, sourceCode),
+                data: {
+                    formerName: astUtils.getFunctionNameWithKind(formerNode.value),
+                    latterName: astUtils.getFunctionNameWithKind(latterNode.value)
+                }
+            });
+        }
+
+        /**
+         * Creates a new `AccessorData` object for the given getter or setter node.
+         * @param {ASTNode} node A getter or setter node.
+         * @returns {AccessorData} New `AccessorData` object that contains the given node.
+         * @private
+         */
+        function createAccessorData(node) {
+            const name = astUtils.getStaticPropertyName(node);
+            const key = (name !== null) ? name : sourceCode.getTokens(node.key);
+
+            return {
+                key,
+                getters: node.kind === "get" ? [node] : [],
+                setters: node.kind === "set" ? [node] : []
+            };
+        }
+
+        /**
+         * Merges the given `AccessorData` object into the given accessors list.
+         * @param {AccessorData[]} accessors The list to merge into.
+         * @param {AccessorData} accessorData The object to merge.
+         * @returns {AccessorData[]} The same instance with the merged object.
+         * @private
+         */
+        function mergeAccessorData(accessors, accessorData) {
+            const equalKeyElement = accessors.find(a => areEqualKeys(a.key, accessorData.key));
+
+            if (equalKeyElement) {
+                equalKeyElement.getters.push(...accessorData.getters);
+                equalKeyElement.setters.push(...accessorData.setters);
+            } else {
+                accessors.push(accessorData);
+            }
+
+            return accessors;
+        }
+
+        /**
+         * Checks accessor pairs in the given list of nodes.
+         * @param {ASTNode[]} nodes The list to check.
+         * @param {Function} shouldCheck – Predicate that returns `true` if the node should be checked.
+         * @returns {void}
+         * @private
+         */
+        function checkList(nodes, shouldCheck) {
+            const accessors = nodes
+                .filter(shouldCheck)
+                .filter(isAccessorKind)
+                .map(createAccessorData)
+                .reduce(mergeAccessorData, []);
+
+            for (const { getters, setters } of accessors) {
+
+                // Don't report accessor properties that have duplicate getters or setters.
+                if (getters.length === 1 && setters.length === 1) {
+                    const [getter] = getters,
+                        [setter] = setters,
+                        getterIndex = nodes.indexOf(getter),
+                        setterIndex = nodes.indexOf(setter),
+                        formerNode = getterIndex < setterIndex ? getter : setter,
+                        latterNode = getterIndex < setterIndex ? setter : getter;
+
+                    if (Math.abs(getterIndex - setterIndex) > 1) {
+                        report("notGrouped", formerNode, latterNode);
+                    } else if (
+                        (order === "getBeforeSet" && getterIndex > setterIndex) ||
+                        (order === "setBeforeGet" && getterIndex < setterIndex)
+                    ) {
+                        report("invalidOrder", formerNode, latterNode);
+                    }
+                }
+            }
+        }
+
+        return {
+            ObjectExpression(node) {
+                checkList(node.properties, n => n.type === "Property");
+            },
+            ClassBody(node) {
+                checkList(node.body, n => n.type === "MethodDefinition" && !n.static);
+                checkList(node.body, n => n.type === "MethodDefinition" && n.static);
+            }
+        };
+    }
+};

package/lib/rules/indent.js

@@ -187,6 +187,7 @@ class BinarySearchTree {
  */
 class TokenInfo {
 
+    // eslint-disable-next-line jsdoc/require-description
     /**
      * @param {SourceCode} sourceCode A SourceCode object
      */
@@ -236,6 +237,7 @@ class TokenInfo {
  */
 class OffsetStorage {
 
+    // eslint-disable-next-line jsdoc/require-description
     /**
      * @param {TokenInfo} tokenInfo a TokenInfo instance
      * @param {number} indentSize The desired size of each indentation level
@@ -329,7 +331,6 @@ class OffsetStorage {
      * Instead, the correct way would be to offset `baz` by 1 level from `bar`, offset `bar` by 1 level from the `)`, and
      * offset the `)` by 0 levels from `foo`. This ensures that the offset between `bar` and the `)` are correctly collapsed
      * in the second case.
-     *
      * @param {Token} token The token
      * @param {Token} fromToken The token that `token` should be offset from
      * @param {number} offset The desired indent level
@@ -358,7 +359,6 @@ class OffsetStorage {
      *
      * The `setDesiredOffsets` methods inserts ranges like the ones above. The third line above would be inserted by using:
      * `setDesiredOffsets([30, 43], fooToken, 1);`
-     *
      * @param {[number, number]} range A [start, end] pair. All tokens with range[0] <= token.start < range[1] will have the offset applied.
      * @param {Token} fromToken The token that this is offset from
      * @param {number} offset The desired indent level
@@ -1492,6 +1492,17 @@ module.exports = {
                 );
             },
 
+            JSXSpreadAttribute(node) {
+                const openingCurly = sourceCode.getFirstToken(node);
+                const closingCurly = sourceCode.getLastToken(node);
+
+                offsets.setDesiredOffsets(
+                    [openingCurly.range[1], closingCurly.range[0]],
+                    openingCurly,
+                    1
+                );
+            },
+
             "*"(node) {
                 const firstToken = sourceCode.getFirstToken(node);
 

package/lib/rules/index.js

@@ -52,6 +52,7 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "generator-star-spacing": () => require("./generator-star-spacing"),
     "getter-return": () => require("./getter-return"),
     "global-require": () => require("./global-require"),
+    "grouped-accessor-pairs": () => require("./grouped-accessor-pairs"),
     "guard-for-in": () => require("./guard-for-in"),
     "handle-callback-err": () => require("./handle-callback-err"),
     "id-blacklist": () => require("./id-blacklist"),
@@ -101,6 +102,7 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "no-console": () => require("./no-console"),
     "no-const-assign": () => require("./no-const-assign"),
     "no-constant-condition": () => require("./no-constant-condition"),
+    "no-constructor-return": () => require("./no-constructor-return"),
     "no-continue": () => require("./no-continue"),
     "no-control-regex": () => require("./no-control-regex"),
     "no-debugger": () => require("./no-debugger"),
@@ -108,6 +110,7 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "no-div-regex": () => require("./no-div-regex"),
     "no-dupe-args": () => require("./no-dupe-args"),
     "no-dupe-class-members": () => require("./no-dupe-class-members"),
+    "no-dupe-else-if": () => require("./no-dupe-else-if"),
     "no-dupe-keys": () => require("./no-dupe-keys"),
     "no-duplicate-case": () => require("./no-duplicate-case"),
     "no-duplicate-imports": () => require("./no-duplicate-imports"),
@@ -186,6 +189,7 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "no-self-assign": () => require("./no-self-assign"),
     "no-self-compare": () => require("./no-self-compare"),
     "no-sequences": () => require("./no-sequences"),
+    "no-setter-return": () => require("./no-setter-return"),
     "no-shadow": () => require("./no-shadow"),
     "no-shadow-restricted-names": () => require("./no-shadow-restricted-names"),
     "no-spaced-func": () => require("./no-spaced-func"),
@@ -238,6 +242,7 @@ module.exports = new LazyLoadingRuleMap(Object.entries({
     "prefer-arrow-callback": () => require("./prefer-arrow-callback"),
     "prefer-const": () => require("./prefer-const"),
     "prefer-destructuring": () => require("./prefer-destructuring"),
+    "prefer-exponentiation-operator": () => require("./prefer-exponentiation-operator"),
     "prefer-named-capture-group": () => require("./prefer-named-capture-group"),
     "prefer-numeric-literals": () => require("./prefer-numeric-literals"),
     "prefer-object-spread": () => require("./prefer-object-spread"),

package/lib/rules/init-declarations.js

@@ -11,7 +11,7 @@
 
 /**
  * Checks whether or not a given node is a for loop.
- * @param {ASTNode} block - A node to check.
+ * @param {ASTNode} block A node to check.
  * @returns {boolean} `true` when the node is a for loop.
  */
 function isForLoop(block) {
@@ -22,7 +22,7 @@ function isForLoop(block) {
 
 /**
  * Checks whether or not a given declarator node has its initializer.
- * @param {ASTNode} node - A declarator node to check.
+ * @param {ASTNode} node A declarator node to check.
  * @returns {boolean} `true` when the node has its initializer.
  */
 function isInitialized(node) {

package/lib/rules/jsx-quotes.js

@@ -65,7 +65,7 @@ module.exports = {
 
         /**
          * Checks if the given string literal node uses the expected quotes
-         * @param {ASTNode} node - A string literal node.
+         * @param {ASTNode} node A string literal node.
          * @returns {boolean} Whether or not the string literal used the expected quotes.
          * @public
          */

package/lib/rules/keyword-spacing.js

@@ -41,8 +41,7 @@ const KEYS = keywords.concat(["as", "async", "await", "from", "get", "let", "of"
 
 /**
  * Checks whether or not a given token is a "Template" token ends with "${".
- *
- * @param {Token} token - A token to check.
+ * @param {Token} token A token to check.
  * @returns {boolean} `true` if the token is a "Template" token ends with "${".
  */
 function isOpenParenOfTemplate(token) {
@@ -51,8 +50,7 @@ function isOpenParenOfTemplate(token) {
 
 /**
  * Checks whether or not a given token is a "Template" token starts with "}".
- *
- * @param {Token} token - A token to check.
+ * @param {Token} token A token to check.
  * @returns {boolean} `true` if the token is a "Template" token starts with "}".
  */
 function isCloseParenOfTemplate(token) {
@@ -88,8 +86,8 @@ module.exports = {
                             retv[key] = {
                                 type: "object",
                                 properties: {
-                                    before: { type: "boolean", default: true },
-                                    after: { type: "boolean", default: true }
+                                    before: { type: "boolean" },
+                                    after: { type: "boolean" }
                                 },
                                 additionalProperties: false
                             };
@@ -114,9 +112,8 @@ module.exports = {
 
         /**
          * Reports a given token if there are not space(s) before the token.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp} pattern - A pattern of the previous token to check.
+         * @param {Token} token A token to report.
+         * @param {RegExp} pattern A pattern of the previous token to check.
          * @returns {void}
          */
         function expectSpaceBefore(token, pattern) {
@@ -141,9 +138,8 @@ module.exports = {
 
         /**
          * Reports a given token if there are space(s) before the token.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp} pattern - A pattern of the previous token to check.
+         * @param {Token} token A token to report.
+         * @param {RegExp} pattern A pattern of the previous token to check.
          * @returns {void}
          */
         function unexpectSpaceBefore(token, pattern) {
@@ -168,9 +164,8 @@ module.exports = {
 
         /**
          * Reports a given token if there are not space(s) after the token.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp} pattern - A pattern of the next token to check.
+         * @param {Token} token A token to report.
+         * @param {RegExp} pattern A pattern of the next token to check.
          * @returns {void}
          */
         function expectSpaceAfter(token, pattern) {
@@ -195,9 +190,8 @@ module.exports = {
 
         /**
          * Reports a given token if there are space(s) after the token.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp} pattern - A pattern of the next token to check.
+         * @param {Token} token A token to report.
+         * @param {RegExp} pattern A pattern of the next token to check.
          * @returns {void}
          */
         function unexpectSpaceAfter(token, pattern) {
@@ -222,8 +216,7 @@ module.exports = {
 
         /**
          * Parses the option object and determines check methods for each keyword.
-         *
-         * @param {Object|undefined} options - The option object to parse.
+         * @param {Object|undefined} options The option object to parse.
          * @returns {Object} - Normalized option object.
          *      Keys are keywords (there are for every keyword).
          *      Values are instances of `{"before": function, "after": function}`.
@@ -263,9 +256,8 @@ module.exports = {
         /**
          * Reports a given token if usage of spacing followed by the token is
          * invalid.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp|undefined} pattern - Optional. A pattern of the previous
+         * @param {Token} token A token to report.
+         * @param {RegExp} [pattern] Optional. A pattern of the previous
          *      token to check.
          * @returns {void}
          */
@@ -276,9 +268,8 @@ module.exports = {
         /**
          * Reports a given token if usage of spacing preceded by the token is
          * invalid.
-         *
-         * @param {Token} token - A token to report.
-         * @param {RegExp|undefined} pattern - Optional. A pattern of the next
+         * @param {Token} token A token to report.
+         * @param {RegExp} [pattern] Optional. A pattern of the next
          *      token to check.
          * @returns {void}
          */
@@ -288,8 +279,7 @@ module.exports = {
 
         /**
          * Reports a given token if usage of spacing around the token is invalid.
-         *
-         * @param {Token} token - A token to report.
+         * @param {Token} token A token to report.
          * @returns {void}
          */
         function checkSpacingAround(token) {
@@ -300,8 +290,7 @@ module.exports = {
         /**
          * Reports the first token of a given node if the first token is a keyword
          * and usage of spacing around the token is invalid.
-         *
-         * @param {ASTNode|null} node - A node to report.
+         * @param {ASTNode|null} node A node to report.
          * @returns {void}
          */
         function checkSpacingAroundFirstToken(node) {
@@ -318,8 +307,7 @@ module.exports = {
          *
          * This is used for unary operators (e.g. `typeof`), `function`, and `super`.
          * Other rules are handling usage of spacing preceded by those keywords.
-         *
-         * @param {ASTNode|null} node - A node to report.
+         * @param {ASTNode|null} node A node to report.
          * @returns {void}
          */
         function checkSpacingBeforeFirstToken(node) {
@@ -333,8 +321,7 @@ module.exports = {
         /**
          * Reports the previous token of a given node if the token is a keyword and
          * usage of spacing around the token is invalid.
-         *
-         * @param {ASTNode|null} node - A node to report.
+         * @param {ASTNode|null} node A node to report.
          * @returns {void}
          */
         function checkSpacingAroundTokenBefore(node) {
@@ -348,8 +335,7 @@ module.exports = {
         /**
          * Reports `async` or `function` keywords of a given node if usage of
          * spacing around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForFunction(node) {
@@ -366,8 +352,7 @@ module.exports = {
         /**
          * Reports `class` and `extends` keywords of a given node if usage of
          * spacing around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForClass(node) {
@@ -378,8 +363,7 @@ module.exports = {
         /**
          * Reports `if` and `else` keywords of a given node if usage of spacing
          * around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForIfStatement(node) {
@@ -390,8 +374,7 @@ module.exports = {
         /**
          * Reports `try`, `catch`, and `finally` keywords of a given node if usage
          * of spacing around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForTryStatement(node) {
@@ -403,8 +386,7 @@ module.exports = {
         /**
          * Reports `do` and `while` keywords of a given node if usage of spacing
          * around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForDoWhileStatement(node) {
@@ -415,8 +397,7 @@ module.exports = {
         /**
          * Reports `for` and `in` keywords of a given node if usage of spacing
          * around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForForInStatement(node) {
@@ -427,8 +408,7 @@ module.exports = {
         /**
          * Reports `for` and `of` keywords of a given node if usage of spacing
          * around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForForOfStatement(node) {
@@ -449,8 +429,7 @@ module.exports = {
          *
          *     import*as A from "./a"; /*error Expected space(s) after "import".
          *                               error Expected space(s) before "as".
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForModuleDeclaration(node) {
@@ -474,8 +453,7 @@ module.exports = {
         /**
          * Reports `as` keyword of a given node if usage of spacing around this
          * keyword is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForImportNamespaceSpecifier(node) {
@@ -487,8 +465,7 @@ module.exports = {
         /**
          * Reports `static`, `get`, and `set` keywords of a given node if usage of
          * spacing around those keywords is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForProperty(node) {
@@ -528,8 +505,7 @@ module.exports = {
         /**
          * Reports `await` keyword of a given node if usage of spacing before
          * this keyword is invalid.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function checkSpacingForAwaitExpression(node) {

package/lib/rules/lines-around-directive.js

@@ -131,7 +131,7 @@ module.exports = {
 
         /**
          * Check lines around directives in node
-         * @param {ASTNode} node - node to check
+         * @param {ASTNode} node node to check
          * @returns {void}
          */
         function checkDirectives(node) {

package/lib/rules/max-len.js

@@ -198,7 +198,6 @@ module.exports = {
 
         /**
          * Ensure that an array exists at [key] on `object`, and add `value` to it.
-         *
          * @param {Object} object the object to mutate
          * @param {string} key the object's key
          * @param {*} value the value to add
@@ -214,7 +213,6 @@ module.exports = {
 
         /**
          * Retrieves an array containing all strings (" or ') in the source code.
-         *
          * @returns {ASTNode[]} An array of string nodes.
          */
         function getAllStrings() {
@@ -224,7 +222,6 @@ module.exports = {
 
         /**
          * Retrieves an array containing all template literals in the source code.
-         *
          * @returns {ASTNode[]} An array of template literal nodes.
          */
         function getAllTemplateLiterals() {
@@ -234,7 +231,6 @@ module.exports = {
 
         /**
          * Retrieves an array containing all RegExp literals in the source code.
-         *
          * @returns {ASTNode[]} An array of RegExp literal nodes.
          */
         function getAllRegExpLiterals() {
@@ -244,7 +240,6 @@ module.exports = {
 
         /**
          * A reducer to group an AST node by line number, both start and end.
-         *
          * @param {Object} acc the accumulator
          * @param {ASTNode} node the AST node in question
          * @returns {Object} the modified accumulator