Add scalafix configuration

Author: Gedochao

.scalafix.conf

@@ -0,0 +1,22 @@
+rules  = [
+  DisableSyntax,
+  RemoveUnused,
+  OrganizeImports,
+  NoValInForComprehension,
+  ProcedureSyntax
+]
+DisableSyntax.noFinalize = true
+DisableSyntax.noIsInstanceOf = true
+DisableSyntax.noReturns = true
+
+// `rules` on compilation
+triggered.rules = [
+  DisableSyntax
+]
+
+OrganizeImports {
+  coalesceToWildcardImportThreshold = 6
+  expandRelative = true
+  groups = ["*", "re:javax?\\.", "scala."]
+  groupedImports = AggressiveMerge
+}

build.mill.scala

@@ -2,6 +2,7 @@ import $ivy.`de.tototec::de.tobiasroeser.mill.vcs.version::0.4.1`
 import $ivy.`io.github.alexarchambault.mill::mill-native-image::0.1.31-1`
 import $ivy.`io.github.alexarchambault.mill::mill-native-image-upload:0.1.31-1`
 import $ivy.`io.get-coursier::coursier-launcher:2.1.24`
+import $ivy.`com.goyeau::mill-scalafix::0.5.1`
 import build_.package_.native0
 import de.tobiasroeser.mill.vcs.version._
 import io.github.alexarchambault.millnativeimage.NativeImage
@@ -15,6 +16,7 @@ import org.jgrapht.graph.DefaultGraphType.simple
 import scala.annotation.unused
 import scala.concurrent.duration._
 import scala.util.Properties.isWin
+import com.goyeau.mill.scalafix.ScalafixModule
 
 object Versions {
   def scala213                = "2.13.16"
@@ -29,9 +31,13 @@ object Versions {
   def scoptVersion            = "4.1.0"
   def ubuntuVersion           = "24.04"
 }
-object cli extends Cli
-trait Cli extends ScalaModule with ScalaJsCliPublishModule {
+trait ScalaJsCliModule extends ScalaModule with ScalafixModule {
+  override def scalacOptions: Target[Seq[String]] = super.scalacOptions.map(_ ++ Seq("-Wunused"))
   def scalaVersion: Target[String] = Versions.scala213
+}
+
+object cli extends Cli
+trait Cli extends ScalaJsCliModule with ScalaJsCliPublishModule {
   def artifactName: Target[String] = "scalajs" + super.artifactName()
   def ivyDeps: Target[Agg[Dep]] = super.ivyDeps() ++ Seq(
     ivy"org.scala-js::scalajs-linker:${Versions.scalaJsVersion}",
@@ -97,8 +103,7 @@ trait Cli extends ScalaModule with ScalaJsCliPublishModule {
   }
 }
 
-trait ScalaJsCliNativeImage extends ScalaModule with NativeImage {
-  def scalaVersion: Target[String] = Versions.scala213
+trait ScalaJsCliNativeImage extends ScalaJsCliModule with NativeImage {
 
   def nativeImageClassPath: Target[Seq[PathRef]] = Task {
     runClasspath()
@@ -176,8 +181,7 @@ trait ScalaJsCliMostlyStaticNativeImage extends ScalaJsCliNativeImage {
 object `native-mostly-static` extends ScalaJsCliMostlyStaticNativeImage
 
 @unused
-object tests extends ScalaModule {
-  def scalaVersion: Target[String] = Versions.scala213
+object tests extends ScalaJsCliModule {
 
   @unused
   object test extends ScalaTests with TestModule.Munit {

cli/src/org/scalajs/cli/Scalajsld.scala

@@ -7,29 +7,21 @@
 \*                                                                                     */
 
 package org.scalajs.cli
-
+import org.scalajs.cli.internal.{EsVersionParser, ImportMapJsonIr, ModuleSplitStyleParser}
 import org.scalajs.ir.ScalaJSVersions
-
-import org.scalajs.logging._
-
 import org.scalajs.linker._
+import org.scalajs.linker.interface.CheckedBehavior.Compliant
 import org.scalajs.linker.interface._
-
-import CheckedBehavior.Compliant
-
-import scala.concurrent.{Await, Future}
-import scala.concurrent.duration.Duration
-import scala.concurrent.ExecutionContext.Implicits.global
+import org.scalajs.logging._
 
 import java.io.File
+import java.lang.NoClassDefFoundError
 import java.net.URI
 import java.nio.file.Path
-import java.lang.NoClassDefFoundError
-import org.scalajs.cli.internal.{EsVersionParser, ModuleSplitStyleParser}
-import org.scalajs.cli.internal.ImportMapJsonIr.ImportMap
 
-import com.github.plokhotnyuk.jsoniter_scala.core._
-import org.scalajs.cli.internal.ImportMapJsonIr
+import scala.concurrent.ExecutionContext.Implicits.global
+import scala.concurrent.duration.Duration
+import scala.concurrent.{Await, Future}
 
 object Scalajsld {
 

cli/src/org/scalajs/cli/Scalajsp.scala

@@ -8,27 +8,20 @@
 
 package org.scalajs.cli
 
-import org.scalajs.ir.ScalaJSVersions
-import org.scalajs.ir.Trees.{Tree, ClassDef}
 import org.scalajs.ir.Printers.IRTreePrinter
-
+import org.scalajs.ir.ScalaJSVersions
 import org.scalajs.linker._
 import org.scalajs.linker.interface._
 import org.scalajs.linker.interface.unstable.IRFileImpl
-import org.scalajs.linker.standard._
 
-import scala.collection.immutable
+import java.io.{Console => _, _}
 
+import scala.collection.immutable
+import scala.concurrent.ExecutionContext.Implicits.global
 import scala.concurrent._
 import scala.concurrent.duration.Duration
-import scala.concurrent.ExecutionContext.Implicits.global
-
 import scala.util.{Failure, Success}
 
-import java.io.{Console => _, _}
-import java.util.zip.{ZipFile, ZipEntry}
-import java.nio.file.Path
-
 object Scalajsp {
 
   private case class Options(

cli/src/org/scalajs/cli/internal/ImportMap.scala

@@ -1,10 +1,11 @@
 package org.scalajs.cli.internal
 
+import com.armanbilge.sjsimportmap.ImportMappedIRFile
 import com.github.plokhotnyuk.jsoniter_scala.core._
 import com.github.plokhotnyuk.jsoniter_scala.macros._
 import org.scalajs.linker.interface.IRFile
+
 import java.io.File
-import com.armanbilge.sjsimportmap.ImportMappedIRFile
 
 object ImportMapJsonIr {