ci: disable Playwright and e2e test suites - #71
Merged
Merged
Conversation
The browser suites and the live worker e2e script need a Kind cluster, built images, and a Claude subscription, so they no longer run in CI or from the default local entry points. - Drop the E2E Tests job from CI; PRs now run lint and image builds only. The build job no longer exports image tarballs, since only that job consumed them. - Guard make test/test-run/test-ci/test-worker-e2e and the frontend pnpm test scripts behind ENABLE_E2E=1 via scripts/e2e-guard.sh - Record the opt-in in AGENTS.md and frontend/specs/README.md Test files, playwright.config.ts, and @playwright/test stay in place for deliberate opt-in runs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR: ci: disable Playwright and e2e test suites
Playwright and the live-agent e2e paths are off by default, in CI and locally.
CI — removed the
E2E Testsjob (Kind cluster, browser install/cache,CLAUDE_CODE_OAUTH_TOKENsecret, deploy, report uploads). PRs now runLint+Buildonly. Also dropped the now-deadKIND_CLUSTER_NAMEenv and the build job's image-tarball export/upload, which only that job consumed — images are still built, so the Dockerfiles stay validated.Local — new
scripts/e2e-guard.shfails with an explanatory message unlessENABLE_E2E=1. Wired intomake test,make test-run,make test-ci,make test-worker-e2e, and the frontendtest,test:ui,test:debugscripts.test:reportis untouched.Docs —
AGENTS.mdverification section andfrontend/specs/README.mdstate the suites are disabled and how to opt in.Nothing is deleted: the test files,
playwright.config.ts, and the@playwright/testdep remain for an opt-in run.Verification — guards confirmed to block by default and pass with
ENABLE_E2E=1; CI YAML and package.json parse;trunk fmt/checkclean on the changed files (one trivy failure from its own broken rego check bundle, unrelated to this diff).