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

Distribute union types over generic function application #52295

Open
5 tasks done
matthewvalentine opened this issue Jan 18, 2023 · 5 comments
Open
5 tasks done

Distribute union types over generic function application #52295

matthewvalentine opened this issue Jan 18, 2023 · 5 comments
Labels
Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature Suggestion An idea for TypeScript

Comments

@matthewvalentine
Copy link

matthewvalentine commented Jan 18, 2023

Suggestion

πŸ” Search Terms

generic union distribute function mapped

βœ… Viability Checklist

My suggestion meets these guidelines:

  • This wouldn't be a breaking change in existing TypeScript/JavaScript code
  • This wouldn't change the runtime behavior of existing JavaScript code
  • This could be implemented without emitting different JS based on the types of the expressions
  • This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
  • This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

It should be possible to call a generic function with a union as input and separately resolve the generics for each member of the union. This mimics the way generic types can be distributed over a union.

πŸ“ƒ Motivating Example

Consider this simple example (playground):

type A<T> = (a: T) => T; // any type invariant on T
function foo<T>(a: A<T>) {}
declare const a: A<1> | A<2>;
foo(a); // error: A<1> | A<2> is not assignable to A<1 | 2>

The function foo is perfectly capable of handling an input of either A<1> or A<2>, but Typescript will not allow you to execute it on the union of those types. That is because it tries to find a single instantiation for T that works, but there is none, because the type A<T> is not covariant.

In the case that foo had an output, foo: <T>(a: A<T>) => B<T>, for input of A<1> | A<2> the output type would be B<1> | B<2>, much the same as how distributing over a union works in a type expression like X extends A<infer T> ? B<T> : never;

πŸ’» Use Cases

This is one of a few issues that make non-covariant types a little bit second-class to work with in Typescript. And some of the other issues might be very hard to resolve, like how to type "An array of A<T> where each element can have a different T" without using any. But in comparison, I don't think this one requires any deep thought for the desired behavior, and while the implementation might be tricky I don't think it requires any truly new capabilities.

Workarounds:

  1. For a function where T doesn't appear in the output, like <T>(a: A<T>) => void, can be typed as (a: A<any>) => void. Then it works with unions as input. However, that introduces anys into the typechecking of the function's implementation, which don't need to be there. Instead, you can explicitly specify foo<any>(x) when calling the function. But if the function has other generics, that will make it so they also have to be explicitly specified instead of inferred.
  2. For a function where T does appear in the output, like <T>(a: A<T>) => B<T> where A<T> and B<T> are both invariant, I am not aware of any workaround except casting. Using any leaks into the output type and therefore the rest of your code. Even casting in that situation is more brittle than usual, as changes to the input union type or to the definition of the function's output type will both be lost.
@RyanCavanaugh
Copy link
Member

I'd be extremely worried about this happening in practice. Unions regularly contain hundreds of members, and resolving a generic function call is not a cheap operation. Worse, if a union appears in the output, then this is very prone to combinatorial explosion. A codebase full of calls like this could take minutes or hours to check (or just run out of memory).

@RyanCavanaugh RyanCavanaugh added Suggestion An idea for TypeScript Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature labels Jan 19, 2023
@matthewvalentine
Copy link
Author

@RyanCavanaugh That makes sense. Though I'd have thought type operators using extends should have the same possible issues. What has made it ok there - is it that it's opt-in via the extends clause, or just that there are fewer usages of type operators than generic functions?

As for performance, it might not have to be implemented as actually doing a separate resolution for each member of the union. Currently, if you call foo(A<1> | A<2>), TS gets as far as inferring that T = 1 | 2. That inference doesn't actually work, but there's clearly something in the resolution already that is seeing the relationship between A<1> | A<2> and A<T>. And it resolves it in a way that works for covariant types. At that same moment, it might be possible to see that since A is not covariant, it should be resolved in a different way instead.

@whzx5byb
Copy link

You can distribute the union explicitly as a workaround:

type A<T> = (a: T) => T; // any type invariant on T
function foo<T>(a: T extends unknown ? A<T> : never) {
}
declare const a: A<1> | A<2>;
foo(a); // ok

@OliverJAsh
Copy link
Contributor

You can distribute the union explicitly as a workaround:

Unfortunately this doesn't work if the parameters are objects:

type AR = { a: string };
type BR = { b: string };
type A<T> = (a: T) => T; // any type invariant on T
function foo<T>(a: T extends unknown ? A<T> : never) {}
declare const a: A<AR> | A<BR>;
foo(a); // error

@ManicardiFrancesco
Copy link

This is preventing me from instantiating dynamic components based on my configuration file in my angular project.

For example, i have a list of components that my client wants:

statsComponents: [
        {
            componentClass: GlobalStatsSearchComponent,
            htmlElementClasses: ['my-5'],
        },
        {
            componentClass: DebtorNumberAndAmountComponent,
            htmlElementClasses: ['my-5'],
        },
....
] 

defined in my environment file for that client.
In my component i then instantiate them in a loop:

ngAfterViewInit() {
        this.container.clear();
        for (const item of this.desiredComponents) {
            //@ts-expect-error workaround to https://github.com/microsoft/TypeScript/issues/52295
            const componentRef = this.container.createComponent(item.componentClass);

            const hostElement = componentRef.location.nativeElement;
            hostElement.classList.add(...item.htmlElementClasses);
        }
    }

this code gives error without the @ts-expect-workaround, but it actually runs fine. The workaround described aboce doesn't change anything in my case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting More Feedback This means we'd like to hear from more people who would be helped by this feature Suggestion An idea for TypeScript
Projects
None yet
Development

No branches or pull requests

5 participants