@nx/playwright 24.0.0-beta.2 → 24.0.0-beta.4

package/dist/index.d.ts

@@ -1,3 +1,3 @@
-export { playwrightExecutor, PlaywrightExecutorSchema, } from './src/executors/playwright/playwright.impl';
+export type { PlaywrightExecutorSchema } from './src/executors/playwright/schema';
 export { initGenerator } from './src/generators/init/init';
 export { configurationGenerator } from './src/generators/configuration/configuration';

package/dist/index.js

@@ -1,8 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.configurationGenerator = exports.initGenerator = exports.playwrightExecutor = void 0;
-var playwright_impl_1 = require("./src/executors/playwright/playwright.impl");
-Object.defineProperty(exports, "playwrightExecutor", { enumerable: true, get: function () { return playwright_impl_1.playwrightExecutor; } });
+exports.configurationGenerator = exports.initGenerator = void 0;
 var init_1 = require("./src/generators/init/init");
 Object.defineProperty(exports, "initGenerator", { enumerable: true, get: function () { return init_1.initGenerator; } });
 var configuration_1 = require("./src/generators/configuration/configuration");

package/dist/src/executors/playwright/{playwright.impl.d.ts → schema.d.ts}

@@ -1,4 +1,3 @@
-import { ExecutorContext } from '@nx/devkit';
 export interface PlaywrightExecutorSchema {
     browser?: 'all' | 'chromium' | 'firefox' | 'webkit' | string;
     config?: string;
@@ -33,7 +32,3 @@ export interface PlaywrightExecutorSchema {
     skipInstall?: boolean;
     cacheDir?: string;
 }
-export declare function playwrightExecutor(options: PlaywrightExecutorSchema, context: ExecutorContext): Promise<{
-    success: boolean;
-}>;
-export default playwrightExecutor;

package/dist/src/executors/playwright/schema.js

@@ -0,0 +1,2 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });

package/dist/src/executors/wait-for-webserver/wait-for-webserver.impl.js

@@ -21,7 +21,7 @@ const FALLBACK_RETRY_DELAY = 1_000;
 // nothing, so it must not replace what the previous probe saw.
 const NO_OBSERVATION = 'the deadline passed before the server responded';
 async function waitForWebserverExecutor(options, context) {
-    const servers = options.servers ?? [];
+    const servers = withReadyUrl(options.servers ?? []);
     if (servers.length === 0) {
         devkit_1.logger.error('@nx/playwright:wait-for-webserver requires at least one server to wait for.');
         return { success: false };
@@ -49,6 +49,15 @@ async function waitForWebserverExecutor(options, context) {
     }
 }
 exports.default = waitForWebserverExecutor;
+// The address the server task reported when it signaled readiness. It names no
+// server, so it only moves the url of a gate waiting for exactly one.
+function withReadyUrl(servers) {
+    const readyUrl = process.env.NX_READY_URL;
+    if (!readyUrl || servers.length !== 1 || servers[0].port != null) {
+        return servers;
+    }
+    return [{ ...servers[0], url: (0, devkit_1.getReadyUrl)(servers[0].url) }];
+}
 // A malformed URL, an out-of-range port and a proxy that cannot be used are all
 // inputs that can never become valid, so they are rejected up front instead of
 // being retried until the timeout and reported as a slow server. An unusable

package/dist/src/generators/configuration/configuration.js

@@ -7,7 +7,6 @@ const internal_1 = require("@nx/devkit/internal");
 const devkit_1 = require("@nx/devkit");
 const js_1 = require("@nx/js");
 const internal_2 = require("@nx/js/internal");
-const deprecation_1 = require("../../utils/deprecation");
 const child_process_1 = require("child_process");
 const path = tslib_1.__importStar(require("path"));
 const add_linter_1 = require("../../utils/add-linter");
@@ -16,7 +15,7 @@ const versions_1 = require("../../utils/versions");
 const init_1 = require("../init/init");
 const internal_3 = require("@nx/eslint/internal");
 function configurationGenerator(tree, options) {
-    return configurationGeneratorInternal(tree, { addPlugin: false, ...options });
+    return configurationGeneratorInternal(tree, options);
 }
 async function configurationGeneratorInternal(tree, rawOptions) {
     (0, assert_supported_playwright_version_1.assertSupportedPlaywrightVersion)(tree);
@@ -25,7 +24,6 @@ async function configurationGeneratorInternal(tree, rawOptions) {
     tasks.push(await (0, init_1.initGenerator)(tree, {
         skipFormat: true,
         skipPackageJson: options.skipPackageJson,
-        addPlugin: options.addPlugin,
     }));
     const projectConfig = (0, devkit_1.readProjectConfiguration)(tree, options.project);
     const offsetFromProjectRoot = (0, devkit_1.offsetFromRoot)(projectConfig.root);
@@ -135,14 +133,6 @@ async function configurationGeneratorInternal(tree, rawOptions) {
         }
         ignoreTestOutput(tree, options);
     }
-    const hasPlugin = (0, devkit_1.readNxJson)(tree).plugins?.some((p) => typeof p === 'string'
-        ? p === '@nx/playwright/plugin'
-        : p.plugin === '@nx/playwright/plugin');
-    if (!hasPlugin) {
-        (0, deprecation_1.warnPlaywrightExecutorGenerating)();
-        addE2eTarget(tree, options);
-        setupE2ETargetDefaults(tree);
-    }
     tasks.push(await (0, add_linter_1.addLinterToPlaywrightProject)(tree, {
         project: options.project,
         linter: options.linter,
@@ -151,7 +141,6 @@ async function configurationGeneratorInternal(tree, rawOptions) {
         directory: options.directory,
         enableTypedLinting: (0, internal_3.isTypedLintingEnabled)(options),
         rootProject: options.rootProject ?? projectConfig.root === '.',
-        addPlugin: options.addPlugin,
     }));
     if (options.js) {
         const { ModuleKind } = (0, internal_2.ensureTypescript)();
@@ -179,10 +168,6 @@ async function configurationGeneratorInternal(tree, rawOptions) {
     return (0, devkit_1.runTasksInSerial)(...tasks);
 }
 async function normalizeOptions(tree, options) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    const addPlugin = options.addPlugin ??
-        (process.env.NX_ADD_PLUGINS !== 'false' &&
-            nxJson.useInferencePlugins !== false);
     const linter = await (0, internal_2.normalizeLinterOption)(tree, options.linter);
     if (!options.webServerCommand || !options.webServerAddress) {
         const { webServerCommand, webServerAddress } = await promptForMissingServeData(options.project);
@@ -191,7 +176,6 @@ async function normalizeOptions(tree, options) {
     }
     return {
         ...options,
-        addPlugin,
         linter,
         directory: options.directory ?? 'e2e',
     };
@@ -248,55 +232,6 @@ function recommendVsCodeExtensions(tree) {
         });
     }
 }
-function setupE2ETargetDefaults(tree) {
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    if (!nxJson.namedInputs) {
-        return;
-    }
-    // E2e targets depend on all their project's sources + production sources of dependencies
-    const productionFileSet = !!nxJson.namedInputs?.production;
-    // Either a `target: 'e2e'` default or a default keyed on the executor
-    // we're about to scaffold will apply to the new target — consider both
-    // before deciding to add cache/inputs. Target-keyed wins when both are
-    // present.
-    const existingForTarget = (0, internal_1.findTargetDefault)(nxJson.targetDefaults, {
-        target: 'e2e',
-    });
-    const existingForExecutor = (0, internal_1.findTargetDefault)(nxJson.targetDefaults, {
-        executor: '@nx/playwright:playwright',
-    });
-    const existingCache = existingForTarget?.cache ?? existingForExecutor?.cache;
-    const existingInputs = existingForTarget?.inputs ?? existingForExecutor?.inputs;
-    const patch = {};
-    if (existingCache === undefined) {
-        patch.cache = true;
-    }
-    if (existingInputs === undefined) {
-        patch.inputs = ['default', productionFileSet ? '^production' : '^default'];
-    }
-    if (Object.keys(patch).length > 0) {
-        (0, internal_1.upsertTargetDefault)(tree, nxJson, { target: 'e2e', ...patch });
-        (0, devkit_1.updateNxJson)(tree, nxJson);
-    }
-}
-function addE2eTarget(tree, options) {
-    const projectConfig = (0, devkit_1.readProjectConfiguration)(tree, options.project);
-    if (projectConfig?.targets?.e2e) {
-        throw new Error(`Project ${options.project} already has an e2e target.
-Rename or remove the existing e2e target.`);
-    }
-    projectConfig.targets ??= {};
-    projectConfig.targets.e2e = {
-        executor: '@nx/playwright:playwright',
-        outputs: [`{workspaceRoot}/dist/.playwright/${projectConfig.root}`],
-        options: {
-            // Generator emits `playwright.config.mts` (`.mjs` for `--js`) so the
-            // legacy executor's `--config` flag must point at the same extension.
-            config: `${projectConfig.root}/playwright.config.${options.js ? 'mjs' : 'mts'}`,
-        },
-    };
-    (0, devkit_1.updateProjectConfiguration)(tree, options.project, projectConfig);
-}
 function ignoreTestOutput(tree, options) {
     // Make sure playwright outputs are not linted.
     if (options.linter === 'eslint') {

package/dist/src/generators/configuration/files/playwright.config.mts.template

@@ -1,9 +1,10 @@
 import { defineConfig, devices } from '@playwright/test';
 import { nxE2EPreset } from '@nx/playwright/preset';
-<% if (webServerCommand && webServerAddress) { %>import { workspaceRoot } from '@nx/devkit';<% } %>
+<% if (webServerCommand && webServerAddress) { %>import { getReadyUrl, workspaceRoot } from '@nx/devkit';<% } %>
 
 // For CI, you may want to set BASE_URL to the deployed application.
-const baseURL = process.env['BASE_URL'] || '<% if(webServerAddress) {%><%= webServerAddress %><% } else {%>http://localhost:3000<% } %>';
+<% if (webServerCommand && webServerAddress) { %>// getReadyUrl moves the address to the port the dev server task reported.
+const baseURL = process.env['BASE_URL'] || getReadyUrl('<%= webServerAddress %>');<% } else { %>const baseURL = process.env['BASE_URL'] || '<% if(webServerAddress) {%><%= webServerAddress %><% } else {%>http://localhost:3000<% } %>';<% } %>
 
 /**
  * Read environment variables from file.
@@ -32,7 +33,7 @@ export default defineConfig({
   /* Run your local dev server before starting the tests */<% if(webServerCommand && webServerAddress) {%>
   webServer: {
     command: '<%= webServerCommand %>',
-    url: '<%= webServerAddress %>',
+    url: getReadyUrl('<%= webServerAddress %>'),
     reuseExistingServer: true,
     cwd: workspaceRoot
   },<% } else {%>

package/dist/src/generators/configuration/schema.d.ts

@@ -27,11 +27,9 @@ export interface ConfigurationGeneratorSchema {
    **/
   webServerAddress?: string;
   rootProject?: boolean;
-  addPlugin?: boolean;
 }
 
 export interface NormalizedGeneratorOptions extends ConfigurationGeneratorSchema {
-  addPlugin: boolean;
   directory: string;
   linter: LinterType;
 }

package/dist/src/generators/init/init.js

@@ -8,24 +8,18 @@ const plugin_1 = require("../../plugins/plugin");
 const assert_supported_playwright_version_1 = require("../../utils/assert-supported-playwright-version");
 const versions_1 = require("../../utils/versions");
 function initGenerator(tree, options) {
-    return initGeneratorInternal(tree, { addPlugin: false, ...options });
+    return initGeneratorInternal(tree, options);
 }
 async function initGeneratorInternal(tree, options) {
     (0, assert_supported_playwright_version_1.assertSupportedPlaywrightVersion)(tree);
     const tasks = [];
-    const nxJson = (0, devkit_1.readNxJson)(tree);
-    const addPluginDefault = process.env.NX_ADD_PLUGINS !== 'false' &&
-        nxJson.useInferencePlugins !== false;
-    options.addPlugin ??= addPluginDefault;
     if (!options.skipPackageJson) {
         tasks.push((0, devkit_1.addDependenciesToPackageJson)(tree, {}, {
             '@nx/playwright': versions_1.nxVersion,
             '@playwright/test': versions_1.playwrightVersion,
         }, undefined, options.keepExistingVersions ?? true));
     }
-    if (options.addPlugin) {
-        await (0, internal_1.addPlugin)(tree, await (0, devkit_1.createProjectGraphAsync)(), '@nx/playwright/plugin', plugin_1.createNodesV2, { targetName: ['e2e', 'playwright:e2e', 'playwright-e2e'] }, options.updatePackageScripts);
-    }
+    await (0, internal_1.addPlugin)(tree, await (0, devkit_1.createProjectGraphAsync)(), '@nx/playwright/plugin', plugin_1.createNodesV2, { targetName: ['e2e', 'playwright:e2e', 'playwright-e2e'] }, options.updatePackageScripts);
     if (!options.skipFormat) {
         await (0, devkit_1.formatFiles)(tree);
     }

package/dist/src/generators/init/schema.d.ts

@@ -3,5 +3,4 @@ export interface InitGeneratorSchema {
   skipPackageJson?: boolean;
   keepExistingVersions?: boolean;
   updatePackageScripts?: boolean;
-  addPlugin?: boolean;
 }

package/dist/src/migrations/update-24-0-0/disable-wait-for-server-ready.d.ts

@@ -0,0 +1,4 @@
+import { type Tree } from '@nx/devkit';
+export default function update(tree: Tree): Promise<{
+    skipAgentic: boolean;
+}>;

package/dist/src/migrations/update-24-0-0/disable-wait-for-server-ready.js

@@ -0,0 +1,35 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = update;
+const devkit_1 = require("@nx/devkit");
+const pluginName = '@nx/playwright/plugin';
+async function update(tree) {
+    const nxJson = (0, devkit_1.readNxJson)(tree);
+    if (!nxJson?.plugins) {
+        return { skipAgentic: true };
+    }
+    let changed = false;
+    nxJson.plugins = nxJson.plugins.map((plugin) => {
+        if (typeof plugin === 'string') {
+            if (plugin !== pluginName) {
+                return plugin;
+            }
+            changed = true;
+            return { plugin, options: { waitForServerReady: false } };
+        }
+        if (plugin.plugin !== pluginName) {
+            return plugin;
+        }
+        const options = (plugin.options ?? {});
+        if (options.waitForServerReady !== undefined) {
+            return plugin;
+        }
+        changed = true;
+        return { ...plugin, options: { ...options, waitForServerReady: false } };
+    });
+    if (!changed) {
+        return { skipAgentic: true };
+    }
+    (0, devkit_1.updateNxJson)(tree, nxJson);
+    await (0, devkit_1.formatFiles)(tree);
+}

package/dist/src/migrations/update-24-0-0/disable-wait-for-server-ready.md

@@ -0,0 +1,40 @@
+#### Keep Playwright e2e tasks from waiting for the dev server to be ready
+
+`@nx/playwright/plugin` has a new `waitForServerReady` option, on by default. With it on, the inferred e2e tasks wait for the dev server task's `readyWhen` and get the address it reported in `NX_READY_URL`. Before, they only waited for the dev server task to start.
+
+This migration sets `waitForServerReady: false` on every `@nx/playwright/plugin` entry in `nx.json`, so existing workspaces keep the old behavior. Entries that already set the option stay as they are. To opt in, remove the option or set it to `true`.
+
+#### Sample code changes
+
+##### Before
+
+```json title="nx.json"
+{
+  "plugins": [
+    "@nx/playwright/plugin",
+    {
+      "plugin": "@nx/playwright/plugin",
+      "include": ["apps/admin-e2e/**"],
+      "options": { "targetName": "e2e" }
+    }
+  ]
+}
+```
+
+##### After
+
+```json title="nx.json"
+{
+  "plugins": [
+    {
+      "plugin": "@nx/playwright/plugin",
+      "options": { "waitForServerReady": false }
+    },
+    {
+      "plugin": "@nx/playwright/plugin",
+      "include": ["apps/admin-e2e/**"],
+      "options": { "targetName": "e2e", "waitForServerReady": false }
+    }
+  ]
+}
+```

package/dist/src/migrations/update-24-0-0/finish-inferred-conversion.md

@@ -0,0 +1,12 @@
+# Replace removed @nx/playwright executors with inferred tasks
+
+Nx 24 removed `@nx/playwright:playwright` (including old `@nrwl` aliases).
+
+Before changing anything, check for leftover work: a project target, a package.json target or a `targetDefaults` entry still uses a removed executor, or the advisory context reports a conversion error. If none applies, make no changes and stop. Treat each advisory-context item as pending work.
+
+1. Inspect all affected projects and workspace defaults. Repair config-loading errors first. Do not restore executor registrations or install older Nx packages.
+2. Preserve Playwright config, browser projects, CI sharding and webServer orchestration.
+3. Register `@nx/playwright/plugin` with options/includes that preserve custom target names and mixed inferred/explicit projects. Replace remaining removed-executor targets, including those in package.json and filtered targetDefaults. Transfer their behavior before removing obsolete defaults. Preserve configurations, dependencies, cache inputs/outputs and environment behavior. Do not silently delete unsupported options or unrelated targets. Migrate old @nrwl aliases too.
+4. Run project graph creation with the daemon and graph cache disabled (`NX_DAEMON=false NX_CACHE_PROJECT_GRAPH=false`), using the workspace package manager. Inspect resolved project targets and verify each migrated entrypoint exists and is inferred (a thin alias to an inferred target is acceptable). Resolve graph errors before completing this migration, even if task validation was skipped. Configs must load for subsequent migrations.
+5. By default run relevant real builds/tests with task cache disabled, including alternate configurations. Honor an existing preference to skip expensive task validation; otherwise offer that option when validation is slow. Skipping expensive tasks never skips conversion or the graph check. Report each skipped task as unverified. Never weaken assertions or change application behavior to make checks pass.
+6. Report converted projects, graph validation, task results and any skipped checks. Unconverted targets or a broken graph mean the migration is incomplete. Investigate task failures and distinguish missing environment prerequisites from conversion regressions; do not claim a working workspace with unresolved regressions.

package/dist/src/migrations/update-24-0-0/remove-deprecated-executors.d.ts

@@ -0,0 +1,6 @@
+import type { Tree } from '@nx/devkit';
+export default function update(tree: Tree): Promise<{
+    nextSteps?: string[];
+    agentContext?: string[];
+    skipAgentic?: boolean;
+}>;

package/dist/src/migrations/update-24-0-0/remove-deprecated-executors.js

@@ -0,0 +1,10 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = update;
+const internal_1 = require("@nx/devkit/internal");
+function update(tree) {
+    return (0, internal_1.migrateRemovedExecutors)(tree, ['@nx/playwright:playwright', '@nrwl/playwright:playwright'], async (tree, options) => {
+        const { convertToInferred } = require('../../generators/convert-to-inferred/convert-to-inferred');
+        return convertToInferred(tree, options);
+    });
+}

package/dist/src/migrations/update-24-0-0/remove-deprecated-executors.md

@@ -0,0 +1,40 @@
+#### Convert `@nx/playwright:playwright` targets to inferred tasks
+
+Nx 24 removes the `@nx/playwright:playwright` executor and its `@nrwl/playwright:playwright` alias. The migration registers `@nx/playwright/plugin` in `nx.json` and converts the targets that use those executors to inferred tasks. Options the plugin does not infer stay on the target. A target with nothing left to override is removed from project configuration. When several projects share the same remaining options, those options move to a `targetDefaults` entry scoped to the plugin. `targetDefaults` entries keyed by a removed executor are removed once no target uses that executor.
+
+The migration lists the targets the converter cannot convert safely. In agentic runs, an AI prompt finishes them and validates the project graph.
+
+#### Sample code changes
+
+##### Before
+
+```json title="myapp-e2e/project.json"
+{
+  "name": "myapp-e2e",
+  "targets": {
+    "e2e": {
+      "executor": "@nx/playwright:playwright",
+      "outputs": ["{workspaceRoot}/dist/.playwright/myapp-e2e"],
+      "options": {
+        "config": "myapp-e2e/playwright.config.ts"
+      }
+    }
+  }
+}
+```
+
+##### After
+
+```json title="nx.json"
+{
+  "plugins": [
+    {
+      "plugin": "@nx/playwright/plugin",
+      "options": {
+        "ciTargetName": "e2e-ci",
+        "targetName": "e2e"
+      }
+    }
+  ]
+}
+```

package/dist/src/plugins/plugin.d.ts

@@ -16,6 +16,12 @@ export interface PlaywrightPluginOptions {
      * true.
      */
     waitForWebServer?: boolean;
+    /**
+     * Whether the inferred dependencies on the web server tasks wait for those
+     * tasks' `readyWhen` (`waitFor: 'ready'`) rather than for them to start.
+     * Defaults to true.
+     */
+    waitForServerReady?: boolean;
 }
 export declare const createNodes: CreateNodes<PlaywrightPluginOptions>;
 export declare const createNodesV2: CreateNodes<PlaywrightPluginOptions>;

package/dist/src/plugins/plugin.js

@@ -205,7 +205,7 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
             },
         },
     };
-    applyChainDependsOn(baseTargetConfig, e2eChain, e2eReadyTargetName);
+    applyChainDependsOn(baseTargetConfig, e2eChain, e2eReadyTargetName, options.waitForServerReady);
     targets[options.targetName] = {
         ...baseTargetConfig,
         cache: true,
@@ -219,10 +219,10 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
         outputs: getTargetOutputs(testOutput, reporterOutputs, context.workspaceRoot, projectRoot),
     };
     if (e2eReadyTargetName) {
-        targets[e2eReadyTargetName] = buildWaitForWebserverTarget(e2eChain, options.webServerTimeout);
+        targets[e2eReadyTargetName] = buildWaitForWebserverTarget(e2eChain, options.webServerTimeout, options.waitForServerReady);
     }
     if (ciReadyTargetName && ciReadyTargetName !== e2eReadyTargetName) {
-        targets[ciReadyTargetName] = buildWaitForWebserverTarget(ciChain, options.webServerTimeout);
+        targets[ciReadyTargetName] = buildWaitForWebserverTarget(ciChain, options.webServerTimeout, options.waitForServerReady);
     }
     if (options.ciTargetName) {
         // ensure the blob reporter output is the directory containing the blob
@@ -245,7 +245,7 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
         // Unconditionally: the chains can differ in ways the shared-gate check
         // deliberately ignores (an extra uncovered server flips parallelism), and
         // re-deriving is a no-op when they match.
-        applyChainDependsOn(ciBaseTargetConfig, ciChain, ciReadyTargetName);
+        applyChainDependsOn(ciBaseTargetConfig, ciChain, ciReadyTargetName, options.waitForServerReady);
         const groupName = 'E2E (CI)';
         metadata = { targetGroups: { [groupName]: [] } };
         const ciTargetGroup = metadata.targetGroups[groupName];
@@ -279,7 +279,7 @@ async function buildPlaywrightTargets(configFilePath, projectRoot, options, cont
             const targetName = `${options.ciTargetName}--${relativeSpecFilePath}`;
             ciTargetGroup.push(targetName);
             targets[targetName] = {
-                ...(0, internal_1.applyTestFileDirectives)(ciBaseTargetConfig, context.workspaceRoot, testFile, ['depends-on', 'ultracache']),
+                ...(0, internal_1.applyTestFileDirectives)(ciBaseTargetConfig, context.workspaceRoot, testFile, { ignoreServerInputs: true }),
                 options: {
                     ...ciBaseTargetConfig.options,
                     env: getAtomizedTaskEnvVars(reporterOutputs, outputSubfolder),
@@ -401,6 +401,7 @@ function normalizeOptions(options) {
         ciTargetName,
         mergeReportsTargetName: `${ciTargetName}--merge-reports`,
         waitForWebServer: options?.waitForWebServer ?? true,
+        waitForServerReady: options?.waitForServerReady !== false,
     };
 }
 function getTestOutput(playwrightConfig) {
@@ -546,7 +547,7 @@ function sameChain(a, b) {
     return (JSON.stringify(a.commandTasks) === JSON.stringify(b.commandTasks) &&
         JSON.stringify(a.readinessServers) === JSON.stringify(b.readinessServers));
 }
-function buildWaitForWebserverTarget(chain, webServerTimeout) {
+function buildWaitForWebserverTarget(chain, webServerTimeout, waitForServerReady) {
     return {
         executor: '@nx/playwright:wait-for-webserver',
         cache: false,
@@ -554,7 +555,7 @@ function buildWaitForWebserverTarget(chain, webServerTimeout) {
             servers: chain.readinessServers,
             ...(webServerTimeout != null ? { timeout: webServerTimeout } : {}),
         },
-        dependsOn: getDependsOn(chain.commandTasks),
+        dependsOn: getDependsOn(chain.commandTasks, waitForServerReady),
         metadata: {
             technologies: ['playwright'],
             description: 'Waits for the E2E web server(s) to be ready before the Playwright test tasks run.',
@@ -693,11 +694,12 @@ function chainRequiresSerialization(chain) {
 // was inferred). Set together with parallelism so a config that inherits one
 // state (the atomized CI base copies the e2e config) cannot end up carrying
 // both.
-function applyChainDependsOn(targetConfig, chain, readyTargetName) {
+function applyChainDependsOn(targetConfig, chain, readyTargetName, waitForServerReady) {
     if (chain.commandTasks.length) {
+        const serverDependsOn = getDependsOn(chain.commandTasks, waitForServerReady);
         targetConfig.dependsOn = readyTargetName
-            ? [...getDependsOn(chain.commandTasks), { target: readyTargetName }]
-            : getDependsOn(chain.commandTasks);
+            ? [...serverDependsOn, { target: readyTargetName }]
+            : serverDependsOn;
     }
     else {
         delete targetConfig.dependsOn;
@@ -709,7 +711,7 @@ function applyChainDependsOn(targetConfig, chain, readyTargetName) {
         delete targetConfig.parallelism;
     }
 }
-function getDependsOn(tasks) {
+function getDependsOn(tasks, waitForServerReady) {
     const projectsPerTask = new Map();
     for (const { project, target } of tasks) {
         if (!projectsPerTask.has(target)) {
@@ -720,6 +722,7 @@ function getDependsOn(tasks) {
     return Array.from(projectsPerTask.entries()).map(([target, projects]) => ({
         projects,
         target,
+        ...(waitForServerReady ? { waitFor: 'ready' } : {}),
     }));
 }
 function normalizeOutput(path, workspaceRoot, projectRoot) {

package/dist/src/utils/add-linter.d.ts

@@ -15,6 +15,5 @@ export interface PlaywrightLinterOptions {
      * Directory from the project root, where the playwright files will be located.
      **/
     directory: string;
-    addPlugin?: boolean;
 }
 export declare function addLinterToPlaywrightProject(tree: Tree, options: PlaywrightLinterOptions): Promise<GeneratorCallback>;

package/dist/src/utils/add-linter.js

@@ -23,7 +23,6 @@ async function addLinterToPlaywrightProject(tree, options) {
             enableTypedLinting,
             skipPackageJson: options.skipPackageJson,
             rootProject: options.rootProject,
-            addPlugin: options.addPlugin,
         }));
     }
     // Everything below configures ESLint — predefined configs, `extends`, ignore

package/executors.json

@@ -1,10 +1,5 @@
 {
   "executors": {
-    "playwright": {
-      "implementation": "./dist/src/executors/playwright/playwright.impl",
-      "schema": "./dist/src/executors/playwright/schema.json",
-      "description": "Run Playwright tests."
-    },
     "merge-reports": {
       "implementation": "./dist/src/executors/merge-reports/merge-reports.impl",
       "schema": "./dist/src/executors/merge-reports/schema.json",

package/migrations.json

@@ -5,6 +5,19 @@
       "description": "Rename imports of `createNodesV2` from `@nx/playwright/plugin` to the canonical `createNodes` export.",
       "implementation": "./dist/src/migrations/update-23-0-0/migrate-create-nodes-v2-to-create-nodes",
       "documentation": "./dist/src/migrations/update-23-0-0/migrate-create-nodes-v2-to-create-nodes.md"
+    },
+    "remove-deprecated-executors": {
+      "version": "24.0.0-beta.3",
+      "description": "Converts targets using the removed `@nx/playwright` executors to inferred tasks; an AI prompt finishes the targets the converter cannot convert safely and validates the project graph.",
+      "implementation": "./dist/src/migrations/update-24-0-0/remove-deprecated-executors",
+      "prompt": "./dist/src/migrations/update-24-0-0/finish-inferred-conversion.md",
+      "documentation": "./dist/src/migrations/update-24-0-0/remove-deprecated-executors.md"
+    },
+    "update-24-0-0-disable-wait-for-server-ready": {
+      "version": "24.0.0-beta.4",
+      "description": "Sets `waitForServerReady: false` on `@nx/playwright/plugin` entries in `nx.json` so existing Playwright e2e tasks keep starting without waiting for the dev server to report it is ready.",
+      "implementation": "./dist/src/migrations/update-24-0-0/disable-wait-for-server-ready",
+      "documentation": "./dist/src/migrations/update-24-0-0/disable-wait-for-server-ready.md"
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/playwright",
-  "version": "24.0.0-beta.2",
+  "version": "24.0.0-beta.4",
   "private": false,
   "type": "commonjs",
   "files": [
@@ -75,16 +75,16 @@
     "supportsOptionalMigrations": true
   },
   "dependencies": {
-    "@nx/devkit": "24.0.0-beta.2",
-    "@nx/eslint": "24.0.0-beta.2",
-    "@nx/js": "24.0.0-beta.2",
+    "@nx/devkit": "24.0.0-beta.4",
+    "@nx/eslint": "24.0.0-beta.4",
+    "@nx/js": "24.0.0-beta.4",
     "https-proxy-agent": "^7.0.6",
     "minimatch": "10.2.5",
     "semver": "^7.6.3",
     "tslib": "^2.3.0"
   },
   "devDependencies": {
-    "nx": "24.0.0-beta.2"
+    "nx": "24.0.0-beta.4"
   },
   "peerDependencies": {
     "@playwright/test": "^1.36.0"

package/dist/src/executors/playwright/playwright.impl.js

@@ -1,83 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.playwrightExecutor = playwrightExecutor;
-const devkit_1 = require("@nx/devkit");
-const child_process_1 = require("child_process");
-const deprecation_1 = require("../../utils/deprecation");
-async function playwrightExecutor(options, context) {
-    (0, deprecation_1.warnPlaywrightExecutorDeprecation)();
-    const projectRoot = context.projectGraph?.nodes?.[context?.projectName]?.data?.root;
-    if (!projectRoot) {
-        throw new Error(`Unable to find the Project Root for ${context.projectName}. Is it set in the project.json?`);
-    }
-    if (!options.skipInstall) {
-        devkit_1.output.log({
-            title: 'Ensuring Playwright is installed.',
-            bodyLines: ['use --skipInstall to skip installation.'],
-        });
-        const pmc = (0, devkit_1.getPackageManagerCommand)();
-        (0, child_process_1.execSync)(`${pmc.exec} playwright install`, {
-            cwd: devkit_1.workspaceRoot,
-            stdio: 'inherit',
-            windowsHide: true,
-        });
-    }
-    const args = createArgs(options);
-    const env = options.cacheDir
-        ? { ...process.env, PWTEST_CACHE_DIR: options.cacheDir }
-        : undefined;
-    const p = runPlaywright(args, context.root, env);
-    p.stdout.on('data', (message) => {
-        process.stdout.write(message);
-    });
-    p.stderr.on('data', (message) => {
-        process.stderr.write(message);
-    });
-    return new Promise((resolve) => {
-        p.on('close', (code) => {
-            resolve({ success: code === 0 });
-        });
-    });
-}
-function createArgs(opts, exclude = ['skipInstall', 'cacheDir']) {
-    const args = [];
-    const { testFiles, ...rest } = opts;
-    if (testFiles) {
-        args.push(...testFiles);
-    }
-    for (const key in rest) {
-        if (exclude.includes(key))
-            continue;
-        const value = opts[key];
-        // NOTE: playwright doesn't accept pascalCase args, only kebab-case
-        const arg = (0, devkit_1.names)(key).fileName;
-        if (Array.isArray(value)) {
-            args.push(...value.map((v) => `--${arg}=${v.trim()}`));
-        }
-        else if (typeof value === 'boolean') {
-            // NOTE: playwright don't accept --arg=false, instead just don't pass the arg.
-            if (value) {
-                args.push(`--${arg}`);
-            }
-        }
-        else {
-            args.push(`--${arg}=${value}`);
-        }
-    }
-    return args;
-}
-function runPlaywright(args, cwd, env) {
-    try {
-        const cli = require.resolve('@playwright/test/cli');
-        return (0, child_process_1.fork)(cli, ['test', ...args], {
-            stdio: ['pipe', 'pipe', 'pipe', 'ipc'],
-            cwd,
-            ...(env ? { env } : {}),
-        });
-    }
-    catch (e) {
-        console.error(e);
-        throw new Error('Unable to run playwright. Is @playwright/test installed?');
-    }
-}
-exports.default = playwrightExecutor;

package/dist/src/executors/playwright/schema.json

@@ -1,166 +0,0 @@
-{
-  "$schema": "https://json-schema.org/schema",
-  "version": 2,
-  "title": "Playwright executor",
-  "description": "Run Playwright tests.",
-  "x-deprecated": "The `@nx/playwright:playwright` executor is deprecated and will be removed in Nx v24. Run `nx g @nx/playwright:convert-to-inferred` to migrate to the `@nx/playwright/plugin` inferred targets. See https://nx.dev/docs/guides/tasks--caching/convert-to-inferred for details.",
-  "type": "object",
-  "properties": {
-    "browser": {
-      "type": "string",
-      "description": "Browser to use for tests, one of 'all', 'chromium', 'firefox' or 'webkit'. If a playwright config is provided/discovered then the browserName value is expected from the configured 'projects'",
-      "x-priority": "important"
-    },
-    "config": {
-      "type": "string",
-      "description": "Configuration file, or a test directory with optional",
-      "x-completion-type": "file",
-      "x-completion-glob": "playwright?(*)@(.js|.cjs|.mjs|.ts|.cts|.mtx)",
-      "x-priority": "important"
-    },
-    "debug": {
-      "type": "boolean",
-      "description": "Run tests with Playwright Inspector. Shortcut for 'PWDEBUG=1' environment variable and '--timeout=0',--max-failures=1 --headed --workers=1' options"
-    },
-    "lastFailed": {
-      "type": "boolean",
-      "description": "Run only the tests that failed in the last run"
-    },
-    "forbidOnly": {
-      "type": "boolean",
-      "description": "Fail if test.only is called"
-    },
-    "fullyParallel": {
-      "type": "boolean",
-      "description": "Run all tests in parallel"
-    },
-    "grep": {
-      "alias": "g",
-      "type": "string",
-      "description": "Only run tests matching this regular expression"
-    },
-    "globalTimeout": {
-      "type": "number",
-      "description": "Maximum time this test suite can run in milliseconds"
-    },
-    "grepInvert": {
-      "alias": "gv",
-      "type": "string",
-      "description": "Only run tests that do not match this regular expression"
-    },
-    "testFiles": {
-      "alias": "t",
-      "type": "array",
-      "description": "Test files to run",
-      "items": {
-        "type": "string"
-      }
-    },
-    "headed": {
-      "type": "boolean",
-      "description": "Run tests in headed browsers",
-      "x-priority": "important"
-    },
-    "ignoreSnapshots": {
-      "type": "boolean",
-      "description": "Ignore screenshot and snapshot expectations"
-    },
-    "workers": {
-      "alias": "j",
-      "oneOf": [{ "type": "number" }, { "type": "string" }],
-      "description": "Number of concurrent workers or percentage of logical CPU cores, use 1 to run in a single worker"
-    },
-    "list": {
-      "type": "boolean",
-      "description": "Collect all the tests and report them, but do not run"
-    },
-    "maxFailures": {
-      "alias": "x",
-      "oneOf": [{ "type": "number" }, { "type": "boolean" }],
-      "description": "Stop after the first N failures"
-    },
-    "noDeps": {
-      "type": "boolean",
-      "description": "Do not run project dependencies"
-    },
-    "output": {
-      "type": "string",
-      "description": "Folder for output artifacts"
-    },
-    "passWithNoTests": {
-      "type": "boolean",
-      "description": "Makes test run succeed even if no tests were found",
-      "default": true
-    },
-    "project": {
-      "description": "Only run tests from the specified list of projects",
-      "type": "array",
-      "items": {
-        "type": "string"
-      }
-    },
-    "quiet": {
-      "alias": "q",
-      "type": "boolean",
-      "description": "Suppress stdio"
-    },
-    "repeatEach": {
-      "type": "number",
-      "description": "Run each test N times"
-    },
-    "reporter": {
-      "type": "string",
-      "description": "Common Reporter values to use, comma-separated, 'list', 'line', 'dot', 'json', 'junit', 'null', 'github', 'html', 'blob'. To configure reporter options, use the playwright configuration."
-    },
-    "retries": {
-      "type": "number",
-      "description": "Maximum retry count for flaky tests, zero for no retries"
-    },
-    "shard": {
-      "type": "string",
-      "description": "Shard tests and execute only the selected shard, specify in the form 'current/all', 1-based, for example '3/5'"
-    },
-    "timeout": {
-      "type": "number",
-      "description": "Specify test timeout threshold in milliseconds, zero for unlimited"
-    },
-    "trace": {
-      "type": "string",
-      "enum": [
-        "on",
-        "off",
-        "on-first-retry",
-        "on-all-retries",
-        "retain-on-failure"
-      ],
-      "description": "Force tracing mode, can be 'on', 'off', 'on-first-retry', 'on-all-retries', 'retain-on-failure'"
-    },
-    "updateSnapshots": {
-      "alias": "u",
-      "type": "boolean",
-      "description": "Update snapshots with actual results. Snapshots will be created if missing."
-    },
-    "ui": {
-      "type": "boolean",
-      "description": "Run tests in interactive UI mode"
-    },
-    "uiHost": {
-      "type": "string",
-      "description": "Host to serve UI on; specifying this option opens UI in a browser tab"
-    },
-    "uiPort": {
-      "type": "number",
-      "description": "Port to serve UI on, 0 for any free port; specifying this option opens UI in a browser tab"
-    },
-    "skipInstall": {
-      "type": "boolean",
-      "description": "Skip running playwright install before running playwright tests. This is to ensure that playwright browsers are installed before running tests.",
-      "default": false
-    },
-    "cacheDir": {
-      "type": "string",
-      "description": "Directory for Playwright internal cache (browser binaries, etc.). Sets the PWTEST_CACHE_DIR environment variable."
-    }
-  },
-  "required": []
-}

package/dist/src/utils/deprecation.d.ts

@@ -1,3 +0,0 @@
-export declare const PLAYWRIGHT_EXECUTOR_DEPRECATION_MESSAGE = "The `@nx/playwright:playwright` executor is deprecated and will be removed in Nx v24. Run `nx g @nx/playwright:convert-to-inferred` to migrate to the `@nx/playwright/plugin` inferred targets. See https://nx.dev/docs/guides/tasks--caching/convert-to-inferred for details.";
-export declare function warnPlaywrightExecutorDeprecation(): void;
-export declare function warnPlaywrightExecutorGenerating(): void;

package/dist/src/utils/deprecation.js

@@ -1,16 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.PLAYWRIGHT_EXECUTOR_DEPRECATION_MESSAGE = void 0;
-exports.warnPlaywrightExecutorDeprecation = warnPlaywrightExecutorDeprecation;
-exports.warnPlaywrightExecutorGenerating = warnPlaywrightExecutorGenerating;
-const devkit_1 = require("@nx/devkit");
-// TODO(v24): Remove the @nx/playwright:playwright executor. The inferred
-// plugin (@nx/playwright/plugin) and the convert-to-inferred generator stay
-// supported.
-exports.PLAYWRIGHT_EXECUTOR_DEPRECATION_MESSAGE = 'The `@nx/playwright:playwright` executor is deprecated and will be removed in Nx v24. Run `nx g @nx/playwright:convert-to-inferred` to migrate to the `@nx/playwright/plugin` inferred targets. See https://nx.dev/docs/guides/tasks--caching/convert-to-inferred for details.';
-function warnPlaywrightExecutorDeprecation() {
-    devkit_1.logger.warn(exports.PLAYWRIGHT_EXECUTOR_DEPRECATION_MESSAGE);
-}
-function warnPlaywrightExecutorGenerating() {
-    devkit_1.logger.warn('Generating a target that uses the deprecated `@nx/playwright:playwright` executor. The executor will be removed in Nx v24. Run `nx g @nx/playwright:convert-to-inferred` next to migrate this target to the `@nx/playwright/plugin` inferred plugin and prevent future generators from emitting executor targets. See https://nx.dev/docs/guides/tasks--caching/convert-to-inferred for details.');
-}