fix: lsp don't log too many errors

Author: vknabel

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - lsp: fix jump to definition
 - lsp: fix multiline block comments
+- lsp: fix logging too many errors
 - fix: extern docs generation
 - stdlib: new markup and markdown library
 

langsrv/handler-completion.go

@@ -9,8 +9,8 @@ import (
 func textDocumentCompletion(context *glsp.Context, params *protocol.CompletionParams) (interface{}, error) {
 	rc := NewReqContextAtPosition(&params.TextDocumentPositionParams)
 	sourceFile, err := rc.parseSourceFile()
-	if err != nil {
-		return nil, err
+	if err != nil && sourceFile == nil {
+		return nil, nil
 	}
 	completionItems := []protocol.CompletionItem{}
 	for _, decl := range sourceFile.Declarations {

langsrv/handler-declaration.go

@@ -8,12 +8,12 @@ import (
 func textDocumentDeclaration(context *glsp.Context, params *protocol.DeclarationParams) (interface{}, error) {
 	rc := NewReqContextAtPosition(&params.TextDocumentPositionParams)
 	sourceFile, err := rc.parseSourceFile()
-	if err != nil {
-		return nil, err
+	if err != nil && sourceFile == nil {
+		return nil, nil
 	}
 	token, _, err := rc.findToken()
 	if err != nil {
-		return nil, err
+		return nil, nil
 	}
 	for _, decl := range sourceFile.Declarations {
 		if string(decl.DeclName()) != token || decl.Meta().Source == nil {

langsrv/handler-definition.go

@@ -8,12 +8,12 @@ import (
 func textDocumentDefinition(context *glsp.Context, params *protocol.DefinitionParams) (interface{}, error) {
 	rc := NewReqContextAtPosition(&params.TextDocumentPositionParams)
 	sourceFile, err := rc.parseSourceFile()
-	if err != nil {
-		return nil, err
+	if err != nil && sourceFile == nil {
+		return nil, nil
 	}
 	token, _, err := rc.findToken()
-	if err != nil {
-		return nil, err
+	if err != nil && token == "" {
+		return nil, nil
 	}
 	for _, decl := range sourceFile.Declarations {
 		if string(decl.DeclName()) != token || decl.Meta().Source == nil {

langsrv/handler-hover.go

@@ -9,12 +9,12 @@ import (
 func textDocumentHover(context *glsp.Context, params *protocol.HoverParams) (*protocol.Hover, error) {
 	rc := NewReqContextAtPosition(&params.TextDocumentPositionParams)
 	sourceFile, err := rc.parseSourceFile()
-	if err != nil {
-		return nil, err
+	if err != nil && sourceFile == nil {
+		return nil, nil
 	}
 	name, tokenRange, err := rc.findToken()
-	if err != nil {
-		return nil, err
+	if err != nil && tokenRange == nil {
+		return nil, nil
 	}
 	for _, decl := range sourceFile.Declarations {
 		if string(decl.DeclName()) != name {

langsrv/handler-type-definition.go

@@ -7,9 +7,9 @@ import (
 
 func textDocumentTypeDefinition(context *glsp.Context, params *protocol.TypeDefinitionParams) (interface{}, error) {
 	rc := NewReqContextAtPosition(&params.TextDocumentPositionParams)
-	_, err := rc.parseSourceFile()
-	if err != nil {
-		return nil, err
+	sourceFile, err := rc.parseSourceFile()
+	if err != nil && sourceFile == nil {
+		return nil, nil
 	}
 	return nil, nil
 }