Remove unsupported overlay property for ReactRefreshWebpackPlugin

docs/guides/hmr-and-hot-reloading-with-the-webpack-dev-server.md

@@ -85,7 +85,7 @@ To enable the HMR functionality, you have to use `./bin/shakapacker-dev-server`
    }
    ```
 
-   We added `overlay.sockPort` option in `ReactRefreshWebpackPlugin` to match the webpack-dev-server port specified in `config/shakapacker.yml`. This makes SockJS works properly and fixes this error in browser console: `GET http://localhost:[port]/sockjs-node/info?t=[xxxxxxxxxx] 404 (Not Found)`.
+   We added `overlay.sockPort` option in `ReactRefreshWebpackPlugin` to match the webpack-dev-server port specified in `config/shakapacker.yml`. This makes SockJS works properly and fixes this error in browser console: `GET http://localhost:[port]/sockjs-node/info?t=[xxxxxxxxxx] 404 (Not Found)`. When using `ReactRefreshWebpackPlugin` with `v0.6.0` and newer, `overlay.sockPort` is [no longer supported](https://github.com/pmmmwh/react-refresh-webpack-plugin/pull/850) and should be omitted from the configurations.
 
 4. Add the react-refresh plugin in `babel.config.js`
 

docs/guides/rails-webpacker-react-integration-options.md

@@ -132,14 +132,7 @@ You can see an example commit of adding this [here](https://github.com/shakacode
 
    //plugins
    if (isWebpackDevServer) {
-     environment.plugins.append(
-       'ReactRefreshWebpackPlugin',
-       new ReactRefreshWebpackPlugin({
-         overlay: {
-           sockPort: 3035,
-         },
-       }),
-     );
+     environment.plugins.append('ReactRefreshWebpackPlugin', new ReactRefreshWebpackPlugin({}));
    }
    ```
 

lib/generators/react_on_rails/templates/base/base/config/webpack/development.js.tt

@@ -13,11 +13,7 @@ const developmentEnvOnly = (clientWebpackConfig, _serverWebpackConfig) => {
     // eslint-disable-next-line global-require
     const ReactRefreshWebpackPlugin = require('@pmmmwh/react-refresh-webpack-plugin');
     clientWebpackConfig.plugins.push(
-      new ReactRefreshWebpackPlugin({
-        overlay: {
-          sockPort: devServer.port,
-        },
-      }),
+      new ReactRefreshWebpackPlugin({}),
     );
   }
 };

spec/dummy/config/webpack/development.js

@@ -1,6 +1,6 @@
 process.env.NODE_ENV = process.env.NODE_ENV || 'development';
 
-const { devServer, inliningCss } = require('shakapacker');
+const { inliningCss } = require('shakapacker');
 
 const webpackConfig = require('./webpackConfig');
 
@@ -12,13 +12,7 @@ const developmentEnvOnly = (clientWebpackConfig, _serverWebpackConfig) => {
 
     // eslint-disable-next-line global-require
     const ReactRefreshWebpackPlugin = require('@pmmmwh/react-refresh-webpack-plugin');
-    clientWebpackConfig.plugins.push(
-      new ReactRefreshWebpackPlugin({
-        overlay: {
-          sockPort: devServer.port,
-        },
-      }),
-    );
+    clientWebpackConfig.plugins.push(new ReactRefreshWebpackPlugin({}));
   }
 };
 

spec/dummy/tests/react-on-rails.import.test.js

@@ -1,5 +1,7 @@
 import ReactOnRails from 'react-on-rails';
 
 test('ReactOnRails', () => {
-  ReactOnRails.register({});
+  expect(() => {
+    ReactOnRails.register({});
+  }).not.toThrow();
 });

spec/dummy/tests/react-on-rails.require.test.js

@@ -1,5 +1,7 @@
 const ReactOnRails = require('react-on-rails').default;
 
 test('ReactOnRails', () => {
-  ReactOnRails.register({});
+  expect(() => {
+    ReactOnRails.register({});
+  }).not.toThrow();
 });