Easier way to migrate observed React props to react-mobx 9?

[marcelgerber]

Hi!

We're currently in the process to migrate our codebase from Mobx 5 & mobx-react 5 to Mobx 6 and mobx-react 9.

While that's a ton of work already, we are also running head-first into React props no longer being observable, and that's really killing us.

We have 210 React components powered by Mobx, and they all make heavy use of the pattern @computed get isReady() { return this.props.isReady } etc. to make props observable.

It would be a lot of manual work to convert all of these into the alternative of manually updating these in componentDidUpdate calls.

Can you think of any other ways to solve this that would be less cumbersome to us? Compatibility with <StrictMode> would be nice, but is not required for us right now.
Would it be possible to write a custom monkeypatch or extension that enables observing props like before, for example?

PS: Our code is public over at https://github.com/owid/owid-grapher, so do take a look if that helps!

[urugator]

There seems to be 46 files using @computed + this.props
https://github.com/search?q=repo%3Aowid%2Fowid-grapher+%2F%40computed+get%5B%5E%7D%5D%2Bthis%5C.props%2F&type=code&p=1

If you don't use these computeds (or this.props directly) in autorun/reaction, I think you can just remove the @computed annotation.
IIRC props are invalidated with every component update and mobx doesn't track each prop individually, therefore there shouldn't be significant (if any) perf loss. You should double check, but I would probably start with that and consider anything more sofisticated later if needed.
Reactive props were introduced mainly for compat with component level reactions/autoruns. These require more attention.