agentic-qe 3.15.9 → 3.15.11

package/dist/init/init-wizard-steps.js

@@ -11,6 +11,7 @@ import { createSkillsInstaller } from './skills-installer.js';
 import { createAgentsInstaller } from './agents-installer.js';
 import { createN8nInstaller } from './n8n-installer.js';
 import { toErrorMessage } from '../shared/error-utils.js';
+import { isNativeModuleLoadFailure, withNativeModuleRemedy } from '../shared/native-module-errors.js';
 import { openDatabase } from '../shared/safe-db.js';
 import { safeJsonParse } from '../shared/safe-json.js';
 // ============================================================================
@@ -112,7 +113,7 @@ export async function initializePersistenceDatabase(projectRoot) {
         const mod = await import('better-sqlite3');
         Database = mod.default;
     }
-    catch (error) {
+    catch {
         throw new Error('SQLite persistence REQUIRED but better-sqlite3 is not installed.\n' +
             'Install it with: npm install better-sqlite3\n' +
             'If you see native compilation errors, ensure build tools are installed:\n' +
@@ -157,6 +158,9 @@ export async function initializePersistenceDatabase(projectRoot) {
         return true;
     }
     catch (error) {
+        if (isNativeModuleLoadFailure(error)) {
+            throw new Error(`SQLite persistence initialization FAILED: ${toErrorMessage(withNativeModuleRemedy(error))}`, { cause: error });
+        }
         throw new Error(`SQLite persistence initialization FAILED: ${error}\n` +
             `Database path: ${dbPath}\n` +
             'Ensure the directory is writable and has sufficient disk space.');

package/dist/init/phases/04-database.js

@@ -17,6 +17,8 @@ import { existsSync, mkdirSync } from 'fs';
 import { join } from 'path';
 import { BasePhase, } from './phase-interface.js';
 import { getAQEVersion } from '../types.js';
+import { toErrorMessage } from '../../shared/error-utils.js';
+import { isNativeModuleLoadFailure, withNativeModuleRemedy } from '../../shared/native-module-errors.js';
 /**
  * Database phase - initializes SQLite persistence via UnifiedMemoryManager
  */
@@ -106,6 +108,11 @@ export class DatabasePhase extends BasePhase {
             };
         }
         catch (error) {
+            // A native-addon load failure (#1173) is not a disk/permission problem:
+            // report the cause and the install fix instead of the generic advice.
+            if (isNativeModuleLoadFailure(error)) {
+                throw new Error(`SQLite persistence initialization FAILED: ${toErrorMessage(withNativeModuleRemedy(error))}`, { cause: error });
+            }
             throw new Error(`SQLite persistence initialization FAILED: ${error}\n` +
                 `Database path: ${dbPath}\n` +
                 'Ensure the directory is writable and has sufficient disk space.');

package/dist/init/phases/10-workers.js

@@ -150,6 +150,29 @@ if (existsSync(pidFile) && isRegularOrAbsent(pidFile)) {
   try { process.kill(pid, 0); console.log('Daemon already running (PID: ' + pid + ')'); process.exit(0); } catch {}
 }
 
+// #1116: honour AQE_WORKERS_ENABLED, from the environment or else the project's
+// .claude/settings.local.json or settings.json env (where \`aqe init
+// --no-database\` writes it; the local file wins, as in Claude Code), the same
+// falsy values as the MCP server. A JSON boolean counts as its string.
+// Disabled means nothing is spawned.
+function workersEnabledSetting() {
+  if (process.env.AQE_WORKERS_ENABLED !== undefined) return process.env.AQE_WORKERS_ENABLED;
+  for (const file of ['settings.local.json', 'settings.json']) {
+    try {
+      const settings = JSON.parse(readFileSync(join(projectRoot, '.claude', file), 'utf-8'));
+      const value = settings && settings.env && settings.env.AQE_WORKERS_ENABLED;
+      if (typeof value === 'string') return value;
+      if (typeof value === 'boolean') return String(value);
+    } catch { /* missing or unreadable: try the next file */ }
+  }
+  return undefined;
+}
+const workersSetting = workersEnabledSetting();
+if (workersSetting !== undefined && ['false', '0', 'off', 'no'].includes(workersSetting.trim().toLowerCase())) {
+  console.log('AQE v3 Worker Daemon not started: background workers are disabled (AQE_WORKERS_ENABLED=' + workersSetting + ')');
+  process.exit(0);
+}
+
 const ts = new Date().toISOString();
 appendFileSync(logFile, '[' + ts + '] Starting AQE v3 Worker Daemon...\\n');
 
@@ -205,17 +228,38 @@ if (binCandidate && binCandidate.endsWith('bundle.js')) {
 appendFileSync(logFile, '[' + ts + '] Using: ' + mcpCmd + ' ' + mcpArgs.join(' ') + '\\n');
 
 // #1066: keep a user's AQE_WORKERS_ENABLED (for example false) instead of forcing it on.
-const env = { ...process.env, AQE_STORAGE_PATH: '.agentic-qe', AQE_LEARNING_ENABLED: 'true', AQE_WORKERS_ENABLED: process.env.AQE_WORKERS_ENABLED ?? 'true', AQE_HTTP_PORT: '0' };
+const env = { ...process.env, AQE_STORAGE_PATH: '.agentic-qe', AQE_LEARNING_ENABLED: 'true', AQE_WORKERS_ENABLED: workersSetting ?? 'true', AQE_HTTP_PORT: '0' };
 const child = spawn(mcpCmd, mcpArgs, { cwd: projectRoot, env, detached: true, stdio: ['ignore', 'pipe', 'pipe'] });
 child.stdout.on('data', d => appendFileSync(logFile, d));
 child.stderr.on('data', d => appendFileSync(logFile, d));
+let exited = null;
+child.on('exit', (code, signal) => { exited = { code, signal }; });
+// #1116: a command that cannot be started (no npx on PATH: spawn ENOENT) emits
+// 'error' instead of 'exit'; without a handler the script crashed and left
+// "undefined" in daemon.pid.
+child.on('error', (error) => {
+  try { fs.unlinkSync(pidFile); } catch {}
+  appendFileSync(logFile, '[' + ts + '] Daemon could not be started: ' + error.message + '\\n');
+  console.error('AQE v3 Worker Daemon not started: ' + error.message + '; see ' + logFile);
+  process.exit(1);
+});
 child.unref();
 
-writeFileSync(pidFile, String(child.pid));
-appendFileSync(logFile, '[' + ts + '] Daemon started with PID: ' + child.pid + '\\n');
-console.log('AQE v3 Worker Daemon started (PID: ' + child.pid + ')');
-console.log('Log file: ' + logFile);
-console.log('To stop: node ' + join(workersDir, 'stop-daemon.cjs'));
+if (child.pid !== undefined) writeFileSync(pidFile, String(child.pid));
+// #1116: say "started" only if the server is still running a moment later.
+setTimeout(() => {
+  if (exited) {
+    try { fs.unlinkSync(pidFile); } catch {}
+    appendFileSync(logFile, '[' + ts + '] Daemon exited right away (code ' + exited.code + ', signal ' + exited.signal + ')\\n');
+    console.error('AQE v3 Worker Daemon not started: the MCP server exited right away (code ' + exited.code + '); see ' + logFile);
+    process.exitCode = 1;
+    return;
+  }
+  appendFileSync(logFile, '[' + ts + '] Daemon started with PID: ' + child.pid + '\\n');
+  console.log('AQE v3 Worker Daemon started (PID: ' + child.pid + ')');
+  console.log('Log file: ' + logFile);
+  console.log('To stop: node ' + join(workersDir, 'stop-daemon.cjs'));
+}, 3000);
 `;
         writeFileIfChanged(daemonScriptPath, daemonScript);
         // Also write a cross-platform stop script

package/dist/init/phases/12-verification.js

@@ -13,6 +13,8 @@ import { BasePhase, } from './phase-interface.js';
 import { openDatabase } from '../../shared/safe-db.js';
 /** The `# <ISO timestamp>` header line written by configToYAML(). */
 const CONFIG_TIMESTAMP_LINE = /^# \d{4}-\d{2}-\d{2}T[\d:.]+Z$/;
+/** #1157: config keys that would reach Object.prototype through a plain object. */
+const UNSAFE_YAML_KEYS = new Set(['__proto__', 'constructor', 'prototype']);
 /**
  * Verification phase - verifies installation and writes markers
  */
@@ -173,8 +175,12 @@ export class VerificationPhase extends BasePhase {
                 // Top-level key (no indentation)
                 const topMatch = line.match(/^(\w+):\s*(.*)$/);
                 if (topMatch) {
-                    currentSection = topMatch[1];
+                    // #1157: keys come from config.yaml; a `__proto__` section (and
+                    // everything under it) is skipped instead of re-prototyping result.
+                    currentSection = UNSAFE_YAML_KEYS.has(topMatch[1]) ? '' : topMatch[1];
                     currentSubSection = '';
+                    if (!currentSection)
+                        continue;
                     const value = topMatch[2].trim();
                     if (value && !value.startsWith('"')) {
                         // Simple value
@@ -191,9 +197,11 @@ export class VerificationPhase extends BasePhase {
                 // Second-level key (2-space indent)
                 const subMatch = line.match(/^ {2}(\w+):\s*(.*)$/);
                 if (subMatch && currentSection) {
-                    currentSubSection = subMatch[1];
+                    currentSubSection = UNSAFE_YAML_KEYS.has(subMatch[1]) ? '' : subMatch[1];
+                    if (!currentSubSection)
+                        continue;
                     const value = subMatch[2].trim();
-                    if (!result[currentSection]) {
+                    if (!Object.hasOwn(result, currentSection) || !result[currentSection]) {
                         result[currentSection] = {};
                     }
                     if (value) {
@@ -210,8 +218,10 @@ export class VerificationPhase extends BasePhase {
                 if (thirdMatch && currentSection && currentSubSection) {
                     const key = thirdMatch[1];
                     const value = thirdMatch[2].trim();
+                    if (UNSAFE_YAML_KEYS.has(key))
+                        continue;
                     const section = result[currentSection];
-                    if (!section[currentSubSection]) {
+                    if (!Object.hasOwn(section, currentSubSection) || !section[currentSubSection]) {
                         section[currentSubSection] = {};
                     }
                     if (typeof section[currentSubSection] === 'object' && !Array.isArray(section[currentSubSection])) {

package/dist/init/settings-merge.js

@@ -8,6 +8,7 @@
  * - Generates full env vars and v3 settings sections
  */
 import { existsSync, readFileSync, writeFileSync } from 'fs';
+import { getOwn, setOwn } from '../shared/utils/safe-record.js';
 /**
  * Create a one-time `.backup` of an existing settings.json before AQE modifies
  * it — mirroring the CLAUDE.md backup convention.
@@ -75,22 +76,24 @@ export function isAqeHookEntry(entry) {
  * 3. Preserve any hook types not in the new config
  */
 export function mergeHooksSmart(existingHooks, newAqeHooks) {
+    // #1157: hook types come from the user's settings.json; read and write own
+    // entries only, so a type named `constructor` or `__proto__` is kept as-is.
     const merged = {};
     // Process hook types that have new AQE entries
     for (const [hookType, newEntries] of Object.entries(newAqeHooks)) {
-        const existing = existingHooks[hookType] || [];
+        const existing = getOwn(existingHooks, hookType) || [];
         // Keep only non-AQE entries from existing config
         const userEntries = Array.isArray(existing)
             ? existing.filter((entry) => !isAqeHookEntry(entry))
             : [];
         // New AQE hooks go after user hooks (except PreToolUse guard which should be first)
-        merged[hookType] = [...newEntries, ...userEntries];
+        setOwn(merged, hookType, [...newEntries, ...userEntries]);
     }
     // Preserve hook types that are not in our new config (user's custom hook types)
     for (const [hookType, hookArray] of Object.entries(existingHooks)) {
-        if (!merged[hookType]) {
+        if (!Object.hasOwn(merged, hookType)) {
             // This hook type isn't one we manage — keep it as-is
-            merged[hookType] = hookArray;
+            setOwn(merged, hookType, hookArray);
         }
     }
     return merged;

package/dist/integrations/agentic-flow/reasoning-bank/experience-replay.d.ts

@@ -113,6 +113,7 @@ export declare class ExperienceReplay {
     private prepared;
     private initialized;
     private initializationPromise;
+    private embedderUnavailableLogged;
     private hnswIndex;
     private idToExperienceId;
     private experienceIdToHnswId;
@@ -150,6 +151,12 @@ export declare class ExperienceReplay {
      * @returns The stored experience
      */
     storeExperience(trajectory: Trajectory, strategy: string, tags?: string[]): Promise<Experience | null>;
+    /**
+     * #1022: embed text, or return null when no embedding runtime is available.
+     * The experience is then stored without a vector (backfilled at a later
+     * start that has an embedder) and similarity search finds no matches.
+     */
+    private embedOrNull;
     /**
      * Get guidance for a new task based on similar experiences
      *

package/dist/integrations/agentic-flow/reasoning-bank/experience-replay.js

@@ -60,6 +60,7 @@ export class ExperienceReplay {
     prepared = new Map();
     initialized = false;
     initializationPromise = null;
+    embedderUnavailableLogged = false;
     // Real HNSW index for O(log n) similarity search (150x-12,500x faster than linear scan)
     hnswIndex;
     // Mapping from HNSW numeric IDs to experience string IDs
@@ -399,7 +400,7 @@ export class ExperienceReplay {
             .map(s => scrubReasoningBlocks(s.action));
         // Generate embedding for similarity search
         const embeddingText = `${task} ${strategy} ${keyActions.join(' ')}`;
-        const embedding = await computeRealEmbedding(embeddingText, this.config.embedding);
+        const embedding = (await this.embedOrNull(embeddingText)) ?? undefined;
         const id = uuidv4();
         const domain = trajectory.domain || 'test-generation';
         const experience = {
@@ -456,6 +457,24 @@ export class ExperienceReplay {
         }
         return experience;
     }
+    /**
+     * #1022: embed text, or return null when no embedding runtime is available.
+     * The experience is then stored without a vector (backfilled at a later
+     * start that has an embedder) and similarity search finds no matches.
+     */
+    async embedOrNull(text) {
+        try {
+            return await computeRealEmbedding(text, this.config.embedding);
+        }
+        catch (error) {
+            if (!this.embedderUnavailableLogged) {
+                this.embedderUnavailableLogged = true;
+                console.warn('[ExperienceReplay] No embedding runtime; experiences are stored without vectors ' +
+                    'and similar-experience guidance is unavailable:', error instanceof Error ? error.message : error);
+            }
+            return null;
+        }
+    }
     /**
      * Get guidance for a new task based on similar experiences
      *
@@ -465,6 +484,13 @@ export class ExperienceReplay {
      */
     async getGuidance(task, domain) {
         this.ensureInitialized();
+        // #1022: the reservoir blend below picks experiences without looking at
+        // the query and gives them a coherence-derived similarity. Without an
+        // embedding of the query nothing measures relevance, so return no guidance
+        // rather than query-independent picks with a made-up similarity.
+        if (!(await this.embedOrNull(scrubReasoningBlocks(task)))) {
+            return null;
+        }
         // Find similar experiences via HNSW
         const similar = await this.findSimilarExperiences(task, domain);
         // Blend in high-coherence experiences from reservoir buffer (R10, ADR-087)
@@ -546,7 +572,10 @@ export class ExperienceReplay {
         // Generate embedding for the task
         // ADR-099: scrub so query embeddings match the scrubbed stored embeddings
         task = scrubReasoningBlocks(task);
-        const taskEmbedding = await computeRealEmbedding(task, this.config.embedding);
+        const taskEmbedding = await this.embedOrNull(task);
+        if (!taskEmbedding) {
+            return [];
+        }
         // Create query embedding for HNSW search
         const queryEmbedding = {
             vector: taskEmbedding,

package/dist/integrations/agentic-flow/reasoning-bank/index.d.ts

@@ -15,7 +15,7 @@
  * - Automatic strategy recommendations
  * - Pattern quality improvement over time
  */
-import { RealQEReasoningBank, type RealQEReasoningBankConfig, type RealQEPatternSearchResult, type RealQERoutingRequest, type RealQERoutingResult, type LearningOutcome } from '../../../learning/real-qe-reasoning-bank.js';
+import { RealQEReasoningBank, type RealQEReasoningBankConfig, type RealQEPatternSearchResult, type PatternSearchStatus, type RealQERoutingRequest, type RealQERoutingResult, type LearningOutcome } from '../../../learning/real-qe-reasoning-bank.js';
 import { TrajectoryTracker, type Trajectory, type TrajectoryStep, type TrajectoryOptions, type TrajectoryTrackerConfig } from './trajectory-tracker.js';
 import { ExperienceReplay, type ExperienceGuidance, type ExperienceReplayConfig } from './experience-replay.js';
 import { PatternEvolution, type PatternVersion, type DriftDetectionResult, type MergeCandidate, type PatternEvolutionConfig } from './pattern-evolution.js';
@@ -183,6 +183,8 @@ export declare class EnhancedReasoningBankAdapter {
         domain?: QEDomain;
         minSimilarity?: number;
     }): Promise<Result<RealQEPatternSearchResult[]>>;
+    /** #1022: whether pattern search uses vectors or text matching only. */
+    getPatternSearchStatus(): PatternSearchStatus;
     /**
      * Record a pattern outcome with evolution tracking
      */

package/dist/integrations/agentic-flow/reasoning-bank/index.js

@@ -297,6 +297,11 @@ export class EnhancedReasoningBankAdapter {
         this.ensureInitialized();
         return this.reasoningBank.searchQEPatterns(query, options);
     }
+    /** #1022: whether pattern search uses vectors or text matching only. */
+    getPatternSearchStatus() {
+        this.ensureInitialized();
+        return this.reasoningBank.getPatternSearchStatus();
+    }
     /**
      * Record a pattern outcome with evolution tracking
      */

package/dist/integrations/browser/agent-browser/client.js

@@ -210,7 +210,7 @@ export class AgentBrowserClient {
             // Extra safety: also terminate via executor if we have one
             if (this.executor) {
                 try {
-                    this.executor.terminateDaemon();
+                    await this.executor.terminateDaemon();
                 }
                 catch (error) {
                     // Non-critical: best effort daemon termination
@@ -251,7 +251,7 @@ export class AgentBrowserClient {
     async navigate(url) {
         const startTime = Date.now();
         try {
-            const result = this.executor.open(url);
+            const result = await this.executor.open(url);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Navigation failed', 'NAVIGATION_FAILED', 'agent-browser'));
             }
@@ -260,7 +260,7 @@ export class AgentBrowserClient {
                 this.sessionManager.updateSessionUrl(this.currentSessionId, url);
             }
             // Get page info for result
-            const evalResult = this.executor.eval('JSON.stringify({ title: document.title, url: window.location.href })');
+            const evalResult = await this.executor.eval('JSON.stringify({ title: document.title, url: window.location.href })');
             const pageInfo = evalResult.success && evalResult.data
                 ? typeof evalResult.data === 'string'
                     ? safeJsonParse(evalResult.data)
@@ -282,7 +282,7 @@ export class AgentBrowserClient {
      */
     async reload() {
         try {
-            const result = this.executor.eval('window.location.reload()');
+            const result = await this.executor.eval('window.location.reload()');
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Reload failed', 'RELOAD_FAILED', 'agent-browser'));
             }
@@ -297,7 +297,7 @@ export class AgentBrowserClient {
      */
     async goBack() {
         try {
-            const result = this.executor.eval('window.history.back()');
+            const result = await this.executor.eval('window.history.back()');
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Go back failed', 'HISTORY_FAILED', 'agent-browser'));
             }
@@ -312,7 +312,7 @@ export class AgentBrowserClient {
      */
     async goForward() {
         try {
-            const result = this.executor.eval('window.history.forward()');
+            const result = await this.executor.eval('window.history.forward()');
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Go forward failed', 'HISTORY_FAILED', 'agent-browser'));
             }
@@ -331,7 +331,7 @@ export class AgentBrowserClient {
     async click(target) {
         try {
             const resolvedTarget = resolveTarget(target);
-            const result = this.executor.click(resolvedTarget);
+            const result = await this.executor.click(resolvedTarget);
             if (!result.success) {
                 return err(new BrowserElementNotFoundError('agent-browser', target, new Error(result.error)));
             }
@@ -347,7 +347,7 @@ export class AgentBrowserClient {
     async fill(target, text) {
         try {
             const resolvedTarget = resolveTarget(target);
-            const result = this.executor.fill(resolvedTarget, text);
+            const result = await this.executor.fill(resolvedTarget, text);
             if (!result.success) {
                 return err(new BrowserElementNotFoundError('agent-browser', target, new Error(result.error)));
             }
@@ -363,7 +363,7 @@ export class AgentBrowserClient {
     async getText(target) {
         try {
             const resolvedTarget = resolveTarget(target);
-            const result = this.executor.getText(resolvedTarget);
+            const result = await this.executor.getText(resolvedTarget);
             if (!result.success) {
                 return err(new BrowserElementNotFoundError('agent-browser', target, new Error(result.error)));
             }
@@ -400,7 +400,7 @@ export class AgentBrowserClient {
     async isVisible(target) {
         try {
             const resolvedTarget = resolveTarget(target);
-            const result = this.executor.isVisible(resolvedTarget);
+            const result = await this.executor.isVisible(resolvedTarget);
             if (!result.success) {
                 // Element not found means not visible
                 return ok(false);
@@ -419,12 +419,12 @@ export class AgentBrowserClient {
      */
     async screenshot(options) {
         try {
-            const result = this.executor.screenshot(options?.path, options?.fullPage);
+            const result = await this.executor.screenshot(options?.path, options?.fullPage);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Screenshot failed', 'SCREENSHOT_FAILED', 'agent-browser'));
             }
             // Get viewport dimensions
-            const dimensionsResult = this.executor.eval('JSON.stringify({ width: window.innerWidth, height: window.innerHeight })');
+            const dimensionsResult = await this.executor.eval('JSON.stringify({ width: window.innerWidth, height: window.innerHeight })');
             let dimensions = { width: 1280, height: 720 };
             if (dimensionsResult.success && dimensionsResult.data) {
                 const parsed = typeof dimensionsResult.data === 'string'
@@ -451,7 +451,7 @@ export class AgentBrowserClient {
      */
     async evaluate(script) {
         try {
-            const result = this.executor.eval(script);
+            const result = await this.executor.eval(script);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Evaluation failed', 'EVAL_FAILED', 'agent-browser'));
             }
@@ -487,7 +487,7 @@ export class AgentBrowserClient {
      */
     async getSnapshot(options) {
         try {
-            const result = this.executor.snapshot({
+            const result = await this.executor.snapshot({
                 interactive: options?.interactive,
                 depth: options?.depth,
             });
@@ -504,8 +504,8 @@ export class AgentBrowserClient {
                 parsed = this.snapshotParser.parse(String(result.data));
             }
             // Get current URL and title
-            const urlResult = this.executor.eval('window.location.href');
-            const titleResult = this.executor.eval('document.title');
+            const urlResult = await this.executor.eval('window.location.href');
+            const titleResult = await this.executor.eval('document.title');
             if (urlResult.success) {
                 parsed.url = String(urlResult.data);
             }
@@ -578,7 +578,7 @@ export class AgentBrowserClient {
                 body: response.body,
                 headers: response.headers,
             };
-            const result = this.executor.mockRoute(urlPattern, mockBody);
+            const result = await this.executor.mockRoute(urlPattern, mockBody);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Mock route failed', 'MOCK_ROUTE_FAILED', 'agent-browser'));
             }
@@ -593,7 +593,7 @@ export class AgentBrowserClient {
      */
     async abortRoute(urlPattern) {
         try {
-            const result = this.executor.abortRoute(urlPattern);
+            const result = await this.executor.abortRoute(urlPattern);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Abort route failed', 'ABORT_ROUTE_FAILED', 'agent-browser'));
             }
@@ -608,7 +608,7 @@ export class AgentBrowserClient {
      */
     async clearRoutes() {
         try {
-            const result = this.executor.clearRoutes();
+            const result = await this.executor.clearRoutes();
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Clear routes failed', 'CLEAR_ROUTES_FAILED', 'agent-browser'));
             }
@@ -626,7 +626,7 @@ export class AgentBrowserClient {
      */
     async setDevice(deviceName) {
         try {
-            const result = this.executor.setDevice(deviceName);
+            const result = await this.executor.setDevice(deviceName);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Set device failed', 'SET_DEVICE_FAILED', 'agent-browser'));
             }
@@ -641,7 +641,7 @@ export class AgentBrowserClient {
      */
     async setViewport(width, height) {
         try {
-            const result = this.executor.setViewport(width, height);
+            const result = await this.executor.setViewport(width, height);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Set viewport failed', 'SET_VIEWPORT_FAILED', 'agent-browser'));
             }
@@ -659,7 +659,7 @@ export class AgentBrowserClient {
      */
     async saveState(path) {
         try {
-            const result = this.executor.saveState(path);
+            const result = await this.executor.saveState(path);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Save state failed', 'SAVE_STATE_FAILED', 'agent-browser'));
             }
@@ -674,7 +674,7 @@ export class AgentBrowserClient {
      */
     async loadState(path) {
         try {
-            const result = this.executor.loadState(path);
+            const result = await this.executor.loadState(path);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Load state failed', 'LOAD_STATE_FAILED', 'agent-browser'));
             }
@@ -693,7 +693,7 @@ export class AgentBrowserClient {
     async waitForElement(target, timeout) {
         try {
             const resolvedTarget = resolveTarget(target);
-            const result = this.executor.waitForElement(resolvedTarget, timeout);
+            const result = await this.executor.waitForElement(resolvedTarget, timeout);
             if (!result.success) {
                 return err(new BrowserTimeoutError('agent-browser', `waitForElement(${resolvedTarget})`, new Error(result.error)));
             }
@@ -711,7 +711,7 @@ export class AgentBrowserClient {
             // The timeout parameter is ignored by the current CLI implementation
             // but we accept it for interface compatibility
             void timeout;
-            const result = this.executor.waitForText(text);
+            const result = await this.executor.waitForText(text);
             if (!result.success) {
                 return err(new BrowserTimeoutError('agent-browser', `waitForText("${text}")`, new Error(result.error)));
             }
@@ -728,7 +728,7 @@ export class AgentBrowserClient {
         try {
             // The timeout parameter is ignored by the current CLI implementation
             void timeout;
-            const result = this.executor.waitForUrl(pattern);
+            const result = await this.executor.waitForUrl(pattern);
             if (!result.success) {
                 return err(new BrowserTimeoutError('agent-browser', `waitForUrl("${pattern}")`, new Error(result.error)));
             }
@@ -745,7 +745,7 @@ export class AgentBrowserClient {
         try {
             // The timeout parameter is ignored by the current CLI implementation
             void timeout;
-            const result = this.executor.waitForNetworkIdle();
+            const result = await this.executor.waitForNetworkIdle();
             if (!result.success) {
                 return err(new BrowserTimeoutError('agent-browser', 'waitForNetworkIdle', new Error(result.error)));
             }
@@ -766,7 +766,7 @@ export class AgentBrowserClient {
      */
     async startTrace() {
         try {
-            const result = this.executor.startTrace();
+            const result = await this.executor.startTrace();
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Start trace failed', 'START_TRACE_FAILED', 'agent-browser'));
             }
@@ -784,7 +784,7 @@ export class AgentBrowserClient {
      */
     async stopTrace(outputPath) {
         try {
-            const result = this.executor.stopTrace(outputPath);
+            const result = await this.executor.stopTrace(outputPath);
             if (!result.success) {
                 return err(new BrowserError(result.error ?? 'Stop trace failed', 'STOP_TRACE_FAILED', 'agent-browser'));
             }
@@ -815,28 +815,13 @@ export function createAgentBrowserClient(config) {
  * Only use in test environments.
  */
 export async function cleanupAllBrowserProcesses() {
-    const { execSync } = await import('child_process');
-    try {
-        // Kill agent-browser daemons
-        execSync('pkill -9 -f "agent-browser.*daemon" 2>/dev/null || true', {
-            timeout: 5000,
-            stdio: 'ignore'
-        });
-    }
-    catch (error) {
-        // Non-critical: daemon cleanup errors
-        console.debug('[AgentBrowserClient] Daemon cleanup error:', error instanceof Error ? error.message : error);
-    }
-    try {
-        // Kill chromium headless shell processes
-        execSync('pkill -9 -f "chromium_headless_shell\\|headless_shell" 2>/dev/null || true', {
-            timeout: 5000,
-            stdio: 'ignore'
-        });
-    }
-    catch (error) {
-        // Non-critical: chromium cleanup errors
-        console.debug('[AgentBrowserClient] Chromium cleanup error:', error instanceof Error ? error.message : error);
-    }
+    const { runTestRunnerProcess } = await import('../../../shared/test-runner-process.js');
+    // pkill exits 1 when nothing matched; cleanup is best effort either way
+    const pkill = async (pattern) => {
+        await runTestRunnerProcess('pkill', ['-9', '-f', pattern], { cwd: process.cwd(), timeout: 5000 });
+    };
+    // Kill agent-browser daemons, then chromium headless shell processes
+    await pkill('agent-browser.*daemon');
+    await pkill('chromium_headless_shell\\|headless_shell');
 }
 //# sourceMappingURL=client.js.map