Fix hover range calculation: eliminate duplicate calls and remove whitespace from ranges

internal/ls/hover.go

@@ -31,6 +31,12 @@
 	if quickInfo == "" {
 		return lsproto.HoverOrNull{}, nil
 	}
+
+	// Calculate the applicable range for the hover
+	rangeNode := getNodeForQuickInfo(node)
+	textRange := core.NewTextRange(rangeNode.Pos(), rangeNode.End())
+	hoverRange := l.converters.ToLSPRange(file, textRange)
+
 	return lsproto.HoverOrNull{
 		Hover: &lsproto.Hover{
 			Contents: lsproto.MarkupContentOrStringOrMarkedStringWithLanguageOrMarkedStrings{
@@ -39,6 +45,7 @@
 					Value: formatQuickInfo(quickInfo) + documentation,
 				},
 			},
+			Range: &hoverRange,
 		},
 	}, nil
 }

internal/ls/hover_test.go

@@ -47,6 +47,10 @@ function myFunction() {
 							Value: "```tsx\nfunction myFunction(): string\n```\nA function with JSDoc links that previously caused panic\n`console.log` and `Array.from` and `Object.keys`",
 						},
 					},
+					Range: &lsproto.Range{
+						Start: lsproto.Position{Line: 6, Character: 1},
+						End:   lsproto.Position{Line: 8, Character: 10},
+					},
 				},
 			},
 		},
@@ -70,6 +74,10 @@ myFunction();`,
 							Value: "```tsx\nfunction myFunction(param: string): string\n```\n\n\n*@param* `param` - the greatest of days\n",
 						},
 					},
+					Range: &lsproto.Range{
+						Start: lsproto.Position{Line: 3, Character: 8},
+						End:   lsproto.Position{Line: 3, Character: 19},
+					},
 				},
 			},
 		},
@@ -93,6 +101,10 @@ function myFunction(param) {
 							Value: "```tsx\nfunction myFunction(param: string): string\n```\n\n\n*@param* `param` - the greatest of days\n",
 						},
 					},
+					Range: &lsproto.Range{
+						Start: lsproto.Position{Line: 5, Character: 1},
+						End:   lsproto.Position{Line: 7, Character: 10},
+					},
 				},
 			},
 		},
@@ -116,6 +128,10 @@ myFunction();`,
 							Value: "```tsx\n(parameter) param: string\n```\n- the greatest of days\n",
 						},
 					},
+					Range: &lsproto.Range{
+						Start: lsproto.Position{Line: 3, Character: 20},
+						End:   lsproto.Position{Line: 3, Character: 25},
+					},
 				},
 			},
 		},
@@ -136,6 +152,28 @@ myFunction();`,
 				"marker": nil,
 			},
 		},
+		{
+			title: "HoverRangeTest",
+			input: `
+// @filename: index.ts
+function /*marker*/testFunction(param: string): string {
+    return param;
+}`,
+			expected: map[string]*lsproto.Hover{
+				"marker": {
+					Contents: lsproto.MarkupContentOrStringOrMarkedStringWithLanguageOrMarkedStrings{
+						MarkupContent: &lsproto.MarkupContent{
+							Kind:  lsproto.MarkupKindMarkdown,
+							Value: "```tsx\nfunction testFunction(param: string): string\n```\n",
+						},
+					},
+					Range: &lsproto.Range{
+						Start: lsproto.Position{Line: 0, Character: 8},
+						End:   lsproto.Position{Line: 0, Character: 21},
+					},
+				},
+			},
+		},
 	}
 
 	for _, testCase := range testCases {

testdata/baselines/reference/fourslash/hover/CompletionDetailsOfContextSensitiveParameterNoCrash.baseline

@@ -83,7 +83,7 @@
 //         ...args
 //     ): StylingFunction => {
 //         return curry(getStylingByKeys, 2)(mergedStyling, ...args);
-//                                                             ^
+//                                                             ^^^^
 // | ----------------------------------------------------------------------
 // | ```tsx
 // | (parameter) args: []
@@ -108,6 +108,16 @@
       "contents": {
         "kind": "markdown",
         "value": "```tsx\n(parameter) args: []\n```\n"
+      },
+      "range": {
+        "start": {
+          "line": 82,
+          "character": 60
+        },
+        "end": {
+          "line": 82,
+          "character": 64
+        }
       }
     }
   }

testdata/baselines/reference/fourslash/hover/DeprecatedInheritedJSDocOverload.baseline

@@ -23,7 +23,7 @@
 // }
 // declare const a: ThingWithDeprecations<void>
 // a.subscribe(() => {
-//            ^
+//   ^^^^^^^^^
 // | ----------------------------------------------------------------------
 // | ```tsx
 // | (method) ThingWithDeprecations.subscribe(observer?: PartialObserver<void>): Subscription
@@ -47,6 +47,16 @@
       "contents": {
         "kind": "markdown",
         "value": "```tsx\n(method) ThingWithDeprecations.subscribe(observer?: PartialObserver<void>): Subscription\n```\n"
+      },
+      "range": {
+        "start": {
+          "line": 22,
+          "character": 2
+        },
+        "end": {
+          "line": 22,
+          "character": 11
+        }
       }
     }
   }