playwright 1.56.0-beta-1759412259000 → 1.56.0-beta-1759451736000

package/lib/agents/generator.md

@@ -46,6 +46,7 @@ application behavior.
   - Test title must match the scenario name
   - Includes a comment with the step text before each step execution. Do not duplicate comments if step requires
     multiple actions.
+  - Always use best practices from the log when generating tests.
 
    <example-generation>
    For following plan:
@@ -79,18 +80,6 @@ application behavior.
    ```
    </example-generation>
 
-# Best practices
-- Each test has clear, descriptive assertions that validate the expected behavior
-- Includes proper error handling and meaningful failure messages
-- Uses Playwright best practices (page.waitForLoadState, expect.toBeVisible, etc.)
-- Do not improvise, do not add directives that were not asked for
-- Uses reliable locators (preferring data-testid, role-based, or text-based selectors over fragile CSS selectors)
-- Uses local variables for locators that are used multiple times
-- Uses explicit waits rather than arbitrary timeouts
-- Never waits for networkidle or use other discouraged or deprecated apis
-- Is self-contained and can run independently
-- Is deterministic and not prone to flaky behavior
-
 <example>
   Context: User wants to test a login flow on their web application.
   user: 'I need a test that logs into my app at localhost:3000 with username admin@test.com and password 123456, then

package/lib/mcp/test/testContext.js

@@ -65,6 +65,7 @@ class GeneratorJournal {
 \`\`\`ts
 ${step.code}
 \`\`\``).join("\n\n"));
+    result.push(bestPracticesMarkdown);
     return result.join("\n\n");
   }
 }
@@ -156,6 +157,18 @@ class TestContext {
   async close() {
   }
 }
+const bestPracticesMarkdown = `
+# Best practices
+- Do not improvise, do not add directives that were not asked for
+- Use clear, descriptive assertions to validate the expected behavior
+- Use reliable locators from this log
+- Use local variables for locators that are used multiple times
+- Use Playwright waiting assertions and best practices from this log
+- NEVER! use page.waitForLoadState()
+- NEVER! use page.waitForNavigation()
+- NEVER! use page.waitForTimeout()
+- NEVER! use page.evaluate()
+`;
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
   GeneratorJournal,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright",
-  "version": "1.56.0-beta-1759412259000",
+  "version": "1.56.0-beta-1759451736000",
   "description": "A high-level API to automate web browsers",
   "repository": {
     "type": "git",
@@ -64,7 +64,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "playwright-core": "1.56.0-beta-1759412259000"
+    "playwright-core": "1.56.0-beta-1759451736000"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"