From 7ffb0ffced8b93b7f4a697ff53b4344eb1c32fa6 Mon Sep 17 00:00:00 2001
From: David Langley <davidl@element.io>
Date: Fri, 31 Jul 2026 14:03:35 +0100
Subject: [PATCH] Stop caching Vite pre-bundled deps in shared-components
 vitest CI (#34472)

Signed-off-by: David Langley <davidl@element.io>
---
 .github/workflows/tests.yml | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 4d5e03d216..748553ec7a 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -129,12 +129,13 @@ jobs:
               run: "pnpm install"
               if: matrix.path == 'packages/shared-components'
 
-            - name: Cache storybook & vitest
+            - name: Cache storybook
               uses: actions/cache@55cc8345863c7cc4c66a329aec7e433d2d1c52a9 # v6
               with:
+                  # NB: node_modules/.vite/vitest is intentionally not cached; a lockfile-keyed stale bundle
+                  # triggers a mid-run re-optimise + page reload that flakes browser-mode tests.
                   path: |
                       ${{ matrix.path }}/node_modules/.cache
-                      ${{ matrix.path }}/node_modules/.vite/vitest
                   key: ${{ matrix.path }}-${{ hashFiles('pnpm-lock.yaml') }}
 
             - name: Setup playwright