microsoft / microsoft/TypeScript
Add more context to unique symbols that appear in type errors
还没有人认领这个 Issue。
- 主要语言
- Go
- 星标
- 111k
- 派生
- 14.3k
- 平均合并
- 2 天 4 小时
- 30 天内合并 PR
- 132
描述
### 🔍 Search Terms
unique symbol type error
I went through every bug i could find to see if someone suggested anything like this - no luck.
### ✅ Viability Checklist
- [x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
- [x] This wouldn't change the runtime behavior of existing JavaScript code
- [x] This could be implemented without emitting different JS based on the types of the expressions
- [x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- [x] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
- [x] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals
### ⭐ Suggestion
Aa part of tracking down https://github.com/sveltejs/svelte/issues/15182 and https://github.com/sveltejs/svelte/issues/13760, you can see the error messages from the typescript compiler around conflicting unique symbols correctly state that the types are different. But they give you no real context about where the unique symbols are from, or even give the unique symbols different numbers.
So you get something like (ignore the annoying string that svelte has added to the type):
And if you are a normal developer, you are just like "uh, okay" - the types look identical because by name, they are! (Both the different unique symbols are named `SnippetReturn`).
The actual part that matters is really that part that states that the unique symbol is not assignable to unique symbol. At the very bottom.
But the *really* important part, that you can't tell here, is that the unique symbol it's talking about in "& unique symbol" and the one in 'type "unique symbol"' are *different unique symbols*.
This was hard enough to tell that nobody actually understood what this means., and just assumed it was breakage elsehwere.
All of this is coming from the typescript compiler/parser, and you can see this with the repro in 15182 that is just a bare ts file. Running tsc on the project will give you a similar message. tsc happens to name the overall containing types at the top, which is nice. But this only looks nice because of the import paths. In the real errors, you either don't get the import statements, or you see ```Type 'import("svelte").Snippet<[]>' is not assignable to type 'import("svelte").Snippet<[]>``` at the top which helps make you think the tooling is broken, not the code.
It would be really nice if, for unique symbols (which are all unique), it gave you more context.
Like ```Type '{ '{@render ...} must be called with a Snippet': "import type { Snippet } from 'svelte'"; } & (unique symbol declared at line 57 of file bar.ts)' is not assignable to type '(unique symbol declared at like 48 of file foo.ts)'```
So it's totally obvious they are not the same unique symbol.
Maybe even add "Different instances of unique symbols are never compatible" at the end to help.
If this is too hard, even just something like:
``` Type '{ '{@render ...} must be called with a Snippet': "import type { Snippet } from 'svelte'"; } & (unique symbol #1)' is not assignable to type '(unique symbol #2)'. #1 and #2 are two different instances of unique symbols, and different instances of unique symbols are never compatible```
So that it was similarly obvious, though not quite as helpful.
Another option would be to point out the `SnippetReturn`'s it's talking about are not the same (`SnippetReturn` is the thing that is `typeof `)
``` Type '{ '{@render ...} must be called with a Snippet': "import type { Snippet } from 'svelte'"; } & typeof SnippetReturn (SnippetReturn is an instance of a unique symbol) ' is not assignable to type '{ '{@render ...} must be called with a Snippet': "import type { Snippet } from 'svelte'"; } & typeof SnippetReturn (SnippetReturn is a *different* instance of a unique symbol)'. Two different types with this name exist, but they are unrelated.```
This is even less helpful, but still IMHO an improvement.
### 📃 Motivating Example
Tracking down a svelte bug - https://github.com/sveltejs/svelte/issues/15182
This bug went unnoticed despite a number of reports in part because the error message didn't clearly help understand what was broken, and so folks wrote it off. it was worsened by the fact that the typeof's of the different unique symbols had the same name, making the error message look like a bug or weird failure.
Had it contained the filenames and declarations, someone would probably have looked and it would have been instantly obvious that it not was random weirdness, but a real bug.
### 💻 Use Cases
1. What do you want to use this for?
Better error messages
2. What shortcomings exist with current approaches?
The current error message is missing enough that there are concrete cases of very smart developers missing the real bug.
3. What workarounds are you using in the meantime?
Using people like me with lots of compiler experience who are willing to jump very far down crazy compiler and language server rabbit holes when bugs look weird.
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先根据链接的 Svelte 示例,在一个空的 .ts 文件中重现冲突并运行 tsc。检查 TypeScript 编译器/parser 对 unique-symbol 类型错误的处理;当诊断信息能够结合有用的声明、文件或实例上下文区分不兼容的 unique symbols 时,即视为完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- typescript
- 领域
- compilers
- Issue 类型
- 功能
- 难度
- 5/5
- 预计耗时
- 一周以上
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 32/100