specweave 1.0.239 → 1.0.241

package/plugins/specweave-testing/lib/playwright-cli-runner.js

@@ -1,21 +1,24 @@
-import { execSync } from "child_process";
+import { execFileSync } from "child_process";
 class PlaywrightCliRunner {
   constructor(config = {}) {
     this.config = {
       headed: config.headed ?? false,
       browser: config.browser ?? "chrome",
-      session: config.session,
       timeout: config.timeout ?? 3e4,
-      ...config
+      session: config.session
     };
   }
-  exec(command) {
-    const sessionFlag = this.config.session ? `-s=${this.config.session} ` : "";
-    const fullCommand = `playwright-cli ${sessionFlag}${command}`;
+  exec(args) {
+    const fullArgs = [];
+    if (this.config.session) {
+      fullArgs.push(`-s=${this.config.session}`);
+    }
+    fullArgs.push(...args);
     try {
-      const output = execSync(fullCommand, {
+      const output = execFileSync("playwright-cli", fullArgs, {
         encoding: "utf-8",
-        timeout: this.config.timeout
+        timeout: this.config.timeout,
+        maxBuffer: 1024 * 1024
       }).trim();
       return { ok: true, output };
     } catch (e) {
@@ -23,34 +26,36 @@ class PlaywrightCliRunner {
     }
   }
   open(url) {
-    const headedFlag = this.config.headed ? " --headed" : "";
-    const urlPart = url ? ` ${url}` : "";
-    return this.exec(`open${urlPart}${headedFlag}`);
+    const args = ["open"];
+    if (url) args.push(url);
+    if (this.config.headed) args.push("--headed");
+    return this.exec(args);
   }
   navigate(url) {
-    return this.exec(`goto ${url}`);
+    return this.exec(["goto", url]);
   }
   snapshot() {
-    return this.exec("snapshot");
+    return this.exec(["snapshot"]);
   }
   screenshot(filename) {
-    const flag = filename ? ` --filename ${filename}` : "";
-    return this.exec(`screenshot${flag}`);
+    const args = ["screenshot"];
+    if (filename) args.push("--filename", filename);
+    return this.exec(args);
   }
   close() {
-    return this.exec("close");
+    return this.exec(["close"]);
   }
   click(ref) {
-    return this.exec(`click ${ref}`);
+    return this.exec(["click", ref]);
   }
   type(text) {
-    return this.exec(`type "${text}"`);
+    return this.exec(["type", text]);
   }
   fill(ref, text) {
-    return this.exec(`fill ${ref} "${text}"`);
+    return this.exec(["fill", ref, text]);
   }
   evaluate(fn) {
-    return this.exec(`eval "${fn}"`);
+    return this.exec(["eval", fn]);
   }
 }
 export {

package/plugins/specweave-testing/lib/playwright-cli-runner.ts

@@ -1,4 +1,4 @@
-import { execSync } from 'child_process';
+import { execFileSync } from 'child_process';
 
 export interface CliRunnerConfig {
   headed?: boolean;
@@ -19,19 +19,22 @@ export class PlaywrightCliRunner {
     this.config = {
       headed: config.headed ?? false,
       browser: config.browser ?? 'chrome',
-      session: config.session,
       timeout: config.timeout ?? 30_000,
-      ...config,
+      session: config.session,
     };
   }
 
-  private exec(command: string): CliResult {
-    const sessionFlag = this.config.session ? `-s=${this.config.session} ` : '';
-    const fullCommand = `playwright-cli ${sessionFlag}${command}`;
+  private exec(args: string[]): CliResult {
+    const fullArgs: string[] = [];
+    if (this.config.session) {
+      fullArgs.push(`-s=${this.config.session}`);
+    }
+    fullArgs.push(...args);
     try {
-      const output = execSync(fullCommand, {
+      const output = execFileSync('playwright-cli', fullArgs, {
         encoding: 'utf-8',
         timeout: this.config.timeout,
+        maxBuffer: 1024 * 1024,
       }).trim();
       return { ok: true, output };
     } catch (e: unknown) {
@@ -40,41 +43,43 @@ export class PlaywrightCliRunner {
   }
 
   open(url?: string): CliResult {
-    const headedFlag = this.config.headed ? ' --headed' : '';
-    const urlPart = url ? ` ${url}` : '';
-    return this.exec(`open${urlPart}${headedFlag}`);
+    const args = ['open'];
+    if (url) args.push(url);
+    if (this.config.headed) args.push('--headed');
+    return this.exec(args);
   }
 
   navigate(url: string): CliResult {
-    return this.exec(`goto ${url}`);
+    return this.exec(['goto', url]);
   }
 
   snapshot(): CliResult {
-    return this.exec('snapshot');
+    return this.exec(['snapshot']);
   }
 
   screenshot(filename?: string): CliResult {
-    const flag = filename ? ` --filename ${filename}` : '';
-    return this.exec(`screenshot${flag}`);
+    const args = ['screenshot'];
+    if (filename) args.push('--filename', filename);
+    return this.exec(args);
   }
 
   close(): CliResult {
-    return this.exec('close');
+    return this.exec(['close']);
   }
 
   click(ref: string): CliResult {
-    return this.exec(`click ${ref}`);
+    return this.exec(['click', ref]);
   }
 
   type(text: string): CliResult {
-    return this.exec(`type "${text}"`);
+    return this.exec(['type', text]);
   }
 
   fill(ref: string, text: string): CliResult {
-    return this.exec(`fill ${ref} "${text}"`);
+    return this.exec(['fill', ref, text]);
   }
 
   evaluate(fn: string): CliResult {
-    return this.exec(`eval "${fn}"`);
+    return this.exec(['eval', fn]);
   }
 }

package/plugins/specweave-testing/lib/playwright-routing.js

@@ -1,15 +1,10 @@
 import { detectPlaywrightCli } from "./playwright-cli-detector.js";
-const MCP_PREFERRED_TASKS = /* @__PURE__ */ new Set([
-  "ui-inspect",
-  "page-exploration",
-  "self-healing-test"
-]);
 function resolvePlaywrightMode(task, config = {}) {
   const { preferCli = true } = config;
   if (!preferCli) return "mcp";
   const detection = detectPlaywrightCli({ useCache: true });
   if (!detection.installed) return "mcp";
-  return MCP_PREFERRED_TASKS.has(task) ? "mcp" : "cli";
+  return "cli";
 }
 export {
   resolvePlaywrightMode

package/plugins/specweave-testing/lib/playwright-routing.ts

@@ -13,26 +13,29 @@ export type TaskType =
   | 'self-healing-test';
 
 export interface RoutingConfig {
+  /** @deprecated CLI is now the only mode. MCP plugin removed from auto-install (0198). */
   preferCli?: boolean;
 }
 
-/** Tasks that benefit from MCP's rich DOM introspection */
-const MCP_PREFERRED_TASKS: ReadonlySet<TaskType> = new Set([
-  'ui-inspect',
-  'page-exploration',
-  'self-healing-test',
-]);
-
+/**
+ * Resolve Playwright mode for a given task.
+ *
+ * v1.0.240 (0198): CLI-only mode. All tasks route to CLI.
+ * MCP fallback only when CLI is not installed (graceful degradation).
+ * Users who want Playwright MCP can install it manually.
+ */
 export function resolvePlaywrightMode(
   task: TaskType,
   config: RoutingConfig = {},
 ): PlaywrightMode {
   const { preferCli = true } = config;
 
+  // Config override: allow forcing MCP if user explicitly installed Playwright MCP
   if (!preferCli) return 'mcp';
 
   const detection = detectPlaywrightCli({ useCache: true });
   if (!detection.installed) return 'mcp';
 
-  return MCP_PREFERRED_TASKS.has(task) ? 'mcp' : 'cli';
+  // All tasks route to CLI — no MCP-preferred tasks
+  return 'cli';
 }