[material-ui][Select] SelectProps suddenly break by making variant required about material-ui HOT 10 CLOSED

I created a typescript playground with all the problems I could find, including this one:

• If only the Value type argument is provided, inference on Variant doesn't work due to microsoft/TypeScript#10571
• If the Variant type arguments are provided with value "standard" or "filled", but the variant prop is not provided, then the "outlined" variant will be rendered. TS should catch this.
• When trying to wrap the Select, variant is incorrectly required and inference stops working (this issue)

from material-ui.

Hey @maapteh, thanks for the report.

Also reported here: #39137 (comment)

I'll look into it this week

from material-ui.

Thank you, i looked but search didnt find anything. I will see if making them optional again will fix my issue.

FYI, this is breaking type change for people using SelectProps 
directly because the variant prop is required now when it wasn't 
before. I guess BaseSelectProps is a suitable replacement 
for most use cases?

This was helpful but not fixing the root cause:)

from material-ui.

I created a fix in the PR attached to this issue report!

from material-ui.

Please look at it @DiegoAndai when you find the time :)

from material-ui.

Hey @maapteh, may I ask you to provide a minimal reproduction? This would help a lot in debugging the issue. A live example would be perfect. This StackBlitz sandbox template may be a good starting point.

from material-ui.

I solved it in #41359 right?

from material-ui.

I'm not sure, I left a comment there

from material-ui.

Ah now i see, but its a union type so i dont see how that would work...
I created the sample at https://stackblitz.com/edit/stackblitz-starters-gfn7kp?file=src%2FApp.tsx

from material-ui.

I'll add the ready to take label to see if we get any contributors with ideas. This is the isolated repro: typescript playground. If you have an idea on how to support wrapping the Select component and avoid the errors in the playground, let's discuss here.

from material-ui.