playwright-test-agent 0.1.7 → 0.1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright-test-agent",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "Initialize Playwright Test agents with a Playwright CLI-first browser workflow.",
   "type": "module",
   "bin": {

package/playwright-test-agent/SKILL.md

@@ -7,19 +7,23 @@ description: Use first when a user asks to investigate or test a website, browse
 
 Use Planner -> Generator -> Healer for durable tests. `AGENTS.md` or `CLAUDE.md` defines the platform-specific role routing. Generated role definitions use `playwright-cli` as their primary browser interface, so MCP tool injection is not required.
 
-## Prepare the objective
+## Fast start
 
-The main agent may inspect the local project for relevant requirements, routes, fixtures, and constraints. Do not require a seed/config audit before planning.
+When the request contains a test objective and deployed URL, enter Planner immediately and run:
 
-Before Planner, ask for information that could materially change purpose, scope, safety, or expected behavior: deployed URL, intended role, success criteria, allowed data mutations, and environment constraints. Do not ask for UI details Planner can safely discover.
+```bash
+playwright-cli open <deployed-url>
+```
+
+Before opening the page, do not inspect the local project and do not locate or read a seed file, Playwright config, test directory, agent definition, fixtures, source tree, or existing tests. Do not ask optional questions about roles, browsers, edge cases, expected results, or test data when Planner can discover or defer them.
 
-If the request is sufficient, begin Planner immediately. If investigation finds a missing, incorrect, or contradictory URL, credential, role, prerequisite, expected result, or authorization boundary, pause and ask for the specific correction. Resume Planner with the correction and blocking observation. Do not guess credentials, probe nearby environments, bypass access controls, or wander through unrelated pages.
+Ask before Planner only when the objective or deployed URL itself is missing. During exploration, ask only when a real blocker is reached: required credentials, an invalid URL, ambiguous expected behavior that changes assertions, or authorization for a consequential action. Resume Planner after the answer. Do not guess credentials, probe nearby environments, bypass access controls, or wander through unrelated pages.
 
 Keep credentials and tokens in environment variables or ignored secret files. Never copy them into plans, tests, screenshots, evidence, or chat.
 
 ## Planner
 
-Planner investigates the live application with `playwright-cli` and saves a human-readable Markdown plan under `specs/`. Use `open`/`goto`, compact `snapshot` or `find`, and refs for interaction. Supply a seed only when explicitly identified; do not perform a separate seed discovery pass.
+Planner investigates the live application with `playwright-cli` and saves a human-readable Markdown plan under `specs/`. Start with the supplied URL, then use compact `snapshot` or `find` output and refs for interaction. A seed is not a Planner prerequisite and must not be located before exploration.
 
 The plan contains prerequisites, test data, independent scenarios, steps, observable expected results, exclusions, and intended output files. Reconnaissance must not mutate durable/shared data or perform consequential actions unless authorized.
 

package/playwright-test-agent/scripts/init-playwright.mjs

@@ -17,20 +17,20 @@ Use the installed \`playwright-cli\` command as the primary browser interface. R
 
 CLI replaces any conflicting requirement above to call Playwright Test MCP setup, browser, save, generation, or debug tools. Do not stop merely because \`planner_*\`, \`generator_*\`, \`test_*\`, or \`browser_*\` MCP tools are absent, and do not spawn a nested or same-role agent.
 
-${role === 'planner' ? 'Explore with CLI and save the completed Markdown test plan directly under `specs/` using filesystem tools.' : ''}${role === 'generator' ? 'Use CLI to validate the confirmed plan against the live UI, then write executable Playwright test files directly with filesystem tools.' : ''}${role === 'healer' ? 'Run failures with `npx playwright test`, use CLI to inspect and reproduce the current UI, patch the tests directly, and rerun within the healer guardrails.' : ''}
+${role === 'planner' ? 'FAST START: when the request contains a test objective and deployed URL, immediately run `playwright-cli open <url>` and investigate. Do not first locate or read a seed file, Playwright config, test directory, agent definition, source tree, fixtures, or existing tests. Do not ask optional planning questions before opening the page. Ask only when exploration reaches a real blocker such as required credentials, missing/incorrect URL, ambiguous expected behavior that changes assertions, or authorization for a consequential action. Save the completed Markdown test plan directly under `specs/` using filesystem tools.' : ''}${role === 'generator' ? 'Use CLI to validate the confirmed plan against the live UI, then write executable Playwright test files directly with filesystem tools.' : ''}${role === 'healer' ? 'Run failures with `npx playwright test`, use CLI to inspect and reproduce the current UI, patch the tests directly, and rerun within the healer guardrails.' : ''}
 
 ${ROLE_BLOCK_END}`;
 const CODEX_INSTRUCTIONS = `${BLOCK_START}
 ## Playwright Test Agent
 
-For website, browser workflow, HTTP API, or application-feature testing, load \`.agents/skills/playwright-test-agent/SKILL.md\` first. Codex must not start Playwright subagents; the main agent performs Planner -> Generator -> Healer in this session. Use \`playwright-cli\` as the primary browser interface and read generated TOML role definitions only for responsibilities. Ask for missing or contradictory purpose, access, role, data, or safety information before or during planning; do not guess or create nested agents.
+For website, browser workflow, HTTP API, or application-feature testing, load \`.agents/skills/playwright-test-agent/SKILL.md\` first. Codex must not start Playwright subagents; the main agent performs Planner -> Generator -> Healer in this session. Use \`playwright-cli\` as the primary browser interface. When the request contains a test objective and deployed URL, begin Planner immediately; do not first inspect the project or locate a seed, config, test directory, agent definition, fixtures, or existing tests. Ask only at a real access, expected-behavior, or safety blocker. Do not guess or create nested agents.
 
 ${BLOCK_END}`;
 
 const CLAUDE_INSTRUCTIONS = `${BLOCK_START}
 ## Playwright Test Agent
 
-For website, browser workflow, HTTP API, or application-feature testing, load \`.claude/skills/playwright-test-agent/SKILL.md\` first. Claude Code starts the official \`playwright-test-planner\`, \`playwright-test-generator\`, and \`playwright-test-healer\` directly. These roles use \`playwright-cli\` as their primary browser interface and must not create nested agents. Ask for missing or contradictory purpose, access, role, data, or safety information before or during planning; do not guess.
+For website, browser workflow, HTTP API, or application-feature testing, load \`.claude/skills/playwright-test-agent/SKILL.md\` first. Claude Code starts the official \`playwright-test-planner\`, \`playwright-test-generator\`, and \`playwright-test-healer\` directly. When the request contains a test objective and deployed URL, start Planner immediately; do not first inspect the project or locate a seed, config, test directory, agent definition, fixtures, or existing tests. These roles use \`playwright-cli\` as their primary browser interface and must not create nested agents. Ask only at a real access, expected-behavior, or safety blocker; do not guess.
 
 ${BLOCK_END}`;
 
@@ -126,13 +126,31 @@ function patchCodexRole(source, role) {
 
 function patchClaudeRole(source, role) {
   let updated = source;
-  if (!/^\s*-\s+Bash\(playwright-cli:\*\)\s*$/m.test(updated)) {
-    const tools = updated.match(/^tools:\s*\r?\n/m);
-    if (!tools) throw new Error('tools list was not found in a Claude role definition');
-    const insertion = '  - Bash(playwright-cli:*)\n  - Bash(npx:*)\n';
-    updated = updated.slice(0, tools.index + tools[0].length) + insertion +
-      updated.slice(tools.index + tools[0].length);
+  const frontmatter = updated.match(/^---\s*\r?\n([\s\S]*?)\r?\n---/);
+  if (!frontmatter) throw new Error('YAML frontmatter was not found in a Claude role definition');
+
+  let header = frontmatter[1];
+  const inlineTools = header.match(/^tools:[ \t]*(\S.*?)[ \t]*$/m);
+  const blockTools = header.match(/^tools:[ \t]*$/m);
+  if (inlineTools) {
+    const tools = inlineTools[1].split(',').map((tool) => tool.trim());
+    if (!tools.includes('Bash')) tools.push('Bash');
+    header = header.replace(inlineTools[0], `tools: ${tools.join(', ')}`);
+  } else if (blockTools) {
+    const blockStart = blockTools.index + blockTools[0].length;
+    const remainder = header.slice(blockStart);
+    const nextKey = remainder.search(/\r?\n(?=[A-Za-z0-9_-]+:\s*)/);
+    const blockEnd = nextKey === -1 ? header.length : blockStart + nextKey;
+    const toolsBlock = header.slice(blockStart, blockEnd);
+    if (!/^\s*-\s+Bash\s*$/m.test(toolsBlock)) {
+      header = header.slice(0, blockStart) + '\n  - Bash' +
+        header.slice(blockStart);
+    }
+  } else {
+    header = `${header.trimEnd()}\ntools: Bash`;
   }
+  updated = updated.slice(0, frontmatter.index) + `---\n${header}\n---` +
+    updated.slice(frontmatter.index + frontmatter[0].length);
   return withRoleBlock(updated, role);
 }