Ignore inline completion request if parameters are out of bounds

Signed-off-by: Remy Suen <remy.suen@docker.com>

Author: rcjsuen

CHANGELOG.md

@@ -15,6 +15,9 @@ All notable changes to the Docker Language Server will be documented in this fil
 - Compose
   - textDocument/completion
     - fix error case triggered by using code completion before the first node ([#314](https://github.com/docker/docker-language-server/issues/314))
+- Bake
+  - textDocument/inlineCompletion
+    - check that the request is within the document's bounds when processing the request ([#318](https://github.com/docker/docker-language-server/issues/318))
 
 ## [0.11.0] - 2025-06-10
 

e2e-tests/inlineCompletion_test.go

@@ -31,12 +31,16 @@ func TestInlineCompletion(t *testing.T) {
 	testCases := []struct {
 		name              string
 		content           string
+		line              protocol.UInteger
+		character         protocol.UInteger
 		dockerfileContent string
 		items             []protocol.InlineCompletionItem
 	}{
 		{
 			name:              "one build stage",
 			content:           "",
+			line:              0,
+			character:         0,
 			dockerfileContent: "FROM scratch AS simple",
 			items: []protocol.InlineCompletionItem{
 				{
@@ -48,6 +52,14 @@ func TestInlineCompletion(t *testing.T) {
 				},
 			},
 		},
+		{
+			name:              "outside document bounds",
+			content:           "",
+			line:              1,
+			character:         0,
+			dockerfileContent: "FROM scratch AS simple",
+			items:             nil,
+		},
 	}
 
 	temporaryDockerfile := fmt.Sprintf("file:///%v", strings.TrimPrefix(filepath.ToSlash(filepath.Join(os.TempDir(), "Dockerfile")), "/"))
@@ -81,7 +93,7 @@ func TestInlineCompletion(t *testing.T) {
 			err = conn.Call(context.Background(), protocol.MethodTextDocumentInlineCompletion, protocol.InlineCompletionParams{
 				TextDocumentPositionParams: protocol.TextDocumentPositionParams{
 					TextDocument: protocol.TextDocumentIdentifier{URI: didOpenBakeFile.TextDocument.URI},
-					Position:     protocol.Position{Line: 0, Character: 0},
+					Position:     protocol.Position{Line: tc.line, Character: tc.character},
 				},
 			}, &result)
 			require.NoError(t, err)

internal/bake/hcl/inlineCompletion.go

@@ -40,6 +40,9 @@ func shouldSuggest(content []byte, body *hclsyntax.Body, position protocol.Posit
 	}
 
 	lines := strings.Split(string(content), "\n")
+	if len(lines) <= int(position.Line) {
+		return false
+	}
 	return strings.TrimSpace(lines[position.Line]) != "}"
 }