Remove unused LSP helpers and fix log verbosity

Author: kindermax

internal/cli/cli.go

@@ -136,6 +136,7 @@ func Main(version string, buildDate string) int {
 		if errors.As(err, &depErr) {
 			executor.PrintDependencyTree(depErr, os.Stderr)
 			log.Errorf("%s", depErr.FailureMessage())
+
 			return getExitCode(err, 1)
 		}
 

internal/lsp/handlers.go

@@ -91,6 +91,7 @@ func (h *definitionHandler) findMixinsDefinition(doc *string, params *lsp.Defini
 
 func (h *definitionHandler) findCommandDefinition(doc *string, params *lsp.DefinitionParams) (any, error) {
 	path := normalizePath(params.TextDocument.URI)
+
 	commandName := h.parser.extractCommandReference(doc, params.Position)
 	if commandName == "" {
 		h.parser.log.Debugf("no command reference resolved at %s:%d:%d", path, params.Position.Line, params.Position.Character)

internal/lsp/server.go

@@ -25,8 +25,8 @@ func (s *lspServer) Run() error {
 	return s.server.RunStdio()
 }
 
-func lspLogVerbosity() (verbosity int) {
-	verbosity = 1
+func lspLogVerbosity() int {
+	verbosity := 1
 
 	defer func() {
 		_ = recover()

internal/lsp/treesitter.go

@@ -106,49 +106,6 @@ func executeYAMLQuery(document *string, queryText string, visit func(capture ts.
 	return false
 }
 
-func getLine(document *string, line uint32) string {
-	lines := strings.Split(*document, "\n")
-	if line >= uint32(len(lines)) {
-		return ""
-	}
-
-	return lines[line]
-}
-
-// position.
-func wordUnderCursor(text string, position *lsp.Position) string {
-	if len(text) == 0 {
-		return ""
-	}
-
-	character := position.Character
-
-	if character >= uint32(len(text)) {
-		return ""
-	}
-
-	if text[character] == ' ' {
-		return ""
-	}
-
-	// Find word boundaries
-	start := position.Character
-	for start > 0 && isWordChar(text[start-1]) {
-		start--
-	}
-
-	end := position.Character
-	for end < uint32(len(text)) && isWordChar(text[end]) {
-		end++
-	}
-
-	return text[start:end]
-}
-
-func isWordChar(c byte) bool {
-	return (c >= 'a' && c <= 'z') || (c >= 'A' && c <= 'Z') || (c >= '0' && c <= '9') || c == '_' || c == '-'
-}
-
 type parser struct {
 	log commonlog.Logger
 }
@@ -457,8 +414,10 @@ func (p *parser) findCommandNameByAnchor(document *string, anchorName string) st
 			break
 		}
 
-		var commandName string
-		var matchedAnchor string
+		var (
+			commandName   string
+			matchedAnchor string
+		)
 
 		for _, capture := range match.Captures {
 			switch capture.Name {

internal/lsp/treesitter_test.go

@@ -694,28 +694,3 @@ commands:
 		t.Fatalf("extractDependsValues() = %#v, want %#v", got, want)
 	}
 }
-
-func TestWordUnderCursor(t *testing.T) {
-	tests := []struct {
-		line     string
-		position lsp.Position
-		want     string
-	}{
-		{"test word here", lsp.Position{Character: 0}, "test"},
-		{"test word here", lsp.Position{Character: 2}, "test"},
-		{"test word here", lsp.Position{Character: 5}, "word"},
-		{"test word here", lsp.Position{Character: 10}, "here"},
-		{"test-word_123", lsp.Position{Character: 5}, "test-word_123"},
-		{"", lsp.Position{Character: 0}, ""},
-		{"test", lsp.Position{Character: 10}, ""},
-		{"test word", lsp.Position{Character: 4}, ""},
-		{"  test  ", lsp.Position{Character: 3}, "test"},
-	}
-
-	for i, tt := range tests {
-		got := wordUnderCursor(tt.line, &tt.position)
-		if got != tt.want {
-			t.Errorf("case %d: got %q, want %q", i, got, tt.want)
-		}
-	}
-}