Domain: check: Type InferenceHelp WantedPossible Improvement
説明
🔍 Search Terms
"type inference", "type narrowing"
✅ Viability Checklist
- 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 isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
- This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals
⭐ Suggestion
Introduction
When I write React codes with TypeScript, I sometimes define Props as the following.
type Props = ({
parentIds: string[];
onChange: (event: { id: string }) => void
onChange2: () => void
} | {
parentIds?: never;
onChange: (event: { id: number }) => void;
})
It says, "When parentId is passed, string id is passed to onChange handler and you can use onChange2. If it is not passed, number is is passed to onChange handler".
But it doesn't work correctly. When I use it, as the following
function Component(props: Props) {
return null
}
function App() {
return (
<Component
parentIds={[]}
// Expected: Parameter e is { id: string }
// Actual: Parameter 'e' implicitly has an 'any' type.(7006)
onChange={e => {
if (e.id) {
// Do nothing
} else {
// Do nothing
}
}}
onChange2={() => {
console.log("parentId is detected as string[]")
}}
/>
)
}
The parameter e of onChange is inferred as any, not { id: string }.
I can pass onChange2 handler so it seems type narrowing works, but type inference of the args of handler doesn't work.
Suggestion
How about enhancing type inference in such case?
📃 Motivating Example
When you want to use same function or component for various purposes.
💻 Use Cases
- What do you want to use this for?
- What shortcomings exist with current approaches?
- What workarounds are you using in the meantime?