feat(biome_js_analyse): added new option to rule to ignore unused function parameters

vladimir-ivanov · vladimir-ivanov · commit 92011a4b28bb · 2025-07-01T21:03:37.000+01:00
diff --git a/crates/biome_js_analyze/src/lint/correctness/no_unused_function_parameters.rs b/crates/biome_js_analyze/src/lint/correctness/no_unused_function_parameters.rs
@@ -57,7 +57,7 @@ declare_lint_rule! {
     ///
     /// ### `ignoreRestSiblings`
     /// **Since `v2.1.0`**
-    /// 
+    ///
     /// Whether to ignore unused variables from an object destructuring with a spread.
     /// Example: `a` and `b` in `function({ a, b, ...rest }) { return rest;}` should be ignored by this rule when set to false.
     ///
diff --git a/crates/biome_rule_options/src/no_unused_function_parameters.rs b/crates/biome_rule_options/src/no_unused_function_parameters.rs
@@ -1,10 +1,18 @@
 use biome_deserialize_macros::Deserializable;
 use serde::{Deserialize, Serialize};
-#[derive(Default, Clone, Debug, Deserialize, Deserializable, Eq, PartialEq, Serialize)]
+#[derive(Clone, Debug, Deserialize, Deserializable, Eq, PartialEq, Serialize)]
 #[cfg_attr(feature = "schema", derive(schemars::JsonSchema))]
 #[serde(rename_all = "camelCase", deny_unknown_fields, default)]
 pub struct NoUnusedFunctionParametersOptions {
     /// Whether to ignore unused variables from an object destructuring with a spread.
     #[serde(default)]
     pub ignore_rest_siblings: bool,
 }
+
+impl Default for NoUnusedFunctionParametersOptions {
+    fn default() -> Self {
+        Self {
+            ignore_rest_siblings: true,
+        }
+    }
+}
