fix(langage_server): correct position for "ignore this rule for this file"

Author: Sysix

crates/oxc_language_server/src/snapshots/fixtures_linter_astro@debugger.astro.snap

@@ -120,11 +120,11 @@ fixed: Multiple(
             code: "// oxlint-disable no-debugger\n",
             range: Range {
                 start: Position {
-                    line: 0,
+                    line: 9,
                     character: 0,
                 },
                 end: Position {
-                    line: 0,
+                    line: 9,
                     character: 0,
                 },
             },
@@ -184,11 +184,11 @@ fixed: Multiple(
             code: "// oxlint-disable no-debugger\n",
             range: Range {
                 start: Position {
-                    line: 0,
+                    line: 13,
                     character: 0,
                 },
                 end: Position {
-                    line: 0,
+                    line: 13,
                     character: 0,
                 },
             },
@@ -248,11 +248,11 @@ fixed: Multiple(
             code: "// oxlint-disable no-debugger\n",
             range: Range {
                 start: Position {
-                    line: 0,
+                    line: 17,
                     character: 0,
                 },
                 end: Position {
-                    line: 0,
+                    line: 17,
                     character: 0,
                 },
             },

crates/oxc_language_server/src/snapshots/fixtures_linter_vue@debugger.vue.snap

@@ -56,11 +56,11 @@ fixed: Multiple(
             code: "// oxlint-disable no-debugger\n",
             range: Range {
                 start: Position {
-                    line: 0,
+                    line: 4,
                     character: 0,
                 },
                 end: Position {
-                    line: 0,
+                    line: 4,
                     character: 0,
                 },
             },
@@ -120,11 +120,11 @@ fixed: Multiple(
             code: "// oxlint-disable no-debugger\n",
             range: Range {
                 start: Position {
-                    line: 0,
+                    line: 8,
                     character: 0,
                 },
                 end: Position {
-                    line: 0,
+                    line: 8,
                     character: 0,
                 },
             },

crates/oxc_linter/src/context/host.rs

@@ -174,7 +174,7 @@ impl<'a> ContextHost<'a> {
     }
 
     /// The current [`ContextSubHost`]
-    fn current_sub_host(&self) -> &ContextSubHost<'a> {
+    pub fn current_sub_host(&self) -> &ContextSubHost<'a> {
         &self.sub_hosts[self.current_sub_host_index.get()]
     }
 

crates/oxc_linter/src/context/mod.rs

@@ -247,7 +247,14 @@ impl<'a> LintContext<'a> {
     /// Use [`LintContext::diagnostic_with_fix`] to provide an automatic fix.
     #[inline]
     pub fn diagnostic(&self, diagnostic: OxcDiagnostic) {
+        #[cfg(not(feature = "language_server"))]
         self.add_diagnostic(Message::new(diagnostic, PossibleFixes::None));
+
+        #[cfg(feature = "language_server")]
+        self.add_diagnostic(
+            Message::new(diagnostic, PossibleFixes::None)
+                .with_section_offset(self.parent.current_sub_host().source_text_offset),
+        );
     }
 
     /// Report a lint rule violation and provide an automatic fix.
@@ -358,7 +365,14 @@ impl<'a> LintContext<'a> {
     {
         let (diagnostic, fix) = self.create_fix(fix_kind, fix, diagnostic);
         if let Some(fix) = fix {
+            #[cfg(not(feature = "language_server"))]
             self.add_diagnostic(Message::new(diagnostic, PossibleFixes::Single(fix)));
+
+            #[cfg(feature = "language_server")]
+            self.add_diagnostic(
+                Message::new(diagnostic, PossibleFixes::Single(fix))
+                    .with_section_offset(self.parent.current_sub_host().source_text_offset),
+            );
         } else {
             self.diagnostic(diagnostic);
         }
@@ -387,7 +401,14 @@ impl<'a> LintContext<'a> {
         if fixes_result.is_empty() {
             self.diagnostic(diagnostic);
         } else {
+            #[cfg(not(feature = "language_server"))]
             self.add_diagnostic(Message::new(diagnostic, PossibleFixes::Multiple(fixes_result)));
+
+            #[cfg(feature = "language_server")]
+            self.add_diagnostic(
+                Message::new(diagnostic, PossibleFixes::Multiple(fixes_result))
+                    .with_section_offset(self.parent.current_sub_host().source_text_offset),
+            );
         }
     }
 

crates/oxc_linter/src/fixer/mod.rs

@@ -224,6 +224,8 @@ pub struct Message<'a> {
     pub fixes: PossibleFixes<'a>,
     span: Span,
     fixed: bool,
+    #[cfg(feature = "language_server")]
+    pub section_offset: u32,
 }
 
 impl<'new> CloneIn<'new> for Message<'_> {
@@ -235,6 +237,8 @@ impl<'new> CloneIn<'new> for Message<'_> {
             fixes: self.fixes.clone_in(allocator),
             span: self.span,
             fixed: self.fixed,
+            #[cfg(feature = "language_server")]
+            section_offset: self.section_offset,
         }
     }
 }
@@ -249,7 +253,20 @@ impl<'a> Message<'a> {
             .map(|span| Span::new(span.offset() as u32, (span.offset() + span.len()) as u32))
             .unwrap_or_default();
 
-        Self { error, span, fixes, fixed: false }
+        Self {
+            error,
+            span,
+            fixes,
+            fixed: false,
+            #[cfg(feature = "language_server")]
+            section_offset: 0,
+        }
+    }
+
+    #[cfg(feature = "language_server")]
+    pub fn with_section_offset(mut self, section_offset: u32) -> Self {
+        self.section_offset = section_offset;
+        self
     }
 
     /// move the offset of all spans to the right

crates/oxc_linter/src/lsp.rs

@@ -109,8 +109,7 @@ pub fn message_to_message_with_position<'a>(
 ) -> MessageWithPosition<'a> {
     let code = message.error.code.clone();
     let error_offset = message.span().start;
-    // TODO: feature flag for knowing the source text section start offset of this message
-    let section_offset = 0;
+    let section_offset = message.section_offset;
 
     let mut result = oxc_diagnostic_to_message_with_position(message.error, source_text, rope);
     let fixes = match &message.fixes {