1949588 - Remove non-ESM version of react modules

Status: RESOLVED FIXED

(DevTools :: General, task)

Description

[Alexandre Poirot [:ochameau]]

Bug 1949486 will introduce ESM versions for React modules, but could yet delete the non-ESM versions because JSONViewer was still using the non-ESM version.
Bug 1929387 will migrate the JSON Viewer to ESM which will allow getting rid of the non-ESM versions.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1949588 - [devtools] Remove React non-ESM modules

Also introduce a temporary workaround in the base loader to force loading
the available ESM from vendor.
This will prevent significantly increasing firefox installer size.

All the duplicated ESM modules are removed from Firefox package,
but some are kept in-tree for some jest tests which can't use ESM.

Comment 2

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1949588 - [devtools] Allow running the leftover debugger jest tests without react.

Node doesn't support ESM, so we have to drop enzyme and prevent loading any frontend module.
There is much more to be cleaned up, but let wait for getting rid of the tests entirely.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fa69a6efd514
[devtools] Remove React non-ESM modules r=nchevobbe,devtools-reviewers
https://hg.mozilla.org/integration/autoland/rev/0585ab26c3b3
[devtools] Allow running the leftover debugger jest tests without react. r=nchevobbe,devtools-reviewers

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/fa69a6efd514
https://hg.mozilla.org/mozilla-central/rev/0585ab26c3b3