fix(completion): no space after cursor

[qvalentin]

#118

[qvalentin]

@CodiumAI-Agent /review

[QodoAI-Agent]

PR Reviewer Guide 🔍

(Review updated until commit c0f1c77)

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis ✅ 118 - Fully compliant Fully compliant requirements: Fix inconsistent auto-completion behavior in NeoVim when editing existing template keys. Ensure .Values. and .Values.f provide correct completions in all contexts. Add relevant tests to validate the fix. Not compliant requirements:

⏱️ Estimated effort to review: 4 🔵🔵🔵🔵⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Possible Issue The function NestedNodeAtPositionForCompletion modifies the logic for determining the current node and its children. Ensure that this change does not introduce regressions in other parts of the codebase that rely on this function. func NestedNodeAtPositionForCompletion(tree *sitter.Tree, position lsp.Position) *sitter.Node { pointToLookUp := sitter.Point{ Row: position.Line, Column: position.Character - 1, } currentNode := tree.RootNode().NamedDescendantForPointRange(pointToLookUp, pointToLookUp) return FindRelevantChildNodeCompletion(currentNode, pointToLookUp) Logic Change The handling of NodeTypeDotSymbol and NodeTypeSelectorExpression has been updated. Verify that these changes correctly address the ticket requirements without breaking existing functionality. case gotemplate.NodeTypeDotSymbol: // DotSymbol appears inside a SelectorExpression or at the end of an UnfinishedSelectorExpression v.symbolTable.AddTemplateContext(append(v.currentContext, ""), GetRangeForNode(node)) case gotemplate.NodeTypeFieldIdentifier: content := node.Content(v.content) v.symbolTable.AddTemplateContext(append(v.currentContext, content), GetRangeForNode(node)) case gotemplate.NodeTypeField: content := node.ChildByFieldName("name").Content(v.content) v.symbolTable.AddTemplateContext(append(v.currentContext, content), GetRangeForNode(node.ChildByFieldName("name"))) case gotemplate.NodeTypeSelectorExpression: operandNode := node.ChildByFieldName("operand") if operandNode != nil && operandNode.Type() == gotemplate.NodeTypeVariable { v.StashContext() v.PushContext(operandNode.Content(v.content)) }

[QodoAI-Agent]

Persistent review updated to latest commit c0f1c77