golang/go

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

开放

#40,871 创建于 2020年8月18日

在 GitHub 查看贡献者指南
 (15 条评论) (25 个反应) (0 位负责人)Go (19,008 个派生)batch import
FeatureRequestToolsgoplshelp wanted

仓库指标

星标
 (133,883 个星标)
PR 合并指标
 (30 天内没有已合并 PR)

描述

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

贡献者指南

研究方向
研究引用的CL（go review.googlesource.com/c/tools/+/208501），该CL将范围从[ident.Pos(), c.pos]更改为[ident.Pos(), ident.End()]，导致后缀被覆盖。理解gopls中当前的补全逻辑，并确定如何根据LSP规范实现InsertReplaceEdit支持，类似于其他语言服务器的处理方式。修复可能需要修改补全项的范围，使其仅覆盖前缀，并可选择提供用于替换的文本编辑。检查仓库中现有的问题或讨论，寻找任何已提出的解决方案。
技术栈
go
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
Go
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70