typescript-eslint · bradzacher · Mar 15, 2023 · Mar 15, 2023 · Mar 15, 2023
diff --git a/packages/eslint-plugin/src/rules/no-misused-promises.ts b/packages/eslint-plugin/src/rules/no-misused-promises.ts
@@ -365,20 +365,27 @@ export default util.createRule<Options, MessageId>({
    }

    function checkJSXAttribute(node: TSESTree.JSXAttribute): void {
-      const tsNode = parserServices.esTreeNodeToTSNodeMap.get(node);
-      const value = tsNode.initializer;
      if (
        node.value == null ||
-        value === undefined ||
-        !ts.isJsxExpression(value) ||
-        value.expression === undefined
+        node.value.type !== AST_NODE_TYPES.JSXExpressionContainer
      ) {
        return;
      }
-      const contextualType = checker.getContextualType(value);
+      const expressionContainer = parserServices.esTreeNodeToTSNodeMap.get(
+        node.value,
+      );
+      const expression = parserServices.esTreeNodeToTSNodeMap.get(
+        node.value.expression,
+      );
+      const contextualType = checker.getContextualType(expressionContainer);
      if (
        contextualType !== undefined &&
-        isVoidReturningFunctionType(checker, value, contextualType)
+        isVoidReturningFunctionType(
+          checker,
+          expressionContainer,
+          contextualType,
+        ) &&
+        returnsThenable(checker, expression)
      ) {
        context.report({
          messageId: 'voidReturnAttribute',

diff --git a/packages/eslint-plugin/tests/rules/no-misused-promises.test.ts b/packages/eslint-plugin/tests/rules/no-misused-promises.test.ts
@@ -460,6 +460,23 @@ restTuple('Hello');
        };
      }
    `,
+    // https://github.com/typescript-eslint/typescript-eslint/issues/6637
+    {
+      code: `
+        type OnSelectNodeFn = (node: string | null) => void;
+
+        interface ASTViewerBaseProps {
+          readonly onSelectNode?: OnSelectNodeFn;
+        }
+
+        declare function ASTViewer(props: ASTViewerBaseProps): null;
+        declare const onSelectFn: OnSelectNodeFn;
+
+        <ASTViewer onSelectNode={onSelectFn} />;
+      `,
+      filename: 'react.tsx',
+      options: [{ checksVoidReturn: { attributes: true } }],
+    },
  ],

  invalid: [