microsoft / microsoft/TypeScript
Unions of assertion functions do not act as assertion functions
オープン
まだ誰も着手していません。
Bug
Domain: This-Typing
Help Wanted
- 主要言語
- Go
- スター
- 111k
- フォーク
- 14.4k
- 平均マージ
- 1日 19時間
- マージ済み PR(30日)
- 117
説明
🔎 Search Terms
narrowing, assertion function
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about everything
⏯ Playground Link
💻 Code
class box<T> {
constructor(public value: T){}
check(): this is box<string> {
return typeof this.value == 'string';
}
assert(): asserts this is box<string> {
if (typeof this.value != 'string') throw new Error();
}
private test() {
this.assert();
// type correctly narrowed
this.value.substring(0);
}
}
function make() : box<string> | box<number> {
return new box('a');
}
function assert(b: box<string> | box<number>): asserts b is box<string> {
if (typeof b.value != 'string') throw new Error();
}
const b = make();
if (b.check()) {
// type correctly narrowed
b.value.substring(0);
}
b.assert();
// type not narrowed (substring does not exist on type 'string | number')
b.value.substring(0);
assert(b);
// type correctly narrowed
b.value.substring(0);
🙁 Actual behavior
Type is not narrowed after method call
🙂 Expected behavior
I expected the type to be narrowed after method call since it is narrowed within other methods or when using a function external to the type.
Additional information about the issue
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
リンクされた TypeScript Playground の再現コードから始め、b.check()、b.assert()、および外部の assert(b) 呼び出しによって行われる絞り込みを比較します。メソッド呼び出しでユニオンが絞り込まれない理由を調査し、そのうえで、既存の成功するケースを維持しながら、メソッドのアサーションが外部のアサーションと一貫して動作することを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- compilers
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 35/100