Fix nil pointer dereference in getAdjustedLocation for type-only exports (#2090)

Copilot · jakebailey · web-flow · commit 7ebe2774a45d · 2025-11-14T22:38:36.000Z
Co-authored-by: copilot-swe-agent[bot] &lt;198982749+Copilot@users.noreply.github.com&gt;
Co-authored-by: jakebailey &lt;5341706+jakebailey@users.noreply.github.com&gt;
diff --git a/internal/fourslash/_scripts/failingTests.txt b/internal/fourslash/_scripts/failingTests.txt
@@ -534,7 +534,6 @@ TestQuickinfoForNamespaceMergeWithClassConstrainedToSelf
 TestQuickinfoForUnionProperty
 TestQuickinfoWrongComment
 TestRecursiveInternalModuleImport
-TestReferencesForStatementKeywords
 TestReferencesInEmptyFile
 TestRegexDetection
 TestRenameForAliasingExport02
diff --git a/internal/fourslash/tests/gen/referencesForStatementKeywords_test.go b/internal/fourslash/tests/gen/referencesForStatementKeywords_test.go
@@ -9,7 +9,7 @@ import (
 
 func TestReferencesForStatementKeywords(t *testing.T) {
 	t.Parallel()
-	t.Skip()
+
 	defer testutil.RecoverAndFail(t, "Panic on fourslash test")
 	const content = `// @filename: /main.ts
 // import ... = ...
diff --git a/internal/ls/utilities.go b/internal/ls/utilities.go
@@ -914,7 +914,7 @@ func getAdjustedLocation(node *ast.Node, forRename bool, sourceFile *ast.SourceF
 		// export /**/type * from "[|module|]";
 		// export /**/type * as ... from "[|module|]";
 		if ast.IsExportDeclaration(parent) && parent.IsTypeOnly() {
-			if location := getAdjustedLocationForExportDeclaration(parent.Parent.AsExportDeclaration(), forRename); location != nil {
+			if location := getAdjustedLocationForExportDeclaration(parent.AsExportDeclaration(), forRename); location != nil {
 				return location
 			}
 		}
diff --git a/testdata/baselines/reference/fourslash/findAllReferences/referencesForStatementKeywords.baseline.jsonc b/testdata/baselines/reference/fourslash/findAllReferences/referencesForStatementKeywords.baseline.jsonc

Original file line number	Diff line number	Diff line change
`@@ -914,7 +914,7 @@ func getAdjustedLocation(node ast.Node, forRename bool, sourceFile ast.SourceF`
`914`	`914`	`// export /*/type from "[\|module\|]";`
`915`	`915`	`// export /*/type as ... from "[\|module\|]";`
`916`	`916`	`if ast.IsExportDeclaration(parent) && parent.IsTypeOnly() {`
`917`		`- if location := getAdjustedLocationForExportDeclaration(parent.Parent.AsExportDeclaration(), forRename); location != nil {`
	`917`	`+ if location := getAdjustedLocationForExportDeclaration(parent.AsExportDeclaration(), forRename); location != nil {`
`918`	`918`	`return location`
`919`	`919`	`}`
`920`	`920`	`}`