explorbot 0.1.32-beta.1 → 0.2.1

package/src/ai/task-agent.ts

@@ -1,10 +1,11 @@
-import dedent from 'dedent';
 import type { ActionResult } from '../action-result.js';
-import { type ExperienceTracker, renderExperienceToc } from '../experience-tracker.js';
+import type { ExplorbotConfig } from '../config.ts';
+import type { ExperienceTracker } from '../experience-tracker.js';
+import type Explorer from '../explorer.ts';
 import type { KnowledgeTracker } from '../knowledge-tracker.js';
-import { createDebug, pluralize, tag } from '../utils/logger.js';
-
-const debugLog = createDebug('explorbot:task-agent');
+import type { StateManager } from '../state-manager.ts';
+import { HooksRunner } from '../utils/hooks-runner.ts';
+import type { AgentDeps, ToolDeps } from './agent.ts';
 import { Historian } from './historian.js';
 import type { Navigator } from './navigator.js';
 import type { Provider } from './provider.js';
@@ -21,6 +22,12 @@ function createNullProxy<T extends object>(): T {
 }
 
 export abstract class TaskAgent {
+  explorer!: Explorer;
+  provider!: Provider;
+  config!: ExplorbotConfig;
+  stateManager!: StateManager;
+  knowledgeTracker!: KnowledgeTracker;
+  protected hooksRunner!: HooksRunner;
   protected consecutiveFailures = 0;
   protected consecutiveEmptyResults = 0;
   protected recentToolCalls: any[] = [];
@@ -29,44 +36,48 @@ export abstract class TaskAgent {
   private _historian: Historian | null = null;
   private _quartermaster: Quartermaster | null = null;
 
-  protected abstract getNavigator(): Navigator;
-  protected abstract getExperienceTracker(): ExperienceTracker;
-  protected abstract getKnowledgeTracker(): KnowledgeTracker;
-  protected abstract getProvider(): Provider;
+  constructor(deps?: AgentDeps) {
+    if (!deps) return;
+    this.explorer = deps.explorer;
+    this.provider = deps.ai;
+    this.config = deps.config;
+    this.stateManager = deps.stateManager;
+    this.knowledgeTracker = deps.knowledgeTracker;
+    this.hooksRunner = new HooksRunner(deps.explorer, deps.config);
+  }
 
-  protected getKnowledge(actionResult: ActionResult): string {
-    const knowledgeFiles = this.getKnowledgeTracker().getRelevantKnowledge(actionResult);
+  setHistorian(historian: Historian): void {
+    this._historian = historian;
+  }
 
-    if (knowledgeFiles.length === 0) return '';
+  setQuartermaster(quartermaster: Quartermaster): void {
+    this._quartermaster = quartermaster;
+  }
 
-    const knowledgeContent = knowledgeFiles
-      .map((k) => k.content)
-      .filter((k) => !!k)
-      .join('\n\n');
+  protected abstract getNavigator(): Navigator;
 
-    tag('operation').log(`Found ${knowledgeFiles.length} relevant knowledge ${pluralize(knowledgeFiles.length, 'file')}`);
-    return dedent`
-      <knowledge>
-      Here is relevant knowledge for this page:
+  protected get toolDeps(): ToolDeps {
+    return { explorer: this.explorer, stateManager: this.stateManager, ai: this.provider };
+  }
 
-      ${knowledgeContent}
-      </knowledge>
-    `;
+  protected getExperienceTracker(): ExperienceTracker {
+    return this.stateManager.getExperienceTracker();
   }
 
-  protected getExperience(actionResult: ActionResult): string {
-    const tracker = this.getExperienceTracker();
-    const toc = tracker.getExperienceTableOfContents(actionResult);
-    if (toc.length === 0) return '';
-
-    const totalSections = toc.reduce((sum, entry) => sum + entry.sections.length, 0);
-    debugLog(`injecting experience TOC (${toc.length} files, ${totalSections} sections)`);
-    tag('operation').log(`Found ${toc.length} experience ${pluralize(toc.length, 'file')} (${totalSections} sections)`);
-    return renderExperienceToc(toc);
+  protected getKnowledgeTracker(): KnowledgeTracker {
+    return this.knowledgeTracker;
   }
 
-  setHistorian(historian: Historian): void {
-    this._historian = historian;
+  protected getProvider(): Provider {
+    return this.provider;
+  }
+
+  protected getKnowledge(actionResult: ActionResult): string {
+    return this.getKnowledgeTracker().renderRelevantKnowledge(actionResult);
+  }
+
+  protected getExperience(actionResult: ActionResult): string {
+    return this.getExperienceTracker().renderExperienceTocFor(actionResult);
   }
 
   protected getHistorian(): Historian {
@@ -74,10 +85,6 @@ export abstract class TaskAgent {
     return createNullProxy<Historian>();
   }
 
-  setQuartermaster(quartermaster: Quartermaster): void {
-    this._quartermaster = quartermaster;
-  }
-
   protected getQuartermaster(): Quartermaster {
     if (this._quartermaster) return this._quartermaster;
     return createNullProxy<Quartermaster>();

package/src/ai/tester.ts

@@ -5,20 +5,17 @@ import dedent from 'dedent';
 import { z } from 'zod';
 import { ActionResult } from '../action-result.ts';
 import { clearActivity, setActivity } from '../activity.ts';
-import { ConfigParser } from '../config.ts';
-import type { ExperienceTracker } from '../experience-tracker.ts';
-import type Explorer from '../explorer.ts';
+import type { RequestStore } from '../api/request-store.ts';
+import type { TestRun } from '../explorer.ts';
 import { Observability } from '../observability.ts';
-import type { StateTransition, WebPageState } from '../state-manager.ts';
+import type { StateTransition } from '../state-manager.ts';
 import { Stats } from '../stats.ts';
-import { type Note, type Test, TestResult, type TestResultType } from '../test-plan.ts';
+import { type Test, TestResult, type TestResultType } from '../test-plan.ts';
 import { detectFocusArea, extractFocusedElement } from '../utils/aria.ts';
 import { ErrorPageError, isErrorPage } from '../utils/error-page.ts';
-import { HooksRunner } from '../utils/hooks-runner.ts';
-import { codeToMarkdown } from '../utils/html.ts';
 import { createDebug, tag } from '../utils/logger.ts';
 import { loop } from '../utils/loop.ts';
-import type { Agent } from './agent.ts';
+import type { Agent, AgentDeps } from './agent.ts';
 import type { Captain } from './captain.ts';
 import type { Conversation } from './conversation.ts';
 import { Navigator } from './navigator.ts';
@@ -27,7 +24,7 @@ import { Provider } from './provider.ts';
 import { Researcher } from './researcher.ts';
 import { actionRule, capabilityGroundingRule, dataProtectionRules, focusedElementRule, formRequirementsRule, locatorRule, multipleTabsRule, sectionContextRule } from './rules.ts';
 import { TaskAgent } from './task-agent.ts';
-import { createCodeceptJSTools, createSpecialContextTools } from './tools.ts';
+import { createCodeceptJSTools, createIframeTools } from './tools.ts';
 
 const debugLog = createDebug('explorbot:tester');
 
@@ -46,8 +43,8 @@ export class Tester extends TaskAgent implements Agent {
   protected readonly ACTION_TOOLS = ['click', 'hover', 'pressKey', 'form'];
   protected readonly SPECIAL_CONTEXT_ACTION_TOOLS = ['exitIframe'];
   emoji = '🧪';
-  private explorer: Explorer;
-  private provider: Provider;
+  private requestStore: RequestStore;
+  private testRun: TestRun | null = null;
   private currentConversation: Conversation | null = null;
   private pilot: Pilot | null = null;
   private captain: Captain | null = null;
@@ -58,43 +55,27 @@ export class Tester extends TaskAgent implements Agent {
   researcher: Researcher;
   navigator: Navigator;
   agentTools: any;
-  executionLogFile: string | null = null;
   private previousUrl: string | null = null;
   private previousStateHash: string | null = null;
   private pageStateHash: string | null = null;
   private pageActionResult: ActionResult | null = null;
-  private hooksRunner: HooksRunner;
   private seenUiMapUrls = new Set<string>();
   private lastAnalyzedStateHash: string | null = null;
   private stalledIterations = 0;
   private readonly MAX_STALLED_ITERATIONS = 3;
 
-  constructor(explorer: Explorer, provider: Provider, researcher: Researcher, navigator: Navigator, agentTools?: any) {
-    super();
-    this.explorer = explorer;
-    this.provider = provider;
+  constructor(deps: AgentDeps, researcher: Researcher, navigator: Navigator, agentTools?: any) {
+    super(deps);
+    this.requestStore = deps.requestStore;
     this.researcher = researcher;
     this.navigator = navigator;
     this.agentTools = agentTools;
-    this.hooksRunner = new HooksRunner(explorer, explorer.getConfig());
   }
 
   protected getNavigator(): Navigator {
     return this.navigator;
   }
 
-  protected getExperienceTracker(): ExperienceTracker {
-    return this.explorer.getStateManager().getExperienceTracker();
-  }
-
-  protected getKnowledgeTracker() {
-    return this.explorer.getKnowledgeTracker();
-  }
-
-  protected getProvider(): Provider {
-    return this.provider;
-  }
-
   setPilot(pilot: Pilot): void {
     this.pilot = pilot;
   }
@@ -104,11 +85,11 @@ export class Tester extends TaskAgent implements Agent {
   }
 
   private getCurrentState(): ActionResult {
-    return ActionResult.fromState(this.explorer.getStateManager().getCurrentState()!);
+    return ActionResult.fromState(this.stateManager.getCurrentState()!);
   }
 
   private get progressCheckInterval(): number {
-    return (this.explorer.getConfig().ai?.agents?.tester as any)?.progressCheckInterval ?? 3;
+    return (this.config.ai?.agents?.tester as any)?.progressCheckInterval ?? 3;
   }
 
   getConversation(): Conversation | null {
@@ -117,7 +98,7 @@ export class Tester extends TaskAgent implements Agent {
 
   async test(task: Test): Promise<{ success: boolean }> {
     Stats.tests++;
-    const state = this.explorer.getStateManager().getCurrentState();
+    const state = this.stateManager.getCurrentState();
     if (!state) throw new Error('No state found');
 
     setActivity(`🧪 Testing: ${task.scenario}`, 'action');
@@ -129,20 +110,20 @@ export class Tester extends TaskAgent implements Agent {
     this.seenUiMapUrls.clear();
     this.lastAnalyzedStateHash = null;
     this.stalledIterations = 0;
-    this.explorer.getStateManager().clearHistory();
+    this.stateManager.clearHistory();
     this.resetFailureCount();
     this.pilot?.reset();
 
-    const requestStore = this.explorer.getRequestStore();
-    requestStore?.clear();
-    const offFailedRequest = requestStore?.onFailedRequest((r) => {
+    const requestStore = this.requestStore;
+    requestStore.clear();
+    const offFailedRequest = requestStore.onFailedRequest((r) => {
       task.addNote(`Network error: ${r.method} ${r.path} → ${r.status}`, TestResult.FAILED);
     });
 
     const initialState = ActionResult.fromState(state);
     if (isErrorPage(initialState)) {
       task.start();
-      await this.explorer.startTest(task);
+      this.testRun = await this.explorer.beginTest(task);
       offFailedRequest?.();
       return await this.abortStartedTestOnErrorPage(task, initialState);
     }
@@ -151,10 +132,6 @@ export class Tester extends TaskAgent implements Agent {
     conversation.markLastMessageCacheable();
     this.currentConversation = conversation;
 
-    const outputDir = ConfigParser.getInstance().getOutputDir();
-    this.executionLogFile = join(outputDir, `tester_${task.sessionName}.md`);
-    // Note: Markdown saving functionality removed from Conversation class
-
     const scenarioBlock = this.buildScenarioBlock(task, initialState);
     conversation.addUserText(scenarioBlock);
     conversation.markLastMessageCacheable();
@@ -203,7 +180,8 @@ export class Tester extends TaskAgent implements Agent {
 
     debugLog('Starting test execution with tools');
 
-    if (!(await this.explorer.startTest(task))) {
+    this.testRun = await this.explorer.beginTest(task);
+    if (!this.testRun.started) {
       offFailedRequest?.();
       await this.cleanupStartedTest(task);
       return { success: task.isSuccessful };
@@ -221,7 +199,7 @@ export class Tester extends TaskAgent implements Agent {
       }
     }
 
-    const startState = this.explorer.getStateManager().getCurrentState();
+    const startState = this.stateManager.getCurrentState();
     if (startState) {
       task.addUrlNote(startState);
       const startActionResult = ActionResult.fromState(startState);
@@ -233,14 +211,14 @@ export class Tester extends TaskAgent implements Agent {
     const currentUrl = startState?.url || task.startUrl || '';
     await this.hooksRunner.runBeforeHook('tester', currentUrl);
 
-    const offStateChange = this.explorer.getStateManager().onStateChange((event: StateTransition) => {
+    const offStateChange = this.stateManager.onStateChange((event: StateTransition) => {
       if (task.hasFinished) return;
       if (event.toState?.url === event.fromState?.url) return;
       if (event.toState) task.addUrlNote(event.toState, event.fromState || undefined);
       task.states.push(event.toState);
     });
 
-    const codeceptjsTools = createCodeceptJSTools(this.explorer, task);
+    const codeceptjsTools = createCodeceptJSTools(this.toolDeps, task);
     let assertionPerformed = false;
     let extensions = 0;
     let shouldContinue = true;
@@ -251,7 +229,7 @@ export class Tester extends TaskAgent implements Agent {
       await loop(
         async ({ stop, pause, iteration, userInput }) => {
           debugLog('iteration', iteration);
-          if (!(await this.explorer.ensurePageAvailable())) {
+          if (!(await this.explorer.recover()).ok) {
             task.addNote('Browser page is unavailable');
             task.finish(TestResult.FAILED);
             stop();
@@ -261,14 +239,16 @@ export class Tester extends TaskAgent implements Agent {
 
           const tools = {
             ...codeceptjsTools,
-            ...(currentState.isInsideIframe ? createSpecialContextTools(this.explorer, 'iframe') : {}),
             ...this.createTestFlowTools(task, currentState, conversation),
             ...this.agentTools,
           };
+          if (currentState.isInsideIframe) {
+            Object.assign(tools, createIframeTools(this.toolDeps));
+          }
 
           debugLog(`Test ${task.scenario} iteration ${iteration}`);
 
-          if (this.explorer.getStateManager().isInDeadLoop()) {
+          if (this.stateManager.isInDeadLoop()) {
             task.addNote('Dead loop detected. Stopped');
             stop();
             return;
@@ -417,7 +397,7 @@ export class Tester extends TaskAgent implements Agent {
 
       if (task.hasFinished) break;
 
-      if (!(await this.explorer.ensurePageAvailable())) break;
+      if (!(await this.explorer.recover()).ok) break;
 
       const finalState = this.getCurrentState();
       const wantsContinue = await this.pilot!.finalReview(task, finalState, conversation, this.navigator);
@@ -436,7 +416,7 @@ export class Tester extends TaskAgent implements Agent {
       shouldContinue = true;
     }
 
-    const finalUrl = this.explorer.getStateManager().getCurrentState()?.url || currentUrl;
+    const finalUrl = this.stateManager.getCurrentState()?.url || currentUrl;
     await this.hooksRunner.runAfterHook('tester', finalUrl);
 
     await this.getHistorian().saveSession(task, initialState, conversation);
@@ -448,7 +428,7 @@ export class Tester extends TaskAgent implements Agent {
     offStateChange();
     offFailedRequest?.();
     await this.finishTest(task);
-    await this.explorer.stopTest(task, this.buildStopTestMeta(task));
+    await this.testRun?.stop(this.buildStopTestMeta(task));
 
     return {
       success: task.isSuccessful,
@@ -526,7 +506,6 @@ export class Tester extends TaskAgent implements Agent {
     const currentStateHash = currentState.hash;
 
     const isNewUrl = this.previousUrl !== currentUrl;
-    const isStateChanged = !isNewUrl && this.previousStateHash !== currentStateHash;
 
     this.previousUrl = currentUrl;
     this.previousStateHash = currentStateHash;
@@ -565,7 +544,7 @@ export class Tester extends TaskAgent implements Agent {
     }
 
     if (currentState.isInsideIframe) {
-      const iframeInfo = currentState.iframeURL || this.explorer.getCurrentIframeInfo() || 'iframe context active';
+      const iframeInfo = currentState.iframeURL || 'iframe context active';
       context += dedent`
         <iframe_context>
         INSIDE IFRAME: ${iframeInfo}
@@ -574,10 +553,10 @@ export class Tester extends TaskAgent implements Agent {
       `;
     }
 
-    if (this.explorer.hasOtherTabs()) {
-      const otherTabs = this.explorer.getOtherTabsInfo();
+    const otherTabs = this.stateManager.otherTabs;
+    if (otherTabs.length > 0) {
       context += multipleTabsRule(otherTabs);
-      this.explorer.clearOtherTabsInfo();
+      this.stateManager.otherTabs = [];
     }
 
     if (isNewUrl) {
@@ -641,27 +620,6 @@ export class Tester extends TaskAgent implements Agent {
       }
     }
 
-    // if (isStateChanged) {
-    //   const combinedHtml = await currentState.combinedHtml();
-    //   context += dedent`
-    //     Context (state changed):
-
-    //     <page>
-    //     CURRENT URL: ${currentState.url}
-    //     CURRENT TITLE: ${currentState.title}
-    //     </page>
-
-    //     <page_html>
-    //     ${combinedHtml}
-    //     </page_html>
-
-    //     <page_aria>
-    //     ${currentState.ariaSnapshot}
-    //     </page_aria>
-    //   `;
-    //   return context;
-    // }
-
     if (context) return context;
 
     if (iteration % 5) return '';
@@ -680,38 +638,6 @@ export class Tester extends TaskAgent implements Agent {
     `;
   }
 
-  private async promptLogStep(task: Test): Promise<string> {
-    let logPrompt = dedent`
-      <task>
-        Add a note explaining what you achieved with previous action.
-        Use tools to interact with the page to achieve the scenario goal or expected outcomes.
-        Call record tool to explain the last action
-        Format: record([<action performed>, <what has changed>, <what you expect to do next>]) 
-      </task>
-    `;
-
-    if (task.getPrintableNotes()) {
-      logPrompt = dedent`
-        Your interaction log notes:
-        <notes>
-        ${task.getPrintableNotes()}
-        </notes>
-
-        <rules>
-        Use your previous interaction notes to guide your next actions.
-        Do not perform the same checks.
-        </rules>
-      `;
-    }
-
-    const remaining = task.getRemainingExpectations();
-    if (remaining.length > 0) {
-      logPrompt += `\nExpected steps to check: ${remaining.join(', ')}`;
-    }
-
-    return logPrompt;
-  }
-
   private finishTest(task: Test): void {
     if (!task.hasFinished) {
       task.finish(TestResult.FAILED);
@@ -734,7 +660,7 @@ export class Tester extends TaskAgent implements Agent {
     task.addNote(error.message, TestResult.FAILED, actionResult.screenshotFile, actionResult.fullUrl || actionResult.url);
     task.finish(TestResult.FAILED);
     this.finishTest(task);
-    await this.explorer.stopTest(task, this.buildStopTestMeta(task));
+    await this.testRun?.stop(this.buildStopTestMeta(task));
     clearActivity(true);
     return { success: false };
   }
@@ -810,11 +736,13 @@ export class Tester extends TaskAgent implements Agent {
     - Before retrying your actions check maybe they already achived expected results. Use see() tool for that
     - If the current URL is already a create/edit/new form and the scenario is about creating/editing that entity, fill and submit that form. Do not click the list-page "New" button again from inside the form.
     - If the scenario is about search/filter/sort/tabs/list inspection and the current URL is a create/edit/new form, go back or reset to the stable list page before interacting with list controls.
-    - When selecting related entities from a list, do not choose rows/options/cards marked as "0 items", "0 tests", or otherwise empty if the scenario requires selecting real content.
-    - In selection pickers, counters such as "Selected 0", "Matched tests 0", or disabled Save/Apply mean the selection did not register. Choose a non-empty item or change filters before submitting.
+    - When selecting related entities from a list, do not choose rows/options/cards marked as "0 items", "0 results", or otherwise empty if the scenario requires selecting real content.
+    - In selection pickers, counters such as "Selected 0", "Matched 0", or disabled Save/Apply mean the selection did not register. Choose a non-empty item or change filters before submitting.
     - A passed form/click command only means the command executed. If a required field remains empty, submit stays disabled, or the expected text is not visible, treat the action as not completed and correct the missing field/state.
-    - For filter/tab scenarios, success requires BOTH: the requested filter/tab is visibly active/selected AND the list content matches that filter. Do not finish from only one of these signals.
-    - Empty-state text such as "No matched items" only proves a filter when the requested filter/tab is active and the empty state belongs to the filtered list.
+    - For filter/tab scenarios, success requires BOTH: the requested state is evidenced by a selected control, URL/query, or another explicit state indicator AND the list content matches that state. Do not finish from only one of these signals.
+    - Once the requested control state and matching content are both visible, finish the scenario instead of repeating the interaction. Do not require an aggregate count change unless a baseline was observed immediately before the action.
+    - Empty-state text such as "No matched items" only proves a filter when the requested filter state is explicit and the empty state belongs to the filtered list.
+    - Associate validation feedback with a field only through explicit evidence such as the field label in the message, an accessibility relationship, focus on the invalid control, or visual confirmation. Do not infer the affected field from DOM order or proximity alone.
     - When filling complex form with lot of actions performed, use see() to look which fields were filled and which are not
     - When verify() fails, use see() to visually confirm the result — visual confirmation is equally valid evidence
     - For visual state verification (active tabs, selected items, counts, colors), prefer see() over DOM-based verify()
@@ -847,7 +775,7 @@ export class Tester extends TaskAgent implements Agent {
 
     ${dataProtectionRules}
 
-    ${this.provider.getSystemPromptForAgent('tester', this.explorer.getStateManager().getCurrentState()?.url) || ''}
+    ${this.provider.getSystemPromptForAgent('tester', this.stateManager.getCurrentState()?.url) || ''}
     `;
   }
 
@@ -875,10 +803,6 @@ export class Tester extends TaskAgent implements Agent {
       When creating or editing items via form() or type() you should include ${task.sessionName} in the value (if it is not restricted by the application logic)
       Initial page URL: ${actionResult.url}
 
-      ${capabilityGroundingRule}
-
-      ${dataProtectionRules}
-
       ${this.buildDeletionScope(task)}
 
       ${this.buildAvailableFiles()}
@@ -898,7 +822,7 @@ export class Tester extends TaskAgent implements Agent {
   }
 
   private buildAvailableFiles(): string {
-    const userFiles = this.explorer.getConfig().files || {};
+    const userFiles = this.config.files || {};
     const codeceptDir = (global as any).codecept_dir || process.cwd();
     const lines: string[] = [];
 
@@ -956,11 +880,9 @@ export class Tester extends TaskAgent implements Agent {
           reason: z.string().optional().describe('Explanation why reset is the only option'),
         }),
         execute: async ({ reason }) => {
-          if (this.getCurrentState().isInsideIframe) {
-            await this.explorer.switchToMainFrame();
-          }
+          await this.explorer.action().exitIframe();
 
-          const currentState = this.explorer.getStateManager().getCurrentState();
+          const currentState = this.stateManager.getCurrentState();
           const currentUrl = currentState?.fullUrl || currentState?.url;
           if (currentUrl === resetUrl!) {
             return {
@@ -988,7 +910,7 @@ export class Tester extends TaskAgent implements Agent {
           const explanation = reason ? `${reason} (RESET)` : 'Resetting to initial page';
           const targetUrl = resetUrl!;
           task.addNote(explanation);
-          const resetAction = this.explorer.createAction();
+          const resetAction = this.explorer.action();
           const success = await resetAction.attempt(`I.amOnPage(${JSON.stringify(targetUrl)})`, explanation);
 
           if (success) {
@@ -1137,7 +1059,7 @@ export class Tester extends TaskAgent implements Agent {
             mappedStatus = TestResult.FAILED;
           }
 
-          const screenshotFile = this.explorer.getStateManager().getCurrentState()?.screenshotFile;
+          const screenshotFile = this.stateManager.getCurrentState()?.screenshotFile;
 
           for (const noteText of input.notes) {
             task.addNote(noteText, mappedStatus, screenshotFile);
@@ -1178,7 +1100,7 @@ export class Tester extends TaskAgent implements Agent {
     const message = error instanceof Error ? error.message : String(error);
     if (!task.hasFinished) task.addNote(`Execution error: ${message}`);
 
-    const result = await this.explorer.handleExecutionError(error);
+    const result = await this.explorer.recover(error);
     tag('info').log(`Browser supervisor: ${result.action} - ${result.message}`);
     task.addNote(result.message);
 
@@ -1213,7 +1135,7 @@ export class Tester extends TaskAgent implements Agent {
 
   private async cleanupStartedTest(task: Test): Promise<void> {
     await this.finishTest(task);
-    await this.explorer.stopTest(task, {
+    await this.testRun?.stop({
       startUrl: task.startUrl,
       style: task.style,
       sessionName: task.sessionName,