ci: try six workers across Playwright suites

Restore serial browser bundling and retain three Node integration shards.

Co-Authored-By: GPT-6 <codex@openai.com>

Author: nicohrubec

dev-packages/browser-integration-tests/playwright.browser.config.ts

@@ -3,7 +3,6 @@ import CorePlaywrightConfig from './playwright.config';
 
 const config: PlaywrightTestConfig = {
   ...CorePlaywrightConfig,
-  workers: process.env.CI ? 6 : CorePlaywrightConfig.workers,
   testDir: './suites',
 };
 

dev-packages/browser-integration-tests/playwright.config.ts

@@ -5,9 +5,7 @@ const config: PlaywrightTestConfig = {
   retries: 0,
   // Run tests inside of a single file in parallel
   fullyParallel: true,
-  // Use 3 workers on CI, else use defaults (based on available CPU cores)
-  // Note that 3 is a random number selected to work well with our CI setup
-  workers: process.env.CI ? 3 : undefined,
+  workers: process.env.CI ? 6 : undefined,
   testMatch: /test.ts/,
 
   use: {

dev-packages/e2e-tests/test-applications/cloudflare-workers-send-to-sentry/playwright.config.ts

@@ -10,7 +10,6 @@ export default defineConfig({
   fullyParallel: true,
   forbidOnly: !!process.env.CI,
   retries: 0,
-  // Every test spends most of its time polling Sentry, so run them all at once.
-  workers: '100%',
+  workers: process.env.CI ? 6 : '100%',
   reporter: process.env.CI ? [['list'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
 });

dev-packages/e2e-tests/test-applications/ember-classic/playwright.config.ts

@@ -30,7 +30,7 @@ const config: PlaywrightTestConfig = {
     timeout: 10000,
   },
   fullyParallel: false,
-  workers: 1,
+  workers: process.env.CI ? 6 : 1,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   retries: 0,

dev-packages/e2e-tests/test-applications/ember-embroider/playwright.config.ts

@@ -30,7 +30,7 @@ const config: PlaywrightTestConfig = {
     timeout: 10000,
   },
   fullyParallel: false,
-  workers: 1,
+  workers: process.env.CI ? 6 : 1,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   retries: 0,

dev-packages/e2e-tests/test-applications/node-express-send-to-sentry-static/playwright.config.mjs

@@ -18,6 +18,7 @@ const config = {
   },
   /* Run tests in files in parallel */
   fullyParallel: true,
+  workers: process.env.CI ? 6 : undefined,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */

dev-packages/e2e-tests/test-applications/node-express-send-to-sentry/playwright.config.mjs

@@ -18,6 +18,7 @@ const config = {
   },
   /* Run tests in files in parallel */
   fullyParallel: true,
+  workers: process.env.CI ? 6 : undefined,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */

dev-packages/e2e-tests/test-applications/react-send-to-sentry/playwright.config.mjs

@@ -20,8 +20,7 @@ const config = {
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
   retries: 0,
-  /* Opt out of parallel tests on CI. */
-  workers: 1,
+  workers: process.env.CI ? 6 : 1,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */
   reporter: process.env.CI ? [['list'], ['junit', { outputFile: 'results.junit.xml' }]] : 'list',
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */

dev-packages/test-utils/src/playwright-config.ts

@@ -31,7 +31,6 @@ export function getPlaywrightConfig(
       timeout: 10000,
     },
     fullyParallel: false,
-    workers: 1,
     /* Fail the build on CI if you accidentally left test.only in the source code. */
     forbidOnly: !!process.env.CI,
     /* In dev mode some apps are flaky, so we allow retry there... */
@@ -96,5 +95,6 @@ export function getPlaywrightConfig(
   return {
     ...config,
     ...overwriteConfig,
+    workers: process.env.CI ? 6 : (overwriteConfig?.workers ?? 1),
   };
 }

packages/browser/package.json

@@ -52,10 +52,7 @@
   "scripts": {
     "build": "run-p build:transpile build:bundle build:types",
     "build:dev": "run-p build:transpile build:types",
-    "build:bundle": "run-p build:bundle:shard:*",
-    "build:bundle:shard:1": "rollup -c rollup.bundle.config.mjs --configShard=1",
-    "build:bundle:shard:2": "rollup -c rollup.bundle.config.mjs --configShard=2",
-    "build:bundle:shard:3": "rollup -c rollup.bundle.config.mjs --configShard=3",
+    "build:bundle": "rollup -c rollup.bundle.config.mjs",
     "build:transpile": "rollup -c rollup.npm.config.mjs",
     "build:types": "tsc -p tsconfig.types.json",
     "build:watch": "run-p build:transpile:watch build:bundle:watch",