lsp: Fix ast error line index bug

Signed-off-by: Charlie Egan <charlie@styra.com>
StyraInc · May 22, 2024 · de68881 · de68881
1 parent 2ca2df8
commit de68881
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/internal/lsp/lint.go b/internal/lsp/lint.go
@@ -67,17 +67,17 @@ func updateParse(cache *cache.Cache, uri string) (bool, error) {
 	diags := make([]types.Diagnostic, 0)
 
 	for _, astError := range astErrors {
-		lineLength := 1
-
-		if astError.Location.Row-1 < len(lines) {
-			lineLength = len(lines[astError.Location.Row-1])
-		}
 
 		line := astError.Location.Row - 1
 		if line < 0 {
 			line = 0
 		}
 
+		lineLength := 1
+		if line < len(lines) {
+			lineLength = len(lines[line])
+		}
+
 		key := "regal/parse"
 		link := "https://docs.styra.com/opa/category/rego-parse-error"