fix(no-callback-in-promise): false triggering by callback

eslint-community · Nov 26, 2024 · 1fdca9e · 1fdca9e
1 parent 24fd90a
commit 1fdca9e
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 13 deletions.
diff --git a/__tests__/no-callback-in-promise.js b/__tests__/no-callback-in-promise.js
@@ -46,6 +46,12 @@ ruleTester.run('no-callback-in-promise', rule, {
       code: 'a.then(next).catch(next)',
       options: [{ exceptions: ['next'] }],
     },
+
+    `
+        while (!(step = call(next, iterator)).done) {
+          if (result !== undefined) break;
+        }
+      `,
   ],
 
   invalid: [

diff --git a/rules/no-callback-in-promise.js b/rules/no-callback-in-promise.js
@@ -7,6 +7,7 @@
 
 const { getAncestors } = require('./lib/eslint-compat')
 const getDocsUrl = require('./lib/get-docs-url')
+const hasPromiseCallback = require('./lib/has-promise-callback')
 const isInsidePromise = require('./lib/is-inside-promise')
 const isCallback = require('./lib/is-callback')
 
@@ -67,20 +68,30 @@ module.exports = {
         const options = context.options[0] || {}
         const exceptions = options.exceptions || []
         if (!isCallback(node, exceptions)) {
-          const callingName = node.callee.name || node.callee.property?.name
-          const name =
-            node.arguments && node.arguments[0] && node.arguments[0].name
-          if (
-            !exceptions.includes(name) &&
-            CB_BLACKLIST.includes(name) &&
-            (timeoutsErr || !TIMEOUT_WHITELIST.includes(callingName))
-          ) {
-            context.report({
-              node: node.arguments[0],
-              messageId: 'callback',
-            })
+          if (hasPromiseCallback(node)) {
+            const callingName = node.callee.name || node.callee.property?.name
+            const name = node.arguments?.[0]?.name
+            if (
+              !exceptions.includes(name) &&
+              CB_BLACKLIST.includes(name) &&
+              (timeoutsErr || !TIMEOUT_WHITELIST.includes(callingName))
+            ) {
+              context.report({
+                node: node.arguments[0],
+                messageId: 'callback',
+              })
+            }
+            return
+          }
+          if (!timeoutsErr) {
+            return
+          }
+
+          const name = node.arguments?.[0]?.name
+          if (!name) {
+            // Will be handled elsewhere
+            return
           }
-          return
         }
 
         const ancestors = getAncestors(context, node)