fix(linter/type-aware): use correct span for disable directives

Author: camc314

apps/oxlint/fixtures/tsgolint/no-unnecessary-condition.ts

@@ -1,3 +1,13 @@
 declare const b1: object;
 declare const b2: boolean;
 export const t1 = b1 && b2;
+
+function checkNumber(x: number) {
+  // eslint-disable-next-line typescript/no-unnecessary-condition
+  if (x != null) {
+    return x;
+  }
+  return 0;
+}
+
+checkNumber(1);

apps/oxlint/src/snapshots/fixtures__tsgolint_--type-aware@oxlint.snap

@@ -165,13 +165,15 @@ working directory: fixtures/tsgolint
  2 | declare const b2: boolean;
  3 | export const t1 = b1 && b2;
    :                   ^^
+ 4 | 
    `----
 
   x typescript-eslint(strict-boolean-expressions): Unexpected object value in conditional. An object is always truthy.
    ,-[no-unnecessary-condition.ts:3:19]
  2 | declare const b2: boolean;
  3 | export const t1 = b1 && b2;
    :                   ^^
+ 4 | 
    `----
 
   x typescript-eslint(no-unnecessary-qualifier): Qualifier is unnecessary since 'B' is in scope.

crates/oxc_linter/src/tsgolint.rs

@@ -1059,7 +1059,10 @@ fn should_skip_diagnostic(
     path: &Path,
     tsgolint_diagnostic: &TsGoLintRuleDiagnostic,
 ) -> bool {
-    let span = tsgolint_diagnostic.span;
+    let span = tsgolint_diagnostic
+        .labeled_ranges
+        .first()
+        .map_or(tsgolint_diagnostic.span, |range| Span::new(range.range.pos, range.range.end));
 
     if let Some(directives) = disable_directives_map.get(path) {
         directives.contains(&tsgolint_diagnostic.rule, span)