KSName doesn't offer a deterministic naming API about ksp HOT 6 CLOSED

KSName is meant to align with qualified names therefore you are seeing this result, but it is also reasonable to provide correct logic to get the qualifier. We will discuss to see what's the path forward for KSName.
The Glide example is not using KotlinPoet in a very practical way, mainly due to there is no KSP support built in KotlinPoet so we took a workaround to address the ClassName, ideally, there should be some extension functions around KSP properties to work with KotlinPoet.

from ksp.

sounds good. I'm one of the maintainers of KotlinPoet, we'd be interested in seeing what a kotlinpoet-ksp artifact could like on our end. Some obvious ones come to mind, like FileSpec.writeTo(CodeGenerator) support and KSName.toClassName(). Open to other suggestions as well for things you'd want to see in an artifact like that, let us know 👍. Will file an issue on our end soon to track ideas

from ksp.

Current KSName implementation is based on the qualified name spec.

To address this issue I will add a new property/function to KSName.

from ksp.

A KSName returned by compiler/KSP is guaranteed to be unambiguous. As for getQualifier, did you mean to get its package name? If so, KSDeclaration.getPackage (to be added) might serve the purpose.

Forcing the notion of package in KSName would require all creation sites to supply package and the remaining identifier(s) separately. Although I agree that specifying package explicitly is much clearer, it is also less convenient.

As for the / separator, it looks like an encoding / implementation choice of Kotlin metadata. Would getPackage be more human friendly?

from ksp.

To add more context, on Kotlin language perspective names are separated by dot, and the case when you see names separated by slash is platform dependent (in this case, it is the JVM runtime name). In my opinion the right way to solve this is to provide KSDeclaration with a getPackage API so that you can inspect package name for a nested class. However, you will have to make some adjustments to the KotlinPoet code you linked with this approach.

from ksp.

Alternative solution for this is shipped with android/kotlin#98

from ksp.