stuebingerb / KGraphQL

Pure Kotlin GraphQL implementation
MIT License
4 stars 1 forks source link

Incorrect built-in scalars #83

Open stuebingerb opened 2 weeks ago

stuebingerb commented 2 weeks ago

KGraphQL currently defines the following built-in scalars:

STRING
SHORT
INT
DOUBLE
FLOAT
BOOLEAN
LONG

However, the spec defines the following:

Int, Float, String, Boolean, and ID

According to the spec, those must be supported and also behave in a special way during introspection (which likely is not correctly implemented either).

So:

IMHO, Short and Long can still be included as built-in "extended" types to save users of the library from having to define them on their own but they must not behave like GraphQL built-in types. We can then also think about adding other common JVM types to those extended built-ins.