Add hooks to be able to extend scripts configuration

Ivan Godorogea · Andrew Luca · commit a66cfc55ce21 · 2020-02-17T10:43:39.000+02:00
diff --git a/packages/react-scripts/config/env.js b/packages/react-scripts/config/env.js
@@ -22,6 +22,12 @@ if (!NODE_ENV) {
   );
 }
 
+// @hook load dynamic environment variables
+const envJs = path.join(paths.appPath, 'config/env.js');
+if (fs.existsSync(envJs)) {
+  require(envJs);
+}
+
 // https://github.com/bkeepers/dotenv#what-other-env-files-can-i-use
 const dotenvFiles = [
   `${paths.dotenv}.${NODE_ENV}.local`,
diff --git a/packages/react-scripts/package.json b/packages/react-scripts/package.json
@@ -1,5 +1,5 @@
 {
-  "name": "react-scripts",
+  "name": "@winify/react-scripts",
   "version": "3.4.0",
   "description": "Configuration and scripts for Create React App.",
   "repository": {
diff --git a/packages/react-scripts/scripts/build.js b/packages/react-scripts/scripts/build.js
@@ -62,6 +62,15 @@ if (!checkRequiredFiles([paths.appHtml, paths.appIndexJs])) {
 // Generate configuration
 const config = configFactory('production');
 
+// @hook override webpack config
+const webpackConfigOverride = path.join(
+  paths.appPath,
+  'config/webpack.config.override.js'
+);
+if (fs.existsSync(webpackConfigOverride)) {
+  require(webpackConfigOverride)(config, 'production');
+}
+
 // We require that you explicitly set browsers and do not fall back to
 // browserslist defaults.
 const { checkBrowsers } = require('react-dev-utils/browsersHelper');
diff --git a/packages/react-scripts/scripts/start.js b/packages/react-scripts/scripts/start.js
@@ -109,6 +109,17 @@ checkBrowsers(paths.appPath, isInteractive)
       errors: errors =>
         devServer.sockWrite(devServer.sockets, 'errors', errors),
     };
+
+    // @hook override webpack config
+    const path = require('path');
+    const webpackConfigOverride = path.join(
+      paths.appPath,
+      'config/webpack.config.override.js'
+    );
+    if (fs.existsSync(webpackConfigOverride)) {
+      require(webpackConfigOverride)(config, 'development');
+    }
+
     // Create a webpack compiler that is configured with custom messages.
     const compiler = createCompiler({
       appName,
@@ -132,6 +143,16 @@ checkBrowsers(paths.appPath, isInteractive)
       proxyConfig,
       urls.lanUrlForConfig
     );
+
+    // @hook override webpackDevServer config
+    const serverConfigOverride = path.join(
+      paths.appPath,
+      'config/webpackDevServer.config.override.js'
+    );
+    if (fs.existsSync(serverConfigOverride)) {
+      require(serverConfigOverride)(serverConfig, process.env.NODE_ENV);
+    }
+
     const devServer = new WebpackDevServer(compiler, serverConfig);
     // Launch WebpackDevServer.
     devServer.listen(port, HOST, err => {
diff --git a/packages/react-scripts/scripts/utils/createJestConfig.js b/packages/react-scripts/scripts/utils/createJestConfig.js
@@ -8,6 +8,7 @@
 'use strict';
 
 const fs = require('fs');
+const path = require('path');
 const chalk = require('react-dev-utils/chalk');
 const paths = require('../../config/paths');
 const modules = require('../../config/modules');
@@ -146,5 +147,15 @@ module.exports = (resolve, rootDir, isEjecting) => {
       process.exit(1);
     }
   }
+
+  // @hook override webpack config
+  const jestConfigOverride = path.join(
+    paths.appPath,
+    'config/jest.config.override.js'
+  );
+  if (fs.existsSync(jestConfigOverride)) {
+    require(jestConfigOverride)(config);
+  }
+
   return config;
 };

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`{`
`2`		`- "name": "react-scripts",`
	`2`	`+ "name": "@winify/react-scripts",`
`3`	`3`	`"version": "3.4.0",`
`4`	`4`	`"description": "Configuration and scripts for Create React App.",`
`5`	`5`	`"repository": {`