ci: Update actions/upload-artifact to v4 (#13489)

It keeps complaining that this is deprecated, so bumping this to v4.

Author: mydea

.github/workflows/build.yml

@@ -185,7 +185,7 @@ jobs:
         with:
           name: build-output
           path: ${{ env.CACHED_BUILD_PATHS }}
-          retention-days: 7
+          retention-days: 4
           compression-level: 6
           overwrite: true
 
@@ -345,6 +345,7 @@ jobs:
         uses: actions/upload-artifact@v4
         with:
           name: ${{ github.sha }}
+          retention-days: 90
           path: |
             ${{ github.workspace }}/packages/browser/build/bundles/**
             ${{ github.workspace }}/packages/replay-internal/build/bundles/**
@@ -597,11 +598,13 @@ jobs:
         run: yarn test:ci${{ matrix.project && format(' --project={0}', matrix.project) || '' }}${{ matrix.shard && format(' --shard={0}/{1}', matrix.shard, matrix.shards) || '' }}
 
       - name: Upload Playwright Traces
-        uses: actions/upload-artifact@v3
-        if: always()
+        uses: actions/upload-artifact@v4
+        if: failure()
         with:
-          name: playwright-traces
+          name: playwright-traces-job_browser_playwright_tests-${{ matrix.bundle}}-${{matrix.project}}-${{matrix.shard || '0'}}
           path: dev-packages/browser-integration-tests/test-results
+          overwrite: true
+          retention-days: 7
 
   job_browser_loader_tests:
     name: PW ${{ matrix.bundle }} Tests
@@ -647,11 +650,13 @@ jobs:
           cd dev-packages/browser-integration-tests
           yarn test:loader
       - name: Upload Playwright Traces
-        uses: actions/upload-artifact@v3
-        if: always()
+        uses: actions/upload-artifact@v4
+        if: failure()
         with:
-          name: playwright-traces
+          name: playwright-traces-job_browser_loader_tests-${{ matrix.bundle}}
           path: dev-packages/browser-integration-tests/test-results
+          overwrite: true
+          retention-days: 7
 
   job_check_for_faulty_dts:
     name: Check for faulty .d.ts files
@@ -1261,6 +1266,7 @@ jobs:
         with:
           name: ${{ steps.process.outputs.artifactName }}
           path: ${{ steps.process.outputs.artifactPath }}
+          retention-days: 7
 
   job_compile_bindings_profiling_node:
     name: Compile & Test Profiling Bindings (v${{ matrix.node }}) ${{ matrix.target_platform || matrix.os }}, ${{ matrix.node || matrix.container }}, ${{ matrix.arch || matrix.container }}, ${{ contains(matrix.container, 'alpine') && 'musl' || 'glibc'  }}