Skip to content

ci: shard local-auth Playwright gate - #2743

Merged
Patrick-Erichsen merged 1 commit into
mainfrom
pe/local-auth-matrix-cache
Jun 19, 2026
Merged

ci: shard local-auth Playwright gate#2743
Patrick-Erichsen merged 1 commit into
mainfrom
pe/local-auth-matrix-cache

Conversation

@Patrick-Erichsen

Copy link
Copy Markdown
Collaborator

Summary

  • Split the playwright-local-auth gate into one matrix shard per local-auth spec, with the shared publish lifecycle spec split by grep so each shard runs one test path.
  • Keep the required playwright-local-auth check name as a lightweight aggregate job over the matrix.
  • Add a shared Playwright browser cache and avoid reinstalling system deps on every Playwright job.

RCA / Intent

The last-run pattern points more at slowness than nondeterministic test behavior: the long pole is the monolithic local-auth job doing repeated setup plus all signed-in browser flows serially. Sharding makes the gate parallel and the aggregate job keeps branch protection semantics stable.

Validation

  • actionlint .github/workflows/ci.yml
  • git diff --check
  • bun run format:check -- .github/workflows/ci.yml
  • bun run ci:static
  • PLAYWRIGHT_LOCAL_AUTH_CONVEX_URL=http://127.0.0.1:3410 PLAYWRIGHT_LOCAL_AUTH_CONVEX_SITE_URL=http://127.0.0.1:3411 PLAYWRIGHT_PORT=4373 bun run test:pw:local-auth -- --project=chromium e2e/local-auth/header-profile-link.pw.test.ts
  • PLAYWRIGHT_LOCAL_AUTH_CONVEX_URL=http://127.0.0.1:3420 PLAYWRIGHT_LOCAL_AUTH_CONVEX_SITE_URL=http://127.0.0.1:3421 PLAYWRIGHT_PORT=4383 bun run test:pw:local-auth -- --project=chromium e2e/local-auth/publish-skill-lifecycle.pw.test.ts --grep "publishing a skill queues scan"
  • .agents/skills/autoreview/scripts/autoreview --mode local

@Patrick-Erichsen
Patrick-Erichsen requested a review from a team as a code owner June 19, 2026 00:37
@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clawhub Ready Ready Preview, Comment Jun 19, 2026 12:37am

@Patrick-Erichsen
Patrick-Erichsen merged commit 6bbdec2 into main Jun 19, 2026
32 checks passed
@Patrick-Erichsen
Patrick-Erichsen deleted the pe/local-auth-matrix-cache branch June 19, 2026 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant