From 9eb5c459dd8010f192116b2d9d2ea0edb35a75e4 Mon Sep 17 00:00:00 2001
From: Paul Jolly <paul@myitcv.io>
Date: Sun, 26 Jan 2020 18:52:46 +0000
Subject: [PATCH] cmd/govim: provide env var to turn on gopls verbose output
 (#718)

Useful for debugging gopls issues.
---
 cmd/govim/config/config.go |  4 ++++
 cmd/govim/gopls_client.go  | 40 ++++++++++++++++++++++----------------
 2 files changed, 27 insertions(+), 17 deletions(-)

diff --git a/cmd/govim/config/config.go b/cmd/govim/config/config.go
index 76298f17b..63b2c64c9 100644
--- a/cmd/govim/config/config.go
+++ b/cmd/govim/config/config.go
@@ -21,6 +21,10 @@ const (
 	// EnvVarGoplsDebug is an environment variable which, when set, will be
 	// used to pass the value as flags to gopls.
 	EnvVarGoplsFlags EnvVar = "GOVIM_GOPLS_FLAGS"
+
+	// EnvVarGoplsVerbose is an environment variable which, when set to the
+	// value "true", configures gopls' verboseOutput option.
+	EnvVarGoplsVerbose EnvVar = "GOVIM_GOPLS_VERBOSE_OUTPUT"
 )
 
 //go:generate go run github.com/govim/govim/cmd/govim/config/internal/applygen Config
diff --git a/cmd/govim/gopls_client.go b/cmd/govim/gopls_client.go
index c4f4eec40..6d0488b34 100644
--- a/cmd/govim/gopls_client.go
+++ b/cmd/govim/gopls_client.go
@@ -3,10 +3,12 @@ package main
 import (
 	"context"
 	"fmt"
+	"os"
 	"reflect"
 	"strings"
 
 	"github.com/govim/govim"
+	"github.com/govim/govim/cmd/govim/config"
 	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/lsp/protocol"
 	"github.com/govim/govim/cmd/govim/internal/golang_org_x_tools/span"
 	"github.com/kr/pretty"
@@ -22,6 +24,7 @@ const (
 	goplsGoImportsLocalPrefix = "local"
 	goplsCompletionBudget     = "completionBudget"
 	goplsTempModfile          = "tempModfile"
+	goplsVerboseOutput        = "verboseOutput"
 )
 
 var _ protocol.Client = (*govimplugin)(nil)
@@ -115,7 +118,7 @@ func (g *govimplugin) Configuration(ctxt context.Context, params *protocol.Param
 	g.logGoplsClientf("Configuration: %v", pretty.Sprint(params))
 
 	g.vimstate.configLock.Lock()
-	config := g.vimstate.config
+	conf := g.vimstate.config
 	defer g.vimstate.configLock.Unlock()
 
 	// gopls now sends params.Items for each of the configured
@@ -126,30 +129,33 @@ func (g *govimplugin) Configuration(ctxt context.Context, params *protocol.Param
 		return nil, fmt.Errorf("govim gopls client: expected at least one item, with the first section \"gopls\"")
 	}
 	res := make([]interface{}, len(params.Items))
-	conf := make(map[string]interface{})
-	conf[goplsConfigHoverKind] = "FullDocumentation"
-	if config.CompletionDeepCompletions != nil {
-		conf[goplsDeepCompletion] = *config.CompletionDeepCompletions
+	goplsConfig := make(map[string]interface{})
+	goplsConfig[goplsConfigHoverKind] = "FullDocumentation"
+	if conf.CompletionDeepCompletions != nil {
+		goplsConfig[goplsDeepCompletion] = *conf.CompletionDeepCompletions
 	}
-	if config.CompletionMatcher != nil {
-		conf[goplsCompletionMatcher] = *config.CompletionMatcher
+	if conf.CompletionMatcher != nil {
+		goplsConfig[goplsCompletionMatcher] = *conf.CompletionMatcher
 	}
-	if config.Staticcheck != nil {
-		conf[goplsStaticcheck] = *config.Staticcheck
+	if conf.Staticcheck != nil {
+		goplsConfig[goplsStaticcheck] = *conf.Staticcheck
 	}
-	if config.CompleteUnimported != nil {
-		conf[goplsCompleteUnimported] = *config.CompleteUnimported
+	if conf.CompleteUnimported != nil {
+		goplsConfig[goplsCompleteUnimported] = *conf.CompleteUnimported
 	}
-	if g.vimstate.config.GoImportsLocalPrefix != nil {
-		conf[goplsGoImportsLocalPrefix] = *config.GoImportsLocalPrefix
+	if conf.GoImportsLocalPrefix != nil {
+		goplsConfig[goplsGoImportsLocalPrefix] = *conf.GoImportsLocalPrefix
 	}
-	if g.vimstate.config.CompletionBudget != nil {
-		conf[goplsCompletionBudget] = *config.CompletionBudget
+	if conf.CompletionBudget != nil {
+		goplsConfig[goplsCompletionBudget] = *conf.CompletionBudget
 	}
 	if g.vimstate.config.ExperimentalTempModfile != nil {
-		conf[goplsTempModfile] = *config.ExperimentalTempModfile
+		goplsConfig[goplsTempModfile] = *conf.ExperimentalTempModfile
 	}
-	res[0] = conf
+	if os.Getenv(string(config.EnvVarGoplsVerbose)) == "true" {
+		goplsConfig[goplsVerboseOutput] = true
+	}
+	res[0] = goplsConfig
 
 	g.logGoplsClientf("Configuration response: %v", pretty.Sprint(res))
 	return res, nil