Side Pane Stage 2: Add baseline content for each major section

.github/workflows/cherryPick.yml

@@ -95,34 +95,42 @@ jobs:
         if: fromJSON(steps.cherryPick.outputs.HAS_CONFLICTS)
         id: createPullRequest
         run: |
-          gh pr create \
-            --title "🍒 Cherry pick PR #${{ github.event.inputs.PULL_REQUEST_NUMBER }} to staging 🍒" \
-            --body \
-              "🍒 Cherry pick https://github.com/Expensify/App/pull/${{ github.event.inputs.PULL_REQUEST_NUMBER }} to staging 🍒
-              This PR had conflicts when we tried to cherry-pick it to staging. You'll need to manually perform the cherry-pick, using the following steps:
+          AUTHOR_CHECKLIST=$(sed -n '/### PR Author Checklist/,$p' .github/PULL_REQUEST_TEMPLATE.md)
+
+          PR_DESCRIPTION=$(cat <<EOF
+          🍒 Cherry pick https://github.com/Expensify/App/pull/56382 to staging 🍒
+
+          This PR had conflicts when we tried to cherry-pick it to staging. You'll need to manually perform the cherry-pick, using the following steps:
+
+          \`\`\`bash
+          git fetch
+          git checkout roryabraham-cherry-pick-staging-56382-1
+          git cherry-pick -S -x --mainline 1 6486b1d1961d4c6de85154905d57af5118b33ff9
+          \`\`\`
 
-              \`\`\`bash
-              git fetch
-              git checkout ${{ github.actor }}-cherry-pick-staging-${{ github.event.inputs.PULL_REQUEST_NUMBER }}-${{ github.run_attempt }}
-              git cherry-pick -S -x --mainline 1 ${{ steps.getCPMergeCommit.outputs.MERGE_COMMIT_SHA }}
-              \`\`\`
+          Then manually resolve conflicts, and commit the change with \`git cherry-pick --continue\`. Lastly, please run:
 
-              Then manually resolve conflicts, and commit the change with \`git cherry-pick --continue\`. Lastly, please run:
+          \`\`\`bash
+          git commit --amend -m "\$(git log -1 --pretty=%B)" -m "(CP triggered by roryabraham)"
+          \`\`\`
 
-              \`\`\`bash
-              git commit --amend -m \"$(git log -1 --pretty=%B)\" -m \"(CP triggered by ${{ github.actor }})\"
-              \`\`\`
+          That will help us keep track of who triggered this CP. Once all that's done, push your changes with \`git push origin roryabraham-cherry-pick-staging-6486b1d1961d4c6de85154905d57af5118b33ff9-1\`, and then open this PR for review.
 
-              That will help us keep track of who triggered this CP. Once all that's done, push your changes with \`git push origin ${{ github.actor }}-cherry-pick-staging-${{ github.event.inputs.PULL_REQUEST_NUMBER }}-${{ github.run_attempt }}\`, and then open this PR for review.
-
-              Note that you **must** test this PR, and both the author and reviewer checklist should be completed, just as if you were merging the PR to main." \
+          Note that you **must** test this PR, and both the author and reviewer checklist should be completed, just as if you were merging the PR to main.
+
+          _Pro-tip:_ If this PR appears to have conflicts against the _staging_ base, it means that the version on staging has been updated. The easiest thing to do if you see this is to close the PR and re-run the CP.
+
+          $AUTHOR_CHECKLIST
+          EOF
+          )
+
+          # Create PR
+          gh pr create \
+            --title "🍒 Cherry pick PR #${{ github.event.inputs.PULL_REQUEST_NUMBER }} to staging 🍒" \
+            --body "$PR_DESCRIPTION" \
             --label "Engineering,Hourly" \
             --base "staging"
           sleep 5
-          gh pr comment --body \
-            "This pull request has merge conflicts and can not be automatically merged. :disappointed:
-             Please manually resolve the conflicts, push your changes, and then request another reviewer to review and merge.
-             **Important:** There may be conflicts that GitHub is not able to detect, so please _carefully_ review this pull request before approving."
           ORIGINAL_PR_AUTHOR="$(gh pr view ${{ github.event.inputs.PULL_REQUEST_NUMBER }} --json author --jq .author.login)"
           gh pr edit --add-assignee "${{ github.actor }},${{ steps.getCPMergeCommit.outputs.MERGE_ACTOR }},$ORIGINAL_PR_AUTHOR"
         env:

.github/workflows/lint.yml

@@ -14,7 +14,7 @@ concurrency:
 jobs:
   lint:
     name: ESLint check
-    if: ${{ github.actor != 'OSBotify' || github.event_name == 'workflow_call' }}
+    if: ${{ github.event.head_commit.author.name != 'OSBotify' || github.event_name == 'workflow_call' }}
     runs-on: ubuntu-latest
     steps:
       - name: Checkout

.github/workflows/test.yml

@@ -13,7 +13,7 @@ concurrency:
 
 jobs:
   jest:
-    if: ${{ github.actor != 'OSBotify' && github.actor != 'imgbot[bot]' || github.event_name == 'workflow_call' }}
+    if: ${{ github.event.head_commit.author.name != 'OSBotify' && github.event.head_commit.author.name != 'imgbot[bot]' || github.event_name == 'workflow_call' }}
     runs-on: ubuntu-latest
     env:
       CI: true
@@ -44,7 +44,7 @@ jobs:
         run: NODE_OPTIONS="$NODE_OPTIONS --experimental-vm-modules" npm test -- --silent --shard=${{ fromJSON(matrix.chunk) }}/${{ strategy.job-total }} --max-workers ${{ steps.cpu-cores.outputs.count }}
 
   storybookTests:
-    if: ${{ github.actor != 'OSBotify' && github.actor != 'imgbot[bot]' || github.event_name == 'workflow_call' }}
+    if: ${{ github.event.head_commit.author.name != 'OSBotify' && github.event.head_commit.author.name != 'imgbot[bot]' || github.event_name == 'workflow_call' }}
     runs-on: ubuntu-latest
     name: Storybook tests
     steps:

.github/workflows/testBuildHybrid.yml

@@ -45,15 +45,18 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Validate that user is an Expensify employee
-        if: github.event_name == 'workflow_dispatch'
+        if: ${{ github.event_name == 'workflow_dispatch' }}
         uses: ./.github/actions/composite/validateActor
         with:
           REQUIRE_APP_DEPLOYER: false
           OS_BOTIFY_TOKEN: ${{ secrets.OS_BOTIFY_COMMIT_TOKEN }}
 
       - name: Validate input
-        if: github.event_name == 'workflow_dispatch'
         run: |
+          if [[ "${{ github.event_name }}" == "workflow_call" ]]; then
+            exit 0
+          fi
+
           if [[ -z "${{ github.event.inputs.APP_PULL_REQUEST_NUMBER }}" && -z "${{ github.event.inputs.HYBRIDAPP_PULL_REQUEST_NUMBER }}" ]]; then
             echo "Invalid input. You have to pass at least one PR number"
             exit 1

android/app/build.gradle

@@ -114,8 +114,8 @@ android {
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
         multiDexEnabled rootProject.ext.multiDexEnabled
-        versionCode 1009010902
-        versionName "9.1.9-2"
+        versionCode 1009010906
+        versionName "9.1.9-6"
         // 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"

contributingGuides/REASSURE_PERFORMANCE_TEST.md

@@ -26,9 +26,9 @@ We use Reassure for monitoring performance regression. It helps us check if our
 ## Running tests locally 
 
 - Checkout your base environment, eg. `git checkout main`.
-- Collect baseline metrics with `npx reassure --baseline`.
+- Collect baseline metrics with `npm run perf-test -- --baseline`.
 - Apply any desired changes (for testing purposes you can eg. try to slow down a list).
-- Collect current metrics with `npx reassure`.
+- Collect current metrics with `npm run perf-test`.
 - Open up the resulting `output.md` / `output.json` (see console output) to compare the results.
 - With all that information, Reassure can present the render duration times as statistically significant or meaningless.
 

desktop/ELECTRON_EVENTS.ts

@@ -14,6 +14,7 @@ const ELECTRON_EVENTS = {
     DOWNLOAD_FAILED: 'download-started',
     DOWNLOAD_CANCELED: 'download-canceled',
     SILENT_UPDATE: 'silent-update',
+    OPEN_LOCATION_SETTING: 'open-location-setting',
 } as const;
 
 export default ELECTRON_EVENTS;

desktop/contextBridge.ts

@@ -18,6 +18,7 @@ const WHITELIST_CHANNELS_RENDERER_TO_MAIN = [
     ELECTRON_EVENTS.LOCALE_UPDATED,
     ELECTRON_EVENTS.DOWNLOAD,
     ELECTRON_EVENTS.SILENT_UPDATE,
+    ELECTRON_EVENTS.OPEN_LOCATION_SETTING,
 ] as const;
 
 const WHITELIST_CHANNELS_MAIN_TO_RENDERER = [