fix(linter/promise/prefer-await-to-callbacks): false positive for addEventListener (#12537)

Fixes #12530

The `promise/prefer-await-to-callbacks` rule was incorrectly flagging
DOM event handlers using `addEventListener` as callback patterns that
should use async/await. This resulted in false positives for legitimate
event handling code.

## Problem

The rule was triggering on code like this:

```js
socket.addEventListener('error', (error) => {
  // snipped
})
```

This is not a callback-style async operation but rather an event
handler, which should not be converted to async/await.

## Solution

Extended the existing exemption logic to include `addEventListener` and
`removeEventListener` methods alongside the already-exempted `on` and
`once` methods. These are all event handler registration methods that
should not be subject to the callback-to-async/await conversion rule.

## Changes

- Added `addEventListener` and `removeEventListener` to the list of
exempted method names
- Added comprehensive test cases covering various event handler
scenarios
- Verified that the rule still correctly flags actual callback patterns

## Testing

The fix ensures that:
- ✅ WebSocket `addEventListener` calls no longer trigger false positives
- ✅ DOM element `addEventListener`/`removeEventListener` calls are
exempted
- ✅ Existing `on`/`once` event handler exemptions continue to work
- ✅ Actual callback patterns are still correctly detected and flagged
- ✅ All existing tests continue to pass

<!-- START COPILOT CODING AGENT TIPS -->
---

💬 Share your feedback on Copilot coding agent for the chance to win a
$200 gift card! Click
[here](https://survey.alchemer.com/s3/8343779/Copilot-Coding-agent) to
start the survey.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Dunqing <29533304+Dunqing@users.noreply.github.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>

Author: 3 people

crates/oxc_linter/src/rules/promise/prefer_await_to_callbacks.rs

@@ -77,7 +77,10 @@ impl Rule for PreferAwaitToCallbacks {
                         .as_member_expression()
                         .and_then(MemberExpression::static_property_name);
 
-                    if matches!(callee_property_name, Some("on" | "once")) {
+                    if matches!(
+                        callee_property_name,
+                        Some("on" | "once" | "addEventListener" | "removeEventListener")
+                    ) {
                         return;
                     }
 
@@ -165,6 +168,10 @@ fn test() {
         r#"map(errors, function(error) { return  err.type === "CoolError" })"#,
         r#"_.find(errors, function(error) { return  err.type === "CoolError" })"#,
         r#"_.map(errors, function(err) { return  err.type === "CoolError" })"#,
+        r#"socket.addEventListener("error", err => { console.error(err) })"#,
+        r#"socket.addEventListener("message", error => { console.log(error) })"#,
+        r#"element.removeEventListener("click", error => { console.log(error) })"#,
+        r#"ws.addEventListener("close", (error) => { console.log(error) })"#,
     ];
 
     let fail = vec![