Skip to content

Commit 2126fc8

Browse files
eakmanrqclaude
andcommitted
Fix CI: run ui-test on ubuntu-latest instead of Playwright container
The Playwright container had slow I/O causing the webServer build to exceed the 120s timeout. Running directly on ubuntu-latest with explicit Playwright browser installation avoids container overhead and enables pnpm caching. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: eakmanrq <6326532+eakmanrq@users.noreply.github.com>
1 parent 6e41807 commit 2126fc8

File tree

1 file changed

+16
-8
lines changed

1 file changed

+16
-8
lines changed

.github/workflows/pr.yaml

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -219,18 +219,26 @@ jobs:
219219
needs.changes.outputs.client == 'true' || needs.changes.outputs.ci ==
220220
'true' || github.ref == 'refs/heads/main'
221221
runs-on: ubuntu-latest
222-
container:
223-
image: mcr.microsoft.com/playwright:v1.54.1-jammy
224222
steps:
225223
- uses: actions/checkout@v5
226-
- name: Install pnpm via corepack
227-
run: |
228-
npm install --global corepack@latest
229-
corepack enable
230-
corepack prepare pnpm@latest-10 --activate
231-
pnpm config set store-dir .pnpm-store
224+
- uses: actions/setup-node@v6
225+
with:
226+
node-version: '20'
227+
- uses: pnpm/action-setup@v4
228+
with:
229+
version: latest
230+
- name: Get pnpm store directory
231+
id: pnpm-cache
232+
run: echo "store=$(pnpm store path)" >> $GITHUB_OUTPUT
233+
- uses: actions/cache@v4
234+
with:
235+
path: ${{ steps.pnpm-cache.outputs.store }}
236+
key: pnpm-store-${{ hashFiles('pnpm-lock.yaml') }}
237+
restore-keys: pnpm-store-
232238
- name: Install dependencies
233239
run: pnpm install
240+
- name: Install Playwright browsers
241+
run: pnpm --prefix web/client exec playwright install --with-deps
234242
- name: Run tests
235243
run: npm --prefix web/client run test
236244

0 commit comments

Comments
 (0)