False positive unused import with version 0.38.0-alpha01

[vRallev]

I'm testing 0.38.0-alpha01 with Kotlin 1.4.0 and see a few false positives for unused imports.

package com.squareup.ktlint.rule

import com.pinterest.ktlint.core.Rule
import com.squareup.psi.deleteFromParentWithWhitespace
import com.squareup.psi.findAnnotation
import dagger.Module
import org.jetbrains.kotlin.com.intellij.lang.ASTNode
import org.jetbrains.kotlin.name.FqName
import org.jetbrains.kotlin.psi.KtObjectDeclaration

internal val DAGGER_MODULE_FQ_NAME = FqName(Module::class.java.canonicalName)

/**
 * KtLint rule that flags redundant @Module annotations on companion objects.
 */
class NoModuleAnnotationOnCompanionObjects : Rule("no-module-annotation-on-companion-objects") {

  override fun visit(
    node: ASTNode,
    autoCorrect: Boolean,
    emit: (offset: Int, errorMessage: String, canBeAutoCorrected: Boolean) -> Unit
  ) {
    val psi = node.psi as? KtObjectDeclaration ?: return
    if (!psi.isCompanion()) return

    val moduleAnnotation = psi.findAnnotation(DAGGER_MODULE_FQ_NAME) ?: return

    emit(
        node.startOffset,
        "Companion objects no longer need to be annotated with @Module. The " +
            "annotation is only required for the surrounding class.",
        true
    )

    if (autoCorrect) {
      moduleAnnotation.deleteFromParentWithWhitespace()
    }
  }
}

/Users/ralf/path/buildSrc/ktlint-rules/src/main/java/com/squareup/ktlint/rule/NoModuleAnnotationOnCompanionObjects.kt
(5, 1): Unused import (no-unused-imports)

The extension function is used on the object psi line 26. Please let me know if you need any further information.

[ZacSweers]

We saw this as well and can also find some examples if need be

[romtsn]

@ZacSweers would appreciate at least one, as the reason might differ

[shashachu]

@vRallev just to confirm - this is code that was passing in ktlint 0.37.2?

[vRallev]

I double checked and confirmed that it's passing with 0.37.2 (we have issues with the continuation indent, though).