Merge branch 'Expensify:main' into fixSearch.ts

Author: allgandalf

.github/scripts/checkCoverageChanges.sh

@@ -1,42 +1,76 @@
 #!/bin/bash
+set -euo pipefail
 
-# Ensure upstream/main exists
-git fetch --depth=50 upstream main:upstream/main
+# -----------------------------
+# 1. Ensure upstream remote exists
+# -----------------------------
+# The PR branch comes from either the base repo or a fork.
+# We need the base repository's main branch for comparison.
+git remote add upstream "https://github.com/${GITHUB_REPOSITORY}.git" 2>/dev/null || true
 
-# Determine diff range safely
-if git merge-base upstream/main HEAD >/dev/null 2>&1; then
-  DIFF_RANGE="upstream/main...HEAD"
-else
-  echo "No merge base with upstream/main; falling back to comparing HEAD against upstream/main"
-  DIFF_RANGE="upstream/main HEAD"
+# -----------------------------
+# 2. Attempt shallow fetch first
+# -----------------------------
+# Start with a depth of 50 commits, which covers most PRs.
+DEPTH=50
+echo "Fetching upstream/main with depth=$DEPTH..."
+git fetch upstream main --depth=$DEPTH
+
+# -----------------------------
+# 3. Check if a merge-base exists
+# -----------------------------
+# merge-base finds the common ancestor between the PR branch and main.
+# If this fails, it means our shallow history didn't go back far enough.
+if ! git merge-base upstream/main HEAD >/dev/null 2>&1; then
+  echo "No merge base found with depth=$DEPTH, fetching full history..."
+  
+  # Try unshallowing the entire repo (pulls full commit history).
+  # If already full, this will be a no-op.
+  git fetch --unshallow upstream || git fetch upstream main
 fi
 
-# Get changed files in src directory
-readarray -t ALL_CHANGED_FILES < <(git diff --name-only "$DIFF_RANGE" | grep '^src/' | grep -E '\.(ts|tsx|js|jsx)$' || true)
+# -----------------------------
+# 4. Define the diff range
+# -----------------------------
+# Using three-dot notation (A...B) shows only the commits in HEAD
+# that aren't in upstream/main (i.e. just the PR changes).
+DIFF_RANGE="upstream/main...HEAD"
+
+# -----------------------------
+# 5. Collect changed src/ files
+# -----------------------------
+readarray -t ALL_CHANGED_FILES < <(
+  git diff --name-only "$DIFF_RANGE" \
+    | grep '^src/' \
+    | grep -E '\.(ts|tsx|js|jsx)$' || true
+)
 
-# Filter out excluded directories and files
+# -----------------------------
+# 6. Filter excluded files/dirs
+# -----------------------------
 CHANGED_FILES=()
 for file in "${ALL_CHANGED_FILES[@]}"; do
-  # Skip excluded directories
+  # Exclude directories
   if [[ "$file" =~ ^src/(CONST|languages|setup|stories|styles|types)/ ]]; then
-    echo "Skipping excluded directory: $file"
+    echo "Skipping excluded directory: \"$file\""
     continue
   fi
-  
-  # Skip excluded files in src root
-  filename=$(basename "$file")
+
+  # Exclude specific files in src root
+  filename="$(basename "$file")"
   if [[ "$filename" =~ ^(App\.tsx|CONFIG\.ts|Expensify\.tsx|HybridAppHandler\.tsx|NAICS\.ts|NAVIGATORS\.ts|ONYXKEYS\.ts|ROUTES\.ts|SCREENS\.ts|SplashScreenStateContext\.tsx|TIMEZONES\.ts)$ ]]; then
-    echo "Skipping excluded file: $file"
+    echo "Skipping excluded file: \"$file\""
     continue
   fi
 
-  # Add to coverage collection
   CHANGED_FILES+=("$file")
 done
 
-# Check if any files remain for coverage
+# -----------------------------
+# 7. Output results
+# -----------------------------
 if [ ${#CHANGED_FILES[@]} -eq 0 ]; then
-  echo "No relevant src files changed (all changes were in excluded directories/files), skipping coverage"
+  echo "No relevant src files changed, skipping coverage"
   echo "run_coverage=false" >> "$GITHUB_OUTPUT"
   exit 0
 fi
@@ -45,5 +79,5 @@ echo "Changed src files for coverage:"
 printf '%s\n' "${CHANGED_FILES[@]}"
 echo "run_coverage=true" >> "$GITHUB_OUTPUT"
 
-# Save changed files for coverage collection
+# Save changed files for later coverage steps
 printf '%s\n' "${CHANGED_FILES[@]}" > changed_files.txt

.github/workflows/testCoverage.yml

@@ -45,11 +45,6 @@ jobs:
           repository: ${{ steps.get_pr.outputs.repo }}
           ref: ${{ steps.get_pr.outputs.ref }}
 
-      - name: Fetch Upstream Remote for Base Repository
-        run: |
-          git remote add upstream https://github.com/${{ github.repository }}.git
-          git fetch upstream main
-
       - name: Setup Git for OSBotify
         uses: Expensify/GitHub-Actions/setupGitForOSBotify@main
         id: setupGitForOSBotify

Mobile-Expensify

@@ -6,6 +6,13 @@ enum EventType {
     PushReceived = 'com.airship.push_received',
 }
 
+// eslint-disable-next-line no-restricted-syntax
+enum PermissionStatus {
+    Granted = 'granted',
+    Denied = 'denied',
+    NotDetermined = 'not_determined',
+}
+
 // eslint-disable-next-line @typescript-eslint/no-namespace
 namespace iOS {
     /**
@@ -71,4 +78,4 @@ const Airship: Partial<AirshipRoot> = {
 
 export default Airship;
 
-export {EventType, iOS};
+export {EventType, iOS, PermissionStatus};

__mocks__/@ua/react-native-airship.ts

@@ -0,0 +1,10 @@
+const NitroModules = {
+    createHybridObject: jest.fn(() => ({
+        getAll: jest.fn(() => Promise.resolve([])),
+    })),
+};
+
+export {
+    // eslint-disable-next-line import/prefer-default-export
+    NitroModules,
+};

__mocks__/react-native-nitro-modules.ts

@@ -0,0 +1,38 @@
+/* eslint-disable no-console */
+import type {Compiler} from 'webpack';
+
+/**
+ * Custom webpack plugin that forces garbage collection every 5 compilations
+ * and logs memory usage to help monitor memory consumption during development.
+ *
+ * Note: Requires Node.js to be started with --expose-gc flag to enable garbage collection.
+ */
+class ForceGarbageCollectionPlugin {
+    private compilationCount = 0;
+
+    apply(compiler: Compiler) {
+        if (gc && typeof gc === 'function') {
+            compiler.hooks.done.tap(this.constructor.name, () => {
+                this.compilationCount++;
+
+                const memUsage = process.memoryUsage();
+                const heapUsedMB = Math.round(memUsage.heapUsed / 1024 / 1024);
+                const heapTotalMB = Math.round(memUsage.heapTotal / 1024 / 1024);
+
+                console.log(`📊 Compilation #${this.compilationCount} - Heap: ${heapUsedMB}MB/${heapTotalMB}MB`);
+                if (this.compilationCount % 5 === 0) {
+                    console.log(`🗑️ Forcing garbage collection after ${this.compilationCount} compilations`);
+                    gc?.();
+
+                    const memAfterGC = process.memoryUsage();
+                    const heapAfterMB = Math.round(memAfterGC.heapUsed / 1024 / 1024);
+                    console.log(`✅ Post-GC heap size: ${heapAfterMB}MB (freed ${heapUsedMB - heapAfterMB}MB)`);
+                }
+            });
+        } else {
+            console.warn('⚠️ ForceGarbageCollectionPlugin: gc() function not available. Start Node.js with --expose-gc flag to enable garbage collection.');
+        }
+    }
+}
+
+export default ForceGarbageCollectionPlugin;