golang/go

x/tools/gopls: Completion: add insert-replace support (insertReplaceEdit)

Aperta

#40.871 aperta il 18 ago 2020

Vedi su GitHubGuida contributor
 (15 commenti) (25 reazioni) (0 assegnatari)Go (19.008 fork)batch import
FeatureRequestToolsgoplshelp wanted

Metriche repository

Star
 (133.883 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Currently gopls is overwriting the surrounding suffix (in addition to prefix) when inserting candidates:

// completing to "food" currently results in "food" instead of "foodoo"
f<>oo

I think it's from https://go-review.googlesource.com/c/tools/+/208501 changing the surrounding's range from [ident.Pos(), c.pos] to [ident.Pos(), ident.End()].

/cc @heschik

Guida contributor

Direzione di ricerca
Indaga il CL di riferimento (go review.googlesource.com/c/tools/+/208501) che ha cambiato l'intervallo da [ident.Pos(), c.pos] a [ident.Pos(), ident.End()], causando la sovrascrittura del suffisso. Comprendi la logica attuale di completamento in gopls e determina come implementare il supporto InsertReplaceEdit secondo le specifiche LSP, in modo simile a come lo gestiscono altri server linguistici. La correzione probabilmente richiede di modificare l'intervallo dell'elemento di completamento per coprire solo il prefisso e opzionalmente fornire una modifica del testo per la sostituzione. Controlla le issue o le discussioni esistenti nel repository per eventuali soluzioni proposte.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70