[openrewrite] initial merge and UpgradeToJava17 (#2636 towards #2634)

Author: nedtwigg

.github/workflows/ci.yml

@@ -13,7 +13,6 @@ concurrency:
   cancel-in-progress: true
 jobs:
   sanityCheck:
-    name: spotlessCheck assemble testClasses
     runs-on: ubuntu-latest
     env:
       buildcacheuser: ${{ secrets.BUILDCACHE_USER }}
@@ -31,6 +30,8 @@ jobs:
         uses: gradle/actions/setup-gradle@v4
       - name: spotlessCheck
         run: ./gradlew spotlessCheck
+      - name: rewriteDryRun
+        run: ./gradlew rewriteDryRun
       - name: assemble testClasses
         run: ./gradlew assemble testClasses
   build:
@@ -66,10 +67,10 @@ jobs:
         uses: gradle/actions/setup-gradle@v4
       - name: build (maven-only)
         if: matrix.kind == 'maven'
-        run: ./gradlew :plugin-maven:build -x spotlessCheck
+        run: ./gradlew :plugin-maven:build -x spotlessCheck -x rewriteDryRun
       - name: build (everything-but-maven)
         if: matrix.kind == 'gradle'
-        run: ./gradlew build -x spotlessCheck -PSPOTLESS_EXCLUDE_MAVEN=true
+        run: ./gradlew build -x spotlessCheck -x rewriteDryRun -PSPOTLESS_EXCLUDE_MAVEN=true
       - name: test npm
         if: matrix.kind == 'npm'
         run: ./gradlew testNpm

CHANGES.md

@@ -28,6 +28,7 @@ We adhere to the [keepachangelog](https://keepachangelog.com/en/1.0.0/) format (
 ### Added
 * Add a `lint` mode to `ReplaceRegexStep` ([#2571](https://github.com/diffplug/spotless/pull/2571))
 * `LintSuppression` now enforces unix-style paths in its `setPath` and `relativizeAsUnix` methods. ([#2629](https://github.com/diffplug/spotless/pull/2629))
+* Add `rewrite` support ([#2588](https://github.com/diffplug/spotless/pull/2588))
 
 ## [3.3.1] - 2025-07-21
 ### Fixed

build.gradle

@@ -12,6 +12,7 @@ repositories {
 apply from: rootProject.file('gradle/java-publish.gradle')
 apply from: rootProject.file('gradle/changelog.gradle')
 allprojects {
+	apply from: rootProject.file('gradle/rewrite.gradle')
 	apply from: rootProject.file('gradle/spotless.gradle')
 }
 apply from: rootProject.file('gradle/spotless-freshmark.gradle')
@@ -27,3 +28,8 @@ spotless {
 		endWithNewline()
 	}
 }
+
+dependencies {
+	rewrite(platform("org.openrewrite.recipe:rewrite-recipe-bom:3.14.1"))
+	rewrite("org.openrewrite.recipe:rewrite-migrate-java:3.17.1")
+}

gradle/java-publish.gradle

@@ -103,9 +103,9 @@ model {
 					from components.java
 				}
 
-				groupId project.group
-				artifactId project.ext.artifactId
-				version project.version
+				groupId = project.group
+				artifactId = project.ext.artifactId
+				version = project.version
 
 				def projectExtArtifactId = project.ext.artifactId
 				def projectDescription = project.description

gradle/rewrite.gradle

@@ -0,0 +1,7 @@
+apply plugin: 'org.openrewrite.rewrite'
+
+rewrite {
+	activeRecipe("org.openrewrite.java.migrate.UpgradeToJava17")
+	exportDatatables = true
+	failOnDryRunResults = true
+}

lib-extra/src/main/java/com/diffplug/spotless/extra/EclipseBasedStepBuilder.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2016-2024 DiffPlug
+ * Copyright 2016-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -175,7 +175,7 @@ private static String convertEclipseVersion(String version) {
 				char lastChar = version.charAt(version.length() - 1);
 				if ('.' != lastChar && 'a' <= lastChar) {
 					semanticVersion = version.substring(0, version.length() - 1);
-					semanticVersion += String.format(".%d", (int) lastChar);
+					semanticVersion += ".%d".formatted((int) lastChar);
 				}
 			}
 			return semanticVersion;

lib-extra/src/main/java/com/diffplug/spotless/extra/cpp/EclipseCdtFormatterStep.java

@@ -69,7 +69,7 @@ private static FormatterFunc apply(EquoBasedStepBuilder.State state) throws Exce
 						return (String) method.invoke(formatter, input);
 					} catch (InvocationTargetException exceptionWrapper) {
 						Throwable throwable = exceptionWrapper.getTargetException();
-						Exception exception = (throwable instanceof Exception) ? (Exception) throwable : null;
+						Exception exception = (throwable instanceof Exception e) ? e : null;
 						throw (null == exception) ? exceptionWrapper : exception;
 					}
 				});

lib-extra/src/main/java/com/diffplug/spotless/extra/groovy/GrEclipseFormatterStep.java

@@ -97,7 +97,7 @@ private static FormatterFunc apply(EquoBasedStepBuilder.State state) throws Exce
 						return (String) method.invoke(formatter, input);
 					} catch (InvocationTargetException exceptionWrapper) {
 						Throwable throwable = exceptionWrapper.getTargetException();
-						Exception exception = (throwable instanceof Exception) ? (Exception) throwable : null;
+						Exception exception = (throwable instanceof Exception e) ? e : null;
 						throw (null == exception) ? exceptionWrapper : exception;
 					}
 				});

lib-extra/src/main/java/com/diffplug/spotless/extra/wtp/EclipseWtpFormatterStep.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2016-2021 DiffPlug
+ * Copyright 2016-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -67,7 +67,7 @@ private static FormatterFunc applyWithoutFile(String className, EclipseBasedStep
 				return (String) method.invoke(formatter, input);
 			} catch (InvocationTargetException exceptionWrapper) {
 				Throwable throwable = exceptionWrapper.getTargetException();
-				Exception exception = (throwable instanceof Exception) ? (Exception) throwable : null;
+				Exception exception = (throwable instanceof Exception e) ? e : null;
 				throw (null == exception) ? exceptionWrapper : exception;
 			}
 		};
@@ -85,7 +85,7 @@ public String applyWithFile(String unix, File file) throws Exception {
 					return (String) method.invoke(formatter, unix, file.getAbsolutePath());
 				} catch (InvocationTargetException exceptionWrapper) {
 					Throwable throwable = exceptionWrapper.getTargetException();
-					Exception exception = (throwable instanceof Exception) ? (Exception) throwable : null;
+					Exception exception = (throwable instanceof Exception e) ? e : null;
 					throw (null == exception) ? exceptionWrapper : exception;
 				}
 			}

lib/src/main/java/com/diffplug/spotless/ConfigurationCacheHackList.java

@@ -16,6 +16,7 @@
 package com.diffplug.spotless;
 
 import java.io.IOException;
+import java.io.Serial;
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -51,6 +52,7 @@
  * to make Spotless work with all of Gradle's cache systems at once.
  */
 public class ConfigurationCacheHackList implements java.io.Serializable {
+	@Serial
 	private static final long serialVersionUID = 6914178791997323870L;
 
 	private boolean optimizeForEquality;
@@ -108,8 +110,8 @@ public void clear() {
 
 	public void addAll(Collection<? extends FormatterStep> c) {
 		for (FormatterStep step : c) {
-			if (step instanceof FormatterStepSerializationRoundtrip) {
-				var clone = ((FormatterStepSerializationRoundtrip) step).hackClone(optimizeForEquality);
+			if (step instanceof FormatterStepSerializationRoundtrip roundtrip) {
+				var clone = roundtrip.hackClone(optimizeForEquality);
 				backingList.add(clone);
 			} else {
 				backingList.add(step);
@@ -120,8 +122,8 @@ public void addAll(Collection<? extends FormatterStep> c) {
 	public List<FormatterStep> getSteps() {
 		var result = new ArrayList<FormatterStep>(backingList.size());
 		for (Object obj : backingList) {
-			if (obj instanceof FormatterStepSerializationRoundtrip.HackClone) {
-				result.add(((FormatterStepSerializationRoundtrip.HackClone) obj).rehydrate());
+			if (obj instanceof FormatterStepSerializationRoundtrip.HackClone clone) {
+				result.add(clone.rehydrate());
 			} else {
 				result.add((FormatterStep) obj);
 			}