From 6cbb83a1e2f4334ab3dc13ba9cb129bd787c8b1a Mon Sep 17 00:00:00 2001 From: Johnpaul Chiwetelu <49923152+Myestery@users.noreply.github.com> Date: Tue, 13 Jan 2026 22:43:32 +0100 Subject: [PATCH] perf(ci): remove unnecessary pnpm install from merge-reports job (#8030) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Summary - Remove `setup-frontend` action from `merge-reports` job - Use `npx @playwright/test` instead of `pnpm exec playwright` ## Why The `merge-reports` job was spending ~16-18s on `pnpm install` just to run a CLI command that takes ~3s. Since `npx` is pre-installed on GitHub runners, we can eliminate the setup overhead entirely. **Expected savings: ~16-18 seconds per CI run** ## Test Plan - [ ] Verify merge-reports job completes successfully - [ ] Verify HTML report is generated and uploaded correctly - [ ] Compare job timing before/after ┆Issue is synchronized with this [Notion page](https://www.notion.so/PR-8030-perf-ci-remove-unnecessary-pnpm-install-from-merge-reports-job-2e76d73d36508134b3e6c11726170f64) by [Unito](https://www.unito.io) --- .github/workflows/ci-tests-e2e.yaml | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/.github/workflows/ci-tests-e2e.yaml b/.github/workflows/ci-tests-e2e.yaml index 90bf4112c..c1e0af411 100644 --- a/.github/workflows/ci-tests-e2e.yaml +++ b/.github/workflows/ci-tests-e2e.yaml @@ -144,9 +144,10 @@ jobs: - name: Checkout repository uses: actions/checkout@v5 - # Setup pnpm/node to run playwright merge-reports (no browsers needed) - - name: Setup frontend - uses: ./.github/actions/setup-frontend + - name: Install pnpm + uses: pnpm/action-setup@v4 + with: + version: 10 - name: Download blob reports uses: actions/download-artifact@v4 @@ -158,10 +159,10 @@ jobs: - name: Merge into HTML Report run: | # Generate HTML report - pnpm exec playwright merge-reports --reporter=html ./all-blob-reports + pnpm dlx @playwright/test merge-reports --reporter=html ./all-blob-reports # Generate JSON report separately with explicit output path PLAYWRIGHT_JSON_OUTPUT_NAME=playwright-report/report.json \ - pnpm exec playwright merge-reports --reporter=json ./all-blob-reports + pnpm dlx @playwright/test merge-reports --reporter=json ./all-blob-reports - name: Upload HTML report uses: actions/upload-artifact@v4