[prone] Apply errorprone

Vincent Potucek · Vincent Potucek · commit e1057d72590e · 2025-10-18T12:48:18.000+02:00
diff --git a/gradle/error-prone.gradle b/gradle/error-prone.gradle
@@ -18,12 +18,91 @@ dependencies {
 
 tasks.withType(JavaCompile).configureEach {
 	options.errorprone {
-		disableAllChecks = true // consider removal to avoid error prone error´s, following convention over configuration.
-		error('RedundantStringConversion')
+		disable( // not patchable
+				'FormatStringConcatenation',
+				'FunctionalInterfaceMethodChanged',
+				'JavaxInjectOnAbstractMethod',
+				'OverridesJavaxInjectableMethod',
+				'ReturnValueIgnored',
+				'Slf4jLogStatement',
+				'MissingSummary',
+				// patchable
+				'ConstantNaming', // fixed: https://github.com/diffplug/spotless/pull/2687
+				'LexicographicalAnnotationAttributeListing',
+				'LexicographicalAnnotationListing',
+				'NonStaticImport',
+				'Slf4jLoggerDeclaration', // logger -> log
+				'StaticImport',
+				// critical, finds lots of bugs... unused configs and stuff.
+				'Unused',
+				'UnusedMethod',
+				'UnusedParameters',
+				'UnusedVariable',
+				)
+		error(
+				'AmbiguousJsonCreator',
+				'AssertJNullnessAssertion',
+				'AutowiredConstructor',
+				'CanonicalAnnotationSyntax',
+				'CollectorMutability',
+				// 'ConstantNaming',
+				'DirectReturn',
+				'EmptyMethod',
+				'ExplicitArgumentEnumeration',
+				'ExplicitEnumOrdering',
+				'IdentityConversion',
+				'ImmutablesSortedSetComparator',
+				'IsInstanceLambdaUsage',
+				// 'LexicographicalAnnotationAttributeListing',
+				// 'LexicographicalAnnotationListing',
+				'MockitoMockClassReference',
+				'MockitoStubbing',
+				'NestedOptionals',
+				'PrimitiveComparison',
+				'RedundantStringConversion',
+				'RedundantStringEscape',
+				'SelfAssignment',
+				'StringJoin',
+				'StringJoining',
+				)
+		// fixme bug: this only happens when the file is dirty!
+		// up2date checking (caching) must consider file changes, as file is currently corrupt!
+		// fix SelfTest.java:L22 ForbidGradleInternal(import org.gradle.api.internal.project.ProjectInternal;)
+		errorproneArgs.add('-XepExcludedPaths:.*/SelfTest.java')
 		if (!getenv().containsKey('CI') && getenv('IN_PLACE')?.toBoolean()) {
 			errorproneArgs.addAll(
 					'-XepPatchLocation:IN_PLACE',
-					'-XepPatchChecks:RedundantStringConversion'
+					'-XepPatchChecks:' +
+					'AmbiguousJsonCreator,' +
+					'ArrayEquals,' +
+					'AssertJNullnessAssertion,' +
+					'AutowiredConstructor,' +
+					'CanonicalAnnotationSyntax,' +
+					'CollectorMutability,' +
+					// 'ConstantNaming,' +
+					'DirectReturn,' +
+					'EmptyMethod,' +
+					'ExplicitArgumentEnumeration,' +
+					'ExplicitEnumOrdering,' +
+					'FormatStringConcatenation,' +
+					'IdentityConversion,' +
+					'ImmutablesSortedSetComparator,' +
+					'IsInstanceLambdaUsage,' +
+					// 'LexicographicalAnnotationAttributeListing,' +
+					// 'LexicographicalAnnotationListing,' +
+					'MockitoMockClassReference,' +
+					'MockitoStubbing,' +
+					'NestedOptionals,' +
+					// 'NonStaticImport,' +
+					'PrimitiveComparison,' +
+					'RedundantStringConversion,' +
+					'RedundantStringEscape,' +
+					'SelfAssignment,' +
+					'Slf4jLogStatement,' +
+					// 'StaticImport,' +
+					'StringJoin,' +
+					'StringJoining,' +
+					'TimeZoneUsage,'
 					)
 		}
 	}
diff --git a/lib-extra/src/test/java/com/diffplug/spotless/extra/java/EclipseJdtEqualityTest.java b/lib-extra/src/test/java/com/diffplug/spotless/extra/java/EclipseJdtEqualityTest.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2024 DiffPlug
+ * Copyright 2024-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,6 +20,7 @@
 import java.io.IOException;
 import java.io.ObjectOutputStream;
 import java.io.Serializable;
+import java.util.Arrays;
 import java.util.List;
 
 import org.junit.jupiter.api.Assertions;
@@ -43,7 +44,7 @@ public void test() throws Exception {
 
 		var serialized1 = toBytes(step1);
 		var serialized2 = toBytes(step2);
-		Assertions.assertFalse(serialized1.equals(serialized2));
+		Assertions.assertFalse(Arrays.equals(serialized1, serialized2));
 	}
 
 	private static FormatterStep withSettingsFile(File settingsFile) {
diff --git a/lib/src/compatKtLint0Dot48Dot0/java/com/diffplug/spotless/glue/ktlint/compat/KtLintCompat0Dot48Dot0Adapter.java b/lib/src/compatKtLint0Dot48Dot0/java/com/diffplug/spotless/glue/ktlint/compat/KtLintCompat0Dot48Dot0Adapter.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2023-2024 DiffPlug
+ * Copyright 2023-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -115,7 +115,7 @@ public String format(
 	private static EditorConfigOverride createEditorConfigOverride(final List<Rule> rules, Map<String, Object> editorConfigOverrideMap) {
 		// Get properties from rules in the rule sets
 		Stream<EditorConfigProperty<?>> ruleProperties = rules.stream()
-				.filter(rule -> rule instanceof UsesEditorConfigProperties)
+				.filter(UsesEditorConfigProperties.class::isInstance)
 				.flatMap(rule -> ((UsesEditorConfigProperties) rule).getEditorConfigProperties().stream());
 
 		// Create a mapping of properties to their names based on rule properties and default properties
diff --git a/lib/src/main/java/com/diffplug/spotless/PaddedCell.java b/lib/src/main/java/com/diffplug/spotless/PaddedCell.java
@@ -153,7 +153,7 @@ public String canonical() {
 		// @formatter:off
 		switch (type) {
 		case CONVERGE:	return steps.get(steps.size() - 1);
-		case CYCLE:		return Collections.min(steps, Comparator.comparing(String::length).thenComparing(Function.identity()));
+		case CYCLE:		return Collections.min(steps, Comparator.comparingInt(String::length).thenComparing(Function.identity()));
 		case DIVERGE:	throw new IllegalArgumentException("No canonical form for a diverging result");
 		default:	throw new IllegalArgumentException("Unknown type: " + type);
 		}
diff --git a/lib/src/main/java/com/diffplug/spotless/biome/BiomeStep.java b/lib/src/main/java/com/diffplug/spotless/biome/BiomeStep.java
@@ -25,7 +25,6 @@
 import java.nio.file.attribute.PosixFilePermission;
 import java.util.ArrayList;
 import java.util.HashSet;
-import java.util.Set;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -299,7 +298,7 @@ private State createState() throws IOException, InterruptedException {
 		validateBiomeExecutable(resolvedPathToExe);
 		validateBiomeConfigPath(configPath, version);
 		logger.debug("Using Biome executable located at  '{}'", resolvedPathToExe);
-		var exeSignature = FileSignature.signAsList(Set.of(new File(resolvedPathToExe)));
+		var exeSignature = FileSignature.signAsList(new File(resolvedPathToExe));
 		makeExecutable(resolvedPathToExe);
 		return new State(resolvedPathToExe, exeSignature, configPath, language);
 	}
diff --git a/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/GitRatchetGradleTest.java b/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/GitRatchetGradleTest.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.
@@ -58,7 +58,7 @@ public GradleRunner gradleRunner() throws IOException {
 
 	@ParameterizedTest
 	//@ValueSource(ints = {0, 1}) // TODO: this is a flaky configuration cache issue that started with Gradle 8.5
-	@ValueSource(ints = {0})
+	@ValueSource(ints = 0)
 	void singleProjectExhaustive(int useConfigCache) throws Exception {
 		try (Git git = initRepo()) {
 			if (useConfigCache == 1) {
@@ -158,7 +158,7 @@ private BuildResultAssertion assertFail(String... tasks) throws Exception {
 
 	@ParameterizedTest
 	//@ValueSource(ints = {0, 1}) // TODO: this is a flaky configuration cache issue that started with Gradle 8.5
-	@ValueSource(ints = {0})
+	@ValueSource(ints = 0)
 	void multiProject(int useConfigCache) throws Exception {
 		try (Git git = initRepo()) {
 			if (useConfigCache == 1) {
diff --git a/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/SpotlessTaskImplTest.java b/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/SpotlessTaskImplTest.java
@@ -29,17 +29,17 @@
 public class SpotlessTaskImplTest {
 	@Test
 	public void testThrowsMessageContainsFilename() throws Exception {
-		SpotlessTaskImpl task = Mockito.mock(SpotlessTaskImpl.class, Mockito.CALLS_REAL_METHODS);
+		SpotlessTaskImpl task = Mockito.mock(Mockito.CALLS_REAL_METHODS);
 		Mockito.when(task.getLogger()).thenReturn(Mockito.mock(Logger.class));
 
 		File projectDir = Path.of("unitTests", "projectDir").toFile();
-		DirectoryProperty projectDirProperty = Mockito.mock(DirectoryProperty.class, Mockito.RETURNS_DEEP_STUBS);
+		DirectoryProperty projectDirProperty = Mockito.mock(Mockito.RETURNS_DEEP_STUBS);
 		Mockito.when(projectDirProperty.get().getAsFile()).thenReturn(projectDir);
 
 		Mockito.when(task.getProjectDir()).thenReturn(projectDirProperty);
 
 		File input = Path.of("unitTests", "projectDir", "someInput").toFile();
-		Formatter formatter = Mockito.mock(Formatter.class);
+		Formatter formatter = Mockito.mock();
 
 		Assertions.assertThatThrownBy(() -> task.processInputFile(null, formatter, input, "someInput")).hasMessageContaining(input.toString());
 	}
diff --git a/plugin-maven/src/test/java/com/diffplug/spotless/maven/FileLocatorTest.java b/plugin-maven/src/test/java/com/diffplug/spotless/maven/FileLocatorTest.java
@@ -40,7 +40,7 @@ class FileLocatorTest extends ResourceHarness {
 
 	@BeforeEach
 	void beforeEach() {
-		resourceManager = mock(ResourceManager.class);
+		resourceManager = mock();
 		fileLocator = new FileLocator(resourceManager, rootFolder(), rootFolder());
 	}
 
diff --git a/plugin-maven/src/test/java/com/diffplug/spotless/maven/incremental/FileIndexHarness.java b/plugin-maven/src/test/java/com/diffplug/spotless/maven/incremental/FileIndexHarness.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2021 DiffPlug
+ * Copyright 2021-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -35,8 +35,8 @@ abstract class FileIndexHarness {
 
 	protected static final PluginFingerprint FINGERPRINT = PluginFingerprint.from("foo");
 
-	protected final FileIndexConfig config = mock(FileIndexConfig.class);
-	protected final Log log = mock(Log.class);
+	protected final FileIndexConfig config = mock();
+	protected final Log log = mock();
 
 	protected Path tempDir;
 
diff --git a/plugin-maven/src/test/java/com/diffplug/spotless/maven/incremental/NoopCheckerTest.java b/plugin-maven/src/test/java/com/diffplug/spotless/maven/incremental/NoopCheckerTest.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2021-2024 DiffPlug
+ * Copyright 2021-2025 DiffPlug
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -59,7 +59,7 @@ void beforeEach() throws Exception {
 
 	@Test
 	void deletesExistingIndexFileWhenCreated() {
-		Log log = mock(Log.class);
+		Log log = mock();
 		try (UpToDateChecker realChecker = UpToDateChecker.forProject(project, indexFile, singletonList(dummyFormatter()), log)) {
 			realChecker.setUpToDate(existingSourceFile);
 		}
@@ -76,7 +76,7 @@ void deletesExistingIndexFileWhenCreated() {
 	void doesNothingWhenIndexFileDoesNotExist() {
 		assertThat(indexFile).doesNotExist();
 
-		Log log = mock(Log.class);
+		Log log = mock();
 		try (UpToDateChecker noopChecker = UpToDateChecker.noop(project, indexFile, log)) {
 			assertThat(noopChecker).isNotNull();
 		}
diff --git a/rewrite.yml b/rewrite.yml
@@ -17,6 +17,9 @@ recipeList:
   - org.openrewrite.java.format.NormalizeLineBreaks
   - org.openrewrite.java.format.RemoveTrailingWhitespace
   - org.openrewrite.java.migrate.UpgradeToJava17
+  - org.openrewrite.java.migrate.lang.StringRulesRecipes
+# - org.openrewrite.java.migrate.lang.UseTextBlocks consider.
+  - org.openrewrite.java.migrate.util.JavaLangAPIs
   - org.openrewrite.java.migrate.util.JavaUtilAPIs
   - org.openrewrite.java.migrate.util.MigrateInflaterDeflaterToClose
   - org.openrewrite.java.migrate.util.ReplaceStreamCollectWithToList
@@ -62,6 +65,7 @@ recipeList:
   - tech.picnic.errorprone.refasterrules.StreamRulesRecipes
   - tech.picnic.errorprone.refasterrules.StringRulesRecipes
   - tech.picnic.errorprone.refasterrules.TimeRulesRecipes
+  # - org.openrewrite.java.migrate.lang.FindVirtualThreadOpportunities # don't want to use: https://github.com/diffplug/spotless/pull/2684#discussion_r2433831887
 ---
 name: com.diffplug.spotless.openrewrite.SpotlessFormat
 styleConfigs:

Original file line number	Diff line number	Diff line change
`@@ -153,7 +153,7 @@ public String canonical() {`
`153`	`153`	`// @formatter:off`
`154`	`154`	`switch (type) {`
`155`	`155`	`case CONVERGE: return steps.get(steps.size() - 1);`
`156`		`- case CYCLE: return Collections.min(steps, Comparator.comparing(String::length).thenComparing(Function.identity()));`
	`156`	`+ case CYCLE: return Collections.min(steps, Comparator.comparingInt(String::length).thenComparing(Function.identity()));`
`157`	`157`	`case DIVERGE: throw new IllegalArgumentException("No canonical form for a diverging result");`
`158`	`158`	`default: throw new IllegalArgumentException("Unknown type: " + type);`
`159`	`159`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ class FileLocatorTest extends ResourceHarness {`
`40`	`40`
`41`	`41`	`@BeforeEach`
`42`	`42`	`void beforeEach() {`
`43`		`- resourceManager = mock(ResourceManager.class);`
	`43`	`+ resourceManager = mock();`
`44`	`44`	`fileLocator = new FileLocator(resourceManager, rootFolder(), rootFolder());`
`45`	`45`	`}`
`46`	`46`