fix(linter/prefer-called-once): avoid panic on trailing comma

[Afsoon]

Fix #17733

[codspeed-hq]

Merging this PR will not alter performance

Summary

✅ 4 untouched benchmarks
⏩ 41 skipped benchmarks¹

---

_{Comparing Afsoon:fix-panic-prefer-called-once (c3b9bdb) with main (0e8127e)}

Footnotes

1. 41 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[connorshea]

Thanks! LGTM :) Jk I missed a typo

[connorshea]

I realized this was using source text checks that won't work in certain cases, so I've added a few test cases where this will fail to fix the code correctly.

(Technically I guess it shouldn't delete the code comments, so the tests I've added are arguably wrong, but I feel less strongly about keeping those)

---- rules::vitest::prefer_called_once::test stdout ----

thread 'rules::vitest::prefer_called_once::test' (12306774) panicked at crates/oxc_linter/src/fixer/mod.rs:395:13:
Linter fixer produced invalid syntax.

Input code: 
```
expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledTimes(/* comment (because why not) */1,);
```

Fixed code: 
```
expect(window.HTMLElement.prototype.scrollIntoView).toHaveBeenCalledOnce(/* comment ();
```

Parse errors: [OxcDiagnostic { inner: OxcDiagnosticInner { message: "Unterminated multiline comment", labels: Some([LabeledSpan { label: None, span: SourceSpan { offset: SourceOffset(73), length: 14 }, primary: false }]), help: None, note: None, severity: Error, code: OxcCode { scope: None, number: None }, url: None } }]

I also added some tests to the other rules with similar fixers just to make sure this isn't a problem on those rules, you can ignore those.

[Afsoon]

I will convert this PR to draft, you are right with those assumptions. In this state, it doesn't solve the problem.

[Afsoon]

I change how I handle the fix. Now instead of remove everything, I'm only removing the argument span and comma span. Or if it's okay to remove the comment, I can replace completly the modifiers and args.

[camc314]

there're still bugs here. I'm going to take a look

[camc314]

Thank you!

[camc314]

there're still bugs here. I'm going to take a look

fixed by c3b9bdb

fixes applied