feat(graphql_analyze): implement useUniqueInputFieldNames

Author: Netail

.changeset/cold-ravens-show.md

@@ -0,0 +1,13 @@
+---
+"@biomejs/biome": patch
+---
+
+Added the nursery rule [`useUniqueInputFieldNames`](https://biomejs.dev/linter/rules/use-unique-input-field-names/). Require fields within an input object to be unique.
+
+**Invalid:**
+
+```graphql
+query A($x: Int, $x: Int) {
+  field
+}
+```

crates/biome_configuration/src/analyzer/linter/rules.rs

@@ -230,6 +230,7 @@ define_categories! {
     "lint/nursery/useSortedClasses": "https://biomejs.dev/linter/rules/use-sorted-classes",
     "lint/nursery/useSpread": "https://biomejs.dev/linter/rules/no-spread",
     "lint/nursery/useUniqueGraphqlOperationName": "https://biomejs.dev/linter/rules/use-unique-graphql-operation-name",
+    "lint/nursery/useUniqueInputFieldNames": "https://biomejs.dev/linter/rules/use-unique-input-field-names",
     "lint/nursery/useUniqueVariableNames": "https://biomejs.dev/linter/rules/use-unique-variable-names",
     "lint/nursery/useVueConsistentDefinePropsDeclaration": "https://biomejs.dev/linter/rules/use-vue-consistent-define-props-declaration",
     "lint/nursery/useVueDefineMacrosOrder": "https://biomejs.dev/linter/rules/use-vue-define-macros-order",

crates/biome_diagnostics_categories/src/categories.rs

@@ -7,5 +7,6 @@ pub mod no_empty_source;
 pub mod use_consistent_graphql_descriptions;
 pub mod use_deprecated_date;
 pub mod use_unique_graphql_operation_name;
+pub mod use_unique_input_field_names;
 pub mod use_unique_variable_names;
-declare_lint_group! { pub Nursery { name : "nursery" , rules : [self :: no_empty_source :: NoEmptySource , self :: use_consistent_graphql_descriptions :: UseConsistentGraphqlDescriptions , self :: use_deprecated_date :: UseDeprecatedDate , self :: use_unique_graphql_operation_name :: UseUniqueGraphqlOperationName , self :: use_unique_variable_names :: UseUniqueVariableNames ,] } }
+declare_lint_group! { pub Nursery { name : "nursery" , rules : [self :: no_empty_source :: NoEmptySource , self :: use_consistent_graphql_descriptions :: UseConsistentGraphqlDescriptions , self :: use_deprecated_date :: UseDeprecatedDate , self :: use_unique_graphql_operation_name :: UseUniqueGraphqlOperationName , self :: use_unique_input_field_names :: UseUniqueInputFieldNames , self :: use_unique_variable_names :: UseUniqueVariableNames ,] } }

crates/biome_graphql_analyze/src/lint/nursery.rs

@@ -0,0 +1,67 @@
+use biome_analyze::{
+    Ast, Rule, RuleDiagnostic, RuleSource, context::RuleContext, declare_lint_rule,
+};
+use biome_console::markup;
+use biome_graphql_syntax::GraphqlRoot;
+use biome_rowan::AstNode;
+use biome_rule_options::use_unique_input_field_names::UseUniqueInputFieldNamesOptions;
+
+declare_lint_rule! {
+    /// Require fields within an input object to be unique.
+    ///
+    /// A GraphQL input object value is only valid if all supplied fields are uniquely named.
+    ///
+    /// ## Examples
+    ///
+    /// ### Invalid
+    ///
+    /// ```graphql,expect_diagnostic
+    /// query {
+    ///   field(arg: { f1: "value", f1: "value" })
+    /// }
+    /// ```
+    ///
+    /// ### Valid
+    ///
+    /// ```graphql
+    /// query {
+    ///   field(arg: { f1: "value", f2: "value" })
+    /// }
+    /// ```
+    ///
+    pub UseUniqueInputFieldNames {
+        version: "next",
+        name: "useUniqueInputFieldNames",
+        language: "graphql",
+        recommended: false,
+        sources: &[RuleSource::EslintGraphql("unique-input-field-names").same()],
+    }
+}
+
+impl Rule for UseUniqueInputFieldNames {
+    type Query = Ast<GraphqlRoot>;
+    type State = ();
+    type Signals = Option<Self::State>;
+    type Options = UseUniqueInputFieldNamesOptions;
+
+    fn run(ctx: &RuleContext<Self>) -> Self::Signals {
+        let _node = ctx.query();
+        None
+    }
+
+    fn diagnostic(ctx: &RuleContext<Self>, _state: &Self::State) -> Option<RuleDiagnostic> {
+        let span = ctx.query().range();
+        Some(
+            RuleDiagnostic::new(
+                rule_category!(),
+                span,
+                markup! {
+                    "Duplicate input field name."
+                },
+            )
+            .note(markup! {
+                "A GraphQL input object value is only valid if all supplied fields are uniquely named. Make sure to name every input field differently."
+            }),
+        )
+    }
+}

crates/biome_graphql_analyze/src/lint/nursery/use_unique_input_field_names.rs

@@ -0,0 +1,12 @@
+# should generate diagnostics
+query A {
+	field(arg: { f1: "value", f1: "value" })
+}
+
+query B {
+	field(arg: { f1: "value", f1: "value", f1: "value" })
+}
+
+query C {
+	field(arg: { f1: {f2: "value", f2: "value" }})
+}

crates/biome_graphql_analyze/tests/specs/nursery/useUniqueInputFieldNames/invalid.graphql

@@ -0,0 +1,24 @@
+# should not generate diagnostics
+query A {
+	field(arg: { f: true })
+}
+
+query B {
+	field(arg1: { f: true }, arg2: { f: true })
+}
+
+query C {
+	field(arg: { f1: "value", f2: "value", f3: "value" })
+}
+
+query D{
+	field(arg: {
+		deep: {
+			deep: {
+				id: 1
+			}
+			id: 1
+		}
+		id: 1
+	})
+}

crates/biome_graphql_analyze/tests/specs/nursery/useUniqueInputFieldNames/valid.graphql

@@ -384,6 +384,7 @@ pub mod use_trim_start_end;
 pub mod use_unified_type_signatures;
 pub mod use_unique_element_ids;
 pub mod use_unique_graphql_operation_name;
+pub mod use_unique_input_field_names;
 pub mod use_unique_variable_names;
 pub mod use_valid_anchor;
 pub mod use_valid_aria_props;

crates/biome_rule_options/src/lib.rs

@@ -0,0 +1,6 @@
+use biome_deserialize_macros::{Deserializable, Merge};
+use serde::{Deserialize, Serialize};
+#[derive(Default, Clone, Debug, Deserialize, Deserializable, Merge, Eq, PartialEq, Serialize)]
+#[cfg_attr(feature = "schema", derive(schemars::JsonSchema))]
+#[serde(rename_all = "camelCase", deny_unknown_fields, default)]
+pub struct UseUniqueInputFieldNamesOptions {}