Narrowing of generic this is inconsistent with variable narrowing

オープン
#64,186 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
45/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
typescript
領域
compilers

調査の方向性

リンクされている TypeScript Playground から始め、direct this、aliased this、parameter narrowing について提示されている generic-union の例を縮小してください。Issue ではリポジトリのファイルも test も特定されていないため、compiler の narrowing のエントリーポイントを見つけ、示されているケースをカバーする regression test を追加してください。direct this と aliased this の narrowing が一貫した diagnostics を生成すれば完了です。

索引モデルが issue の本文から書いたものです。

説明

Possible Improvement
🔎 Search Terms

generic union constraint narrowing this

🕗 Version & Regression Information
  • This is the behavior in every version I tried
⏯ Playground Link

https://www.typescriptlang.org/play/?noImplicitReturns=false#code/C4TwDgpgBAKu0F4oEkDMAmKAfKBlYATgNwBQAlgHbAQEBmAhgMbT4FQDeJUUA5gQPYBXMAC4oAIgr8KEcaW4BrSgBMx4gM6E5JAL7kqNBsxQYOXXgOFr1jegBt6BbYpVqyGbXpLKIjBwWhaQQpGYDJpKGB6BQg0dAAKADd7QQgxOIBKMUT+MmVSHz9HQODQ8IooSloaAGkVAB4YAD4klLSOKCUKVVgoHSzYUhIgkLCI6k0YAAsydUaoCAAPam71WHgW4Bn1MRgMs251AHcyYEYpqHit2YA6PiEwfc5ublt1aElpWRFzF6gA4CCAgUeQvN4fGz2RziH5-Q4nM4XK7bG5dZRPX5w8ESTROMQAenxUCkCwIAgIAH5MX9seJ3OgYdS+uY9F4RmVxhBNAAFRz0AC2EGoBHmSxWyjWcEgLX4uwx8NO50u-DulkeBzB9HeEikMkZcP+QqBIMxtMh-n1cOOiqRKrR8oNUDNWgJRJo5IAhEzXlqPvTLX89NxWSRhqUxhUJsAAIJ2Mi+5TTWai5YQVbraXI2ZyjWMaSaKD8KBIa7qUHWxHK1UPB0+7WfPWwuEA42gusQ2wWpt-CtK+J2lS1rG+nEuqCE0me71Okd0jzdl5B5ls8PlSJc4BJ9ToFPiyUbLM7WADOLYKDBHy0SgQZQasi0S6l6vCYsIJAaTvQod5igF5J2VJ0lMEttiIcc3TJfgCExKIYjiQ8MjAid3SgmDtigdR6DCdQry5Ex0CQiDyVNfNgCgRwBCOYtLhPYCmkiUDwMnVC-hbYEGNmQjmOg4NdFDdkI3XHk+UFYUdxgBZU3TKUIBlOUgMwHALwgK8ZFvZ5KgfftnzAV933NL8NVeUioH-QD8Oo-guP4BQYOiWIMH7RCmJszEi0w7DcLWOJrNsmkTIo-gqKQeJaMwBB6Ksly-JeNiKiiidXN4ldRjXKNY3jd5E22cTJL3DNZMPeSLKU7oVOvdTzB-AsixAzjzHvKt7hfN99M-JxvxMsz2lPJAEqJJKXlghyEn4ZzEpi7h3Kw2YvPw3ySN-MjAuCmiFOLSKFtYo12P6wsYpDIA

💻 Code
type Type = I32 | Str;
interface Str {
  group: "none";
  kind: "str";
}
interface I32 {
  group: "scalar";
  kind: "i32";
}

declare function takeI32(value: I32): void;
declare function inferKind<T>(value: { kind: T }): T;

function testThis<T extends Type>(this: T) {
  switch (this.group) {
    case "none":
      return;
    case "scalar":
      switch (this.kind) {
        case "str": // no error?
        case "i32":
      }
  }
}

function testParameter<T extends Type>(o: T) {
  switch (o.group) {
    case "none":
      return;
    case "scalar":
      switch (o.kind) {
        case "str": // error!
        case "i32":
      }
  }
}

function testAliasedThis<T extends Type>(this: T) {
  const o = this;
  switch (o.group) {
    case "none":
      return;
    case "scalar":
      switch (o.kind) {
        case "str": // error!
        case "i32":
      }
  }
}

function testThis2<T extends Type>(this: T): I32 | undefined {
  if (this.group === "scalar") {
    const value: I32 = this; // error
    takeI32(this); // error
    this satisfies I32; // error
    const arrow = (): I32 => this; // error
    return this; // error
  }
}

function testParameter2<T extends Type>(o: T): I32 | undefined {
  if (o.group === "scalar") {
    const value: I32 = o; // ok
    takeI32(o); // ok
    o satisfies I32; // ok
    const arrow = (): I32 => o; // ok
    return o; // ok
  }
}

function testAliasedThis2<T extends Type>(this: T): I32 | undefined {
  const o = this;
  if (o.group === "scalar") {
    const value: I32 = o; // ok
    takeI32(o); // ok
    o satisfies I32; // ok
    const arrow = (): I32 => o; // ok
    return o; // ok
  }
}

🙁 Actual behavior

The errors (or lack of them) are inconsistent between narrowing this directly vs narrowing an "aliased" this (const alias = this;)

🙂 Expected behavior

They should be consistent

Additional information about the issue

No response

主要言語
Go
スター
111k
フォーク
14.4k
平均マージ
1日 15時間
マージ済み PR(30日)
106

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

microsoft/TypeScript のほかの issue

microsoft/TypeScript の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。