unintended escaped string for hover

[Hajime-san]

🔎 Search Terms

"lsp"

🕗 Version & Regression Information

version "typescript": "5.3.3"

⏯ Playground Link

No response

💻 Code

// 🦁.ts
export const foo = "bar";

// main.ts
import { bar } from './🦁.ts';

🙁 Actual behavior

The lsp message shows Module '"./\uD83E\uDD81.ts"' has no exported member 'bar'.ts(2305).

🙂 Expected behavior

It should be Module '"./🦁.ts"' has no exported member 'bar'.ts(2305).

Additional information about the issue

[Andarist]

This likely falls under the umbrella here (it's worked on here)

[fatcerberus]

Mateusz Burzyński (@Andarist) Wait, is that even related? That looks like it's about arbitrary import names, e.g.

import { "😊" as yay } from "foo";

...which doesn't work at all today (i.e. it's a feature request), whereas this issue is about unicode characters in module specifiers, which does work today, the filename is just not being rendered property in error messages.

[Andarist]

You are totally right. I'm trying to do to much lately and it seems my circuits are overheating :v

[Andarist]

My penance: #57550