2019-04-05 13:56:08 -06:00
|
|
|
// Copyright 2018 The Go Authors. All rights reserved.
|
|
|
|
// Use of this source code is governed by a BSD-style
|
|
|
|
// license that can be found in the LICENSE file.
|
|
|
|
|
|
|
|
package lsp
|
|
|
|
|
|
|
|
import (
|
|
|
|
"context"
|
2019-06-27 19:26:42 -06:00
|
|
|
"fmt"
|
2019-04-05 13:56:08 -06:00
|
|
|
"strings"
|
|
|
|
|
|
|
|
"golang.org/x/tools/internal/lsp/protocol"
|
|
|
|
"golang.org/x/tools/internal/lsp/source"
|
2019-07-14 21:08:10 -06:00
|
|
|
"golang.org/x/tools/internal/lsp/telemetry"
|
|
|
|
"golang.org/x/tools/internal/lsp/telemetry/log"
|
2019-04-05 13:56:08 -06:00
|
|
|
"golang.org/x/tools/internal/span"
|
|
|
|
)
|
|
|
|
|
|
|
|
func (s *Server) codeAction(ctx context.Context, params *protocol.CodeActionParams) ([]protocol.CodeAction, error) {
|
2019-06-27 19:26:42 -06:00
|
|
|
|
|
|
|
// The Only field of the context specifies which code actions the client wants.
|
|
|
|
// If Only is empty, assume that the client wants all of the possible code actions.
|
|
|
|
var wanted map[protocol.CodeActionKind]bool
|
|
|
|
if len(params.Context.Only) == 0 {
|
|
|
|
wanted = s.supportedCodeActions
|
|
|
|
} else {
|
|
|
|
wanted = make(map[protocol.CodeActionKind]bool)
|
|
|
|
for _, only := range params.Context.Only {
|
|
|
|
wanted[only] = s.supportedCodeActions[only]
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2019-04-05 13:56:08 -06:00
|
|
|
uri := span.NewURI(params.TextDocument.URI)
|
2019-06-27 19:26:42 -06:00
|
|
|
if len(wanted) == 0 {
|
|
|
|
return nil, fmt.Errorf("no supported code action to execute for %s, wanted %v", uri, params.Context.Only)
|
|
|
|
}
|
|
|
|
|
2019-05-15 10:24:49 -06:00
|
|
|
view := s.session.ViewOf(uri)
|
2019-06-20 14:57:45 -06:00
|
|
|
gof, m, err := getGoFile(ctx, view, uri)
|
2019-04-05 13:56:08 -06:00
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
|
|
|
spn, err := m.RangeSpan(params.Range)
|
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
2019-06-27 19:26:42 -06:00
|
|
|
|
2019-04-05 13:56:08 -06:00
|
|
|
var codeActions []protocol.CodeAction
|
2019-06-27 19:26:42 -06:00
|
|
|
|
2019-04-16 15:28:20 -06:00
|
|
|
edits, err := organizeImports(ctx, view, spn)
|
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
2019-06-27 19:26:42 -06:00
|
|
|
|
|
|
|
// If the user wants to see quickfixes.
|
|
|
|
if wanted[protocol.QuickFix] {
|
|
|
|
// First, add the quick fixes reported by go/analysis.
|
|
|
|
// TODO: Enable this when this actually works. For now, it's needless work.
|
|
|
|
if s.wantSuggestedFixes {
|
|
|
|
qf, err := quickFixes(ctx, view, gof)
|
|
|
|
if err != nil {
|
2019-07-14 21:08:10 -06:00
|
|
|
log.Error(ctx, "quick fixes failed", err, telemetry.File.Of(uri))
|
2019-06-27 19:26:42 -06:00
|
|
|
}
|
|
|
|
codeActions = append(codeActions, qf...)
|
|
|
|
}
|
|
|
|
|
|
|
|
// If we also have diagnostics for missing imports, we can associate them with quick fixes.
|
2019-04-16 15:28:20 -06:00
|
|
|
if findImportErrors(params.Context.Diagnostics) {
|
|
|
|
// TODO(rstambler): Separate this into a set of codeActions per diagnostic,
|
|
|
|
// where each action is the addition or removal of one import.
|
|
|
|
// This can only be done when https://golang.org/issue/31493 is resolved.
|
2019-04-16 10:50:59 -06:00
|
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
2019-04-16 15:28:20 -06:00
|
|
|
Title: "Organize All Imports", // clarify that all imports will change
|
|
|
|
Kind: protocol.QuickFix,
|
2019-04-16 10:50:59 -06:00
|
|
|
Edit: &protocol.WorkspaceEdit{
|
|
|
|
Changes: &map[string][]protocol.TextEdit{
|
2019-04-16 15:28:20 -06:00
|
|
|
string(uri): edits,
|
2019-04-16 10:50:59 -06:00
|
|
|
},
|
|
|
|
},
|
|
|
|
})
|
2019-04-05 13:56:08 -06:00
|
|
|
}
|
|
|
|
}
|
2019-06-27 19:26:42 -06:00
|
|
|
|
|
|
|
// Add the results of import organization as source.OrganizeImports.
|
|
|
|
if wanted[protocol.SourceOrganizeImports] {
|
|
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
|
|
|
Title: "Organize Imports",
|
|
|
|
Kind: protocol.SourceOrganizeImports,
|
|
|
|
Edit: &protocol.WorkspaceEdit{
|
|
|
|
Changes: &map[string][]protocol.TextEdit{
|
|
|
|
string(spn.URI()): edits,
|
|
|
|
},
|
|
|
|
},
|
|
|
|
})
|
|
|
|
}
|
|
|
|
|
2019-04-05 13:56:08 -06:00
|
|
|
return codeActions, nil
|
|
|
|
}
|
|
|
|
|
2019-06-21 15:00:02 -06:00
|
|
|
func organizeImports(ctx context.Context, view source.View, s span.Span) ([]protocol.TextEdit, error) {
|
|
|
|
f, m, rng, err := spanToRange(ctx, view, s)
|
2019-04-05 13:56:08 -06:00
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
2019-06-28 14:21:07 -06:00
|
|
|
edits, err := source.Imports(ctx, view, f, rng)
|
2019-04-05 13:56:08 -06:00
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
2019-04-16 10:50:59 -06:00
|
|
|
return ToProtocolEdits(m, edits)
|
|
|
|
}
|
|
|
|
|
2019-04-16 15:28:20 -06:00
|
|
|
// findImports determines if a given diagnostic represents an error that could
|
|
|
|
// be fixed by organizing imports.
|
|
|
|
// TODO(rstambler): We need a better way to check this than string matching.
|
|
|
|
func findImportErrors(diagnostics []protocol.Diagnostic) bool {
|
|
|
|
for _, diagnostic := range diagnostics {
|
|
|
|
// "undeclared name: X" may be an unresolved import.
|
|
|
|
if strings.HasPrefix(diagnostic.Message, "undeclared name: ") {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
// "could not import: X" may be an invalid import.
|
|
|
|
if strings.HasPrefix(diagnostic.Message, "could not import: ") {
|
|
|
|
return true
|
|
|
|
}
|
|
|
|
// "X imported but not used" is an unused import.
|
2019-07-30 09:10:00 -06:00
|
|
|
// "X imported but not used as Y" is an unused import.
|
|
|
|
if strings.Contains(diagnostic.Message, " imported but not used") {
|
2019-04-16 15:28:20 -06:00
|
|
|
return true
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return false
|
2019-04-05 13:56:08 -06:00
|
|
|
}
|
2019-06-27 19:26:42 -06:00
|
|
|
|
|
|
|
func quickFixes(ctx context.Context, view source.View, gof source.GoFile) ([]protocol.CodeAction, error) {
|
|
|
|
var codeActions []protocol.CodeAction
|
|
|
|
|
|
|
|
// TODO: This is technically racy because the diagnostics provided by the code action
|
|
|
|
// may not be the same as the ones that gopls is aware of.
|
|
|
|
// We need to figure out some way to solve this problem.
|
|
|
|
diags := gof.GetPackage(ctx).GetDiagnostics()
|
|
|
|
for _, diag := range diags {
|
|
|
|
pdiag, err := toProtocolDiagnostic(ctx, view, diag)
|
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
|
|
|
for _, ca := range diag.SuggestedFixes {
|
|
|
|
_, m, err := getGoFile(ctx, view, diag.URI())
|
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
|
|
|
edits, err := ToProtocolEdits(m, ca.Edits)
|
|
|
|
if err != nil {
|
|
|
|
return nil, err
|
|
|
|
}
|
|
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
|
|
|
Title: ca.Title,
|
|
|
|
Kind: protocol.QuickFix, // TODO(matloob): Be more accurate about these?
|
|
|
|
Edit: &protocol.WorkspaceEdit{
|
|
|
|
Changes: &map[string][]protocol.TextEdit{
|
|
|
|
string(diag.URI()): edits,
|
|
|
|
},
|
|
|
|
},
|
|
|
|
Diagnostics: []protocol.Diagnostic{pdiag},
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return codeActions, nil
|
|
|
|
}
|