Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Static method on abstract class with generic this parameter causes "excessively deep" error when called on concrete class #60992

Open
lucacasonato opened this issue Jan 17, 2025 · 0 comments

Comments

@lucacasonato
Copy link

πŸ”Ž Search Terms

  • abstract class
  • static method
  • this parameter
  • excessively deep
  • concrete class
  • generic

πŸ•— Version & Regression Information

This is broken in 5.6, 5.7, and nightly at least.

⏯ Playground Link

https://www.typescriptlang.org/play/?target=99&ts=5.8.0-dev.20250117&ssl=6&ssc=12&pln=6&pc=15#code/IYIwzgLgTsDGEAJYBthjAgguacIGFV0AeAEQD4EBvAKAXoQHMBTRAE2AmAAoBKALgSlqdBmKisArlAB21AL4I0CSTIDWMgPYB3OctKj68moYSROAS1gIAHgE8AXsQAqCZjYjMZbDBDsAHZk0AMywcGHhCNDBiYBk7cnJuUzEIAAsLMEFnABoUhg4ubLcPLx8EP0CQsMgIgiIYixlg5ighSgB+IQRBGWYAN1bTXgUaYxoUaIR8TRlYCU8o9BLPbwxsWrwlmNqmxkpaMZMZuYXmbYA6ewduAEYAJgBmXiA

πŸ’» Code

abstract class AbstractClass<D> {
    get data(): D {
        return {} as unknown as D
    }

    static xyz<T extends typeof AbstractClass<any>>(
        this: T,
        data: T extends typeof AbstractClass<infer D> ? D : never
    ) {}
}

class ConcreteClass extends AbstractClass<string> {

}

ConcreteClass.xyz(123)

πŸ™ Actual behavior

The ConcreteClass.xyz results in Type instantiation is excessively deep and possibly infinite. and as a result infers D as never.

πŸ™‚ Expected behavior

This to work fine. A similar case, but where InstanceType<T> extends AbstractClass<infer D> ? D : never works fine: https://www.typescriptlang.org/play/?target=99&ts=5.8.0-dev.20250117#code/IYIwzgLgTsDGEAJYBthjAgguacIGFV0AeAEQD4EBvAKAXoQHMBTRAE2AmAAoBKALgSlqdBmKisArlAB21AL4I0CSTIDWMgPYB3OctKj68moYSROAS1gIAHgE8AXsQAqCZjYjMZbDBDsAHZk0AMywcGHhCNDBiYBk7cnJuUzEIAAsLMEFnABoUhg4uQQBJGXMZWGZnAOYXSndPbwxsSAiCIhiLGWDmKCFKAH4hBEEZZgA3XtNeBRpjGhRohHxNColPKPQ3Dy8fMNa8TZjWrsZKWjmTFbXWZiOAOnsHbgBGACYAZl4gA

Additional information about the issue

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant