Keep array-based entries relative-order during injectRefreshEntry (#165)

Author: miguel-silva

lib/utils/injectRefreshEntry.js

@@ -79,12 +79,12 @@ function injectRefreshEntry(originalEntry, options) {
   if (Array.isArray(originalEntry)) {
     const socketEntryIndex = originalEntry.findIndex(isSocketEntry);
 
-    let socketEntry = [];
+    let socketAndPreceedingEntries = [];
     if (socketEntryIndex !== -1) {
-      socketEntry = originalEntry.splice(socketEntryIndex, 1);
+      socketAndPreceedingEntries = originalEntry.splice(0, socketEntryIndex + 1);
     }
 
-    return [...prependEntries, ...socketEntry, ...overlayEntries, ...originalEntry];
+    return [...prependEntries, ...socketAndPreceedingEntries, ...overlayEntries, ...originalEntry];
   }
   // Multiple entry points
   if (typeof originalEntry === 'object') {

test/unit/injectRefreshEntry.test.js

@@ -146,10 +146,16 @@ describe('injectRefreshEntry', () => {
     ]);
   });
 
-  it('should append overlay entry for an array after socket-related entries', () => {
+  it('should append overlay entry for an array after socket-related entries, while keeping relative order on the original entries', () => {
     expect(
-      injectRefreshEntry(['webpack-dev-server/client', 'test.js'], DEFAULT_OPTIONS)
-    ).toStrictEqual([ReactRefreshEntry, 'webpack-dev-server/client', ErrorOverlayEntry, 'test.js']);
+      injectRefreshEntry(['setup-env.js', 'webpack-dev-server/client', 'test.js'], DEFAULT_OPTIONS)
+    ).toStrictEqual([
+      ReactRefreshEntry,
+      'setup-env.js',
+      'webpack-dev-server/client',
+      ErrorOverlayEntry,
+      'test.js',
+    ]);
   });
 
   it('should throw when non-parsable entry is received', () => {