fix: Set INIT_CWD env in all before-* hooks (#480)

In a long living process, where the project can be changed, we need to set
the INIT_CWD environment variable in all before-* hooks. Currently it is
set only in the `before-watchPatterns` hook. This way, when we use webpack
for a project and later we try to use webpack for another project, we try
to execute some operations based on the INIT_CWD directory, which still
points to our old project. In order to fix this unexpected behavior, set
the INIT_CWD in all before-* hooks, as some of them may be skipped based
on the state of the project.

lib/before-cleanApp.js

@@ -1,8 +1,11 @@
 const { cleanSnapshotArtefacts } = require("../snapshot/android/project-snapshot-generator");
 const { isAndroid } = require("../projectHelpers");
+const { setProcessInitDirectory } = require("./utils");
 
 module.exports = function (hookArgs) {
     if (isAndroid(hookArgs.platformInfo.platform)) {
-        cleanSnapshotArtefacts(hookArgs.platformInfo.projectData.projectDir);
+        const projectDir = hookArgs.platformInfo.projectData.projectDir;
+        setProcessInitDirectory(projectDir);
+        cleanSnapshotArtefacts(projectDir);
     }
 }

lib/before-prepareJS.js

@@ -1,4 +1,5 @@
 const { runWebpackCompiler } = require("./compiler");
+const { setProcessInitDirectory } = require("./utils");
 
 module.exports = function ($logger, hookArgs) {
     const env = hookArgs.config.env || {};
@@ -10,6 +11,10 @@ module.exports = function ($logger, hookArgs) {
         bundle: appFilesUpdaterOptions.bundle,
         release: appFilesUpdaterOptions.release,
     };
-    const result = config.bundle && runWebpackCompiler.bind(runWebpackCompiler, config, hookArgs.config.projectData, $logger, hookArgs);
+
+    const projectData = hookArgs.config.projectData;
+    setProcessInitDirectory(projectData.projectDir);
+
+    const result = config.bundle && runWebpackCompiler.bind(runWebpackCompiler, config, projectData, $logger, hookArgs);
     return result;
 }

lib/before-shouldPrepare.js

@@ -1,10 +1,13 @@
 const { join } = require("path");
 const { readFileSync, existsSync, writeFileSync } = require("fs");
+const { setProcessInitDirectory } = require("./utils");
 const envOptionsCacheFileLocation = join(__dirname, "env.cache.json");
 
 module.exports = function (hookArgs) {
 	const platformInfo = hookArgs.shouldPrepareInfo && hookArgs.shouldPrepareInfo.platformInfo;
 	if (platformInfo && platformInfo.appFilesUpdaterOptions && platformInfo.appFilesUpdaterOptions.bundle) {
+		setProcessInitDirectory(platformInfo.projectData.projectDir);
+
 		return (args, originalMethod) => {
 			return originalMethod(...args).then(originalShouldPrepare => {
 				const currentEnvString = JSON.stringify(platformInfo.env || {});

lib/before-watch.js

@@ -1,4 +1,5 @@
 const { runWebpackCompiler } = require("./compiler");
+const { setProcessInitDirectory } = require("./utils");
 
 module.exports = function ($logger, hookArgs) {
 	if (hookArgs.config) {
@@ -15,7 +16,9 @@ module.exports = function ($logger, hookArgs) {
 					watch: true
 				};
 
-				return runWebpackCompiler(config, hookArgs.projectData, $logger, hookArgs);
+				const projectData = hookArgs.projectData;
+				setProcessInitDirectory(projectData.projectDir);
+				return runWebpackCompiler(config, projectData, $logger, hookArgs);
 			}));
 		}
 	}