Improper setup of obsolete package react-hot-loader

[lindapaiste]

While working on other features I have encountered a bunch of errors, warnings, and problems relating to the react-hot-loader package. This is a hindrance to the future development of this project and provides unnecessary obstacles to development.

Package is obsolete

The best way to fix all of these issues is to not use this package. The package itself recommends that you don't use it anymore:

React-Hot-Loader has been your friendly neighbour, living outside of React. But it has been limiting its powers and causing not the greatest experience. It's time to make a next step.

React-Hot-Loader is expected to be replaced by React Fast Refresh. Please remove React-Hot-Loader if Fast Refresh is currently supported on your environment.
source

Doesn't work most of the time

I usually see the "hot update was not successful" message and need to manually refresh the page.

Improper setup

I have seen an error related to using hot(App) and ReactDOM.render() in the same file. We are definitely doing that!

p5.js-web-editor/client/index.jsx

Lines 31 to 33 in 9073487

	const HotApp = hot(App);
	render(

Missing peer dependency

Here's the discussion on that issue

Can create other problems

Here's an issue that I encountered while working on the react-router code. The react-hot-loader package converts all components to a proxied version which breaks the router (discussion). I am able to work around this by writing the router code differently, but it's annoying.