From 7998593510e2b0ea858bd691e3c07e5cd0c2445b Mon Sep 17 00:00:00 2001
From: Sysix <sysix@sysix-coding.de>
Date: Sat, 28 Dec 2024 19:38:36 +0100
Subject: [PATCH] fix(linter): disable rule by file extension

---
 crates/oxc_linter/src/rules/react/rules_of_hooks.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/crates/oxc_linter/src/rules/react/rules_of_hooks.rs b/crates/oxc_linter/src/rules/react/rules_of_hooks.rs
index ac670d111b1f3..15422b18f9727 100644
--- a/crates/oxc_linter/src/rules/react/rules_of_hooks.rs
+++ b/crates/oxc_linter/src/rules/react/rules_of_hooks.rs
@@ -16,7 +16,7 @@ use crate::{
     context::LintContext,
     rule::Rule,
     utils::{is_react_component_or_hook_name, is_react_function_call, is_react_hook},
-    AstNode, FrameworkFlags,
+    AstNode,
 };
 
 mod diagnostics {
@@ -110,8 +110,9 @@ declare_oxc_lint!(
 impl Rule for RulesOfHooks {
     fn should_run(&self, ctx: &crate::rules::ContextHost) -> bool {
         // disable this rule in vue/nuxt and svelte(kit) files
-        // top level useFunction are very common
-        !ctx.frameworks().contains(FrameworkFlags::SvelteKit | FrameworkFlags::Nuxt)
+        // react hook can be build in only `.ts` files,
+        // but `useX` functions are popular and can be false positive in other frameworks
+        !ctx.file_path().extension().is_some_and(|ext| ext == "vue" || ext == "svelte")
     }
 
     fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {