context receivers support

[JavierSegoviaCordoba]

Currently, context add an unnecessary breakline:

context(ScanResult)

internal val ClassInfo.contextClasses: List<KClass<out Any>>
  get() =
    getClassesWithAnnotation(this.name).map { contextClassInfo ->
      contextClassInfo.loadClass().kotlin
    }

should be:

context(ScanResult)
internal val ClassInfo.contextClasses: List<KClass<out Any>>
  get() =
    getClassesWithAnnotation(this.name).map { contextClassInfo ->
      contextClassInfo.loadClass().kotlin
    }

[strulovich]

It is likely we will only fix this when this comes out of preview. Happy to accept a PR once the dependencies sort themselves out (though it looks it might be future compatible from the fact Ktfmt doesn't crash on this)

[JavierSegoviaCordoba]

I would like to know how it will be combined with annotations.

1

context(Persistence)
@Provider class Repo

2

context(Persistence)
@Provider
class Repo

3

@Provider
context(Persistence)
class Repo

The first one is which I like more but if there are a lot of annotations it will not look very good, so I think the third one is the best.

[JavierSegoviaCordoba]

@strulovich I tried this myself but after updating Kotlin to 1.7.10 and 1.7.20 beta, I cannot get the context receivers in visitNamedFunction, I got an empty list even though the KotlinInput shows clearly they are there.

Not sure if I am missing something or if there is a way to try to get them tokenized without using the context receivers from PSI.

[JavierSegoviaCordoba]

@strulovich as they are not going to be on 1.8.0, can this be prioritized a bit, please?

[hick209]

Fixed by @bddckr (#400)