review: checkstyle: Spotless configuration

pom.xml

@@ -232,28 +232,21 @@
         <artifactId>spotless-maven-plugin</artifactId>
         <version>2.43.0</version>
         <configuration>
-          <formats>
-            <format>
-              <includes>
-                <include>src/test/java/spoon/testing/assertions/**/*.java</include>
-              </includes>
-              <trimTrailingWhitespace/>
-              <endWithNewline/>
-              <indent>
-                <tabs>true</tabs>
-                <spacesPerTab>4</spacesPerTab>
-              </indent>
-            </format>
-          </formats>
           <java>
-            <includes>
-              <include>src/test/java/spoon/testing/assertions/**/*.java</include>
-            </includes>
-            <palantirJavaFormat>
-              <version>2.40.0</version>
-              <style>PALANTIR</style>
-              <formatJavadoc>true</formatJavadoc>
-            </palantirJavaFormat>
+            <excludes>
+              <!-- Exclude the test classes to prevent any impact on the current tests. -->
+              <exclude>**/testclasses*/**</exclude>
+            </excludes>
+            <importOrder>
+              <wildcardsLast>true</wildcardsLast>
+              <order>,\#javax|\#java,\#</order>
+            </importOrder>
+            <removeUnusedImports />
+            <trimTrailingWhitespace/>
+            <endWithNewline/>
+            <indent>
+              <tabs>true</tabs>
+            </indent>
           </java>
         </configuration>
       </plugin>

src/main/java/spoon/ContractVerifier.java

@@ -8,6 +8,15 @@
 package spoon;
 
 
+import java.io.PrintWriter;
+import java.io.StringWriter;
+import java.util.ArrayDeque;
+import java.util.Collection;
+import java.util.Deque;
+import java.util.HashSet;
+import java.util.IdentityHashMap;
+import java.util.Map;
+import java.util.Set;
 import spoon.reflect.CtModelImpl;
 import spoon.reflect.code.CtArrayWrite;
 import spoon.reflect.code.CtAssignment;
@@ -49,16 +58,6 @@
 import spoon.support.sniper.internal.ElementSourceFragment;
 import spoon.support.visitor.equals.EqualsVisitor;
 
-import java.io.PrintWriter;
-import java.io.StringWriter;
-import java.util.ArrayDeque;
-import java.util.Collection;
-import java.util.Deque;
-import java.util.HashSet;
-import java.util.IdentityHashMap;
-import java.util.Map;
-import java.util.Set;
-
 import static spoon.testing.utils.Check.assertNotNull;
 
 /**

src/main/java/spoon/FluentLauncher.java

@@ -9,7 +9,6 @@
 
 import java.io.File;
 import java.nio.charset.Charset;
-
 import spoon.processing.Processor;
 import spoon.reflect.CtModel;
 import spoon.reflect.declaration.CtElement;

src/main/java/spoon/IncrementalLauncher.java

@@ -24,13 +24,11 @@
 import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Set;
-
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.filefilter.DirectoryFileFilter;
 import org.apache.commons.io.filefilter.SuffixFileFilter;
 import org.apache.commons.io.filefilter.TrueFileFilter;
 import org.codehaus.plexus.util.CollectionUtils;
-
 import spoon.reflect.cu.CompilationUnit;
 import spoon.reflect.declaration.CtPackage;
 import spoon.reflect.declaration.CtType;

src/main/java/spoon/Launcher.java

@@ -14,6 +14,17 @@
 import com.martiansoftware.jsap.Switch;
 import com.martiansoftware.jsap.stringparsers.EnumeratedStringParser;
 import com.martiansoftware.jsap.stringparsers.FileStringParser;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
+import java.lang.invoke.MethodHandles;
+import java.nio.charset.Charset;
+import java.nio.file.Path;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.List;
+import java.util.ResourceBundle;
 import org.apache.commons.io.FileUtils;
 import org.apache.commons.io.filefilter.IOFileFilter;
 import org.apache.commons.lang3.StringUtils;
@@ -43,18 +54,6 @@
 import spoon.support.compiler.jdt.JDTBasedSpoonCompiler;
 import spoon.support.gui.SpoonModelTree;
 
-import java.io.File;
-import java.io.FileNotFoundException;
-import java.io.IOException;
-import java.lang.invoke.MethodHandles;
-import java.nio.charset.Charset;
-import java.nio.file.Path;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.List;
-import java.util.ResourceBundle;
-
 import static spoon.support.StandardEnvironment.DEFAULT_CODE_COMPLIANCE_LEVEL;
 
 /**
@@ -94,7 +93,7 @@ enum CLASSPATH_MODE {
 
 	/**
 	 * This field is used to ensure that {@link #setArgs(String[])} is only called once.
- 	 */
+	*/
 	private boolean processed = false;
 
 	/**

src/main/java/spoon/MavenLauncher.java

@@ -7,14 +7,13 @@
  */
 package spoon;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import spoon.support.compiler.SpoonPom;
-
 import java.io.File;
 import java.lang.invoke.MethodHandles;
 import java.util.List;
 import java.util.regex.Pattern;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import spoon.support.compiler.SpoonPom;
 
 /**
  * Create a Spoon launcher from a maven pom file

src/main/java/spoon/OutputType.java

@@ -8,7 +8,6 @@
 package spoon;
 
 import java.util.Locale;
-
 import org.jspecify.annotations.Nullable;
 
 /**

src/main/java/spoon/SpoonAPI.java

@@ -8,7 +8,6 @@
 package spoon;
 
 import java.io.File;
-
 import spoon.compiler.Environment;
 import spoon.processing.Processor;
 import spoon.reflect.CtModel;

src/main/java/spoon/SpoonModelBuilder.java

@@ -7,6 +7,10 @@
  */
 package spoon;
 
+import java.io.File;
+import java.util.Collection;
+import java.util.List;
+import java.util.Set;
 import spoon.compiler.SpoonResource;
 import spoon.compiler.builder.JDTBuilder;
 import spoon.processing.Processor;
@@ -19,11 +23,6 @@
 import spoon.support.compiler.jdt.FileCompilerConfig;
 import spoon.support.compiler.jdt.JDTBatchCompiler;
 
-import java.io.File;
-import java.util.Collection;
-import java.util.List;
-import java.util.Set;
-
 /**
  * Responsible for building a spoon model from Java source code.
  *

src/main/java/spoon/compiler/Environment.java

@@ -7,6 +7,11 @@
  */
 package spoon.compiler;
 
+import java.io.File;
+import java.nio.charset.Charset;
+import java.util.List;
+import java.util.function.Supplier;
+import org.jspecify.annotations.Nullable;
 import spoon.OutputType;
 import spoon.compiler.builder.EncodingProvider;
 import spoon.processing.FileGenerator;
@@ -23,13 +28,6 @@
 import spoon.support.modelobs.FineModelChangeListener;
 import spoon.support.sniper.SniperJavaPrettyPrinter;
 
-import java.io.File;
-import java.nio.charset.Charset;
-import java.util.List;
-import java.util.function.Supplier;
-
-import org.jspecify.annotations.Nullable;
-
 /**
  * This interface represents the environment in which Spoon is launched -
  * accessible through {@link spoon.reflect.factory.Factory#getEnvironment()}. Its

src/main/java/spoon/compiler/SpoonFile.java

@@ -7,12 +7,11 @@
  */
 package spoon.compiler;
 
-import spoon.SpoonException;
-
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.nio.charset.Charset;
+import spoon.SpoonException;
 
 /**
  * This interface represents files that can be used as resources for the Spoon

src/main/java/spoon/compiler/SpoonResourceHelper.java

@@ -7,16 +7,15 @@
  */
 package spoon.compiler;
 
-import spoon.Launcher;
-import spoon.support.compiler.FileSystemFile;
-import spoon.support.compiler.FileSystemFolder;
-import spoon.support.compiler.ZipFolder;
-
 import java.io.File;
 import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
+import spoon.Launcher;
+import spoon.support.compiler.FileSystemFile;
+import spoon.support.compiler.FileSystemFolder;
+import spoon.support.compiler.ZipFolder;
 
 /**
  * This class defines a helper for manipulating resources.

src/main/java/spoon/compiler/builder/EncodingProvider.java

@@ -7,8 +7,8 @@
  */
 package spoon.compiler.builder;
 
-import spoon.compiler.SpoonFile;
 import java.nio.charset.Charset;
+import spoon.compiler.SpoonFile;
 
 public interface EncodingProvider {
 

src/main/java/spoon/compiler/builder/SourceOptions.java

@@ -7,14 +7,13 @@
  */
 package spoon.compiler.builder;
 
-import spoon.compiler.SpoonFile;
-
 import java.io.File;
 import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.util.Arrays;
 import java.util.List;
+import spoon.compiler.SpoonFile;
 
 public class SourceOptions<T extends SourceOptions<T>> extends Options<T> {
 	public SourceOptions() {

src/main/java/spoon/experimental/SpoonifierVisitor.java

@@ -7,6 +7,13 @@
  */
 package spoon.experimental;
 
+import java.util.ArrayDeque;
+import java.util.ArrayList;
+import java.util.Deque;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
 import org.apache.commons.lang3.StringEscapeUtils;
 import spoon.reflect.code.CtBinaryOperator;
 import spoon.reflect.code.CtComment;
@@ -32,14 +39,6 @@
 import spoon.reflect.visitor.CtInheritanceScanner;
 import spoon.reflect.visitor.CtScanner;
 
-import java.util.ArrayDeque;
-import java.util.ArrayList;
-import java.util.Deque;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-
 /**
  * Visitor that generates factory calls to recreate the AST visited.
  *

src/main/java/spoon/experimental/package-info.java

@@ -9,4 +9,3 @@
  * this is unstable code, subject to potential complete redesign, not only in snapshots but also in releases. you can use this code but you'll probably be impacted upon future changes.
  */
 package spoon.experimental;
-

src/main/java/spoon/javadoc/internal/Javadoc.java

@@ -14,7 +14,6 @@
  */
 package spoon.javadoc.internal;
 
-import static spoon.javadoc.internal.JavadocInlineTag.nextWord;
 import java.io.Serializable;
 import java.util.Arrays;
 import java.util.Collections;
@@ -25,6 +24,8 @@
 import org.apache.commons.lang3.tuple.Pair;
 import spoon.reflect.code.CtComment;
 
+import static spoon.javadoc.internal.JavadocInlineTag.nextWord;
+
 /**
  * The structured content of a single Javadoc comment.
  *

src/main/java/spoon/javadoc/internal/JavadocBlockTag.java

@@ -14,9 +14,8 @@
  */
 package spoon.javadoc.internal;
 
-import spoon.reflect.code.CtJavaDocTag;
-
 import java.io.Serializable;
+import spoon.reflect.code.CtJavaDocTag;
 
 /**
 * A block tag.

src/main/java/spoon/metamodel/MMMethod.java

@@ -7,15 +7,14 @@
  */
 package spoon.metamodel;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
 import spoon.SpoonException;
 import spoon.reflect.declaration.CtMethod;
 import spoon.reflect.reference.CtTypeReference;
 import spoon.support.adaption.TypeAdaptor;
 
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.List;
-
 /**
  * Represents a method used to get or set a {@link MetamodelProperty} of a {@link MetamodelConcept}.
  */

src/main/java/spoon/metamodel/MMMethodKind.java

@@ -8,7 +8,6 @@
 package spoon.metamodel;
 
 import java.util.function.Predicate;
-
 import spoon.SpoonException;
 import spoon.reflect.declaration.CtField;
 import spoon.reflect.declaration.CtMethod;

src/main/java/spoon/metamodel/Metamodel.java

@@ -18,9 +18,7 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
-
 import org.jspecify.annotations.Nullable;
-
 import spoon.Launcher;
 import spoon.SpoonException;
 import spoon.reflect.annotations.PropertyGetter;