fix: optional ARIA properties for role="separator" in useAriaPropsForRole (#3856)

Author: anthonyshew

CHANGELOG.md

@@ -133,6 +133,8 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
 - Fix edge case [#3791](https://github.com/biomejs/biome/issues/3791) for rule `noFocusedTests` being used with non-string-like expressions ([#3793](https://github.com/biomejs/biome/pull/3793)). Contributed by @h-a-n-a
 
+- Fix optional ARIA properties for `role="separator"` in `useAriaPropsForRole` ([#3856](https://github.com/biomejs/biome/pull/3856)). Contributed by @anthonyshew
+
 ### Configuration
 
 - Add support for loading configuration from `.editorconfig` files ([#1724](https://github.com/biomejs/biome/issues/1724)).
@@ -362,7 +364,7 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
 - [noConsole](https://biomejs.dev/linter/rules/no-console/) now accepts an option that specifies some allowed calls on `console`. Contributed by @Conaclos
 
-- Add an `ignoreNull` option for [noDoubleEquals](https://biomejs.dev/linter/rules/no-double-equals/). 
+- Add an `ignoreNull` option for [noDoubleEquals](https://biomejs.dev/linter/rules/no-double-equals/).
 
   By default the rule allows loose comparisons against `null`.
   The option `ignoreNull` can be set to `false` for reporting loose comparison against `null`.
@@ -847,7 +849,7 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
 #### New features
 
-- Add [noUnknownPseudoClass](https://biomejs.dev/linter/rules/no-unknown-pseudo-class/). Contributed by  @tunamaguro 
+- Add [noUnknownPseudoClass](https://biomejs.dev/linter/rules/no-unknown-pseudo-class/). Contributed by  @tunamaguro
 
 #### Bug fixes
 

crates/biome_aria/src/roles.rs

@@ -152,11 +152,7 @@ define_role! {
 define_role! {
     /// https://www.w3.org/TR/wai-aria-1.1/#separator
     SeparatorRole {
-        PROPS:  [
-            ("aria-valuemax", true),
-            ("aria-valuemin", true),
-            ("aria-valuenow", true),
-        ],
+        PROPS:  [],
         ROLES: ["structure", "widget"],
         CONCEPTS: &[("hr", &[])],
     }

crates/biome_js_analyze/tests/specs/a11y/useAriaPropsForRole/invalid.jsx

@@ -7,9 +7,6 @@ var a = <span role="spinbutton" aria-valuemin="0" aria-valuemax="100"></span>;
 var a = <span role="slider" aria-valuemin="0"></span>;
 var a = <span role="slider" aria-valuemax="100"></span>;
 var a = <span role="slider" aria-valuemin="0" aria-valuemax="100"></span>;
-var a = <span role="separator" aria-valuemin="0"></span>;
-var a = <span role="separator" aria-valuemax="100"></span>;
-var a = <span role="separator" aria-valuemin="0" aria-valuemax="100"></span>;
 var a = <span role="scrollbar" aria-valuemin="0"></span>;
 var a = <span role="scrollbar" aria-valuemax="100"></span>;
 var a = <span role="scrollbar" aria-valuemin="0" aria-valuemax="100"></span>;
@@ -21,4 +18,4 @@ var a = <span role="combobox"></span>;
 var a = <span role="combobox" aria-expanded="true"></span>;
 var a = <span role="combobox" aria-controls="true"></span>;
 var a = <span role="menuitemcheckbox" ></span>;
-var a = <span role="menuitemradio" ></span>;
+var a = <span role="menuitemradio" ></span>;