fix: tailcall generation schema for non required fields

generated/.tailcallrc.graphql

@@ -71,7 +71,7 @@ directive @graphQL(
   argument to batch several requests into a single batch request.Make sure you have 
   also specified batch settings to the `@upstream` and to the `@graphQL` operator.
   """
-  batch: Boolean!
+  batch: Boolean
   """
   Enables deduplication of IO operations to enhance performance.This flag prevents 
   duplicate IO requests from being executed concurrently, reducing resource load. Caution: 
@@ -269,7 +269,7 @@ directive @link(
   The source of the link. It can be a URL or a path to a file. If a path is provided, 
   it is relative to the file that imports the link.
   """
-  src: String!
+  src: String
   """
   The type of the link. It can be `Config`, or `Protobuf`.
   """
@@ -728,8 +728,8 @@ input Headers {
 }
 
 input Routes {
-  graphQL: String!
-  status: String!
+  graphQL: String
+  status: String
 }
 
 input ScriptOptions {
@@ -772,7 +772,7 @@ Output the telemetry metrics data to prometheus server
 """
 input PrometheusExporter {
   format: PrometheusFormat
-  path: String!
+  path: String
 }
 
 """
@@ -782,7 +782,7 @@ input StdoutExporter {
   """
   Output to stdout in pretty human-readable format
   """
-  pretty: Boolean!
+  pretty: Boolean
 }
 
 input TelemetryExporter {
@@ -793,7 +793,7 @@ input TelemetryExporter {
 }
 
 input Batch {
-  delay: Int!
+  delay: Int
   headers: [String!]
   maxSize: Int
 }
@@ -816,7 +816,7 @@ input GraphQL {
   argument to batch several requests into a single batch request.Make sure you have 
   also specified batch settings to the `@upstream` and to the `@graphQL` operator.
   """
-  batch: Boolean!
+  batch: Boolean
   """
   Enables deduplication of IO operations to enhance performance.This flag prevents 
   duplicate IO requests from being executed concurrently, reducing resource load. Caution: 

generated/.tailcallrc.schema.json

@@ -522,10 +522,7 @@
         },
         "dedupe": {
           "description": "Enables deduplication of IO operations to enhance performance.\n\nThis flag prevents duplicate IO requests from being executed concurrently, reducing resource load. Caution: May lead to issues with APIs that expect unique results for identical inputs, such as nonce-based APIs.",
-          "type": [
-            "boolean",
-            "null"
-          ]
+          "type": "boolean"
         },
         "headers": {
           "description": "The headers parameter allows you to customize the headers of the GraphQL request made by the `@graphQL` operator. It is used by specifying a key-value map of header names and their values.",

src/core/blueprint/operators/graphql.rs

@@ -84,7 +84,7 @@ pub fn compile_graphql(
         .map(|req_template| {
             let field_name = graphql.name.clone();
             let batch = graphql.batch;
-            let dedupe = graphql.dedupe.unwrap_or_default();
+            let dedupe = graphql.dedupe;
             IR::IO(IO::GraphQL { req_template, field_name, batch, dl_id: None, dedupe })
         })
 }

src/core/config/directives/graphql.rs

@@ -55,5 +55,5 @@ pub struct GraphQL {
     /// concurrently, reducing resource load. Caution: May lead to issues
     /// with APIs that expect unique results for identical inputs, such as
     /// nonce-based APIs.
-    pub dedupe: Option<bool>,
+    pub dedupe: bool,
 }

tailcall-typedefs-common/src/directive_definition.rs

@@ -80,7 +80,7 @@ pub fn into_directive_definition(
         TypeSystemDefinition::Directive(pos(async_graphql::parser::types::DirectiveDefinition {
             description: description.map(|inner| pos(inner.clone())),
             name: pos(Name::new(name)),
-            arguments: into_input_value_definition(&schema),
+            arguments: into_input_value_definition(schema),
             is_repeatable: attrs.repeatable,
             locations: attrs
                 .locations

tailcall-typedefs-common/src/input_definition.rs

@@ -14,27 +14,27 @@
 }
 
 pub fn into_input_definition(schema: SchemaObject, name: &str) -> TypeSystemDefinition {
-    let description = get_description(&schema);
+    let description = get_description(&schema).cloned();
 
     TypeSystemDefinition::Type(pos(TypeDefinition {
         name: pos(Name::new(name)),
         kind: TypeKind::InputObject(InputObjectType {
-            fields: into_input_value_definition(&schema),
+            fields: into_input_value_definition(schema),
         }),
-        description: description.map(|inner| pos(inner.clone())),
+        description: description.map(pos),
         directives: vec![],
         extend: false,
     }))
 }
 
-pub fn into_input_value_definition(schema: &SchemaObject) -> Vec<Positioned<InputValueDefinition>> {
+pub fn into_input_value_definition(schema: SchemaObject) -> Vec<Positioned<InputValueDefinition>> {
     let mut arguments_type = vec![];
     if let Some(subschema) = schema.subschemas.clone() {
         let list = subschema.any_of.or(subschema.all_of).or(subschema.one_of);
         if let Some(list) = list {
             for schema in list {
                 let schema_object = schema.into_object();
-                arguments_type.extend(build_arguments_type(&schema_object));
+                arguments_type.extend(build_arguments_type(schema_object));
             }
 
             return arguments_type;
@@ -44,22 +44,19 @@
     build_arguments_type(schema)
 }
 
-fn build_arguments_type(schema: &SchemaObject) -> Vec<Positioned<InputValueDefinition>> {
+fn build_arguments_type(schema: SchemaObject) -> Vec<Positioned<InputValueDefinition>> {
     let mut arguments = vec![];
-    if let Some(properties) = schema
-        .object
-        .as_ref()
-        .map(|object| object.properties.clone())
-    {
-        for (name, property) in properties.into_iter() {
+    if let Some(obj) = schema.object {
+        let required = obj.required;
+        for (name, property) in obj.properties.into_iter() {
             let property = property.into_object();
             let description = get_description(&property);
+            let nullable = !required.contains(&name);
             let definition = pos(InputValueDefinition {
                 description: description.map(|inner| pos(inner.to_owned())),
                 name: pos(Name::new(&name)),
                 ty: pos(determine_input_value_type_from_schema(
-                    name,
-                    property.clone(),
+                    name, &property, nullable,
                 )),
                 default_value: None,
                 directives: Vec::new(),
@@ -72,7 +69,11 @@
     arguments
 }
 
-fn determine_input_value_type_from_schema(mut name: String, schema: SchemaObject) -> Type {
+fn determine_input_value_type_from_schema(
+    mut name: String,
+    schema: &SchemaObject,
+    nullable: bool,
+) -> Type {
     first_char_to_upper(&mut name);
     if let Some(instance_type) = &schema.instance_type {
         match instance_type {
@@ -82,25 +83,25 @@
                 | InstanceType::Number
                 | InstanceType::String
                 | InstanceType::Integer => Type {
-                    nullable: false,
+                    nullable,
                     base: BaseType::Named(Name::new(get_instance_type_name(typ))),
                 },
-                _ => determine_type_from_schema(name, &schema),
+                _ => determine_type_from_schema(name, schema),
             },
             SingleOrVec::Vec(typ) => match typ.first().unwrap() {
                 InstanceType::Null
                 | InstanceType::Boolean
                 | InstanceType::Number
                 | InstanceType::String
                 | InstanceType::Integer => Type {
-                    nullable: true,
+                    nullable,
                     base: BaseType::Named(Name::new(get_instance_type_name(typ.first().unwrap()))),
                 },
-                _ => determine_type_from_schema(name, &schema),
+                _ => determine_type_from_schema(name, schema),
             },
         }
     } else {
-        determine_type_from_schema(name, &schema)
+        determine_type_from_schema(name, schema)
     }
 }
 
@@ -145,14 +146,16 @@
                 nullable: true,
                 base: BaseType::List(Box::new(determine_input_value_type_from_schema(
                     name,
-                    schema.clone().into_object(),
+                    &schema.clone().into_object(),
+                    false,
                 ))),
             },
             SingleOrVec::Vec(schemas) => Type {
                 nullable: true,
                 base: BaseType::List(Box::new(determine_input_value_type_from_schema(
                     name,
-                    schemas[0].clone().into_object(),
+                    &schemas[0].clone().into_object(),
+                    false,
                 ))),
             },
         }