AMP-31085: Webpack js issue

Author: brianbrix

amp/TEMPLATE/reampv2/packages/ampoffline/scripts/overrides/webpack.dev.js

@@ -1,7 +1,8 @@
 const { ModuleFederationPlugin } = require('webpack').container;
 const packageJson = require('../../package.json');
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackDev = require(webpackConfigPath);
 
@@ -48,7 +49,7 @@ const override = config => {
   return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = env => override(webpackDev(env));
+require.cache[webpackConfigPath].exports = env => override(webpackDev(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/ampoffline/scripts/overrides/webpack.prod.js

@@ -1,6 +1,7 @@
 const {ModuleFederationPlugin} = require('webpack').container;
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackConfig = require(webpackConfigPath);
 const packageJson = require("../../package.json");
@@ -55,7 +56,7 @@ const override = config => {
     return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = env => override(webpackConfig(env));
+require.cache[webpackConfigPath].exports = env => override(webpackConfig(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/container/scripts/overrides/webpack.dev.js

@@ -1,7 +1,8 @@
 const {ModuleFederationPlugin} = require('webpack').container;
 const packageJson = require('../../package.json');
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackDev = require(webpackConfigPath);
 
@@ -57,7 +58,7 @@ const override = config => {
     return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = env => override(webpackDev(env));
+require.cache[webpackConfigPath].exports = env => override(webpackDev(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/container/scripts/overrides/webpack.prod.js

@@ -1,7 +1,8 @@
 const {ModuleFederationPlugin} = require('webpack').container;
 const Dotenv = require('dotenv-webpack');
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackConfig = require(webpackConfigPath);
 const ppackageJson = require("../../package.json");
@@ -76,7 +77,7 @@ const override = config => {
     return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = env => override(webpackConfig(env));
+require.cache[webpackConfigPath].exports = env => override(webpackConfig(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/reampv2-app/scripts/overrides/webpack.dev.js

@@ -1,7 +1,8 @@
 const { ModuleFederationPlugin } = require('webpack').container;
 const packageJson = require('../../package.json');
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackDev = require(webpackConfigPath);
 
@@ -79,7 +80,7 @@ const override = (config) => {
   return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = (env) => override(webpackDev(env));
+require.cache[webpackConfigPath].exports = (env) => override(webpackDev(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/reampv2-app/scripts/overrides/webpack.prod.js

@@ -1,6 +1,7 @@
 const {ModuleFederationPlugin} = require('webpack').container;
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackConfig = require(webpackConfigPath);
 const packageJson = require("../../package.json");
@@ -83,7 +84,7 @@ const override = config => {
     return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = env => override(webpackConfig(env));
+require.cache[webpackConfigPath].exports = env => override(webpackConfig(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/user-manager/scripts/overrides/webpack.dev.js

@@ -1,7 +1,8 @@
 const { ModuleFederationPlugin } = require('webpack').container;
 const packageJson = require('../../package.json');
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackDev = require(webpackConfigPath);
 
@@ -74,7 +75,7 @@ const override = (config) => {
   return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = (env) => override(webpackDev(env));
+require.cache[webpackConfigPath].exports = (env) => override(webpackDev(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);

amp/TEMPLATE/reampv2/packages/user-manager/scripts/overrides/webpack.prod.js

@@ -1,6 +1,7 @@
 const { ModuleFederationPlugin } = require('webpack').container;
 
-const webpackConfigPath = 'react-scripts/config/webpack.config';
+// Use require.resolve to find react-scripts regardless of workspace hoisting
+const webpackConfigPath = require.resolve('react-scripts/config/webpack.config');
 // eslint-disable-next-line import/no-dynamic-require
 const webpackConfig = require(webpackConfigPath);
 const packageJson = require('../../package.json');
@@ -81,7 +82,7 @@ const override = (config) => {
   return config;
 };
 
-require.cache[require.resolve(webpackConfigPath)].exports = (env) => override(webpackConfig(env));
+require.cache[webpackConfigPath].exports = (env) => override(webpackConfig(env));
 
 // eslint-disable-next-line import/no-dynamic-require
 module.exports = require(webpackConfigPath);