explorbot 0.1.32-beta.1 → 0.2.0

package/src/ai/historian/playwright.ts

@@ -2,7 +2,7 @@ import { mkdirSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { ActionResult } from '../../action-result.ts';
 import { ConfigParser } from '../../config.ts';
-import { KnowledgeTracker } from '../../knowledge-tracker.ts';
+import type { StateManager } from '../../state-manager.ts';
 import { type PlaywrightRecorder, type TraceCall, renderAssertion, renderCall } from '../../playwright-recorder.ts';
 import type { Plan } from '../../test-plan.ts';
 import { tag } from '../../utils/logger.ts';
@@ -25,6 +25,7 @@ export function WithPlaywright<T extends Constructor>(Base: T) {
   return class extends Base {
     declare playwright: { recorder: PlaywrightRecorder; helper: any } | undefined;
     declare savedFiles: Set<string>;
+    declare stateManager?: StateManager;
 
     async toPlaywrightCode(conversation: Conversation, scenario: string): Promise<string> {
       const toolExecutions = conversation.getToolExecutions();
@@ -146,7 +147,8 @@ export function WithPlaywright<T extends Constructor>(Base: T) {
     }
 
     private getPlaywrightKnowledgeLines(url: string, indent = '    '): string[] {
-      const knowledgeTracker = new KnowledgeTracker();
+      const knowledgeTracker = this.stateManager?.getKnowledgeTracker();
+      if (!knowledgeTracker) return [];
       const state = new ActionResult({ url });
       const { wait, waitForElement } = knowledgeTracker.getStateParameters(state, ['wait', 'waitForElement']);
 

package/src/ai/historian/utils.ts

@@ -1,5 +1,4 @@
 import type { ToolExecution } from '../conversation.ts';
-export { isNonReusableCode, stripComments } from '../../utils/step-analyzer.ts';
 
 export function escapeString(str: string): string {
   return str.replace(/'/g, "\\'").replace(/\n/g, ' ');

package/src/ai/historian.ts

@@ -1,6 +1,6 @@
 import { readFileSync, writeFileSync } from 'node:fs';
 import type { ExplorbotConfig } from '../config.ts';
-import { ExperienceTracker } from '../experience-tracker.ts';
+import type { ExperienceTracker } from '../experience-tracker.ts';
 import type { PlaywrightRecorder } from '../playwright-recorder.ts';
 import type { Reporter } from '../reporter.ts';
 import type { StateManager } from '../state-manager.ts';
@@ -13,8 +13,6 @@ import { type PlaywrightMethods, WithPlaywright } from './historian/playwright.t
 import { type ScreencastMethods, WithScreencast } from './historian/screencast.ts';
 import type { Provider } from './provider.ts';
 
-export { isNonReusableCode } from '../utils/step-analyzer.ts';
-
 const HistorianBase = WithScreencast(WithPlaywright(WithCodeceptJS(WithExperience(Object as unknown as new (...args: any[]) => object))));
 
 export interface Historian extends ExperienceMethods, CodeceptJSMethods, PlaywrightMethods, ScreencastMethods {}
@@ -28,10 +26,10 @@ export class Historian extends HistorianBase {
   declare playwright: { recorder: PlaywrightRecorder; helper: any } | undefined;
   declare savedFiles: Set<string>;
 
-  constructor(provider: Provider, experienceTracker?: ExperienceTracker, reporter?: Reporter, stateManager?: StateManager, config?: ExplorbotConfig, playwright?: { recorder: PlaywrightRecorder; helper: any }) {
+  constructor(provider: Provider, experienceTracker: ExperienceTracker, reporter?: Reporter, stateManager?: StateManager, config?: ExplorbotConfig, playwright?: { recorder: PlaywrightRecorder; helper: any }) {
     super();
     this.provider = provider;
-    this.experienceTracker = experienceTracker || new ExperienceTracker();
+    this.experienceTracker = experienceTracker;
     this.reporter = reporter;
     this.stateManager = stateManager;
     this.config = config;
@@ -52,11 +50,10 @@ export class Historian extends HistorianBase {
     return this.isPlaywrightFramework() ? this.savePlaywrightPlanToFile(plan) : this.saveCodeceptPlanToFile(plan);
   }
 
-  rewriteScenarioInFile(filePath: string, healedSteps: Array<{ test: string; original: string; healed: string }>): void {
+  rewriteScenarioInFile(filePath: string, healedSteps: Array<{ original: string; healed: string }>): void {
     let content = readFileSync(filePath, 'utf-8');
 
     for (const step of healedSteps) {
-      if (!content.includes(step.original)) continue;
       content = content.replace(step.original, step.healed);
     }
 

package/src/ai/navigator.ts

@@ -3,10 +3,10 @@ import dedent from 'dedent';
 import { z } from 'zod';
 import { ActionResult } from '../action-result.js';
 import type Action from '../action.ts';
-import { ExperienceTracker, renderExperienceToc } from '../experience-tracker.js';
+import type { ExperienceTracker } from '../experience-tracker.js';
 import Explorer from '../explorer.ts';
-import { KnowledgeTracker } from '../knowledge-tracker.js';
-import { type WebPageState, normalizeUrl } from '../state-manager.js';
+import type { KnowledgeTracker } from '../knowledge-tracker.js';
+import { normalizeUrl } from '../state-manager.js';
 import { extractCodeBlocks } from '../utils/code-extractor.js';
 import { HooksRunner } from '../utils/hooks-runner.ts';
 import { createDebug, pluralize, tag } from '../utils/logger.js';
@@ -15,19 +15,17 @@ import { RulesLoader } from '../utils/rules-loader.ts';
 import { extractStatePath } from '../utils/url-matcher.js';
 import type { Agent } from './agent.js';
 import type { Conversation } from './conversation.js';
-import { ExperienceCompactor } from './experience-compactor.js';
 import type { Provider } from './provider.js';
 import { Researcher } from './researcher.ts';
 import { actionRule, locatorRule, unexpectedPopupRule } from './rules.js';
 import { isInteractive } from './task-agent.js';
-import { createAgentTools } from './tools.ts';
+import { createLearnExperienceTool } from './tools.ts';
 
 const debugLog = createDebug('explorbot:navigator');
 
 class Navigator implements Agent {
   emoji = '🧭';
   private provider: Provider;
-  private experienceCompactor: ExperienceCompactor;
   private knowledgeTracker: KnowledgeTracker;
   private experienceTracker: ExperienceTracker;
   private hooksRunner: HooksRunner;
@@ -73,12 +71,11 @@ class Navigator implements Agent {
   `;
   private explorer: Explorer;
 
-  constructor(explorer: Explorer, provider: Provider, experienceCompactor: ExperienceCompactor, experienceTracker?: ExperienceTracker) {
+  constructor(explorer: Explorer, provider: Provider) {
     this.provider = provider;
     this.explorer = explorer;
-    this.experienceCompactor = experienceCompactor;
-    this.knowledgeTracker = new KnowledgeTracker();
-    this.experienceTracker = experienceTracker || new ExperienceTracker();
+    this.knowledgeTracker = explorer.getKnowledgeTracker();
+    this.experienceTracker = explorer.getStateManager().getExperienceTracker();
     this.hooksRunner = new HooksRunner(explorer, explorer.getConfig());
   }
 
@@ -194,19 +191,9 @@ class Navigator implements Agent {
     const action = opts?.action ?? this.explorer.createAction();
     const expectedUrl = opts?.expectedUrl;
 
-    let knowledge = '';
+    const knowledge = this.knowledgeTracker.renderRelevantKnowledge(actionResult);
     let experience = '';
 
-    const relevantKnowledge = this.knowledgeTracker.getRelevantKnowledge(actionResult);
-    if (relevantKnowledge.length > 0) {
-      const knowledgeContent = relevantKnowledge.map((k) => k.content).join('\n\n');
-      knowledge = `
-      <hint>
-      Here is relevant knowledge for this page:
-      ${knowledgeContent}
-      </hint>`;
-    }
-
     if (!actionResult.isInsideIframe) {
       const successful = this.experienceTracker.getSuccessfulExperience(actionResult);
       if (successful.length > 0) {
@@ -397,7 +384,6 @@ class Navigator implements Agent {
             if (freshState.getStateHash() !== prevHash) {
               try {
                 const diff = await freshState.diff(prevActionResult);
-                await diff.calculate();
                 ariaChanges = diff.ariaChanged;
               } catch (err) {
                 debugLog('Failed to compute pageDiff for failed URL verification:', err);
@@ -486,20 +472,13 @@ class Navigator implements Agent {
   }
 
   private buildExperienceTools(): { learnExperience: unknown } | undefined {
+    if (!this.experienceTracker) return undefined;
     const stateManager = this.explorer.getStateManager();
     const getState = () => {
       const s = stateManager.getCurrentState();
       return s ? ActionResult.fromState(s) : null;
     };
-    const { learnExperience } = createAgentTools({
-      explorer: this.explorer,
-      researcher: null as unknown as Researcher,
-      navigator: this,
-      experienceTracker: this.experienceTracker,
-      getState,
-    });
-    if (!learnExperience) return undefined;
-    return { learnExperience };
+    return { learnExperience: createLearnExperienceTool({ getExperienceTracker: () => this.experienceTracker, getState }) };
   }
 
   async freeSail(opts?: { strategy?: 'deep' | 'shallow'; scope?: string; visitedUrls?: Set<string> }, actionResult?: ActionResult): Promise<{ target: string; reason: string } | null> {
@@ -641,26 +620,11 @@ class Navigator implements Agent {
       return { verified: cachedVerification, successfulCodes: [], assertionSteps: [], totalAttempted: 0 };
     }
 
-    let knowledge = '';
+    const knowledge = this.knowledgeTracker.renderRelevantKnowledge(actionResult);
     let experience = '';
 
-    const relevantKnowledge = this.knowledgeTracker.getRelevantKnowledge(actionResult);
-    if (relevantKnowledge.length > 0) {
-      const knowledgeContent = relevantKnowledge.map((k) => k.content).join('\n\n');
-      knowledge = `
-      <hint>
-      Here is relevant knowledge for this page:
-      ${knowledgeContent}
-      </hint>`;
-    }
-
     if (!actionResult.isInsideIframe) {
-      const toc = this.experienceTracker.getExperienceTableOfContents(actionResult);
-      if (toc.length > 0) {
-        const totalSections = toc.reduce((sum, entry) => sum + entry.sections.length, 0);
-        tag('operation').log(`Found ${toc.length} experience ${pluralize(toc.length, 'file')} (${totalSections} sections) for: ${actionResult.url}`);
-        experience = renderExperienceToc(toc);
-      }
+      experience = this.experienceTracker.renderExperienceTocFor(actionResult);
     }
 
     const priorVerifications = Object.entries(actionResult.verifications ?? {});
@@ -764,7 +728,7 @@ class Navigator implements Agent {
 
           await this.explorer.switchToMainFrame();
 
-          const verified = await action.attempt(codeBlock, message, false);
+          const verified = await action.attempt(codeBlock, message);
 
           if (verified) {
             tag('success').log('Verification passed');

package/src/ai/pilot.ts

@@ -3,7 +3,6 @@ import dedent from 'dedent';
 import { z } from 'zod';
 import { ActionResult } from '../action-result.ts';
 import { ConfigParser } from '../config.ts';
-import { type ExperienceTracker, renderExperienceToc } from '../experience-tracker.ts';
 import type Explorer from '../explorer.ts';
 import { type Test, TestResult } from '../test-plan.ts';
 import { collectInteractiveNodes, detectFocusArea, extractFocusedElement } from '../utils/aria.ts';
@@ -32,14 +31,12 @@ export class Pilot implements Agent {
   private researcher: Researcher;
   private explorer: Explorer;
   private fisherman: Fisherman | null = null;
-  private experienceTracker: ExperienceTracker | null;
 
-  constructor(provider: Provider, agentTools: any, researcher: Researcher, explorer: Explorer, experienceTracker?: ExperienceTracker) {
+  constructor(provider: Provider, agentTools: any, researcher: Researcher, explorer: Explorer) {
     this.provider = provider;
     this.agentTools = agentTools;
     this.researcher = researcher;
     this.explorer = explorer;
-    this.experienceTracker = experienceTracker || null;
   }
 
   setFisherman(fisherman: Fisherman): void {
@@ -111,7 +108,7 @@ export class Pilot implements Agent {
         .string()
         .nullable()
         .describe(
-          'REQUIRED whenever decision is "pass" — a one-sentence natural-language claim about the current page that, if true, proves the scenario goal (e.g., "New test suite \\"Foo\\" is visible in the suites list"). NOT code: do not write I.*, expect(), .then(), grabTitle, or any JavaScript. Navigator translates the claim into CodeceptJS assertions and runs them; passing assertions are saved to the generated test file. Also use when evidence is insufficient before deciding pass/fail. Leave null for "continue", "fail", or "skipped".'
+          'REQUIRED whenever decision is "pass" — a one-sentence natural-language claim about the current page that, if true, proves the scenario goal (e.g., "New item \\"Foo\\" is visible in the items list"). NOT code: do not write I.*, expect(), .then(), grabTitle, or any JavaScript. Navigator translates the claim into CodeceptJS assertions and runs them; passing assertions are saved to the generated test file. Also use when evidence is insufficient before deciding pass/fail. Leave null for "continue", "fail", or "skipped".'
         ),
     });
 
@@ -147,7 +144,7 @@ export class Pilot implements Agent {
       - Mixed evidence + final state shows success → pass. Mixed + final state unclear → continue with guidance.
 
       When deciding "pass", you MUST also set requestVerification to a one-sentence natural-language
-      claim about the current page (e.g., "New test suite Foo is visible in the suites list"). NOT
+      claim about the current page (e.g., "New item Foo is visible in the items list"). NOT
       code — do not write I.*, expect(), .then(), or any JavaScript. Choose the strongest single
       piece of evidence (a unique element/text that exists ONLY because the scenario succeeded).
       Navigator translates the claim into CodeceptJS assertions; without it the generated test has
@@ -429,10 +426,6 @@ export class Pilot implements Agent {
 
         FIRST: Decide if precondition() is needed.
 
-        ${capabilityGroundingRule}
-
-        ${dataProtectionRules}
-
         Call precondition() WHEN:
         - The scenario edits/deletes/modifies an item, and you want a DISPOSABLE item to act on safely
         - The scenario needs specific data clearly NOT on the current page (e.g., items with specific statuses for filtering)
@@ -603,12 +596,9 @@ export class Pilot implements Agent {
   }
 
   private getExperienceToc(): string {
-    if (!this.experienceTracker) return '';
     const state = this.explorer.getStateManager().getCurrentState();
     if (!state) return '';
-    const actionResult = ActionResult.fromState(state);
-    const toc = this.experienceTracker.getExperienceTableOfContents(actionResult);
-    return renderExperienceToc(toc);
+    return this.explorer.getStateManager().getExperienceTracker().renderExperienceTocFor(ActionResult.fromState(state));
   }
 
   private pickPlanningTools() {
@@ -1056,8 +1046,8 @@ export class Pilot implements Agent {
 
       ${dataProtectionRules}
 
-      Describe WHAT to create, not what exists. RIGHT: precondition("1 test"). WRONG:
-      precondition("1 test suite named Updated Suite with existing tests"). Keep descriptions short.
+      Describe WHAT to create, not what exists. RIGHT: precondition("1 post"). WRONG:
+      precondition("1 post named Updated Post with existing comments"). Keep descriptions short.
 
       Response format:
       PROGRESS: <1 sentence assessment>

package/src/ai/planner.ts

@@ -3,7 +3,7 @@ import { z } from 'zod';
 import { ActionResult } from '../action-result.ts';
 import { setActivity } from '../activity.ts';
 import { ConfigParser } from '../config.ts';
-import { ExperienceTracker } from '../experience-tracker.ts';
+import type { ExperienceTracker } from '../experience-tracker.ts';
 import type Explorer from '../explorer.ts';
 import { Observability } from '../observability.ts';
 import type { StateManager } from '../state-manager.js';
@@ -18,10 +18,10 @@ import type { Agent } from './agent.js';
 import { Conversation } from './conversation.ts';
 import type { Fisherman } from './fisherman.ts';
 import { WithSessionDedup } from './planner/session-dedup.ts';
-import { getActiveStyle, getStyles } from './planner/styles.ts';
+import { getActiveStyle } from './planner/styles.ts';
 import { WithSubPages, getPlannedByStateHash, getRegisteredPlan, registerPlan } from './planner/subpages.ts';
 import type { Provider } from './provider.js';
-import { POSSIBLE_SECTIONS, Researcher } from './researcher.ts';
+import { POSSIBLE_SECTIONS, type Researcher } from './researcher.ts';
 import { findSimilarStateHash } from './researcher/cache.ts';
 import { hasFocusedSection } from './researcher/focus.ts';
 import { capabilityGroundingRule, dataProtectionRules, fileUploadRule } from './rules.ts';
@@ -63,13 +63,13 @@ export class Planner extends PlannerBase implements Agent {
   researcher: Researcher;
   private fisherman: Fisherman | null = null;
 
-  constructor(explorer: Explorer, provider: Provider) {
+  constructor(explorer: Explorer, provider: Provider, researcher: Researcher) {
     super();
     this.explorer = explorer;
     this.provider = provider;
-    this.researcher = new Researcher(explorer, provider);
+    this.researcher = researcher;
     this.stateManager = explorer.getStateManager();
-    this.experienceTracker = new ExperienceTracker();
+    this.experienceTracker = explorer.getStateManager().getExperienceTracker();
   }
 
   setFisherman(fisherman: Fisherman): void {
@@ -98,7 +98,7 @@ export class Planner extends PlannerBase implements Agent {
     You are ISTQB certified senior manual QA planning exploratory testing session of a web application.
     Each test scenario must complete a meaningful user workflow — not just open a UI element and verify it exists.
     Bad: "Open the Help panel" — just opens something.
-    Good: "Create a new suite and verify it appears in the list" — completes a business action.
+    Good: "Create a new post and verify it appears in the list" — completes a business action.
     </role>
     <task>
       List possible testing scenarios for the web page.
@@ -171,6 +171,9 @@ export class Planner extends PlannerBase implements Agent {
 
     const tags = ['planner'];
     if (style) tags.push(style);
+
+    const existingTestScenarios = this.getExistingTestFileScenarios(state.url);
+
     const result = await Observability.run(`planner: ${state.url}`, { tags, sessionId: state.url }, async () => {
       const actionResult = ActionResult.fromState(state);
       const conversation = await this.buildConversation(actionResult, style, parentPlan, feature);
@@ -211,7 +214,6 @@ export class Planner extends PlannerBase implements Agent {
       const defaultStartUrl = this.getDefaultStartUrl(state);
       if (parentPlan) this.currentPlan.parentPlan = parentPlan;
       const allPreviousScenarios = this.getPreviousSessionScenarios();
-      const existingTestScenarios = this.getExistingTestFileScenarios(state.url);
       for (const s of existingTestScenarios) allPreviousScenarios.add(s);
       for (const t of tests) {
         if (allPreviousScenarios.has(t.scenario.toLowerCase())) continue;
@@ -266,6 +268,7 @@ export class Planner extends PlannerBase implements Agent {
   }
 
   private getExistingTestFileScenarios(currentUrl?: string): Set<string> {
+    this.lastSuite = null;
     if (!currentUrl) return new Set<string>();
     try {
       this.lastSuite = new Suite(currentUrl);
@@ -277,29 +280,36 @@ export class Planner extends PlannerBase implements Agent {
   }
 
   private cleanExperienceFlows(text: string): string | null {
-    const seenTitles = new Set<string>();
     let result = text;
+    const seenTitles = new Set<string>();
+    for (const selector of ['section2', 'section3']) {
+      result = mdq(result)
+        .query(selector)
+        .replaceEach((section) => {
+          const heading = section.query('heading').text().trim();
+          const withoutHeadings = mdq(section.text()).query('heading').replace('');
+          const body = mdq(withoutHeadings).query('hr').replace('').trim();
+          if (body && !seenTitles.has(heading)) {
+            seenTitles.add(heading);
+            return section.text();
+          }
+          return '';
+        });
+    }
 
-    for (const section of [...mdq(text).query('section2').each(), ...mdq(text).query('section3').each()]) {
-      const heading = section.query('heading').text().trim();
-      const body = mdq(section.text())
-        .query('heading')
-        .replace('')
-        .replace(/^---\s*$/gm, '')
-        .trim();
-
-      if (!body || seenTitles.has(heading)) {
-        result = result.replace(section.text(), '');
-        continue;
-      }
-      seenTitles.add(heading);
-
-      const blockquotes = section.query('blockquote').each();
-      if (blockquotes.length <= 10) continue;
-      for (const bq of blockquotes.slice(10)) {
-        result = result.replace(bq.text(), '');
-      }
-      result = result.replace(section.text().trim(), `${section.text().trim()}\n> ... and ${blockquotes.length - 10} more discoveries`);
+    const trimmedTitles = new Set<string>();
+    for (const selector of ['section2', 'section3']) {
+      result = mdq(result)
+        .query(selector)
+        .replaceEach((section) => {
+          const heading = section.query('heading').text().trim();
+          if (trimmedTitles.has(heading)) return section.text();
+          const count = section.query('blockquote').count();
+          if (count <= 10) return section.text();
+          const kept = mdq(section.text()).query('blockquote[10:]').replace('');
+          trimmedTitles.add(heading);
+          return `${kept.trimEnd()}\n> ... and ${count - 10} more discoveries\n`;
+        });
     }
 
     return result.trim() || null;
@@ -338,6 +348,7 @@ export class Planner extends PlannerBase implements Agent {
       If there are subpages (pages with same URL path) plan testing of those subpages as well
       If you plan to test CRUD operations, plan them in correct order: create, read, update.
       Do not invent specific route names, success messages, validation texts, badge counts, or welcome messages unless they are visible in research, visited pages, or prior observed flows.
+      When validation placement or wording was not observed, require feedback associated with the invalid input without inventing a specific location or message.
       If exact wording is unknown, describe the expected result generically, for example "an authentication error is shown" or "the user stays on the login page" instead of guessing the literal text.
       If exact redirect destination is unknown, describe the destination by visible page identity, for example "the dashboard page opens" or "the current workspace home page opens" instead of inventing a URL slug.
       Only propose scenarios whose prerequisites are evident from page research, visited pages, or API data preparation context.
@@ -378,16 +389,17 @@ export class Planner extends PlannerBase implements Agent {
       deep: true,
     });
     let plannerResearch = mdq(research).query('code').replace('');
-    for (const table of mdq(plannerResearch).query('table').each()) {
-      const rawTable = table.text();
-      const rows = table.toJson();
-      if (rows.length === 0 || !rows[0].Element) continue;
-      const elementWithType = rows.map((r) => ({
-        Element: r.Element,
-        Type: r.Type || '',
-      }));
-      plannerResearch = plannerResearch.replace(rawTable, jsonToTable(elementWithType, ['Element', 'Type']));
-    }
+    plannerResearch = mdq(plannerResearch)
+      .query('table')
+      .replaceEach((table) => {
+        const rows = table.toJson();
+        if (rows.length === 0 || !rows[0].Element) return table.text();
+        const elementWithType = rows.map((r) => ({
+          Element: r.Element,
+          Type: r.Type || '',
+        }));
+        return jsonToTable(elementWithType, ['Element', 'Type']);
+      });
 
     const hasFocusedOverlay = hasFocusedSection(plannerResearch);
     const focusNote = hasFocusedOverlay ? "IMPORTANT: One section is marked as **Focused** — this is the user's current focus area. Concentrate testing on the Focused section FIRST — test all interactions inside it before planning tests for the rest of the page." : '';
@@ -576,17 +588,17 @@ export class Planner extends PlannerBase implements Agent {
 
          STEPS - actionable commands:
          - Each step should be a specific action to perform
-         - Good: "Click Create Suite button", "Enter 'My New Suite' as suite name", "Click Save button"
-         - Good: "Click Star icon on 'Template API Testing' suite"
+         - Good: "Click Create Post button", "Enter 'My New Post' as post title", "Click Save button"
+         - Good: "Click Star icon on 'Quarterly Report' item"
          - Bad: "Open the dropdown menu" (opening is not a goal, it's a means)
          - Bad: "Verify modal appears" (verification belongs in expected outcomes, not steps)
          - Steps should form a complete workflow, not stop at opening a UI element
          - NEVER split a workflow across two tests. One test = one complete action + its verification
 
          EXPECTED OUTCOMES - verifiable results:
-         - Good: "New suite 'My New Suite' appears in the suite list"
-         - Good: "Suite appears under Starred filter tab"
-         - Good: "Success message 'Suite created' is displayed"
+         - Good: "New post 'My New Post' appears in the posts list"
+         - Good: "Item appears under Starred filter tab"
+         - Good: "Success message 'Post created' is displayed"
          - Good when wording is unknown: "An authentication error is displayed"
          - Good when route is unknown: "The workspace home page is displayed"
          - Bad: "Modal is displayed" (just verifying existence, no business value)