Make BSP 1st class citizen

contrib/bsp/src/mill/contrib/BSP.scala → 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

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
+}

contrib/bsp/src/mill/contrib/bsp/BspLoggedReporter.scala → 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

contrib/bsp/src/mill/contrib/bsp/BspTestReporter.scala → bsp/src/mill/bsp/BspTestReporter.scala

@@ -1,4 +1,4 @@
-package mill.contrib.bsp
+package mill.bsp
 
 import java.io.{PrintWriter, StringWriter}
 

contrib/bsp/src/mill/contrib/bsp/MillBspLogger.scala → 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

contrib/bsp/src/mill/contrib/bsp/MillBuildServer.scala → 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

contrib/bsp/src/mill/contrib/bsp/ModuleUtils.scala → 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

contrib/bsp/src/mill/contrib/bsp/TaskParameters.scala → 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)
  }
-}
+}

contrib/bsp/src/mill/contrib/bsp/Utils.scala → 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

contrib/bsp/test/src/BspInstallTests.scala → 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"))
  }

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 =

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/), such as IntelliJ. 
+Use `mill mill.BSP/install` to generate the BSP project config for your build.
+
+It also enables Intellij to provide navigation & code-completion features 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

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: