Update crates/oxc_linter/src/rules/react/jsx_handler_names.rs

takuji · Copilot · web-flow · commit d4a9191f53d2 · 2025-08-16T00:43:55.000+09:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/crates/oxc_linter/src/rules/react/jsx_handler_names.rs b/crates/oxc_linter/src/rules/react/jsx_handler_names.rs
@@ -369,7 +369,8 @@ fn get_member_expression_name(member_expr: &JSXMemberExpression) -> CompactStr {
 
 fn normalize_handler_name(s: &str) -> CompactStr {
     let s1 = regex!(r"\s*").replace_all(s, "");
-    regex!(r"^this\.|\S*::").replace(s1.as_ref(), "").into()
+    // Combine whitespace removal and prefix stripping into a single regex operation.
+    regex!(r"\s+|^this\.|\S*::").replace_all(s, "").into()
 }
 
 // Tests for the normalize_handler_name function to ensure it correctly strips prefixes and whitespace.

Original file line number	Diff line number	Diff line change
`@@ -369,7 +369,8 @@ fn get_member_expression_name(member_expr: &JSXMemberExpression) -> CompactStr {`
`369`	`369`
`370`	`370`	`fn normalize_handler_name(s: &str) -> CompactStr {`
`371`	`371`	`let s1 = regex!(r"\s*").replace_all(s, "");`
`372`		`- regex!(r"^this\.\|\S*::").replace(s1.as_ref(), "").into()`
	`372`	`+ // Combine whitespace removal and prefix stripping into a single regex operation.`
	`373`	`+ regex!(r"\s+\|^this\.\|\S*::").replace_all(s, "").into()`
`373`	`374`	`}`
`374`	`375`
`375`	`376`	`// Tests for the normalize_handler_name function to ensure it correctly strips prefixes and whitespace.`