Update: add 2nd parameter to getPropertyName

Author: mysticatea

docs/api/ast-utils.md

@@ -231,15 +231,19 @@ module.exports = {
 
 ```js
 const name = utils.getPropertyName(node)
+const name = utils.getPropertyName(node, initialScope)
 ```
 
 Get the property name of a given property node.
 
+If the node is a computed property, this tries to compute the property name by the [getStringIfConstant](#getstringifconstant) function.
+
 ### Parameters
 
  Name | Type | Description
 :-----|:-----|:------------
 node | Node | The node to get that name. This shuld be any of `MemberExpression`, `Property`, and `MethodDefinition` node.
+initialScope | Scope or undefined | Optional. The scope object to find variables.
 
 ### Return value
 

src/get-property-name.js

@@ -3,20 +3,21 @@ import { getStringIfConstant } from "./get-string-if-constant"
 /**
  * Get the property name from a MemberExpression node or a Property node.
  * @param {Node} node The node to get.
+ * @param {Scope} [initialScope] The scope to start finding variable. Optional. If the node is a computed property node and this scope was given, this checks the computed property name by the `getStringIfConstant` function with the scope, and returns the value of it.
  * @returns {string|null} The property name of the node.
  */
-export function getPropertyName(node) {
+export function getPropertyName(node, initialScope) {
     switch (node.type) {
         case "MemberExpression":
             if (node.computed) {
-                return getStringIfConstant(node.property)
+                return getStringIfConstant(node.property, initialScope)
             }
             return node.property.name
 
         case "Property":
         case "MethodDefinition":
             if (node.computed) {
-                return getStringIfConstant(node.key)
+                return getStringIfConstant(node.key, initialScope)
             }
             if (node.key.type === "Literal") {
                 return String(node.key.value)