Address review comments

Author: Duhemm

project/Build.scala

@@ -980,14 +980,13 @@ object Build {
         val defaultIDEConfig = baseDirectory.value / "out" / "default-dotty-ide-config"
         IO.write(defaultIDEConfig, dottyVersion)
         val dottyPluginSbtFile = baseDirectory.value / "out" / "dotty-plugin.sbt"
-        IO.write(dottyPluginSbtFile, s"""addSbtPlugin("$dottyOrganization" % $sbtDottyName % "$sbtDottyVersion")""")
+        IO.write(dottyPluginSbtFile, s"""addSbtPlugin("$dottyOrganization" % "$sbtDottyName" % "$sbtDottyVersion")""")
         Seq(defaultIDEConfig, dottyPluginSbtFile)
       },
       compile in Compile := Def.task {
         val workingDir = baseDirectory.value
         val coursier = workingDir / "out" / "coursier"
         val packageJson = workingDir / "package.json"
-        // val _ = (managedResources in Compile).value
         if (!coursier.exists || packageJson.lastModified > coursier.lastModified)
           runProcess(Seq("npm", "install"), wait = true, directory = workingDir)
         val tsc = workingDir / "node_modules" / ".bin" / "tsc"

vscode-dotty/src/extension.ts

@@ -16,9 +16,10 @@ export function activate(context: ExtensionContext) {
   extensionContext = context
   outputChannel = vscode.window.createOutputChannel('Dotty Language Client');
 
-  const sbtArtifact = "org.scala-sbt:sbt-launch:1.2.0"
+  const sbtArtifact = "org.scala-sbt:sbt-launch:1.2.3"
   const buildSbtFile = `${vscode.workspace.rootPath}/build.sbt`
   const dottyPluginSbtFile = path.join(extensionContext.extensionPath, './out/dotty-plugin.sbt')
+  const disableDottyIDEFile = `${vscode.workspace.rootPath}/.dotty-ide-disabled`
   const languageServerArtifactFile = `${vscode.workspace.rootPath}/.dotty-ide-artifact`
   const languageServerDefaultConfigFile = path.join(extensionContext.extensionPath, './out/default-dotty-ide-config')
   const coursierPath = path.join(extensionContext.extensionPath, './out/coursier');
@@ -42,9 +43,9 @@ export function activate(context: ExtensionContext) {
     // otherwise, try propose to start it if there's no build.sbt
     if (fs.existsSync(languageServerArtifactFile)) {
       runLanguageServer(coursierPath, languageServerArtifactFile)
-    } else if (!fs.existsSync(buildSbtFile)) {
+    } else if (!fs.existsSync(disableDottyIDEFile) && !fs.existsSync(buildSbtFile)) {
       vscode.window.showInformationMessage(
-          "This looks like an unconfigured project. Would you like to start Dotty IDE?",
+          "This looks like an unconfigured Scala project. Would you like to start the Dotty IDE?",
           "Yes", "No"
       ).then(choice => {
         if (choice == "Yes") {
@@ -57,6 +58,8 @@ export function activate(context: ExtensionContext) {
               })
             }
           })
+        } else {
+          fs.appendFile(disableDottyIDEFile, "", _ => {})
         }
       })
     }
@@ -118,7 +121,7 @@ function fetchWithCoursier(coursierPath: string, artifact: string, extra: string
 function configureIDE(sbtClasspath: string, languageServerScalaVersion: string, dottyPluginSbtFile: string) {
   return vscode.window.withProgress({
     location: vscode.ProgressLocation.Window,
-    title: 'Configuring IDE...'
+    title: 'Configuring the IDE for Dotty...'
   }, (progress) => {
 
     // Run sbt to configure the IDE. If the `DottyPlugin` is not present, dynamically load it and