microsoft / microsoft/TypeScript
Callback argument type not inferred for union of interfaces
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 111k
- Forks
- 14.4k
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 117
Description
🔎 Search Terms
"implicitly has an" inference
🕗 Version & Regression Information
- This is the behavior in every version I tried, and I reviewed the FAQ for entries about "inference"
⏯ Playground Link
💻 Code
export interface AsyncResultCallback<T, E = Error> {
(err?: E | null, result?: T): void;
}
export interface AsyncResultIterator<T, R, E = Error> {
(item: T, callback: AsyncResultCallback<R, E>): void;
}
export interface AsyncResultIteratorPromise<T, R> {
(item: T): Promise<R>;
}
declare function mapLimit<T, R, E = Error>(
arr: T[],
limit: number,
iterator: AsyncResultIteratorPromise<T, R> | AsyncResultIterator<T, R, E>,
): Promise<R[]>;
mapLimit([1,2,3], 3, async (n) => {
return n ** 2;
});
🙁 Actual behavior
The type of n in the mapLimit callback is inferred as any:
Parameter 'n' implicitly has an 'any' type. (7006)
🙂 Expected behavior
I would expect the type of n to be inferred as number.
Additional information about the issue
This example is based on the types for the async package:
Note that if the type for iterator in mapLimit is changed to either just AsyncResultIteratorPromise<T, R> or just AsyncResultIterator<T, R, E> (that is, no longer a union type), things work as expected and n has type number.
I found a very old discussion of what looks to be the same issue in the DefinitelyTyped repo: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/24297. The issue described there is exactly the one I'm facing. One commenter claimed that things worked correctly, and another commenter claimed that changing the way that async was imported had some effect? Neither of those sounds plausible to me.
I found some issues that may describe the same or a similar problem:
- https://github.com/microsoft/TypeScript/issues/56881
- https://github.com/microsoft/TypeScript/issues/55632
- https://github.com/microsoft/TypeScript/issues/52047
- https://github.com/microsoft/TypeScript/issues/47599
However, my knowledge of TypeScript internals/terminology isn't strong enough to say for sure that this isn't a novel issue I'm facing. If this is in fact a duplicate, please close it as such and I'll add this example to the relevant issue. Thank you!
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the linked Playground and minimal mapLimit example, then compare inference when iterator is each interface alone versus their union. Review the related TypeScript issues and the linked async type definition for matching context. Done means the union case infers callback parameter n as number without an implicit-any error.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- compilers
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100