Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again! The code looks good.

I thought about whether it's worth to useMemo the inputProps objects but I actually think it's not necessary here. In almost all use cases the memo will not apply.

The only use case I could think of is when someone updates the JSON Schema at runtime with a new description. This is really an edge case, and if someone does that they probably don't care that the input is also rerendered (maybe there is even some additional memo barrier further within Material UI)

So the code looks good! However it would be nice if you could add test cases