- apply transformations only after succesfull operation

src/core/config/transformer/rename_args.rs

@@ -33,41 +33,8 @@ impl Transform for RenameArgs {
             let type_name = location.type_name.as_str();
             let field_name = location.field_name.as_str();
             let new_argument_name = location.new_argument_name.as_str();
-            if config.is_root_operation_type(type_name) {
 
-                // We need to ensure we are doing the changes only if `existing_arg_name` is not query param type.
-                let is_safe_operation = config.types.get(type_name)
-                .and_then(|base_type| base_type.fields.get(field_name))
-                .map(|base_field| {
-                    if let Some(Resolver::Http(http)) = &base_field.resolver {
-                        !http.query.iter().any(|q| &q.key == existing_arg_name)
-                    } else {
-                        true
-                    }
-                })
-                .unwrap_or(false);
-
-                if is_safe_operation {
-                    config.types.values_mut().for_each(|type_| {
-                        type_.fields.values_mut().for_each(|field_| {
-                            if let Some(Resolver::Call(call)) = field_.resolver.as_mut() {
-                                call.steps.iter_mut().for_each(|step| {
-                                    let new_f = field_name.to_owned();
-                                    let is_field_name_matched = step.query.as_ref().eq(&Some(&new_f))
-                                        || step.mutation.as_ref().eq(&Some(&new_f));
-                                    if is_field_name_matched {
-                                        if let Some(arg) = step.args.remove(existing_arg_name) {
-                                            step.args.insert(new_argument_name.to_string(), arg);
-                                        }
-                                    }
-                                })
-                            }
-                        })
-                    });
-                }
-            }
-
-            config
+            let result = config
                 .types
                 .get_mut(type_name)
                 .and_then(|type_| type_.fields.get_mut(field_name))
@@ -140,7 +107,41 @@ impl Transform for RenameArgs {
                             ))
                         }
                     },
-                )
+                );
+
+            if result.is_succeed() && config.is_root_operation_type(type_name) {
+                // We need to ensure we are doing the changes only if `existing_arg_name` is not query param type.
+                let is_safe_operation = config.types.get(type_name)
+                .and_then(|base_type| base_type.fields.get(field_name))
+                .map(|base_field| {
+                    if let Some(Resolver::Http(http)) = &base_field.resolver {
+                        !http.query.iter().any(|q| &q.key == existing_arg_name)
+                    } else {
+                        true
+                    }
+                })
+                .unwrap_or(false);
+
+                if is_safe_operation {
+                    config.types.values_mut().for_each(|type_| {
+                        type_.fields.values_mut().for_each(|field_| {
+                            if let Some(Resolver::Call(call)) = field_.resolver.as_mut() {
+                                call.steps.iter_mut().for_each(|step| {
+                                    let new_f = field_name.to_owned();
+                                    let is_field_name_matched = step.query.as_ref().eq(&Some(&new_f))
+                                        || step.mutation.as_ref().eq(&Some(&new_f));
+                                    if is_field_name_matched {
+                                        if let Some(arg) = step.args.remove(existing_arg_name) {
+                                            step.args.insert(new_argument_name.to_string(), arg);
+                                        }
+                                    }
+                                })
+                            }
+                        })
+                    });
+                }
+            }
+            result
         })
         .map(|_| config)
     }