mirror of
https://github.com/golang/go
synced 2024-11-18 21:05:02 -07:00
d7101b74a4
The early return logic for didOpen events in (*snapshot).invalidateContent was preventing the creation of a new snapshot, which was in turn stopping the versions from being updated. This exposed a fundamental issue in the way we were calculating workspace diagnostics. Since we weren't waiting for diagnostics to be completed for an entire snapshot before replying that the server had been initialized, snapshots were being cloned without any type information. For quickfix code actions, we assume that we have all information cached (since we need to have sent the diagnostics that the quickfix is mapped to), so we were not finding the cached analysis results. To handle this in the short-term, we key analyses by their names, and then regenerate results as-needed for code actions. This is technically more correct than simply assuming that we have the analyses cached. In a follow-up CL, I will send a follow-up that will make sure that snapshots "wait" on each other to be fully constructed before being cloned. Change-Id: Ie89fcdb438b6b8b675f87335561bf47b768641ac Reviewed-on: https://go-review.googlesource.com/c/tools/+/208265 Run-TryBot: Rebecca Stambler <rstambler@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Cottrell <iancottrell@google.com> Reviewed-by: Heschi Kreinick <heschi@google.com>
260 lines
8.4 KiB
Go
260 lines
8.4 KiB
Go
// 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"
|
|
"fmt"
|
|
"sort"
|
|
"strings"
|
|
|
|
"golang.org/x/tools/internal/imports"
|
|
"golang.org/x/tools/internal/lsp/protocol"
|
|
"golang.org/x/tools/internal/lsp/source"
|
|
"golang.org/x/tools/internal/lsp/telemetry"
|
|
"golang.org/x/tools/internal/span"
|
|
"golang.org/x/tools/internal/telemetry/log"
|
|
errors "golang.org/x/xerrors"
|
|
)
|
|
|
|
func (s *Server) codeAction(ctx context.Context, params *protocol.CodeActionParams) ([]protocol.CodeAction, error) {
|
|
uri := span.NewURI(params.TextDocument.URI)
|
|
view, err := s.session.ViewOf(uri)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
f, err := view.GetFile(ctx, uri)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
snapshot := view.Snapshot()
|
|
fh := snapshot.Handle(ctx, f)
|
|
|
|
// Determine the supported actions for this file kind.
|
|
supportedCodeActions, ok := view.Options().SupportedCodeActions[fh.Identity().Kind]
|
|
if !ok {
|
|
return nil, fmt.Errorf("no supported code actions for %v file kind", fh.Identity().Kind)
|
|
}
|
|
|
|
// 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 = supportedCodeActions
|
|
} else {
|
|
wanted = make(map[protocol.CodeActionKind]bool)
|
|
for _, only := range params.Context.Only {
|
|
wanted[only] = supportedCodeActions[only]
|
|
}
|
|
}
|
|
if len(wanted) == 0 {
|
|
return nil, errors.Errorf("no supported code action to execute for %s, wanted %v", uri, params.Context.Only)
|
|
}
|
|
|
|
var codeActions []protocol.CodeAction
|
|
switch fh.Identity().Kind {
|
|
case source.Mod:
|
|
if !wanted[protocol.SourceOrganizeImports] {
|
|
return nil, nil
|
|
}
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
|
Title: "Tidy",
|
|
Kind: protocol.SourceOrganizeImports,
|
|
Command: protocol.Command{
|
|
Title: "Tidy",
|
|
Command: "tidy",
|
|
Arguments: []interface{}{
|
|
f.URI(),
|
|
},
|
|
},
|
|
})
|
|
case source.Go:
|
|
edits, editsPerFix, err := source.AllImportsFixes(ctx, snapshot, f)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
if diagnostics := params.Context.Diagnostics; wanted[protocol.QuickFix] && len(diagnostics) > 0 {
|
|
// First, add the quick fixes reported by go/analysis.
|
|
qf, err := quickFixes(ctx, snapshot, f, diagnostics)
|
|
if err != nil {
|
|
log.Error(ctx, "quick fixes failed", err, telemetry.File.Of(uri))
|
|
}
|
|
codeActions = append(codeActions, qf...)
|
|
|
|
// If we also have diagnostics for missing imports, we can associate them with quick fixes.
|
|
if findImportErrors(diagnostics) {
|
|
// Separate this into a set of codeActions per diagnostic, where
|
|
// each action is the addition, removal, or renaming of one import.
|
|
for _, importFix := range editsPerFix {
|
|
// Get the diagnostics this fix would affect.
|
|
if fixDiagnostics := importDiagnostics(importFix.Fix, diagnostics); len(fixDiagnostics) > 0 {
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
|
Title: importFixTitle(importFix.Fix),
|
|
Kind: protocol.QuickFix,
|
|
Edit: protocol.WorkspaceEdit{
|
|
DocumentChanges: documentChanges(fh, importFix.Edits),
|
|
},
|
|
Diagnostics: fixDiagnostics,
|
|
})
|
|
}
|
|
}
|
|
}
|
|
}
|
|
if wanted[protocol.SourceOrganizeImports] && len(edits) > 0 {
|
|
codeActions = append(codeActions, protocol.CodeAction{
|
|
Title: "Organize Imports",
|
|
Kind: protocol.SourceOrganizeImports,
|
|
Edit: protocol.WorkspaceEdit{
|
|
DocumentChanges: documentChanges(fh, edits),
|
|
},
|
|
})
|
|
}
|
|
default:
|
|
// Unsupported file kind for a code action.
|
|
return nil, nil
|
|
}
|
|
return codeActions, nil
|
|
}
|
|
|
|
func (s *Server) getSupportedCodeActions() []protocol.CodeActionKind {
|
|
allCodeActionKinds := make(map[protocol.CodeActionKind]struct{})
|
|
for _, kinds := range s.session.Options().SupportedCodeActions {
|
|
for kind := range kinds {
|
|
allCodeActionKinds[kind] = struct{}{}
|
|
}
|
|
}
|
|
var result []protocol.CodeActionKind
|
|
for kind := range allCodeActionKinds {
|
|
result = append(result, kind)
|
|
}
|
|
sort.Slice(result, func(i, j int) bool {
|
|
return result[i] < result[j]
|
|
})
|
|
return result
|
|
}
|
|
|
|
type protocolImportFix struct {
|
|
fix *imports.ImportFix
|
|
edits []protocol.TextEdit
|
|
}
|
|
|
|
// 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.
|
|
// "X imported but not used as Y" is an unused import.
|
|
if strings.Contains(diagnostic.Message, " imported but not used") {
|
|
return true
|
|
}
|
|
}
|
|
return false
|
|
}
|
|
|
|
func importFixTitle(fix *imports.ImportFix) string {
|
|
var str string
|
|
switch fix.FixType {
|
|
case imports.AddImport:
|
|
str = fmt.Sprintf("Add import: %s %q", fix.StmtInfo.Name, fix.StmtInfo.ImportPath)
|
|
case imports.DeleteImport:
|
|
str = fmt.Sprintf("Delete import: %s %q", fix.StmtInfo.Name, fix.StmtInfo.ImportPath)
|
|
case imports.SetImportName:
|
|
str = fmt.Sprintf("Rename import: %s %q", fix.StmtInfo.Name, fix.StmtInfo.ImportPath)
|
|
}
|
|
return str
|
|
}
|
|
|
|
func importDiagnostics(fix *imports.ImportFix, diagnostics []protocol.Diagnostic) (results []protocol.Diagnostic) {
|
|
for _, diagnostic := range diagnostics {
|
|
switch {
|
|
// "undeclared name: X" may be an unresolved import.
|
|
case strings.HasPrefix(diagnostic.Message, "undeclared name: "):
|
|
ident := strings.TrimPrefix(diagnostic.Message, "undeclared name: ")
|
|
if ident == fix.IdentName {
|
|
results = append(results, diagnostic)
|
|
}
|
|
// "could not import: X" may be an invalid import.
|
|
case strings.HasPrefix(diagnostic.Message, "could not import: "):
|
|
ident := strings.TrimPrefix(diagnostic.Message, "could not import: ")
|
|
if ident == fix.IdentName {
|
|
results = append(results, diagnostic)
|
|
}
|
|
// "X imported but not used" is an unused import.
|
|
// "X imported but not used as Y" is an unused import.
|
|
case strings.Contains(diagnostic.Message, " imported but not used"):
|
|
idx := strings.Index(diagnostic.Message, " imported but not used")
|
|
importPath := diagnostic.Message[:idx]
|
|
if importPath == fmt.Sprintf("%q", fix.StmtInfo.ImportPath) {
|
|
results = append(results, diagnostic)
|
|
}
|
|
}
|
|
}
|
|
return results
|
|
}
|
|
|
|
func quickFixes(ctx context.Context, snapshot source.Snapshot, f source.File, diagnostics []protocol.Diagnostic) ([]protocol.CodeAction, error) {
|
|
var codeActions []protocol.CodeAction
|
|
cphs, err := snapshot.PackageHandles(ctx, f)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
// We get the package that source.Diagnostics would've used. This is hack.
|
|
// TODO(golang/go#32443): The correct solution will be to cache diagnostics per-file per-snapshot.
|
|
cph, err := source.WidestCheckPackageHandle(cphs)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
for _, diag := range diagnostics {
|
|
// This code assumes that the analyzer name is the Source of the diagnostic.
|
|
// If this ever changes, this will need to be addressed.
|
|
srcErr, err := snapshot.FindAnalysisError(ctx, cph.ID(), diag.Source, diag.Message, diag.Range)
|
|
if err != nil {
|
|
continue
|
|
}
|
|
for _, fix := range srcErr.SuggestedFixes {
|
|
action := protocol.CodeAction{
|
|
Title: fix.Title,
|
|
Kind: protocol.QuickFix,
|
|
Diagnostics: []protocol.Diagnostic{diag},
|
|
Edit: protocol.WorkspaceEdit{},
|
|
}
|
|
for uri, edits := range fix.Edits {
|
|
f, err := snapshot.View().GetFile(ctx, uri)
|
|
if err != nil {
|
|
log.Error(ctx, "no file", err, telemetry.URI.Of(uri))
|
|
continue
|
|
}
|
|
fh := snapshot.Handle(ctx, f)
|
|
action.Edit.DocumentChanges = append(action.Edit.DocumentChanges, documentChanges(fh, edits)...)
|
|
}
|
|
codeActions = append(codeActions, action)
|
|
}
|
|
}
|
|
return codeActions, nil
|
|
}
|
|
|
|
func documentChanges(fh source.FileHandle, edits []protocol.TextEdit) []protocol.TextDocumentEdit {
|
|
return []protocol.TextDocumentEdit{
|
|
{
|
|
TextDocument: protocol.VersionedTextDocumentIdentifier{
|
|
Version: fh.Identity().Version,
|
|
TextDocumentIdentifier: protocol.TextDocumentIdentifier{
|
|
URI: protocol.NewURI(fh.Identity().URI),
|
|
},
|
|
},
|
|
Edits: edits,
|
|
},
|
|
}
|
|
}
|