org.openrewrite.staticanalysis.UseCollectionInterfaces does not properly remove import

[blipper]

import java.util.concurrent.ConcurrentLinkedQueue;
class A {
    private final ConcurrentLinkedQueue<String> widgetDataList;
    public foo() {
      widgetDataList.add("TEST");
    }
}

What did you expect to see?

import java.util.Queue;
class A {
    private final Queue<String> widgetDataList;
    public foo() {
      widgetDataList.add("TEST");
    }

}

What did you see instead?

import java.util.concurrent.ConcurrentLinkedQueue;
import java.util.Queue;
class A {
    private final Queue<String> widgetDataList;
    public foo() {
      widgetDataList.add("TEST");
    }

}

https://github.com/openrewrite/rewrite-static-analysis/blob/main/src/main/java/org/openrewrite/staticanalysis/UseCollectionInterfaces.java#L166 has the removal but is doesn't sem to work?

Are you interested in contributing a fix to OpenRewrite?

[timtebeek]

Thanks for the clear example! I suspect there's a dangling type reference on the field, identifier or method that still refers to the old type, and thus blocks removal of the import. Might be easy to spot if the above is converted into a runnable test of a draft PR. Once we clear out any remaining references the import should be dropped.