Does Scala CLI support the workspace/reload BSP endpoint?

[julienrf]

Please see the detailed description here: https://youtrack.jetbrains.com/issue/SCL-20667/Re-importing-a-BSP-project-does-not-work

[tgodzik]

I commented on the issues. It's supported but it does nothing. The build definition should refresh automatically and we send a notification about it. @Gedochao Any idea what might be wrong? We had the same issue in Metals for sbt BSP

[Gedochao]

It actually doesn't do nothing, we should be supporting workspace/reload.

scala-cli/modules/build/src/main/scala/scala/build/bsp/BuildServerProxy.scala

Line 80 in adca9c2

override def workspaceReload(): CompletableFuture[AnyRef] =

scala-cli/modules/build/src/main/scala/scala/build/bsp/BspImpl.scala

Line 557 in 01960be

private def onReload(): CompletableFuture[AnyRef] = {

After testing it a bit just now, it technically does work, but in this use case it's a bit wonky.

Screen.Recording.2022-12-09.at.10.05.31.mov

after adding a new directive/dependency in the code, workspace/reload doesn't seem to pick it up until the first re-compilation is triggered. It doesn't matter if it's from the command line, or if it's a re-run from IntelliJ and the re-run should go through BSP).

Not sure where's the bug, but it's worth investigating on our side. I'll create a ticket.

[Gedochao]

can be tracked under #1673

[Gedochao]

Should be fixed in #1681