Merge pull request #44589 from Expensify/Rory-VerifyPatches

[No QA] Fail patch-package on errors or warnings

scripts/applyPatches.sh

@@ -0,0 +1,39 @@
+#!/bin/bash
+
+# This script is a simple wrapper around patch-package that fails if any errors or warnings are detected.
+# This is useful because patch-package does not fail on errors or warnings by default,
+# which means that broken patches are easy to miss, and leads to developer frustration and wasted time.
+
+SCRIPTS_DIR=$(dirname "${BASH_SOURCE[0]}")
+source "$SCRIPTS_DIR/shellUtils.sh"
+
+# Run patch-package and capture its output and exit code, while still displaying the original output to the terminal
+# (we use `script -q /dev/null` to preserve colorization in the output)
+TEMP_OUTPUT="$(mktemp)"
+script -q /dev/null npx patch-package --error-on-fail 2>&1 | tee "$TEMP_OUTPUT"
+EXIT_CODE=${PIPESTATUS[0]}
+OUTPUT="$(cat "$TEMP_OUTPUT")"
+rm -f "$TEMP_OUTPUT"
+
+# Check if the output contains a warning message
+echo "$OUTPUT" | grep -q "Warning:"
+WARNING_FOUND=$?
+
+printf "\n";
+
+# Determine the final exit code
+if [ "$EXIT_CODE" -eq 0 ]; then
+  if [ $WARNING_FOUND -eq 0 ]; then
+    # patch-package succeeded but warning was found
+    error "It looks like you upgraded a dependency without upgrading the patch. Please review the patch, determine if it's still needed, and port it to the new version of the dependency."
+    exit 1
+  else
+    # patch-package succeeded and no warning was found
+    success "patch-package succeeded without errors or warnings"
+    exit 0
+  fi
+else
+  # patch-package failed
+  error "patch-package failed to apply a patch"
+  exit "$EXIT_CODE"
+fi

scripts/postInstall.sh

@@ -1,11 +1,14 @@
 #!/bin/bash
 
+# Exit immediately if any command exits with a non-zero status
+set -e
+
 # Go to project root
 ROOT_DIR=$(dirname "$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")" &> /dev/null && pwd)")
 cd "$ROOT_DIR" || exit 1
 
-# Run patch-package
-npx patch-package
+# Apply packages using patch-package
+scripts/applyPatches.sh
 
 # Install node_modules in subpackages, unless we're in a CI/CD environment,
 # where the node_modules for subpackages are cached separately.