Fix crash by removing getNameFromImportDeclaration in favor of Node.Name()

[ahejlsberg]

Fixes #1731.

[Copilot]

Pull Request Overview

This PR adds test coverage for import alias resolution errors when referencing declarations exported using export type, and simplifies related checker code by removing an unused helper function.

• Adds a new compiler test case demonstrating that import aliases cannot reference type-only exports
• Removes the unused getNameFromImportDeclaration helper function
• Simplifies the error reporting code to directly call .Name().Text() on the type-only declaration node

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
testdata/tests/cases/compiler/importAliasTypeOnlyExport.ts	New test case for import alias referencing a type-only export
testdata/baselines/reference/compiler/importAliasTypeOnlyExport.types	Baseline for type information output
testdata/baselines/reference/compiler/importAliasTypeOnlyExport.symbols	Baseline for symbol information output
testdata/baselines/reference/compiler/importAliasTypeOnlyExport.errors.txt	Expected error diagnostic for the test case
internal/checker/utilities.go	Removes unused getNameFromImportDeclaration helper function
internal/checker/checker.go	Simplifies error reporting by directly calling .Name().Text()

[jakebailey]

Test does reproduce the crash, thanks!