Fixes to URLConstructorToURICreate recipe

Declare as context-free and also remove the `URL` import if it isn't used anymore.

Author: knutwannheden

src/main/java/org/openrewrite/java/migrate/net/URLConstructorToURICreate.java

@@ -38,7 +38,7 @@ public class URLConstructorToURICreate extends Recipe {
 
     private static final String URI_FQN = "java.net.URI";
     private static final String URL_FQN = "java.net.URL";
-    private static final MethodMatcher methodMatcherSingleArg = new MethodMatcher(URL_FQN + " <constructor>(java.lang.String)");
+    private static final MethodMatcher methodMatcherSingleArg = new MethodMatcher(URL_FQN + "#<init>(java.lang.String)");
 
     @Override
     public String getDisplayName() {
@@ -47,7 +47,7 @@ public String getDisplayName() {
 
     @Override
     public String getDescription() {
-        return "Converts `new URL(String)` constructor to `URI.create(String).toURL()`.";
+        return "Converts `new URL(String)` constructor to `URI.create(String).toURL()`. The URL constructor has been deprecated due to security vulnerabilities when handling malformed URLs. Using `URI.create(String)` provides stronger validation and safer URL handling in modern Java applications.";
     }
 
     @Override
@@ -64,10 +64,10 @@ public J visitNewClass(J.NewClass nc, ExecutionContext ctx) {
 
                             JavaTemplate template = JavaTemplate.builder("URI.create(#{any(String)}).toURL()")
                                     .imports(URI_FQN)
-                                    .contextSensitive()
                                     .javaParser(JavaParser.fromJavaVersion())
                                     .build();
                             maybeAddImport(URI_FQN);
+                            maybeRemoveImport(URL_FQN);
 
                             return template.apply(getCursor(),
                                     nc.getCoordinates().replace(),

src/test/java/org/openrewrite/java/migrate/net/URLConstructorToURICreateTest.java

@@ -21,6 +21,7 @@
 import org.openrewrite.test.RecipeSpec;
 import org.openrewrite.test.RewriteTest;
 
+import static org.openrewrite.gradle.Assertions.buildGradle;
 import static org.openrewrite.java.Assertions.java;
 
 class URLConstructorToURICreateTest implements RewriteTest {
@@ -78,6 +79,58 @@ void urlConstructor() {
         );
     }
 
+    @Test
+    void removeUrlImport() {
+        rewriteRun(
+          //language=java
+          java(
+            """
+              import java.net.URL;
+
+              class Test {
+                  void foo() {
+                      System.out.println(new URL("https://test.com"));
+                  }
+              }
+              """,
+            """
+              import java.net.URI;
+
+              class Test {
+                  void foo() {
+                      System.out.println(URI.create("https://test.com").toURL());
+                  }
+              }
+              """
+          )
+        );
+    }
+
+    @Test
+    @Issue("https://github.com/openrewrite/rewrite-migrate-java/issues/620")
+    void gradleUrl() {
+        rewriteRun(
+          //language=groovy
+          buildGradle(
+            """
+              import java.net.URL
+              import java.util.concurrent.atomic.AtomicReference
+
+              def url = new AtomicReference<URL>()
+              url.set(new URL('https://www.reactive-streams.org/reactive-streams-1.0.3-javadoc/'))
+              """,
+            """
+              import java.net.URI
+              import java.net.URL
+              import java.util.concurrent.atomic.AtomicReference
+
+              def url = new AtomicReference<URL>()
+              url.set(URI.create('https://www.reactive-streams.org/reactive-streams-1.0.3-javadoc/').toURL())
+              """
+          )
+        );
+    }
+
     @Test
     @Issue("https://github.com/openrewrite/rewrite-migrate-java/issues/620")
     void urlCheckConstantAbsolutePath() {