From c906ac0cd3b9f3efa8a97ab9eb8f7dc71c34365c Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Mon, 27 Oct 2025 23:27:03 +0530
Subject: [PATCH 1/6] feat(linter): generic interface for framework-provided
 globals check

---
 crates/oxc_linter/src/frameworks.rs           | 73 +++++++++++++++++++
 .../oxc_linter/src/rules/eslint/no_undef.rs   |  5 ++
 2 files changed, 78 insertions(+)

diff --git a/crates/oxc_linter/src/frameworks.rs b/crates/oxc_linter/src/frameworks.rs
index 2098994212107..09f82903121ca 100644
--- a/crates/oxc_linter/src/frameworks.rs
+++ b/crates/oxc_linter/src/frameworks.rs
@@ -102,3 +102,76 @@ pub enum FrameworkOptions {
     Default,  // default
     VueSetup, // context is inside `<script setup>`
 }
+
+/// Vue 3 compiler macros available in `<script setup>`
+/// Reference: https://github.com/vuejs/vue-eslint-parser/blob/5ff1a4fda76b07608cc17687a976c2309f5648e2/src/script-setup/scope-analyzer.ts#L86
+static VUE_SETUP_GLOBALS: [&str; 7] = [
+    "defineProps",
+    "defineEmits",
+    "defineExpose",
+    "withDefaults",
+    "defineOptions",
+    "defineSlots",
+    "defineModel",
+];
+
+impl FrameworkOptions {
+    /// Check if a variable is a framework-specific global in this context.
+    ///
+    /// Returns `true` if the variable is a framework global, `false` otherwise.
+    /// Framework globals are always readonly, as they are compiler macros or
+    /// special identifiers provided by the framework.
+    ///
+    /// # Examples
+    /// ```
+    /// use oxc_linter::frameworks::FrameworkOptions;
+    ///
+    /// // In a Vue <script setup> context
+    /// let options = FrameworkOptions::VueSetup;
+    /// assert!(options.has_global("defineProps") == true);
+    /// assert!(options.has_global("defineEmits") == true);
+    /// assert!(options.has_global("console") == false);
+    ///
+    /// // No framework globals by default
+    /// let options = FrameworkOptions::Default;
+    /// assert!(options.has_global("defineProps") == false);
+    /// assert!(options.has_global("console") == false);
+    /// ```
+    pub fn has_global(self, var: &str) -> bool {
+        match self {
+            Self::Default => false,
+            Self::VueSetup => VUE_SETUP_GLOBALS.contains(&var),
+        }
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_vue_setup_globals() {
+        // Test all Vue compiler macros
+        let options = FrameworkOptions::VueSetup;
+        assert!(options.has_global("defineProps"));
+        assert!(options.has_global("defineEmits"));
+        assert!(options.has_global("defineExpose"));
+        assert!(options.has_global("defineOptions"));
+        assert!(options.has_global("defineSlots"));
+        assert!(options.has_global("defineModel"));
+        assert!(options.has_global("withDefaults"));
+
+        // Test that non-Vue globals are not included
+        assert!(!options.has_global("console"));
+        assert!(!options.has_global("window"));
+        assert!(!options.has_global("randomVariable"));
+    }
+
+    #[test]
+    fn test_default_no_globals() {
+        // Default context has no framework globals
+        let options = FrameworkOptions::Default;
+        assert!(!options.has_global("defineProps"));
+        assert!(!options.has_global("console"));
+    }
+}
diff --git a/crates/oxc_linter/src/rules/eslint/no_undef.rs b/crates/oxc_linter/src/rules/eslint/no_undef.rs
index 376f5ff35c11a..8b9773159f9bf 100644
--- a/crates/oxc_linter/src/rules/eslint/no_undef.rs
+++ b/crates/oxc_linter/src/rules/eslint/no_undef.rs
@@ -67,6 +67,11 @@ impl Rule for NoUndef {
                     continue;
                 }
 
+                // Check framework-specific globals (e.g., Vue's defineProps in <script setup>)
+                if ctx.frameworks_options().has_global(name) {
+                    continue;
+                }
+
                 // Skip reporting error for 'arguments' if it's in a function scope
                 if name == "arguments"
                     && ctx

From 21c2f979c4c0add1a2264dee0b654849610c667c Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Tue, 28 Oct 2025 00:10:27 +0530
Subject: [PATCH 2/6] define runes as globals for svelte scripts

---
 crates/oxc_linter/src/frameworks.rs           | 37 ++++++++++++++++++-
 .../src/loader/partial_loader/svelte.rs       | 10 ++++-
 2 files changed, 43 insertions(+), 4 deletions(-)

diff --git a/crates/oxc_linter/src/frameworks.rs b/crates/oxc_linter/src/frameworks.rs
index 09f82903121ca..548cdc7bc6e40 100644
--- a/crates/oxc_linter/src/frameworks.rs
+++ b/crates/oxc_linter/src/frameworks.rs
@@ -99,8 +99,10 @@ pub fn has_jest_imports(module_record: &ModuleRecord) -> bool {
 #[derive(Debug, Clone, Copy, Eq, PartialEq)]
 
 pub enum FrameworkOptions {
-    Default,  // default
-    VueSetup, // context is inside `<script setup>`
+    Default,      // default
+    VueSetup,     // context is inside `<script setup>`
+    SvelteModule, // `<script module>`
+    Svelte,       // `<script>`
 }
 
 /// Vue 3 compiler macros available in `<script setup>`
@@ -115,6 +117,14 @@ static VUE_SETUP_GLOBALS: [&str; 7] = [
     "defineModel",
 ];
 
+/// Svelte runes available in `<script>` context
+/// Reference: https://github.com/sveltejs/svelte/blob/da00abe1162a8e56455e92b79020c4e33290e10e/packages/svelte/src/ambient.d.ts#L23
+static SVELTE_GLOBALS: [&str; 7] =
+    ["$state", "$derived", "$effect", "$props", "$bindable", "$inspect", "$host"];
+
+/// A subset of Svelte runes is available in `<script module>` context.
+static SVELTE_MODULE_GLOBALS: [&str; 4] = ["$state", "$derived", "$effect", "$inspect"];
+
 impl FrameworkOptions {
     /// Check if a variable is a framework-specific global in this context.
     ///
@@ -141,6 +151,8 @@ impl FrameworkOptions {
         match self {
             Self::Default => false,
             Self::VueSetup => VUE_SETUP_GLOBALS.contains(&var),
+            Self::SvelteModule => SVELTE_MODULE_GLOBALS.contains(&var),
+            Self::Svelte => SVELTE_GLOBALS.contains(&var),
         }
     }
 }
@@ -167,6 +179,27 @@ mod tests {
         assert!(!options.has_global("randomVariable"));
     }
 
+    #[test]
+    fn test_svelte_globals() {
+        let options = FrameworkOptions::Svelte;
+        assert!(options.has_global("$state"));
+        assert!(options.has_global("$derived"));
+        assert!(options.has_global("$effect"));
+        assert!(options.has_global("$props"));
+        assert!(options.has_global("$bindable"));
+        assert!(options.has_global("$inspect"));
+        assert!(options.has_global("$host"));
+
+        let globals = FrameworkOptions::SvelteModule;
+        assert!(globals.has_global("$state"));
+        assert!(globals.has_global("$derived"));
+        assert!(globals.has_global("$effect"));
+        assert!(globals.has_global("$inspect"));
+        assert!(!globals.has_global("$props"));
+        assert!(!globals.has_global("$bindable"));
+        assert!(!globals.has_global("$host"));
+    }
+
     #[test]
     fn test_default_no_globals() {
         // Default context has no framework globals
diff --git a/crates/oxc_linter/src/loader/partial_loader/svelte.rs b/crates/oxc_linter/src/loader/partial_loader/svelte.rs
index ea5cc53851327..ba45b44c638aa 100644
--- a/crates/oxc_linter/src/loader/partial_loader/svelte.rs
+++ b/crates/oxc_linter/src/loader/partial_loader/svelte.rs
@@ -2,7 +2,7 @@ use memchr::memmem::Finder;
 
 use oxc_span::SourceType;
 
-use crate::loader::JavaScriptSource;
+use crate::{frameworks::FrameworkOptions, loader::JavaScriptSource};
 
 use super::{SCRIPT_END, SCRIPT_START, find_script_closing_angle};
 
@@ -48,6 +48,7 @@ impl<'a> SveltePartialLoader<'a> {
         // get lang="ts" attribute
         let content = &self.source_text[*pointer..*pointer + offset];
         let is_ts = content.contains("ts");
+        let is_module = content.contains("module");
 
         *pointer += offset + 1;
         let js_start = *pointer;
@@ -62,7 +63,12 @@ impl<'a> SveltePartialLoader<'a> {
 
         // NOTE: loader checked that source_text.len() is less than u32::MAX
         #[expect(clippy::cast_possible_truncation)]
-        Some(JavaScriptSource::partial(source_text, source_type, js_start as u32))
+        Some(JavaScriptSource::partial_with_framework_options(
+            source_text,
+            source_type,
+            if is_module { FrameworkOptions::SvelteModule } else { FrameworkOptions::Svelte },
+            js_start as u32,
+        ))
     }
 }
 

From 5d32ccf6ec76f6f738e6aab5457b4b3329d9a9fc Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Tue, 28 Oct 2025 00:18:24 +0530
Subject: [PATCH 3/6] define `Astro` namespace for astro frontmatter

---
 crates/oxc_linter/src/frameworks.rs             | 17 +++++++++++++----
 .../src/loader/partial_loader/astro.rs          |  9 +++++++--
 2 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/crates/oxc_linter/src/frameworks.rs b/crates/oxc_linter/src/frameworks.rs
index 548cdc7bc6e40..1e2e89cdd0ec1 100644
--- a/crates/oxc_linter/src/frameworks.rs
+++ b/crates/oxc_linter/src/frameworks.rs
@@ -99,10 +99,11 @@ pub fn has_jest_imports(module_record: &ModuleRecord) -> bool {
 #[derive(Debug, Clone, Copy, Eq, PartialEq)]
 
 pub enum FrameworkOptions {
-    Default,      // default
-    VueSetup,     // context is inside `<script setup>`
-    SvelteModule, // `<script module>`
-    Svelte,       // `<script>`
+    Default,          // default
+    VueSetup,         // context is inside `<script setup>`
+    SvelteModule,     // `<script module>`
+    Svelte,           // `<script>`
+    AstroFrontmatter, // within `---`-delimited block
 }
 
 /// Vue 3 compiler macros available in `<script setup>`
@@ -153,6 +154,8 @@ impl FrameworkOptions {
             Self::VueSetup => VUE_SETUP_GLOBALS.contains(&var),
             Self::SvelteModule => SVELTE_MODULE_GLOBALS.contains(&var),
             Self::Svelte => SVELTE_GLOBALS.contains(&var),
+            // All of Astro's utilities are grouped under the `Astro` namespace.
+            Self::AstroFrontmatter => var == "Astro",
         }
     }
 }
@@ -200,6 +203,12 @@ mod tests {
         assert!(!globals.has_global("$host"));
     }
 
+    #[test]
+    fn test_astro_frontmatter_globals() {
+        let options = FrameworkOptions::AstroFrontmatter;
+        assert!(options.has_global("Astro"));
+    }
+
     #[test]
     fn test_default_no_globals() {
         // Default context has no framework globals
diff --git a/crates/oxc_linter/src/loader/partial_loader/astro.rs b/crates/oxc_linter/src/loader/partial_loader/astro.rs
index 7a92cc5b7c373..984451a89ea1f 100644
--- a/crates/oxc_linter/src/loader/partial_loader/astro.rs
+++ b/crates/oxc_linter/src/loader/partial_loader/astro.rs
@@ -2,7 +2,7 @@ use memchr::memmem::Finder;
 
 use oxc_span::{SourceType, Span};
 
-use crate::loader::JavaScriptSource;
+use crate::{frameworks::FrameworkOptions, loader::JavaScriptSource};
 
 use super::{SCRIPT_END, SCRIPT_START};
 
@@ -47,7 +47,12 @@ impl<'a> AstroPartialLoader<'a> {
         // move start to the end of the ASTRO_SPLIT
         let start = start + ASTRO_SPLIT.len() as u32;
         let js_code = Span::new(start, end).source_text(self.source_text);
-        Some(JavaScriptSource::partial(js_code, SourceType::ts(), start))
+        Some(JavaScriptSource::partial_with_framework_options(
+            js_code,
+            SourceType::ts(),
+            FrameworkOptions::AstroFrontmatter,
+            start,
+        ))
     }
 
     /// In .astro files, you can add client-side JavaScript by adding one (or more) `<script>` tags.

From 4239a7e62905a9d045c32d73d8223fde53fc3170 Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Tue, 28 Oct 2025 00:33:33 +0530
Subject: [PATCH 4/6] diagnostics on redeclaring framework globals

---
 crates/oxc_linter/src/rules/eslint/no_redeclare.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/crates/oxc_linter/src/rules/eslint/no_redeclare.rs b/crates/oxc_linter/src/rules/eslint/no_redeclare.rs
index 108c45ad4a9ed..a01e9cf571e59 100644
--- a/crates/oxc_linter/src/rules/eslint/no_redeclare.rs
+++ b/crates/oxc_linter/src/rules/eslint/no_redeclare.rs
@@ -85,7 +85,9 @@ impl Rule for NoRedeclare {
             let name = ctx.scoping().symbol_name(symbol_id);
             let decl_span = ctx.scoping().symbol_span(symbol_id);
             let is_builtin = self.built_in_globals
-                && (GLOBALS["builtin"].contains_key(name) || ctx.globals().is_enabled(name));
+                && (GLOBALS["builtin"].contains_key(name)
+                    || ctx.globals().is_enabled(name)
+                    || ctx.frameworks_options().has_global(name));
 
             if is_builtin {
                 ctx.diagnostic(no_redeclare_as_builtin_in_diagnostic(name, decl_span));

From 7ecde65d2e2aeea5b3f9b78f099497cddc0100c3 Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Tue, 28 Oct 2025 01:00:16 +0530
Subject: [PATCH 5/6] use automatic links in static variable docs

---
 crates/oxc_linter/src/frameworks.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/crates/oxc_linter/src/frameworks.rs b/crates/oxc_linter/src/frameworks.rs
index 1e2e89cdd0ec1..0ba54335ab75c 100644
--- a/crates/oxc_linter/src/frameworks.rs
+++ b/crates/oxc_linter/src/frameworks.rs
@@ -107,7 +107,7 @@ pub enum FrameworkOptions {
 }
 
 /// Vue 3 compiler macros available in `<script setup>`
-/// Reference: https://github.com/vuejs/vue-eslint-parser/blob/5ff1a4fda76b07608cc17687a976c2309f5648e2/src/script-setup/scope-analyzer.ts#L86
+/// Reference: <https://github.com/vuejs/vue-eslint-parser/blob/5ff1a4fda76b07608cc17687a976c2309f5648e2/src/script-setup/scope-analyzer.ts#L86>
 static VUE_SETUP_GLOBALS: [&str; 7] = [
     "defineProps",
     "defineEmits",
@@ -119,7 +119,7 @@ static VUE_SETUP_GLOBALS: [&str; 7] = [
 ];
 
 /// Svelte runes available in `<script>` context
-/// Reference: https://github.com/sveltejs/svelte/blob/da00abe1162a8e56455e92b79020c4e33290e10e/packages/svelte/src/ambient.d.ts#L23
+/// Reference: <https://github.com/sveltejs/svelte/blob/da00abe1162a8e56455e92b79020c4e33290e10e/packages/svelte/src/ambient.d.ts#L23>
 static SVELTE_GLOBALS: [&str; 7] =
     ["$state", "$derived", "$effect", "$props", "$bindable", "$inspect", "$host"];
 

From d74a56d3176efc57cfb385d1a2c896c9f6f9e44f Mon Sep 17 00:00:00 2001
From: Arsh <69170106+lilnasy@users.noreply.github.com>
Date: Tue, 28 Oct 2025 01:07:24 +0530
Subject: [PATCH 6/6] remove private use statement from docs

---
 crates/oxc_linter/src/frameworks.rs | 2 --
 1 file changed, 2 deletions(-)

diff --git a/crates/oxc_linter/src/frameworks.rs b/crates/oxc_linter/src/frameworks.rs
index 0ba54335ab75c..fd57c16e4b010 100644
--- a/crates/oxc_linter/src/frameworks.rs
+++ b/crates/oxc_linter/src/frameworks.rs
@@ -135,8 +135,6 @@ impl FrameworkOptions {
     ///
     /// # Examples
     /// ```
-    /// use oxc_linter::frameworks::FrameworkOptions;
-    ///
     /// // In a Vue <script setup> context
     /// let options = FrameworkOptions::VueSetup;
     /// assert!(options.has_global("defineProps") == true);