Merge branch 'main' into @zfurtak/migrate-WorkspaceMembersSelectionList

Author: zfurtak

.github/workflows/claude-review.yml

@@ -39,7 +39,7 @@ jobs:
 
       - name: Run Claude Code (code)
         if: steps.filter.outputs.code == 'true'
-        uses: anthropics/claude-code-action@f30f5eecfce2f34fa72e40fa5f7bcdbdcad12eb8 # v1.0.14
+        uses: anthropics/claude-code-action@a7e4c51380c42dd89b127f5e5f9be7b54020bc6b # v1.0.21
         with:
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
           github_token: ${{ secrets.GITHUB_TOKEN }}
@@ -50,7 +50,7 @@ jobs:
 
       - name: Run Claude Code (docs)
         if: steps.filter.outputs.docs == 'true'
-        uses: anthropics/claude-code-action@f30f5eecfce2f34fa72e40fa5f7bcdbdcad12eb8 # v1.0.14
+        uses: anthropics/claude-code-action@a7e4c51380c42dd89b127f5e5f9be7b54020bc6b # v1.0.21
         with:
           anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
           github_token: ${{ secrets.GITHUB_TOKEN }}

Mobile-Expensify

@@ -114,8 +114,8 @@ android {
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
         multiDexEnabled rootProject.ext.multiDexEnabled
-        versionCode 1009026401
-        versionName "9.2.64-1"
+        versionCode 1009026500
+        versionName "9.2.65-0"
         // Supported language variants must be declared here to avoid from being removed during the compilation.
         // This also helps us to not include unnecessary language variants in the APK.
         resConfigs "en", "es"

android/app/build.gradle

@@ -23,7 +23,7 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>9.2.64</string>
+	<string>9.2.65</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleURLTypes</key>
@@ -44,7 +44,7 @@
 		</dict>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>9.2.64.1</string>
+	<string>9.2.65.0</string>
 	<key>FullStory</key>
 	<dict>
 		<key>OrgId</key>

assets/images/simple-illustrations/simple-illustration__concierge-bot.svg

@@ -11,9 +11,9 @@
 	<key>CFBundleName</key>
 	<string>$(PRODUCT_NAME)</string>
 	<key>CFBundleShortVersionString</key>
-	<string>9.2.64</string>
+	<string>9.2.65</string>
 	<key>CFBundleVersion</key>
-	<string>9.2.64.1</string>
+	<string>9.2.65.0</string>
 	<key>NSExtension</key>
 	<dict>
 		<key>NSExtensionPointIdentifier</key>

assets/images/simple-illustrations/simple-illustration__magnifyingglass-receipt.svg

@@ -11,9 +11,9 @@
 	<key>CFBundleName</key>
 	<string>$(PRODUCT_NAME)</string>
 	<key>CFBundleShortVersionString</key>
-	<string>9.2.64</string>
+	<string>9.2.65</string>
 	<key>CFBundleVersion</key>
-	<string>9.2.64.1</string>
+	<string>9.2.65.0</string>
 	<key>NSExtension</key>
 	<dict>
 		<key>NSExtensionAttributes</key>

docs/assets/images/ExpensifyHelp_ExpenseRules_02.png

@@ -1,7 +1,9 @@
 /* eslint-disable max-classes-per-file */
 import * as core from '@actions/core';
 import '@shopify/flash-list/jestSetup';
+import {useMemo} from 'react';
 import type * as RNAppLogs from 'react-native-app-logs';
+import type {ReadDirItem} from 'react-native-fs';
 import 'react-native-gesture-handler/jestSetup';
 import type * as RNKeyboardController from 'react-native-keyboard-controller';
 import mockStorage from 'react-native-onyx/dist/storage/__mocks__';
@@ -69,6 +71,12 @@ jest.mock('react-native-fs', () => ({
                 res();
             }),
     ),
+    readDir: jest.fn(
+        () =>
+            new Promise<ReadDirItem[]>((res) => {
+                res([]);
+            }),
+    ),
     CachesDirectoryPath: jest.fn(),
 }));
 
@@ -135,60 +143,70 @@ jest.mock('../modules/hybrid-app/src/NativeReactNativeHybridApp', () => ({
     clearOldDotAfterSignOut: jest.fn(),
 }));
 
+const mockUseMemo = useMemo;
+
 // Mock lazy asset loading to be synchronous in tests
 jest.mock('../src/hooks/useLazyAsset.ts', () => ({
-    useMemoizedLazyAsset: jest.fn(() => {
-        // Return a mock asset immediately to avoid async loading in tests
-        const mockAsset = {
-            src: 'mock-icon',
-            testID: 'mock-asset',
-            // Add common icon properties that tests might expect
-            height: 20,
-            width: 20,
-        };
-
-        return {
-            asset: mockAsset,
-            isLoaded: true,
-            isLoading: false,
-            hasError: false,
-        };
-    }),
-    useMemoizedLazyIllustrations: jest.fn((names: readonly string[]) => {
-        // Return a Record with all requested illustration names
-        const mockIllustrations: Record<string, unknown> = {};
-        for (const name of names) {
-            mockIllustrations[name] = {
-                src: `mock-${name}`,
-                testID: `mock-illustration-${name}`,
+    useMemoizedLazyAsset: jest.fn((importFn) =>
+        mockUseMemo(() => {
+            // Return a mock asset immediately to avoid async loading in tests
+            const mockAsset = {
+                src: 'mock-icon',
+                testID: 'mock-asset',
+                // Add common icon properties that tests might expect
                 height: 20,
                 width: 20,
             };
-        }
-        return mockIllustrations;
-    }),
-    useMemoizedLazyExpensifyIcons: jest.fn((names: readonly string[]) => {
-        // Return a Record with all requested icon names
-        const mockIcons: Record<string, unknown> = {};
-        for (const name of names) {
-            mockIcons[name] = {
-                src: `mock-${name}`,
-                testID: `mock-expensify-icon-${name}`,
-                height: 20,
-                width: 20,
+
+            return {
+                asset: mockAsset,
+                isLoaded: true,
+                isLoading: false,
+                hasError: false,
             };
-        }
-        return mockIcons;
-    }),
-    default: jest.fn(() => {
-        const mockAsset = {src: 'mock-icon', testID: 'mock-asset'};
-        return {
-            asset: mockAsset,
-            isLoaded: true,
-            isLoading: false,
-            hasError: false,
-        };
-    }),
+        }, [importFn]),
+    ),
+    useMemoizedLazyIllustrations: jest.fn((names: readonly string[]) =>
+        mockUseMemo(() => {
+            // Return a Record with all requested illustration names
+            const mockIllustrations: Record<string, unknown> = {};
+            for (const name of names) {
+                mockIllustrations[name] = {
+                    src: `mock-${name}`,
+                    testID: `mock-illustration-${name}`,
+                    height: 20,
+                    width: 20,
+                };
+            }
+            return mockIllustrations;
+        }, [names]),
+    ),
+    useMemoizedLazyExpensifyIcons: jest.fn((names: readonly string[]) =>
+        mockUseMemo(() => {
+            // Return a Record with all requested icon names
+            const mockIcons: Record<string, unknown> = {};
+            for (const name of names) {
+                mockIcons[name] = {
+                    src: `mock-${name}`,
+                    testID: `mock-expensify-icon-${name}`,
+                    height: 20,
+                    width: 20,
+                };
+            }
+            return mockIcons;
+        }, [names]),
+    ),
+    default: jest.fn((importFn) =>
+        mockUseMemo(() => {
+            const mockAsset = {src: 'mock-icon', testID: 'mock-asset'};
+            return {
+                asset: mockAsset,
+                isLoaded: true,
+                isLoading: false,
+                hasError: false,
+            };
+        }, [importFn]),
+    ),
 }));
 
 // Mock icon loading functions to resolve immediately