x/tools/gopls: crash due to invalid protocol.Range in CodeAction

[adonovan]

This stack YPXbAQ was reported by telemetry:

golang.org/x/tools/gopls/internal/bug.report:35
golang.org/x/tools/gopls/internal/bug.Reportf:1
golang.org/x/tools/gopls/internal/lsp/source.FindParam:4
golang.org/x/tools/gopls/internal/lsp.canRemoveParameter:1
golang.org/x/tools/gopls/internal/lsp.refactorRewrite:14
golang.org/x/tools/gopls/internal/lsp.(*Server).codeAction:220
golang.org/x/tools/gopls/internal/lsp.(*Server).CodeAction:1
golang.org/x/tools/gopls/internal/lsp/protocol.serverDispatch:142
golang.org/x/tools/gopls/internal/lsp/lsprpc.(*StreamServer).ServeStream.ServerHandler.func3:5
golang.org/x/tools/gopls/internal/lsp/lsprpc.(*StreamServer).ServeStream.handshaker.func4:52
golang.org/x/tools/gopls/internal/lsp/protocol.Handlers.MustReplyHandler.func1:2
golang.org/x/tools/gopls/internal/lsp/protocol.Handlers.AsyncHandler.func2.2:3

This indicates that the Start or End field of the params.Range provided to CodeAction was invalid, as defined by Mapper.PositionOffset. This is surprising because editor clients presumably pass in-range values (and there's no command for code actions that could provide arbitrary values).

golang.org/x/tools/[email protected] go1.21.3 darwin/arm64 vscode (1)

Duplicates:

This stack 42OsdA was reported by telemetry:

go/ast.Walk:184
go/ast.Inspect:?397
golang.org/x/tools/gopls/internal/lsp.canRemoveParameter:22
golang.org/x/tools/gopls/internal/lsp.refactorRewrite:14
golang.org/x/tools/gopls/internal/lsp.(*Server).codeAction:220
golang.org/x/tools/gopls/internal/lsp.(*Server).CodeAction:1
golang.org/x/tools/gopls/internal/lsp/protocol.serverDispatch:142
golang.org/x/tools/gopls/internal/lsp/lsprpc.(*StreamServer).ServeStream.ServerHandler.func3:5
golang.org/x/tools/gopls/internal/lsp/lsprpc.(*StreamServer).ServeStream.handshaker.func4:52
golang.org/x/tools/gopls/internal/lsp/protocol.Handlers.MustReplyHandler.func1:2

golang.org/x/tools/[email protected] go1.21.3 windows/amd64 vscode (1)

Issue created by golang.org/x/tools/gopls/internal/telemetry/cmd/stacks.

[findleyr]

This bug report was removed in https://go.dev/cl/548738, since it's technically out of our control.