Report on standalone function calls too

Author: davidmyersdev

lib/rules/no-setup-props-destructure.js

@@ -35,9 +35,8 @@ module.exports = {
      */
     /** @type {Map<FunctionDeclaration | FunctionExpression | ArrowFunctionExpression | Program, ScopePropsReferences>} */
     const setupScopePropsReferenceIds = new Map()
-    const outerExpressionTypes = new Set([
+    const wrapperExpressionTypes = new Set([
       'ArrayExpression',
-      'CallExpression',
       'ObjectExpression'
     ])
 
@@ -68,27 +67,11 @@ module.exports = {
 
       const rightNode = utils.skipChainExpression(right)
 
-      if (outerExpressionTypes.has(rightNode.type)) {
-        const propRefs = [...propsReferences.refs.values()]
-        const isPropsMemberAccessed = propRefs.some((props) => {
-          const isPropsInCallExpression = utils.inRange(rightNode.range, props)
-
-          if (isPropsInCallExpression) {
-            const isPropMemberExpression =
-              props.parent.type === 'MemberExpression' &&
-              props.parent.object === props
-
-            if (isPropMemberExpression) {
-              return true
-            }
-          }
-
-          return false
-        })
-
-        if (isPropsMemberAccessed) {
-          return report(left, 'getProperty', propsReferences.scopeName)
-        }
+      if (
+        wrapperExpressionTypes.has(rightNode.type) &&
+        isPropsMemberAccessed(rightNode, propsReferences)
+      ) {
+        return report(rightNode, 'getProperty', propsReferences.scopeName)
       }
 
       if (
@@ -108,6 +91,34 @@ module.exports = {
         report(left, 'getProperty', propsReferences.scopeName)
       }
     }
+    /**
+     * @param {Expression} node
+     * @param {ScopePropsReferences} propsReferences
+     */
+    function isPropsMemberAccessed(node, propsReferences) {
+      const propRefs = [...propsReferences.refs.values()]
+      const isMemberAccessed = propRefs.some((props) => {
+        const isPropsInExpressionRange = utils.inRange(node.range, props)
+
+        if (isPropsInExpressionRange) {
+          const isPropsMemberExpression =
+            props.parent.type === 'MemberExpression' &&
+            props.parent.object === props
+
+          if (isPropsMemberExpression) {
+            return true
+          }
+        }
+
+        return false
+      })
+
+      if (isMemberAccessed) {
+        return true
+      }
+
+      return false
+    }
     /**
      * @typedef {object} ScopeStack
      * @property {ScopeStack | null} upper
@@ -182,6 +193,26 @@ module.exports = {
 
           setupScopePropsReferenceIds.delete(node)
         },
+        /**
+         * @param {CallExpression} node
+         */
+        CallExpression(node) {
+          if (!scopeStack) {
+            return
+          }
+
+          const propsReferenceIds = setupScopePropsReferenceIds.get(
+            scopeStack.scopeNode
+          )
+
+          if (!propsReferenceIds) {
+            return
+          }
+
+          if (isPropsMemberAccessed(node, propsReferenceIds)) {
+            report(node, 'getProperty', propsReferenceIds.scopeName)
+          }
+        },
         /**
          * @param {VariableDeclarator} node
          */

tests/lib/rules/no-setup-props-destructure.js

@@ -660,6 +660,23 @@ tester.run('no-setup-props-destructure', rule, {
           line: 4
         }
       ]
+    },
+    {
+      filename: 'test.vue',
+      code: `
+      <script setup>
+      const props = defineProps({ count: Number })
+      const buildCounter = (count) => ({ count })
+
+      buildCounter(props.count)
+      </script>
+      `,
+      errors: [
+        {
+          messageId: 'getProperty',
+          line: 6
+        }
+      ]
     }
   ]
 })