fix: ci clippy and fmt

Author: MedHeikelBouzayene

graphql_client_cli/src/generate.rs

@@ -93,11 +93,13 @@ pub(crate) fn generate_code(params: CliCodegenParams) -> CliResult<()> {
 
         options.set_custom_scalars_module(custom_scalars_module);
     }
-    
+
     if let Some(custom_variable_types) = custom_variable_types {
-        options.set_custom_variable_types(custom_variable_types.split(",").map(String::from).collect());
+        options.set_custom_variable_types(
+            custom_variable_types.split(",").map(String::from).collect(),
+        );
     }
-    
+
     if let Some(custom_response_type) = custom_response_type {
         options.set_custom_response_type(custom_response_type);
     }

graphql_client_cli/src/main.rs

@@ -139,8 +139,8 @@ fn main() -> CliResult<()> {
             selected_operation,
             custom_scalars_module,
             fragments_other_variant,
-            external_enums, 
-            custom_variable_types, 
+            external_enums,
+            custom_variable_types,
             custom_response_type,
         } => generate::generate_code(generate::CliCodegenParams {
             query_path,

graphql_client_codegen/src/codegen/inputs.rs

@@ -19,11 +19,13 @@ pub(super) fn generate_input_object_definitions(
     all_used_types
         .inputs(query.schema)
         .map(|(input_id, input)| {
-            let custom_variable_type = query.query.variables.iter()
+            let custom_variable_type = query
+                .query
+                .variables
+                .iter()
                 .enumerate()
-                .find(|(_, v) | v.r#type.id.as_input_id().is_some_and(|i| i == input_id))
-                .map(|(index, _)| custom_variable_types.get(index))
-                .flatten();
+                .find(|(_, v)| v.r#type.id.as_input_id().is_some_and(|i| i == input_id))
+                .and_then(|(index, _)| custom_variable_types.get(index));
             if let Some(custom_type) = custom_variable_type {
                 generate_type_def(input, options, custom_type)
             } else if input.is_one_of {
@@ -38,7 +40,7 @@ pub(super) fn generate_input_object_definitions(
 fn generate_type_def(
     input: &StoredInputType,
     options: &GraphQLClientCodegenOptions,
-    custom_type: &String,
+    custom_type: &str,
 ) -> TokenStream {
     let custom_type = syn::parse_str::<syn::Path>(custom_type).unwrap();
     let normalized_name = options.normalization().input_name(input.name.as_str());

graphql_client_codegen/src/codegen/selection.rs

@@ -12,8 +12,7 @@ use crate::{
     },
     schema::{Schema, TypeId},
     type_qualifiers::GraphqlTypeQualifier,
-    GraphQLClientCodegenOptions,
-    GeneralError,
+    GeneralError, GraphQLClientCodegenOptions,
 };
 use heck::*;
 use proc_macro2::{Ident, Span, TokenStream};
@@ -43,12 +42,27 @@ pub(crate) fn render_response_data_fields<'a>(
     if let Some(custom_response_type) = options.custom_response_type() {
         if operation.selection_set.len() == 1 {
             let selection_id = operation.selection_set[0];
-            let selection_field = query.query.get_selection(selection_id).as_selected_field()
-                .ok_or_else(|| GeneralError(format!("Custom response type {custom_response_type} will only work on fields")))?;
-            calculate_custom_response_type_selection(&mut expanded_selection, response_data_type_id, custom_response_type, selection_id, selection_field);
+            let selection_field = query
+                .query
+                .get_selection(selection_id)
+                .as_selected_field()
+                .ok_or_else(|| {
+                    GeneralError(format!(
+                        "Custom response type {custom_response_type} will only work on fields"
+                    ))
+                })?;
+            calculate_custom_response_type_selection(
+                &mut expanded_selection,
+                response_data_type_id,
+                custom_response_type,
+                selection_id,
+                selection_field,
+            );
             return Ok(expanded_selection);
         } else {
-            return Err(GeneralError(format!("Custom response type {custom_response_type} requires single selection field")));
+            return Err(GeneralError(format!(
+                "Custom response type {custom_response_type} requires single selection field"
+            )));
         }
     }
 
@@ -66,10 +80,10 @@ pub(crate) fn render_response_data_fields<'a>(
 fn calculate_custom_response_type_selection<'a>(
     context: &mut ExpandedSelection<'a>,
     struct_id: ResponseTypeId,
-    custom_response_type: &'a String,
+    custom_response_type: &'a str,
     selection_id: SelectionId,
-    field: &'a SelectedField)
-{
+    field: &'a SelectedField,
+) {
     let (graphql_name, rust_name) = context.field_name(field);
     let struct_name_string = full_path_prefix(selection_id, context.query);
     let field = context.query.schema.get_field(field.field_id);
@@ -88,7 +102,7 @@ fn calculate_custom_response_type_selection<'a>(
         name: struct_name_string.into(),
     });
     context.push_type_alias(TypeAlias {
-        name: custom_response_type.as_str(),
+        name: custom_response_type,
         struct_id,
         boxed: false,
     });

graphql_client_codegen/src/tests/mod.rs

@@ -62,7 +62,8 @@ fn blended_custom_types_works() {
     match r {
         Ok(_) => {
             // Variables and returns should be replaced with custom types
-            assert!(generated_code.contains("pub type SearchQuerySearch = external_crate :: Transaction"));
+            assert!(generated_code
+                .contains("pub type SearchQuerySearch = external_crate :: Transaction"));
             assert!(generated_code.contains("pub type extern_ = external_crate :: ID"));
         }
         Err(e) => {

graphql_query_derive/src/lib.rs

@@ -106,7 +106,7 @@ fn build_graphql_client_derive_options(
     if let Some(custom_variable_types) = custom_variable_types {
         options.set_custom_variable_types(custom_variable_types);
     }
-    
+
     if let Some(custom_response_type) = custom_response_type {
         options.set_custom_response_type(custom_response_type);
     }