Merge branch 'main' into fix/84022

Author: dukenv0307

.claude/skills/coding-standards/SKILL.md

@@ -13,8 +13,9 @@ Coding standards for the Expensify App. Each standard is a standalone file in `r
 | Category | Prefix | Focus |
 |----------|--------|-------|
 | Performance | `PERF-*` | Render optimization, memo patterns, useEffect hygiene, data selection |
-| Consistency | `CONSISTENCY-*` | Platform checks, magic values, unused props, ESLint discipline |
-| Clean React Patterns | `CLEAN-REACT-PATTERNS-*` | Composition, component ownership, state structure |
+| Consistency | `CONSISTENCY-*` | Platform checks, magic values, unused props, ESLint discipline, localization, file naming, JSDoc |
+| Clean React Patterns | `CLEAN-REACT-PATTERNS-*` | Composition, component ownership, state structure, prop typing, function components |
+| UI | `UI-*` | Loading indicators, scrollable pages, styling conventions |
 
 ## Quick Reference
 
@@ -42,6 +43,11 @@ Coding standards for the Expensify App. Each standard is a standalone file in `r
 - [CONSISTENCY-4](rules/consistency-4-no-unused-props.md) — No unused props
 - [CONSISTENCY-5](rules/consistency-5-justify-eslint-disable.md) — Justify ESLint disables
 - [CONSISTENCY-6](rules/consistency-6-proper-error-handling.md) — Proper error handling
+- [CONSISTENCY-7](rules/consistency-7-localize-copy.md) — Localize all user-visible copy
+- [CONSISTENCY-8](rules/consistency-8-localize-numbers-dates.md) — Localize numbers, amounts, dates and phone numbers
+- [CONSISTENCY-9](rules/consistency-9-file-naming.md) — Name files after what they export
+- [CONSISTENCY-10](rules/consistency-10-jsdoc.md) — Follow the JSDoc style guidelines
+- [CONSISTENCY-11](rules/consistency-11-no-todo-comments.md) — Track future work in an issue, not a TODO comment
 
 ### Clean React Patterns
 - [CLEAN-REACT-PATTERNS-0](rules/clean-react-0-compiler.md) — React Compiler compliance
@@ -50,6 +56,15 @@ Coding standards for the Expensify App. Each standard is a standalone file in `r
 - [CLEAN-REACT-PATTERNS-3](rules/clean-react-3-context-free-contracts.md) — Context-free component contracts
 - [CLEAN-REACT-PATTERNS-4](rules/clean-react-4-no-side-effect-spaghetti.md) — No side-effect spaghetti
 - [CLEAN-REACT-PATTERNS-5](rules/clean-react-5-narrow-state.md) — Keep state narrow
+- [CLEAN-REACT-PATTERNS-6](rules/clean-react-6-no-componentprops.md) — Import the exported prop type instead of ComponentProps
+- [CLEAN-REACT-PATTERNS-7](rules/clean-react-7-no-inline-prop-types.md) — Do not inline prop types on exported components
+- [CLEAN-REACT-PATTERNS-8](rules/clean-react-8-no-class-components.md) — Use function components, not class components
+- [CLEAN-REACT-PATTERNS-9](rules/clean-react-9-no-proptypes.md) — Use TypeScript types, not propTypes or defaultProps
+
+### UI
+- [UI-1](rules/ui-1-correct-loading-indicator.md) — Use the correct loading indicator based on navigation context
+- [UI-2](rules/ui-2-new-page-scrollview.md) — New pages must be scrollable
+- [UI-3](rules/ui-3-no-inline-styles.md) — Do not use inline style objects
 
 ## Usage
 

.claude/skills/coding-standards/rules/clean-react-6-no-componentprops.md

@@ -1,9 +1,9 @@
 ---
-ruleId: CLEAN-REACT-6
+ruleId: CLEAN-REACT-PATTERNS-6
 title: Import the exported prop type instead of ComponentProps
 ---
 
-## [CLEAN-REACT-6] Import the exported prop type instead of ComponentProps
+## [CLEAN-REACT-PATTERNS-6] Import the exported prop type instead of ComponentProps
 
 ### Reasoning
 

.claude/skills/coding-standards/rules/clean-react-7-no-inline-prop-types.md

@@ -1,9 +1,9 @@
 ---
-ruleId: CLEAN-REACT-7
+ruleId: CLEAN-REACT-PATTERNS-7
 title: Do not inline prop types on exported components
 ---
 
-## [CLEAN-REACT-7] Do not inline prop types on exported components
+## [CLEAN-REACT-PATTERNS-7] Do not inline prop types on exported components
 
 ### Reasoning
 

.claude/skills/coding-standards/rules/clean-react-8-no-class-components.md

@@ -1,9 +1,9 @@
 ---
-ruleId: CLEAN-REACT-8
+ruleId: CLEAN-REACT-PATTERNS-8
 title: Use function components, not class components
 ---
 
-## [CLEAN-REACT-8] Use function components, not class components
+## [CLEAN-REACT-PATTERNS-8] Use function components, not class components
 
 ### Reasoning
 

.claude/skills/coding-standards/rules/clean-react-9-no-proptypes.md

@@ -1,9 +1,9 @@
 ---
-ruleId: CLEAN-REACT-9
+ruleId: CLEAN-REACT-PATTERNS-9
 title: Use TypeScript types, not propTypes or defaultProps
 ---
 
-## [CLEAN-REACT-9] Use TypeScript types, not propTypes or defaultProps
+## [CLEAN-REACT-PATTERNS-9] Use TypeScript types, not propTypes or defaultProps
 
 ### Reasoning
 

.github/actions/composite/uploadSentrySizeAnalysis/action.yml

@@ -0,0 +1,71 @@
+name: Upload build to Sentry for size analysis
+description: |
+  Download a platform binary and upload it to Sentry for size analysis. The binary is sourced
+  from either the current workflow run's artifacts (cherry-pick prod deploys, which build natively)
+  or the matching staging GitHub Release (re-promotion prod deploys, which do not build natively).
+
+inputs:
+  platform:
+    description: "Platform label used in the GH Actions notice (e.g. 'Android' or 'iOS')"
+    required: true
+  asset-name:
+    description: "Filename of the binary on disk and as the GH Release asset (e.g. 'Expensify-release.aab')"
+    required: true
+  artifact-name:
+    description: "Name of the GH Actions artifact to download for cherry-pick deploys (e.g. 'androidBuild-artifact')"
+    required: true
+  staging-release-tag:
+    description: "Tag of the staging GH Release to download from for re-promotion deploys (e.g. '9.4.13-0-staging')"
+    required: true
+  is-cherry-pick:
+    description: "'true' when this deploy is a cherry-pick (binary is in the current run); 'false' otherwise (binary is on the staging release)"
+    required: true
+
+outputs:
+  SENTRY_URL:
+    description: URL to the Sentry size analysis for the uploaded build (empty if upload failed or quota exhausted)
+    value: ${{ steps.upload.outputs.SENTRY_URL }}
+
+runs:
+  using: composite
+  steps:
+    - name: Setup Node
+      uses: ./.github/actions/composite/setupNode
+
+    - name: Download binary from current workflow run (cherry-pick path)
+      if: ${{ inputs.is-cherry-pick == 'true' }}
+      continue-on-error: true
+      # v7
+      uses: actions/download-artifact@37930b1c2abaa49bbe596cd826c3c89aef350131
+      with:
+        name: ${{ inputs.artifact-name }}
+        path: ./
+
+    - name: Download binary from the staging GitHub Release (re-promotion path)
+      if: ${{ inputs.is-cherry-pick != 'true' }}
+      continue-on-error: true
+      shell: bash
+      env:
+        TAG: ${{ inputs.staging-release-tag }}
+        ASSET: ${{ inputs.asset-name }}
+      run: |
+        gh release download "$TAG" --pattern "$ASSET" --output "$ASSET"
+        [ -s "$ASSET" ] || { echo "::error::Failed to obtain $ASSET from $TAG"; exit 1; }
+
+    - name: Upload to Sentry
+      id: upload
+      if: ${{ hashFiles(inputs.asset-name) != '' }}
+      continue-on-error: true
+      shell: bash
+      env:
+        ASSET: ${{ inputs.asset-name }}
+        PLATFORM: ${{ inputs.platform }}
+      run: |
+        OUTPUT=$(npx sentry-cli build upload "$ASSET" \
+          --org expensify --project app --build-configuration Release --log-level info 2>&1)
+        echo "$OUTPUT"
+        SENTRY_URL=$(echo "$OUTPUT" | grep -oE 'https://expensify\.sentry\.io/[^ )"}]+' | head -1)
+        if [ -n "$SENTRY_URL" ]; then
+          echo "::notice::${PLATFORM} Sentry size analysis: $SENTRY_URL"
+          echo "SENTRY_URL=$SENTRY_URL" >> "$GITHUB_OUTPUT"
+        fi

.github/workflows/buildAdHoc.yml

@@ -54,7 +54,7 @@ jobs:
               owner: context.repo.owner,
               repo: context.repo.repo,
               issue_number: ${{ inputs.APP_PR_NUMBER || 0 }},
-              body: `🚧 @${{ github.actor }} has triggered a test Expensify/App build. You can view the [workflow run here](${workflowURL}).`
+              body: `🚧 ${{ github.actor }} has triggered a test Expensify/App build. You can view the [workflow run here](${workflowURL}).`
               });
 
       - name: Add build start comment to Expensify/Mobile-Expensify PR
@@ -69,7 +69,7 @@ jobs:
               owner: context.repo.owner,
               repo: 'Mobile-Expensify',
               issue_number: ${{ inputs.MOBILE_EXPENSIFY_PR || 0 }},
-              body: `🚧 @${{ github.actor }} has triggered a test Expensify/Mobile-Expensify build. You can view the [workflow run here](${workflowURL}).`
+              body: `🚧 ${{ github.actor }} has triggered a test Expensify/Mobile-Expensify build. You can view the [workflow run here](${workflowURL}).`
               });
 
   buildAndroid:

.github/workflows/buildAndroid.yml

@@ -35,9 +35,6 @@ on:
       ROCK_ARTIFACT_URL:
         description: URL to download the ad-hoc build artifact (adhoc only)
         value: ${{ jobs.build.outputs.ROCK_ARTIFACT_URL }}
-      SENTRY_URL:
-        description: URL to Sentry size analysis
-        value: ${{ jobs.build.outputs.SENTRY_URL }}
       AAB_FILENAME:
         description: Filename of the AAB artifact (empty if no AAB was produced)
         value: ${{ jobs.build.outputs.AAB_FILENAME }}
@@ -60,7 +57,6 @@ jobs:
     outputs:
       VERSION_CODE: ${{ steps.getAndroidVersion.outputs.VERSION_CODE }}
       ROCK_ARTIFACT_URL: ${{ steps.set-artifact-url.outputs.ARTIFACT_URL }}
-      SENTRY_URL: ${{ steps.sentry-upload.outputs.SENTRY_URL }}
       AAB_FILENAME: ${{ steps.collectArtifacts.outputs.AAB_FILENAME }}
       APK_FILENAME: ${{ steps.collectArtifacts.outputs.APK_FILENAME }}
       SOURCEMAP_FILENAME: index.android.bundle.map
@@ -195,22 +191,6 @@ jobs:
         if: ${{ inputs.variant == 'Adhoc' }}
         run: echo "ARTIFACT_URL=$ARTIFACT_URL" >> "$GITHUB_OUTPUT"
 
-      - name: Upload Android build to Sentry for size analysis
-        id: sentry-upload
-        if: ${{ inputs.variant == 'Release' && env.ARTIFACT_PATH != '' }}
-        continue-on-error: true
-        timeout-minutes: 5
-        run: |
-          OUTPUT=$(npx sentry-cli build upload "$ARTIFACT_PATH" --org expensify --project app --build-configuration Release --log-level debug 2>&1)
-          echo "$OUTPUT"
-          SENTRY_URL=$(echo "$OUTPUT" | grep -oE 'https://expensify\.sentry\.io/[^ ]+' | head -1)
-          if [ -n "$SENTRY_URL" ]; then
-            echo "::notice::Android Sentry size analysis: $SENTRY_URL"
-            echo "SENTRY_URL=$SENTRY_URL" >> "$GITHUB_OUTPUT"
-          fi
-        env:
-          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
-
       - name: Upload Gradle profile report
         if: always()
         # v6

.github/workflows/buildIOS.yml

@@ -35,9 +35,6 @@ on:
       ROCK_ARTIFACT_URL:
         description: URL to download the ad-hoc build artifact (adhoc only)
         value: ${{ jobs.build.outputs.ROCK_ARTIFACT_URL }}
-      SENTRY_URL:
-        description: URL to Sentry size analysis
-        value: ${{ jobs.build.outputs.SENTRY_URL }}
       IPA_FILENAME:
         description: Filename of the IPA artifact produced by the build
         value: ${{ jobs.build.outputs.IPA_FILENAME }}
@@ -58,7 +55,6 @@ jobs:
     outputs:
       IOS_VERSION: ${{ steps.getIOSVersion.outputs.IOS_VERSION }}
       ROCK_ARTIFACT_URL: ${{ steps.set-artifact-url.outputs.ARTIFACT_URL }}
-      SENTRY_URL: ${{ steps.sentry-upload.outputs.SENTRY_URL }}
       IPA_FILENAME: ${{ steps.set-ipa-filename.outputs.IPA_FILENAME }}
       DSYM_FILENAME: ${{ steps.set-ipa-filename.outputs.DSYM_FILENAME }}
       SOURCEMAP_FILENAME: main.jsbundle.map
@@ -244,22 +240,6 @@ jobs:
           comment-bot: false
           custom-identifier: ${{ steps.computeIdentifier.outputs.IDENTIFIER }}
 
-      - name: Upload iOS build to Sentry for size analysis
-        id: sentry-upload
-        if: ${{ inputs.variant == 'Release' && env.ARTIFACT_PATH != '' }}
-        continue-on-error: true
-        timeout-minutes: 5
-        run: |
-          OUTPUT=$(npx sentry-cli build upload "$ARTIFACT_PATH" --org expensify --project app --build-configuration Release --log-level debug 2>&1)
-          echo "$OUTPUT"
-          SENTRY_URL=$(echo "$OUTPUT" | grep -oE 'https://expensify\.sentry\.io/[^ ]+' | head -1)
-          if [ -n "$SENTRY_URL" ]; then
-            echo "::notice::iOS Sentry size analysis: $SENTRY_URL"
-            echo "SENTRY_URL=$SENTRY_URL" >> "$GITHUB_OUTPUT"
-          fi
-        env:
-          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
-
       - name: Set artifact URL output
         id: set-artifact-url
         if: ${{ inputs.variant == 'Adhoc' }}

.github/workflows/buildVictoryChartRenderer.yml

@@ -51,7 +51,7 @@ jobs:
       - name: Log victory-chart-renderer build failure
         if: failure() || cancelled()
         run: |
-          echo "::error::victory-chart-renderer Linux x64 build did not succeed for release ${{ inputs.release-tag }}. GitHub Release will not include victory-chart-renderer-linux-x64."
+          echo "::error::victory-chart-renderer Linux x64 build did not succeed for release ${{ inputs.release-tag }}. The deploy GitHub Release will not be created."
 
       - name: Announce victory-chart-renderer build failure in Slack
         if: failure() || cancelled()
@@ -63,8 +63,26 @@ jobs:
             {
               channel: '#announce',
               attachments: [{
-                color: 'warning',
-                text: `⚠️ ${process.env.AS_REPO} <https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}|victory-chart-renderer build> failed for deploy release <https://github.com/Expensify/App/releases/tag/${{ inputs.release-tag }}|${{ inputs.release-tag }}>. victory-chart-renderer-linux-x64 will not be attached.`,
+                color: '#DB4545',
+                text: `💥 ${process.env.AS_REPO} <https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}|victory-chart-renderer build> failed for deploy release <https://github.com/Expensify/App/releases/tag/${{ inputs.release-tag }}|${{ inputs.release-tag }}>. The deploy GitHub Release was blocked.`,
+              }]
+            }
+        env:
+          GITHUB_TOKEN: ${{ github.token }}
+          SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK }}
+
+      - name: Announce victory-chart-renderer build failure in deployer Slack
+        if: failure()
+        # v3
+        uses: 8398a7/action-slack@1750b5085f3ec60384090fb7c52965ef822e869e
+        with:
+          status: custom
+          custom_payload: |
+            {
+              channel: '#deployer',
+              attachments: [{
+                color: '#DB4545',
+                text: `💥 ${process.env.AS_REPO} <https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}|victory-chart-renderer build> failed for deploy release <https://github.com/Expensify/App/releases/tag/${{ inputs.release-tag }}|${{ inputs.release-tag }}>. The deploy GitHub Release was blocked.`,
               }]
             }
         env: