Make BSP 1st class citizen

joan38 · joan38 · commit 9ec3a4c873e9 · 2020-10-05T01:15:08.000-07:00
diff --git a/bsp/src/mill/BSP.scala b/bsp/src/mill/BSP.scala
@@ -1,37 +1,21 @@
-package mill.contrib
+package mill
 
+import ch.epfl.scala.bsp4j.BuildClient
 import java.io.PrintWriter
 import java.nio.file.FileAlreadyExistsException
 import java.util.concurrent.Executors
-import ch.epfl.scala.bsp4j._
-import mill._
-import mill.contrib.bsp.MillBuildServer
+import mill.bsp.{BspConfigJson, MillBuildServer}
 import mill.define.{Command, Discover, ExternalModule}
 import mill.eval.Evaluator
 import mill.modules.Util
 import org.eclipse.lsp4j.jsonrpc.Launcher
-import upickle.default._
-import scala.collection.JavaConverters._
 import scala.concurrent.CancellationException
-import scala.util.Try
-
-case class BspConfigJson(
-    name: String,
-    argv: Seq[String],
-    millVersion: String,
-    bspVersion: String,
-    languages: Seq[String]
-) extends BspConnectionDetails(name, argv.asJava, millVersion, bspVersion, languages.asJava)
-
-object BspConfigJson {
-  implicit val rw: ReadWriter[BspConfigJson] = macroRW
-}
+import upickle.default.write
 
 object BSP extends ExternalModule {
-
   implicit def millScoptEvaluatorReads[T] = new mill.main.EvaluatorScopt[T]()
 
-  lazy val millDiscover: Discover[BSP.this.type] = Discover[this.type]
+  lazy val millDiscover: Discover[this.type] = Discover[this.type]
   val bspProtocolVersion = "2.0.0"
   val languages = Seq("scala", "java")
 
@@ -73,15 +57,16 @@ object BSP extends ExternalModule {
   def createBspConnectionJson(): String = {
     val millPath = sys.props
       .get("java.class.path")
-      .getOrElse(throw new IllegalStateException("System property java.class.path not set"))
+      .getOrElse(throw new IllegalStateException(
+        "System property java.class.path not set"))
 
     write(
       BspConfigJson(
         "mill-bsp",
         Seq(
           "sh",
           "-c",
-          s"env ${sys.env.map { case (k, v) => s""""$k=$v"""" }.toSeq.mkString(" ")} $millPath -i mill.contrib.BSP/start"
+          s"env ${sys.env.map { case (k, v) => s""""$k=$v"""" }.toSeq.mkString(" ")} $millPath -i mill.bsp/start"
         ),
         Util.millProperty("MILL_VERSION").getOrElse(BuildInfo.millVersion),
         bspProtocolVersion,
@@ -113,7 +98,10 @@ object BSP extends ExternalModule {
         ev.env,
         false
       )
-      val millServer = new MillBuildServer(evaluator, bspProtocolVersion, BuildInfo.millVersion)
+      val millServer = new MillBuildServer(
+        evaluator,
+        bspProtocolVersion,
+        BuildInfo.millVersion)
       val executor = Executors.newCachedThreadPool()
 
       val stdin = System.in
diff --git a/bsp/src/mill/bsp/BspConfigJson.scala b/bsp/src/mill/bsp/BspConfigJson.scala
@@ -0,0 +1,17 @@
+package mill.bsp
+
+import ch.epfl.scala.bsp4j.BspConnectionDetails
+import scala.jdk.CollectionConverters._
+import upickle.default._
+
+case class BspConfigJson(
+    name: String,
+    argv: Seq[String],
+    millVersion: String,
+    bspVersion: String,
+    languages: Seq[String]
+) extends BspConnectionDetails(name, argv.asJava, millVersion, bspVersion, languages.asJava)
+
+object BspConfigJson {
+  implicit val rw: ReadWriter[BspConfigJson] = macroRW
+}
diff --git a/bsp/src/mill/bsp/BspLoggedReporter.scala b/bsp/src/mill/bsp/BspLoggedReporter.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import java.io.File
 import java.util.concurrent.ConcurrentHashMap
diff --git a/bsp/src/mill/bsp/BspTestReporter.scala b/bsp/src/mill/bsp/BspTestReporter.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import java.io.{PrintWriter, StringWriter}
 
diff --git a/bsp/src/mill/bsp/MillBspLogger.scala b/bsp/src/mill/bsp/MillBspLogger.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import ch.epfl.scala.bsp4j._
 import mill.api.Logger
diff --git a/bsp/src/mill/bsp/MillBuildServer.scala b/bsp/src/mill/bsp/MillBuildServer.scala
@@ -1,13 +1,13 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import ammonite.runtime.SpecialClassLoader
 import ch.epfl.scala.bsp4j._
 import com.google.gson.JsonObject
 import java.util.concurrent.CompletableFuture
 import mill._
 import mill.api.{DummyTestReporter, Result, Strict}
-import mill.contrib.bsp.ModuleUtils._
-import mill.contrib.bsp.Utils._
+import mill.bsp.ModuleUtils._
+import mill.bsp.Utils._
 import mill.define.Segment.Label
 import mill.define.{Discover, ExternalModule}
 import mill.eval.Evaluator
diff --git a/bsp/src/mill/bsp/ModuleUtils.scala b/bsp/src/mill/bsp/ModuleUtils.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import ch.epfl.scala.bsp4j._
 import java.net.URL
diff --git a/bsp/src/mill/bsp/TaskParameters.scala b/bsp/src/mill/bsp/TaskParameters.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import ch.epfl.scala.bsp4j.{BuildTargetIdentifier, CompileParams, RunParams, TestParams}
 
@@ -124,4 +124,4 @@ object TaskParameters {
   def fromTestParams(testParams: TestParams): Parameters = {
     TParams(testParams)
   }
-}
+}
diff --git a/bsp/src/mill/bsp/Utils.scala b/bsp/src/mill/bsp/Utils.scala
@@ -1,10 +1,10 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import ch.epfl.scala.bsp4j._
 import mill._
 import mill.api.Result.{Skipped, Success}
 import mill.api.{BuildProblemReporter, Result}
-import mill.contrib.bsp.ModuleUtils._
+import mill.bsp.ModuleUtils._
 import mill.eval.Evaluator
 import mill.modules.Jvm
 import mill.scalalib.Lib.discoverTests
diff --git a/bsp/test/resources/gen-idea-hello-world/build.sc b/bsp/test/resources/gen-idea-hello-world/build.sc
diff --git a/bsp/test/src/BspInstallTests.scala b/bsp/test/src/BspInstallTests.scala
@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import mill.util.ScriptTestSuite
 import os._
@@ -11,7 +11,7 @@ object BspInstallTests extends ScriptTestSuite(false) {
   def tests: Tests = Tests {
     "BSP install" - {
       val workspacePath = initWorkspace()
-      eval("mill.contrib.BSP/install")
+      eval("mill.bsp/install")
 
       assert(exists(workspacePath / ".bsp" /"mill.json"))
     }
diff --git a/build.sc b/build.sc
@@ -69,8 +69,7 @@ object Deps {
   def scalaReflect(scalaVersion: String) = ivy"org.scala-lang:scala-reflect:${scalaVersion}"
   def scalacScoveragePlugin = ivy"org.scoverage::scalac-scoverage-plugin:1.4.1"
   val sourcecode = ivy"com.lihaoyi::sourcecode:0.2.1"
-  val ujsonCirce = ivy"com.lihaoyi::ujson-circe:1.2.0"
-  val upickle = ivy"com.lihaoyi::upickle:1.2.0"
+  val upickle = ivy"com.lihaoyi::upickle:1.2.1"
   val utest = ivy"com.lihaoyi::utest:0.7.4"
   val zinc = ivy"org.scala-sbt::zinc:1.4.0-M1"
   val bsp = ivy"ch.epfl.scala:bsp4j:2.0.0-M4"
@@ -480,16 +479,6 @@ object contrib extends MillModule {
     def testArgs = T(scalanativelib.testArgs())
   }
 
-  object bsp extends MillModule {
-
-    override def compileModuleDeps = Seq(scalalib, scalajslib, main, scalanativelib)
-    def ivyDeps = Agg(
-      Deps.bsp,
-      Deps.ujsonCirce,
-      Deps.sbtTestInterface
-    )
-  }
-
   object artifactory extends MillModule {
     override def compileModuleDeps = Seq(scalalib)
   }
@@ -548,6 +537,14 @@ object scalanativelib extends MillModule {
   }
 }
 
+object bsp extends MillModule {
+  override def compileModuleDeps = Seq(scalalib, scalajslib, scalanativelib)
+  def ivyDeps = Agg(
+    Deps.bsp,
+    Deps.sbtTestInterface
+  )
+}
+
 def testRepos = T{
   Seq(
     "MILL_ACYCLIC_REPO" ->
@@ -677,7 +674,7 @@ def launcherScript(shellJvmArgs: Seq[String],
 }
 
 object dev extends MillModule{
-  def moduleDeps = Seq(scalalib, scalajslib, scalanativelib)
+  def moduleDeps = Seq(scalalib, scalajslib, scalanativelib, bsp)
 
 
   def forkArgs =
diff --git a/docs/pages/1 - Intro to Mill.md b/docs/pages/1 - Intro to Mill.md
@@ -701,9 +701,16 @@ mill mill.scalalib.Dependency/showUpdates
 mill mill.scalalib.Dependency/showUpdates --allowPreRelease true # also show pre-release versions
 ```
 
-## IntelliJ Support
+## IDE Support
 
-Mill supports IntelliJ by default. Use `mill mill.scalalib.GenIdea/idea` to
+Mill supports any IDE that is compatible with [BSP](https://build-server-protocol.github.io/), sush as IntelliJ, VSCode
+or Emacs. Use `mill mill.BSP/install` to generate the BSP project config for your build.
+
+This also configures your IDE to allow easy navigate & code-completion within your build file itself.
+
+## IntelliJ Support (legacy)
+
+Mill supports IntelliJ configuration generation. Use `mill mill.scalalib.GenIdea/idea` to
 generate an IntelliJ project config for your build.
 
 This also configures IntelliJ to allow easy navigate & code-completion within
diff --git a/docs/pages/9 - Contrib Modules.md b/docs/pages/9 - Contrib Modules.md
@@ -175,24 +175,11 @@ object project extends BuildInfo {
 
 ## BSP - Build Server Protocol
 
-The contrib.bsp module was created in order to integrate the Mill build tool
-with IntelliJ IDEA via the Build Server Protocol (BSP). It implements most of
-the server side functionality described in BSP, and can therefore connect to a
-BSP client, including the one behind IntelliJ IDEA. This allows a lot of mill
-tasks to be executed from the IDE.
-
-### Importing an existing mill project in IntelliJ via BSP
-
-1) Add the following import statement in the `build.sc` of your project:
-
-```scala
-import $ivy.`com.lihaoyi::mill-contrib-bsp:$MILL_VERSION`
-```
-
-2) Run the following command in the working directory of your project:
+The contrib.bsp module is now included in mill by default and will eventually replace GenIdea.
 
+Run the following command in the working directory of your project to generate the BSP config:
 ```
-mill mill.contrib.BSP/install
+mill mill.BSP/install
 ```
 
 ### Known Issues:

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-package mill.contrib.bsp`
	`1`	`+package mill.bsp`
`2`	`2`
`3`	`3`	`import java.io.File`
`4`	`4`	`import java.util.concurrent.ConcurrentHashMap`