golang/go

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

Offen

#40.871 geöffnet am 18.08.2020

Auf GitHub ansehenContributor Guide
 (15 Kommentare) (25 Reaktionen) (0 zugewiesene Personen)Go (19.008 Forks)batch import
FeatureRequestToolsgoplshelp wanted

Repository-Metriken

Stars
 (133.883 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche den referenzierten CL (go review.googlesource.com/c/tools/+/208501), der den Bereich von [ident.Pos(), c.pos] zu [ident.Pos(), ident.End()] geändert hat, was zur Überschreibung des Suffixes führte. Verstehe die aktuelle Vervollständigungslogik in gopls und bestimme, wie InsertReplaceEdit Unterstützung gemäß der LSP Spezifikation implementiert werden kann, ähnlich wie andere Sprachserver es handhaben. Die Korrektur erfordert wahrscheinlich, den Bereich des Vervollständigungselements so zu ändern, dass er nur das Präfix abdeckt und optional eine Textbearbeitung für den Ersatz bereitstellt. Überprüfe vorhandene Issues oder Diskussionen im Repository auf etwaige vorgeschlagene Lösungen.
Tech Stack
go
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70