test(linter): fix offset for partical source texts

Author: Sysix

Cargo.lock

@@ -291,3 +291,6 @@ overflow-checks = true # Catch arithmetic overflow errors
 [profile.dev-no-debug-assertions]
 inherits = "dev"
 debug-assertions = false
+
+[patch.crates-io]
+oxc-miette = { git = "https://github.com/oxc-project/oxc-miette/", rev = "8522fe33a69115cf4b4ed7d2ee2c2cd8a1511c2b" }

Cargo.toml

@@ -291,6 +291,23 @@ impl<'a> Message<'a> {
         };
         Self { error, span: Span::new(start, end), fixes, fixed: false }
     }
+
+    pub fn with_offset(&mut self, offset: u32) -> &mut Self {
+        if offset == 0 {
+            return self;
+        }
+
+        self.span =
+            Span::new(self.span.start.saturating_add(offset), self.span.end.saturating_add(offset));
+
+        if let Some(labels) = &mut self.error.labels {
+            for label in labels {
+                label.set_span_offset(label.offset().saturating_add(offset as usize));
+            }
+        }
+
+        self
+    }
 }
 
 impl From<Message<'_>> for OxcDiagnostic {

crates/oxc_linter/src/fixer/mod.rs

@@ -762,22 +762,30 @@ impl Runtime {
                             .into_iter()
                             .zip(section_contents.drain(..))
                         {
+                            let lint_result = match record_result {
+                                Ok(module_record) => me.linter.run(
+                                    Path::new(&module.path),
+                                    Rc::new(section.semantic.unwrap()),
+                                    Arc::clone(&module_record),
+                                    section.source.framework_options,
+                                    allocator_guard,
+                                ),
+                                Err(errors) => errors
+                                    .into_iter()
+                                    .map(|err| Message::new(err, PossibleFixes::None))
+                                    .collect(),
+                            };
+
                             messages.lock().unwrap().extend(
-                                match record_result {
-                                    Ok(module_record) => me.linter.run(
-                                        Path::new(&module.path),
-                                        Rc::new(section.semantic.unwrap()),
-                                        Arc::clone(&module_record),
-                                        section.source.framework_options,
-                                        allocator_guard,
-                                    ),
-                                    Err(errors) => errors
-                                        .into_iter()
-                                        .map(|err| Message::new(err, PossibleFixes::None))
-                                        .collect(),
-                                }
+                                lint_result
                                 .into_iter()
-                                .map(|message| message.clone_in(allocator)),
+                                .map(|mut message| {
+                                    // let mut message = message;
+                                    if section.source.start != 0 {
+                                        message.with_offset(section.source.start);
+                                    }
+                                    message.clone_in(allocator)
+                                }),
                             );
                         }
                     },

crates/oxc_linter/src/service/runtime.rs

@@ -2,38 +2,37 @@
 source: crates/oxc_linter/src/tester.rs
 ---
   ⚠ vue(valid-define-emits): `defineEmits` is referencing locally declared variables.
-   ╭─[valid_define_emits.tsx:4:28]
- 3 │                         /* ✗ BAD */
- 4 │ ╭─▶                     const def = { notify: null }
- 5 │ ╰─▶                     defineEmits(def)
- 6 │                       </script>
+   ╭─[valid_define_emits.tsx:5:12]
+ 4 │                     const def = { notify: null }
+ 5 │                     defineEmits(def)
+   ·                     ────────────────
+ 6 │                   </script>
    ╰────
   help: inline the variable or import it from another module.
 
   ⚠ vue(valid-define-emits): `defineEmits` has both a type-only emit and an argument.
-   ╭─[valid_define_emits.tsx:3:3]
- 2 │                       <script setup lang="ts">
- 3 │ ╭─▶                     /* ✗ BAD */
- 4 │ ╰─▶                     defineEmits<(e: 'notify')=>void>({ submit: null })
- 5 │                       </script>
+   ╭─[valid_define_emits.tsx:4:12]
+ 3 │                     /* ✗ BAD */
+ 4 │                     defineEmits<(e: 'notify')=>void>({ submit: null })
+   ·                     ──────────────────────────────────────────────────
+ 5 │                   </script>
    ╰────
   help: remove the argument for better type inference.
 
   ⚠ vue(valid-define-emits): `defineEmits` has been called multiple times.
-   ╭─[valid_define_emits.tsx:4:29]
- 3 │                         /* ✗ BAD */
- 4 │ ╭─▶                     defineEmits({ notify: null })
- 5 │ ╰─▶                     defineEmits({ submit: null })
- 6 │                       </script>
+   ╭─[valid_define_emits.tsx:5:12]
+ 4 │                     defineEmits({ notify: null })
+ 5 │                     defineEmits({ submit: null })
+   ·                     ─────────────────────────────
+ 6 │                   </script>
    ╰────
   help: combine all events into a single `defineEmits` call.
 
   ⚠ vue(valid-define-emits): Custom events are not defined.
-   ╭─[valid_define_emits.tsx:3:13]
- 2 │                   <script setup>
+   ╭─[valid_define_emits.tsx:4:12]
  3 │                     /* ✗ BAD */
-   ·                      ───────────
  4 │                     defineEmits()
+   ·                     ─────────────
  5 │                   </script>
    ╰────
   help: Define at least one event in `defineEmits`.