Revert

rcosta358 · rcosta358 · commit 3220f5b6d0e1 · 2026-03-27T15:00:59.000Z
diff --git a/client/src/services/events.ts b/client/src/services/events.ts
@@ -3,7 +3,7 @@ import { extension } from '../state';
 import { updateStateMachine } from './state-machine';
 import { SELECTION_DEBOUNCE_MS } from '../utils/constants';
 import { getSelectionContextVariables, normalizeRange } from './context';
-import { normalizeFilePath } from '../utils/utils';
+import { normalizeFilePath, toRange } from '../utils/utils';
 import { updateErrorAtCursor } from './diagnostics';
 import type { Range } from '../types/context';
 
@@ -61,13 +61,8 @@ export async function onSelectionChange(event: vscode.TextEditorSelectionChangeE
  */
 function handleContextUpdate(selection: vscode.Selection) {
     if (!extension.file || !extension.context) return;
-    const range: Range = {
-        lineStart: selection.start.line,
-        colStart: selection.start.character,
-        lineEnd: selection.end.line,
-        colEnd: selection.end.character
-    }
-    const normalizedRange = normalizeRange(range);
+    
+    const normalizedRange = normalizeRange(toRange(selection));
     const { allVars, visibleVars } = getSelectionContextVariables(extension.file, normalizedRange);
     extension.currentSelection = normalizedRange;
     extension.context.visibleVars = visibleVars;
diff --git a/client/src/services/hover.ts b/client/src/services/hover.ts
@@ -2,7 +2,7 @@ import * as vscode from 'vscode';
 import { extension } from '../state';
 import type { Range, LJVariable } from '../types/context';
 import { getSelectionContextVariables } from './context';
-import { getOriginalVariableName, normalizeFilePath } from '../utils/utils';
+import { getOriginalVariableName, normalizeFilePath, toRange } from '../utils/utils';
 
 /**
  * Initializes hover provider for LiquidJava diagnostics
@@ -37,12 +37,7 @@ function getHoveredVariable(document: vscode.TextDocument, position: vscode.Posi
 
     const hoveredWord = document.getText(wordRange);
     const file = normalizeFilePath(document.uri.fsPath);
-    const hoveredRange: Range = {
-        lineStart: position.line + 1,
-        colStart: position.character + 1,
-        lineEnd: position.line + 1,
-        colEnd: position.character + 1
-    }
+    const hoveredRange = toRange(wordRange);
     const { allVars } = getSelectionContextVariables(file, hoveredRange);
     return allVars.find(variable => getOriginalVariableName(variable.name) === hoveredWord);
 }
diff --git a/client/src/utils/utils.ts b/client/src/utils/utils.ts
@@ -142,3 +142,12 @@ export function normalizeFilePath(fsPath: string): string {
     // uppercase windows drive letter (c:\ -> C:\)
     return path.normalize(fsPath).replace(/^([a-z]):\\/, (_, drive) => drive.toUpperCase() + ':\\');
 }
+
+export function toRange(selection: vscode.Selection | vscode.Range): Range {
+    return {
+        lineStart: selection.start.line,
+        colStart: selection.start.character,
+        lineEnd: selection.end.line,
+        colEnd: selection.end.character
+    };
+}
