type with then field cause deprecated warning in caliban-sbt

[ngbinh]

Using caliban-sbt on a schema with type like this:

type Order {
 then: String
}

would cause then is a reserved word (since 2.10.0); usage as an identifier is deprecated on the generated code.

is there a way to fix this?

Thanks

[ghostdogpr]

I think it might be enough just to add it to reservedKeywords here

caliban/tools/src/main/scala/caliban/tools/ClientWriter.scala

Line 836 in 8c52365

val reservedKeywords = Set(

I don't remember where I found the initial list of keywords but then was not in there.

[RhnSharma]

I would like to take this one. Is it up for grabs?

[ghostdogpr]

Sure!

[ngbinh]

would be great if the list is configurable with sensible defaults.

[ghostdogpr]

In theory it's a finite list, but I can't find it. Even on the 2.13 language spec then is not included:

[ngbinh]

then is reserved in dotty. More context here: scala/bug#10309

[ghostdogpr]

Oh, interesting, the error message is actually incorrect.