Add changelog ahead of 0.15.0

Author: tomhoule

CHANGELOG.md

@@ -5,6 +5,20 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+
+- Support for `deprecated` directive on InputValue fields (#553)
+- Support for custom variable and response types (#536)
+- Allow using `#[derive(GraphQLQuery)]` without depending on `serde` directly (#487)
+- CLI option for extern enums (#520)
+- Support deserializing IDs from integers or strings (#476)
+- Introspection schema now includes `oneOf` and `specifiedByUrl` (#501)
+- Update `reqwest` to 0.12 (#499)
+- Fix required ID deserialization (#523)
+- Fix `skip_serializing_none` for root level variables (#485)
+- Use consistent reference to `graphql_client` crate in codegen (#484)
+- Fix multiple operations example in README (#497)
+
 ## 0.14.0 - 2024-03-26
 
 - Add support for GraphQL’s `extend type` directive

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/query/selection.rs

@@ -35,9 +35,7 @@ pub(super) fn validate_type_conditions(
             let union = query.schema.get_union(union_id);
 
             if !union
-                .variants
-                .iter()
-                .any(|variant| *variant == selected_type)
+                .variants.contains(&selected_type)
             {
                 return Err(QueryValidationError::new(format!(
                     "The spread {}... on {} is not valid.",

graphql_client_codegen/src/schema/json_conversion.rs

@@ -150,7 +150,6 @@ fn ingest_enum(schema: &mut Schema, enm: &mut FullType) {
             std::mem::take(
                 v.name
                     .as_mut()
-                    .take()
                     .expect("variant.name.as_mut().take()"),
             )
         })