Remove GraphQLID annotation for ID class (#700)

* Add ID wrapper class

Add a simple wrapper class that will allow us to support List<ID>

Fixes #698

* Remove @GraphQLID annotation

* Add more unit tests for coverage

* Update docs/writing-schemas/scalars.md

Co-authored-by: Dariusz Kuc <[email protected]>

* Update docs to match example code

* Update docs language

Co-authored-by: Shane Myrick <[email protected]>
Co-authored-by: Dariusz Kuc <[email protected]>

Author: 3 people

docs/writing-schemas/annotations.md

@@ -6,10 +6,8 @@ title: Annotations
 `graphql-kotlin-schema-generator` ships with a number of annotation classes to allow you to enhance your GraphQL schema
 for things that can't be directly derived from Kotlin reflection.
 
-* [@GraphQLContext](../execution/contextual-data) - Autowire `GraphQLContext` from the environment
 * [@GraphQLDescription](../customizing-schemas/documenting-fields) - Provide a description for a GraphQL field
 * [@GraphQLDirective](../customizing-schemas/directives) - Registers directive on a GraphQL field
-* [@GraphQLID](scalars#id) - Marks given field as GraphQL `ID`
 * [@GraphQLIgnore](../customizing-schemas/excluding-fields) - Exclude field from the GraphQL schema
 * [@GraphQLName](../customizing-schemas/renaming-fields) - Override the name used for the type
 * Kotlin built in [@Deprecated](../customizing-schemas/deprecating-schema) - Apply the GraphQL `@deprecated` directive on the field

docs/writing-schemas/scalars.md

@@ -24,40 +24,39 @@ extended scalar types provided by `graphql-java`.
 
 ## ID
 
-GraphQL supports a the scalar type ID, a unique identifier that is not intended to be human readable. ID's are
-serialized as a String. To mark given field as an ID field you have to annotate it with `@GraphQLID` annotation.
+GraphQL supports the scalar type `ID`, a unique identifier that is not intended to be human readable. IDs are
+serialized as a `String`. To expose a GraphQL `ID` field, you must use the `com.expediagroup.graphql.types.ID` class, which wraps the underlying `String` value.
+
 
 > NOTE: `graphql-java` supports additional types (`String`, `Int`, `Long`, or `UUID`) but [due to serialization issues](https://github.com/ExpediaGroup/graphql-kotlin/issues/317) we can only directly support Strings. You can still use a type like UUID internally just as long as you convert or parse the value yourself and handle the errors.
 
 ```kotlin
 data class Person(
-  @GraphQLID
-  val id: String,
-  val name: String
+    val id: ID,
+    val name: String
 )
 
-fun locatePerson(@GraphQLID id: String) = Person(id, "Jane Doe")
+fun findPersonById(id: ID) = Person(id, "John Smith")
 
-@GraphQLID
-fun generateRandomId() = UUID.randomUUID().toString()
+fun generateRandomId(): ID = ID(UUID.randomUUID().toString())
 ```
 
 This would produce the following schema:
 
 ```graphql
 schema {
-  query: Query
+    query: Query
 }
 
 
 type Query {
-  locatePerson(id: ID!): Person!
-  generateRandomId: ID!
+    findPersonById(id: ID!): Person!
+    generateRandomId: ID!
 }
 
 type Person {
-  id: ID!
-  name: String!
+    id: ID!
+    name: String!
 }
 ```
 

examples/spring/src/main/kotlin/com/expediagroup/graphql/examples/query/ScalarQuery.kt

@@ -17,9 +17,9 @@
 package com.expediagroup.graphql.examples.query
 
 import com.expediagroup.graphql.annotations.GraphQLDescription
-import com.expediagroup.graphql.annotations.GraphQLID
 import com.expediagroup.graphql.spring.operations.Mutation
 import com.expediagroup.graphql.spring.operations.Query
+import com.expediagroup.graphql.types.ID
 import org.springframework.stereotype.Component
 import java.util.UUID
 
@@ -35,11 +35,10 @@ class ScalarQuery : Query {
     @GraphQLDescription("Prints a string with a custom scalar as input")
     fun printUuids(uuids: List<UUID>) = "You sent $uuids"
 
-    fun findPersonById(@GraphQLID id: String) = Person(id, "Nelson")
+    fun findPersonById(id: ID) = Person(id, "Nelson")
 
     @GraphQLDescription("generates random GraphQL ID")
-    @GraphQLID
-    fun generateRandomId() = UUID.randomUUID().toString()
+    fun generateRandomId() = ID(UUID.randomUUID().toString())
 }
 
 @Component
@@ -48,7 +47,6 @@ class ScalarMutation : Mutation {
 }
 
 data class Person(
-    @GraphQLID
-    val id: String,
+    val id: ID,
     val name: String
 )

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/annotations/GraphQLID.kt

@@ -17,7 +17,6 @@
 package com.expediagroup.graphql.generator.extensions
 
 import com.expediagroup.graphql.annotations.GraphQLDescription
-import com.expediagroup.graphql.annotations.GraphQLID
 import com.expediagroup.graphql.annotations.GraphQLIgnore
 import com.expediagroup.graphql.annotations.GraphQLName
 import kotlin.reflect.KAnnotatedElement
@@ -31,8 +30,6 @@ internal fun KAnnotatedElement.getDeprecationReason(): String? = this.findAnnota
 
 internal fun KAnnotatedElement.isGraphQLIgnored(): Boolean = this.findAnnotation<GraphQLIgnore>() != null
 
-internal fun KAnnotatedElement.isGraphQLID(): Boolean = this.findAnnotation<GraphQLID>() != null
-
 internal fun Deprecated.getReason(): String? {
     val builder = StringBuilder()
     builder.append(this.message)

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/generator/extensions/annotationExtensions.kt

@@ -24,12 +24,6 @@ import kotlin.reflect.KProperty
  * For properties we need to check both the property for annotations and the constructor argument
  */
 
-internal fun KProperty<*>.isPropertyGraphQLID(parentClass: KClass<*>): Boolean = when {
-    this.isGraphQLID() -> true
-    getConstructorParameter(parentClass)?.isGraphQLID().isTrue() -> true
-    else -> false
-}
-
 internal fun KProperty<*>.isPropertyGraphQLIgnored(parentClass: KClass<*>): Boolean = when {
     this.isGraphQLIgnored() -> true
     getConstructorParameter(parentClass)?.isGraphQLIgnored().isTrue() -> true

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/generator/extensions/kPropertyExtensions.kt

@@ -20,7 +20,6 @@ import com.expediagroup.graphql.exceptions.InvalidInputFieldTypeException
 import com.expediagroup.graphql.generator.SchemaGenerator
 import com.expediagroup.graphql.generator.extensions.getGraphQLDescription
 import com.expediagroup.graphql.generator.extensions.getName
-import com.expediagroup.graphql.generator.extensions.isGraphQLID
 import com.expediagroup.graphql.generator.extensions.isInterface
 import com.expediagroup.graphql.generator.extensions.isList
 import com.expediagroup.graphql.generator.extensions.safeCast
@@ -34,7 +33,7 @@ internal fun generateArgument(generator: SchemaGenerator, parameter: KParameter)
         throw InvalidInputFieldTypeException(parameter)
     }
 
-    val graphQLType = generateGraphQLType(generator, parameter.type, inputType = true, annotatedAsID = parameter.isGraphQLID())
+    val graphQLType = generateGraphQLType(generator = generator, type = parameter.type, inputType = true)
 
     // Deprecation of arguments is currently unsupported: https://github.com/facebook/graphql/issues/197
     val builder = GraphQLArgument.newArgument()

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/generator/types/generateArgument.kt

@@ -22,7 +22,6 @@ import com.expediagroup.graphql.generator.extensions.getDeprecationReason
 import com.expediagroup.graphql.generator.extensions.getFunctionName
 import com.expediagroup.graphql.generator.extensions.getGraphQLDescription
 import com.expediagroup.graphql.generator.extensions.getValidArguments
-import com.expediagroup.graphql.generator.extensions.isGraphQLID
 import com.expediagroup.graphql.generator.extensions.safeCast
 import com.expediagroup.graphql.generator.types.utils.getWrappedReturnType
 import graphql.schema.FieldCoordinates
@@ -51,7 +50,7 @@ internal fun generateFunction(generator: SchemaGenerator, fn: KFunction<*>, pare
 
     val typeFromHooks = generator.config.hooks.willResolveMonad(fn.returnType)
     val returnType = getWrappedReturnType(typeFromHooks)
-    val graphQLOutputType = generateGraphQLType(generator, returnType, annotatedAsID = fn.isGraphQLID()).safeCast<GraphQLOutputType>()
+    val graphQLOutputType = generateGraphQLType(generator = generator, type = returnType).safeCast<GraphQLOutputType>()
     val graphQLType = builder.type(graphQLOutputType).build()
     val coordinates = FieldCoordinates.coordinates(parentName, functionName)
 

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/generator/types/generateFunction.kt

@@ -33,10 +33,10 @@ import kotlin.reflect.KType
 /**
  * Return a basic GraphQL type given all the information about the kotlin type.
  */
-internal fun generateGraphQLType(generator: SchemaGenerator, type: KType, inputType: Boolean = false, annotatedAsID: Boolean = false): GraphQLType {
+internal fun generateGraphQLType(generator: SchemaGenerator, type: KType, inputType: Boolean = false): GraphQLType {
     val hookGraphQLType = generator.config.hooks.willGenerateGraphQLType(type)
     val graphQLType = hookGraphQLType
-        ?: generateScalar(generator, type, annotatedAsID)
+        ?: generateScalar(generator, type)
         ?: objectFromReflection(generator, type, inputType)
 
     // Do not call the hook on GraphQLTypeReference as we have not generated the type yet

graphql-kotlin-schema-generator/src/main/kotlin/com/expediagroup/graphql/generator/types/generateGraphQLType.kt

@@ -19,7 +19,6 @@ package com.expediagroup.graphql.generator.types
 import com.expediagroup.graphql.generator.SchemaGenerator
 import com.expediagroup.graphql.generator.extensions.getPropertyDescription
 import com.expediagroup.graphql.generator.extensions.getPropertyName
-import com.expediagroup.graphql.generator.extensions.isPropertyGraphQLID
 import com.expediagroup.graphql.generator.extensions.safeCast
 import graphql.schema.GraphQLInputObjectField
 import graphql.schema.GraphQLInputType
@@ -28,7 +27,7 @@ import kotlin.reflect.KProperty
 
 internal fun generateInputProperty(generator: SchemaGenerator, prop: KProperty<*>, parentClass: KClass<*>): GraphQLInputObjectField {
     val builder = GraphQLInputObjectField.newInputObjectField()
-    val graphQLInputType = generateGraphQLType(generator, prop.returnType, true, prop.isPropertyGraphQLID(parentClass)).safeCast<GraphQLInputType>()
+    val graphQLInputType = generateGraphQLType(generator = generator, type = prop.returnType, inputType = true).safeCast<GraphQLInputType>()
 
     builder.description(prop.getPropertyDescription(parentClass))
     builder.name(prop.getPropertyName(parentClass))