org.openrewrite.java.RemoveUnusedImports should not removed imports for public static inner class usage

[iparadiso]

org.openrewrite.java.RemoveUnusedImports doesn't appear to spot static public inner class usage that's used with wildcard imports statements.

What version of OpenRewrite are you using?

I am using version: 7.40.6

What is the smallest, simplest way to reproduce the problem?

Consider the following class

package com.a.b.c;
public final class ParentBaseClass {
  public static abstract class BaseImplClass {
     void foo() {
      } 
  }
}

Which is then used in the following class

import com.a.b.c.ParentBaseClass.*
import java.util.List;
public class MyClass extends BaseImplClass {
   @Override
   public void foo() {
   }
}

What did you expect to see?

import com.a.b.c.ParentBaseClass.*
public class MyClass extends BaseImplClass {
   @Override
   public void foo() {
   }
}

What did you see instead?

public class MyClass extends BaseImplClass {
   @Override
   public void foo() {
   }
}

[joanvr]

Hi, thanks for reporting this issue! Would you be open to contribute to write a unit test for it or even a fix?

[timtebeek]

Confirmed with this (slightly different) test.

@Test
void nestedImports() {
    rewriteRun(
      java(
        """
          package foo;
                        
          public interface J {
              final class One implements J {}
              final class Two implements J {}
              final class Three implements J {}
              final class Four implements J {}
              final class Five implements J {}
              final class Six implements J {}
          }
          """
      ),
      java(
        """
          package bar;
          
          import foo.J;
          import foo.J.*;
                        
          class Quz {
            void test() {
              J j = null;
              One one = null;
              Two two = null;
              Three three = null;
              Four four = null;
              Five five = null;
              Six six = null;
            }
          }
          """
      )
    );
}

[timtebeek]

@Laurens-W Since you evaluated RemoveUnusedImports in #4450 it might make sense to briefly look at this one as well. The test is already there; perhaps the fix is similar?