Aliased imports not working for nested classes and generate unused imports and usage of FQNs

[mcarleio]

Describe the bug
An aliased import added via addAliasedImport for a nested class seems to be never used, but still generates unused imports.

To Reproduce

package org.example

import com.squareup.kotlinpoet.FileSpec
import com.squareup.kotlinpoet.FunSpec

class Person {
    class Id
}

fun main() {
    println(
        FileSpec.builder("org.example", "SomeFile")
            .addAliasedImport(Person.Id::class, "PID")
            .addFunction(
                FunSpec.builder("pid")
                    .returns(Person.Id::class)
                    .addCode("return %T()", Person.Id::class)
                    .build()
            )
            .build()
    )
}

Expected behavior

package org.example

import org.example.Person.Id as PID

public fun pid(): PID = PID()

Actual behavior

package org.example

import org.example.Person.Id as PID

public fun pid(): Person.Id = Person.Id()

Additional context

If you change the package of the FileSpec to a different package than the nested class (e.g. FileSpec.builder("org.elsewhere", "SomeFile")) and run the same again, you will even get two unused imports and FQNs instead of the alias are used:

expected

package org.elsewhere

import org.example.Person.Id as PID

public fun pid(): PID = PID()

actual

package org.elsewhere

import org.example.Person
import org.example.Person.Id as PID

public fun pid(): org.example.Person.Id = org.example.Person.Id()

[mcarleio]

As @Egorand recognized in #1839, I was not using the latest version 1.16.0 (I was using 1.15.3) - sorry!

But this issue is still valid, although the behavior already changed a bit for the first case. The actual behavior there is now the same as for the last case:

package org.example

import org.example.Person
import org.example.Person.Id as PID

public fun pid(): org.example.Person.Id = org.example.Person.Id()

[Egorand]

Yep, thanks @mcarleio, I was able to reproduce this one. The team will take a look!