@olafurpg thanks, I've attempted to add the correct information to specification.md

I think we should already be able to use canRun/canDebug - I didn't realise we had that. And since we implement both that and debug I don't imagine it would be super useful in case of Metals. I do believe it's a worthwhile addition and might be useful in the future.

I think we should already be able to use canRun/canDebug - I didn't realise we had that. And since we implement both that and debug I don't imagine it would be super useful in case of Metals. I do believe it's a worthwhile addition and might be useful in the future.

FYI: canDebug is only added/available as part of this PR, so I'm not sure I understand your comment about it not being useful to Metals. The example is https://github.com/adpi2/metals/pull/1/files#diff-2249f2a7f1d0baedcc556404e0eb12adcd38f5fdb4d0d640d662643bc3de5b1fR514 - which needs to be updated to be based upon the BSP capabilities rather than just .isBloop.

I got a deserialization failure when trying

    val legacyJson =
      """
        |{
        |  "canCompile": true,
        |  "canTest": true,
        |  "canRun": true
        |}""".stripMargin

against the scala code - java worked, but still need to look into that issue, so please don't merge until I can add this backward compatible test

Update

I got this fixed and added an explicit test. I had to add in circe-generic-extras that adds support for default values on the scala side.