Merge pull request #1074 from julia-vscode/fe/fix-executecommand

Fix missing URI conversion in code action execution.

Author: pfitzseb

src/URIs2/URIs2.jl

@@ -30,6 +30,8 @@ function URI(value::AbstractString)
     )
 end
 
+URI(uri::URI) = uri
+
 function URI(;
     scheme::Union{AbstractString,Nothing}=nothing,
     authority::Union{AbstractString,Nothing}=nothing,

src/requests/actions.jl

@@ -75,11 +75,11 @@ function textDocument_codeAction_request(params::CodeActionParams, server::Langu
 end
 
 function workspace_executeCommand_request(params::ExecuteCommandParams, server::LanguageServerInstance, conn)
-    uri = params.arguments[1]
-    offset = params.arguments[2]
-    doc = getdocument(server, uri)
-    x = get_expr(getcst(doc), offset)
     if haskey(LSActions, params.command)
+        uri = URI(params.arguments[1])
+        offset = params.arguments[2]
+        doc = getdocument(server, uri)
+        x = get_expr(getcst(doc), offset)
         LSActions[params.command].handler(x, server, conn)
     end
 end