fix: revert #129

Author: pmmmwh

lib/utils/injectRefreshLoader.js

@@ -24,15 +24,11 @@ function injectRefreshLoader(data, matchObject) {
     // Check to prevent double injection
     !data.loaders.find(({ loader }) => loader === resolvedLoader)
   ) {
-    // It is important to have inject loader before anything,
-    // especially if runtime transforms (e.g. @babel/plugin-transform-runtime) are being used.
-    // Runtime transforms might make use of cache mechanisms (e.g. WeakMaps) to handle imports,
-    // so if we do things out of their loop,
-    // we might risk referencing a different version of the same dependency in the runtime.
-    // This in its worst form will break HMR referential equality checks,
-    // making us bail out on every change.
-    // e.g. React in the file will not be equal to React as consumed by react-refresh.
-    data.loaders.push({
+    // As we inject runtime code for each module,
+    // it is important to run the injected loader after everything.
+    // This way we can ensure that all code-processing have been done,
+    // and we won't risk breaking tools like Flow or ESLint.
+    data.loaders.unshift({
       loader: resolvedLoader,
       options: undefined,
     });