Skip to content

ci: concurrency groups, e2e sharding, React Doctor cache fix, branch cache cleanup #2377

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
malinskibeniamin merged 7 commits into from
Apr 14, 2026
+19 −11
Merged

ci: concurrency groups, e2e sharding, React Doctor cache fix, branch cache cleanup #2377

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
7268b24
ci: add concurrency groups, shard e2e tests, add Blacksmith audit
malinskibeniamin Apr 13, 2026
a40b3c2
fix(e2e): use exact match for quota rate value to prevent flaky test
malinskibeniamin Apr 14, 2026
8662c5f
fix(ci): stop caching node_modules in React Doctor workflow
malinskibeniamin Apr 14, 2026
1475ee9
ci: add cache cleanup workflow for deleted branches
malinskibeniamin Apr 14, 2026
33fb769
fix(ci): address review feedback — runner, pagination, shard format
malinskibeniamin Apr 14, 2026
932982b
fix(ci): use strategy.job-index for integration test artifact name
malinskibeniamin Apr 14, 2026
eb0b60a
chore: remove cleanup-branch-cache workflow
malinskibeniamin Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions .github/workflows/frontend-react-doctor.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,9 @@ on:
paths:
- "frontend/**"
- ".github/workflows/frontend-react-doctor.yml"
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
jobs:
react-doctor:
timeout-minutes: 5
Expand All @@ -22,12 +25,10 @@ jobs:
- name: Cache dependencies
uses: actions/cache@v4
with:
path: |
~/.bun/install/cache
frontend/node_modules
key: ${{ runner.os }}-bun-frontend-${{ hashFiles('frontend/bun.lock') }}
path: ~/.bun/install/cache
key: ${{ runner.os }}-bun-cache-${{ hashFiles('frontend/bun.lock') }}
restore-keys: |
${{ runner.os }}-bun-frontend-
${{ runner.os }}-bun-cache-
- name: Install dependencies
run: bun install --frozen-lockfile
- name: Run React Doctor
Expand Down
14 changes: 9 additions & 5 deletions .github/workflows/frontend-verify.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -176,14 +176,18 @@ jobs:
- uses: actions/upload-artifact@v4
if: failure()
with:
name: integration-test-results-${{ matrix.shard }}
name: integration-test-results-${{ strategy.job-index }}
path: frontend/test-results
retention-days: 2

e2e-test:
needs: ["lint", "type-check", "build"]
timeout-minutes: 30
runs-on: blacksmith-16vcpu-ubuntu-2404
timeout-minutes: 20
runs-on: blacksmith-8vcpu-ubuntu-2404
strategy:
fail-fast: false
matrix:
shard: [1/2, 2/2]
defaults:
run:
working-directory: frontend
Expand Down Expand Up @@ -226,11 +230,11 @@ jobs:
if: steps.playwright-cache.outputs.cache-hit != 'true'
run: bun run install:chromium
- name: Run Playwright tests
run: bun run e2e-test
run: bun run e2e-test -- --shard=${{ matrix.shard }}
- name: Upload Playwright report artifact
uses: actions/upload-artifact@v4
if: ${{ failure() }}
with:
name: playwright-report
name: playwright-report-${{ strategy.job-index }}
path: frontend/playwright-report/
retention-days: 2
3 changes: 3 additions & 0 deletions .github/workflows/repository-dispatch.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ on:
- '*'
branches:
- '**'
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
permissions:
id-token: write
contents: read
Expand Down
2 changes: 1 addition & 1 deletion frontend/tests/test-variant-console/quotas/quota-display.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ test.describe('Quotas - Display and Data Verification', () => {
const row = page.locator('tr').filter({ hasText: quotaClientId });
await expect(row.locator('td').filter({ hasText: '10 MiB' })).toBeVisible();
await expect(row.locator('td').filter({ hasText: '5 MiB' })).toBeVisible();
await expect(row.locator('td').filter({ hasText: '100' })).toBeVisible();
await expect(row.locator('td').filter({ hasText: /^100$/ })).toBeVisible();
});

await test.step('Cleanup', async () => {
Expand Down
Loading