explorbot 0.4.9 → 0.4.10

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "explorbot",
-  "version": "0.4.9",
+  "version": "0.4.10",
   "description": "CLI app built with React Ink, CodeceptJS, and Playwright",
   "license": "Elastic-2.0",
   "type": "module",

package/dist/src/action.js

@@ -12,7 +12,9 @@ import { captureHtmlForSnapshot, htmlCombinedSnapshot, minifyHtml } from './util
 import { createDebug, setStepSpanParent, tag } from './utils/logger.js';
 import { Overlay, OverlayPage } from './utils/overlay.js';
 import { sleep, waitForPageReadiness } from "./utils/page-readiness.js";
+import { isInternalStep } from "./utils/step-analyzer.js";
 import { safeFilename } from "./utils/strings.js";
+import { isSameHostFamily } from './utils/url-matcher.js';
 import { codeceptJSSandbox, hasPlaywrightCommands, playwrightSandbox, sanitizeCodeBlock } from "./utils/web-sandbox.js";
 const debugLog = createDebug('explorbot:action');
 const CAPTURE_NAVIGATION_TRANSITION_ATTEMPTS = 3;
@@ -303,7 +305,7 @@ class Action {
         const url = URL.parse(request.url());
         if (!url)
             return;
-        if (url.origin !== this.baseOrigin)
+        if (!isSameHostFamily(url.href, this.baseOrigin))
             return;
         const call = { method: request.method(), path: url.pathname, status };
         if (this.networkRequests.some((r) => r.method === call.method && r.path === call.path && r.status === call.status))
@@ -523,7 +525,7 @@ export const attachStepLogger = (target, assertionsTarget) => {
     const listener = (step, error) => {
         if (!step?.toCode)
             return;
-        if (step.name?.startsWith('grab'))
+        if (isInternalStep(step))
             return;
         const existing = recorded.get(step);
         if (existing) {

package/dist/src/ai/navigator.d.ts

@@ -26,7 +26,6 @@ declare class Navigator implements Agent {
     constructor(deps: AgentDeps);
     get verifyAttempts(): number;
     get verifyTimeout(): number;
-    getBaseOrigin(): string | null;
     getComparableCurrentUrl(stateManager: any, expectedUrl: string): string;
     comparableUrl(state: {
         url?: string;

package/dist/src/ai/navigator.js

@@ -12,7 +12,7 @@ import { createDebug, pluralize, tag } from '../utils/logger.js';
 import { loop, pause } from '../utils/loop.js';
 import { RulesLoader } from "../utils/rules-loader.js";
 import { normalizeInlineText } from "../utils/strings.js";
-import { extractStatePath, matchesNavigationUrl } from '../utils/url-matcher.js';
+import { extractStatePath, isSameHostFamily, matchesNavigationUrl } from '../utils/url-matcher.js';
 import { Researcher } from "./researcher.js";
 import { actionRule, locatorRule, unexpectedPopupRule } from './rules.js';
 import { isInteractive } from './task-agent.js';
@@ -82,15 +82,6 @@ class Navigator {
     get verifyTimeout() {
         return this.config.ai?.agents?.navigator?.verifyTimeout ?? 1500;
     }
-    getBaseOrigin() {
-        const baseUrl = this.config.playwright.url;
-        try {
-            return new URL(baseUrl).origin;
-        }
-        catch {
-            return null;
-        }
-    }
     getComparableCurrentUrl(stateManager, expectedUrl) {
         const currentState = stateManager.getCurrentState();
         if (!currentState)
@@ -109,19 +100,14 @@ class Navigator {
         const currentFullUrl = currentState.fullUrl || currentState.url || '';
         if (!currentFullUrl)
             return false;
-        try {
-            const currentOrigin = new URL(currentFullUrl).origin;
-            if (/^https?:\/\//i.test(expectedUrl)) {
-                return currentOrigin === new URL(expectedUrl).origin;
-            }
-            const baseOrigin = this.getBaseOrigin();
-            if (!baseOrigin)
-                return true;
-            return currentOrigin === baseOrigin;
-        }
-        catch {
+        if (!/^https?:\/\//i.test(currentFullUrl))
             return !/^https?:\/\//i.test(expectedUrl);
-        }
+        if (/^https?:\/\//i.test(expectedUrl))
+            return isSameHostFamily(currentFullUrl, expectedUrl);
+        const baseUrl = this.config.playwright.url;
+        if (!baseUrl)
+            return true;
+        return isSameHostFamily(currentFullUrl, baseUrl);
     }
     isOnExpectedPage(expectedUrl, stateManager) {
         if (!this.isSameExpectedOrigin(expectedUrl, stateManager)) {
@@ -289,8 +275,9 @@ class Navigator {
                         tag('warning').log(`Page state did not change at ${check.freshState.url}`);
                     }
                     else {
-                        lastFailure = `Reached ${check.freshState.url}, expected ${expectedUrl}`;
-                        tag('warning').log(`URL verification failed: expected ${expectedUrl}, got ${check.freshState.url}`);
+                        const reachedUrl = check.freshState.fullUrl || check.freshState.url;
+                        lastFailure = `Reached ${reachedUrl}, expected ${expectedUrl}`;
+                        tag('warning').log(`URL verification failed: expected ${expectedUrl}, got ${reachedUrl}`);
                     }
                     batchFailures.push({
                         code: codeBlock,

package/dist/src/ai/pilot.js

@@ -95,7 +95,9 @@ export class Pilot {
             }
         }
         const schema = z.object({
-            decision: z.enum(['pass', 'fail', 'continue', 'skipped']).describe('pass = test succeeded, fail = test failed, continue = tester should keep going, skipped = scenario is irrelevant OR systematic execution failures prevented testing'),
+            decision: z
+                .enum(['pass', 'fail', 'continue', 'skipped'])
+                .describe('pass = scenario goal accomplished, fail = the app misbehaved, continue = tester should keep going, skipped = the scenario cannot be judged against this app (its premise does not hold, it is irrelevant, or systematic execution failures prevented testing)'),
             reason: z.string().describe('Concise user-facing reason, maximum 1 short sentence and 120 characters. Do NOT repeat the decision status; explain only the evidence. For continue: explain why rejected and suggest alternatives.'),
             guidance: z.string().nullable().describe('Required for "continue": specific actionable instruction for the tester — what exactly to verify, retry differently, or complete next. Be concrete.'),
             requestVerification: z
@@ -358,9 +360,13 @@ export class Pilot {
         DOM assertion can't be made.
         Do not pass when Tester achieved only a related navigation/filter/tab/status outcome instead of the
         requested action, workflow, or entity detail goal.
-      - "fail": goal not achieved and no further step toward it is available on the current page.
-      - "skipped": scenario is irrelevant to the app, OR systematic infrastructure failures (LLM errors,
-        crashes) prevented testing. NOT for "test failed to interact" — that's "fail" or "continue".
+      - "fail": the app misbehaved — the scenario's action ran against the right target and the app
+        produced a wrong, broken, or missing outcome. Not reaching the goal is not by itself a fail.
+      - "skipped": the scenario cannot be judged against this app — the page shows its premise does not
+        hold (the assumed constraint, field, or behaviour is designed differently), the target entity or
+        feature is not the one here, the scenario is irrelevant, OR systematic infrastructure failures
+        (LLM errors, crashes) prevented testing. NOT for "test failed to interact" — that's "fail" or
+        "continue".
       - "continue": goal incomplete but the control for the NEXT step is present on the current page, or a
         concrete missing check would change your verdict. Guidance must name that step.
         If a verify() asserted a state that was ALREADY TRUE before the test, it proves nothing — reject.

package/dist/src/ai/rerunner.js

@@ -18,6 +18,7 @@ import { formatHeadings } from "../utils/context-formatter.js";
 import { createDebug, tag } from "../utils/logger.js";
 import { loop } from "../utils/loop.js";
 import { RulesLoader } from "../utils/rules-loader.js";
+import { isInternalStep } from "../utils/step-analyzer.js";
 import { toolExecutionLabel } from "./conversation.js";
 import { actionRule, locatorRule, sectionContextRule } from "./rules.js";
 import { TaskAgent } from "./task-agent.js";
@@ -70,6 +71,8 @@ export class Rerunner extends TaskAgent {
         const onStepStarted = (step) => {
             if (!step.toCode)
                 return;
+            if (isInternalStep(step))
+                return;
             const code = highlight(step.toCode(), { language: 'javascript' });
             console.log(chalk.dim(`    ${code}`));
         };
@@ -77,12 +80,16 @@ export class Rerunner extends TaskAgent {
             const task = this.getCurrentTask(testMap);
             if (!task || !step.toCode)
                 return;
+            if (isInternalStep(step))
+                return;
             task.addStep(step.toCode(), step.duration, 'passed');
         };
         const onStepFailed = (step, error) => {
             const task = this.getCurrentTask(testMap);
             if (!task || !step.toCode)
                 return;
+            if (isInternalStep(step))
+                return;
             task.addStep(step.toCode(), step.duration, 'failed', error?.message);
             console.log(chalk.red(`    ${figureSet.cross} ${step.toCode()} — ${error?.message || 'failed'}`));
         };

package/dist/src/api/xhr-capture.js

@@ -1,3 +1,4 @@
+import { isSameHostFamily } from '../utils/url-matcher.js';
 import { RequestResult, generateRequestId } from "./request-result.js";
 const WRITE_METHODS = new Set(['POST', 'PUT', 'PATCH', 'DELETE']);
 const JSON_CONTENT_TYPES = /application\/json|application\/.*\+json/i;
@@ -33,7 +34,7 @@ export class XhrCapture {
             return;
         const method = request.method();
         const url = request.url();
-        if (!url.startsWith(this.baseOrigin))
+        if (!isSameHostFamily(url, this.baseOrigin))
             return;
         const status = response.status();
         if (status >= 400) {

package/dist/src/explorer.js

@@ -13,6 +13,7 @@ import { ConfigParser } from './config.js';
 import { BrowserRecoveryError, browserErrorMessage, isFatalBrowserError, isNavigationTransitionError } from "./utils/browser-errors.js";
 import { createDebug, log, tag } from './utils/logger.js';
 import { sleep, waitForPageReadiness } from "./utils/page-readiness.js";
+import { isInternalStep } from "./utils/step-analyzer.js";
 const debugLog = createDebug('explorbot:explorer');
 const RECOVERABLE_NAVIGATION_ERRORS = /net::ERR_ABORTED|page\.screenshot.*Timeout|waiting for fonts to load/i;
 const RECOVERY_NAVIGATION = { waitUntil: 'domcontentloaded', timeout: 10000 };
@@ -161,9 +162,7 @@ class Explorer {
         const stepHandler = (step, status, error, log) => {
             if (!step.toCode)
                 return;
-            if (step?.name?.startsWith('grab'))
-                return;
-            if (step?.name?.startsWith('save'))
+            if (isInternalStep(step))
                 return;
             test.addStep(step.toCode(), step.duration, status, error, log);
             if (!this.stateManager.getCurrentState())

package/dist/src/reporter.js

@@ -111,6 +111,7 @@ export class Reporter {
     }
     combineStepsAndNotes(test, lastScreenshotFile) {
         const noteEntries = Object.entries(test.notes)
+            .filter(([, note]) => !note.observation)
             .map(([timestampKey, note]) => ({
             startTime: note.startTime,
             endTime: note.endTime,
@@ -233,9 +234,12 @@ export class Reporter {
                 description: test.description,
                 code: test.generatedCode || '',
                 steps,
-                logs: Object.values(test.steps)
-                    .map((stepData) => stepData.text)
-                    .join('\n'),
+                logs: [
+                    ...Object.values(test.steps).map((stepData) => stepData.text),
+                    ...Object.values(test.notes)
+                        .filter((note) => note.observation)
+                        .map((note) => note.message),
+                ].join('\n'),
                 files: Object.values(test.artifacts) || [],
                 message: test.summary || this.extractLastNoteMessage(test) || '',
                 meta,
@@ -300,7 +304,7 @@ export class Reporter {
         }
     }
     extractLastNoteMessage(test) {
-        const notes = Object.values(test.notes);
+        const notes = Object.values(test.notes).filter((note) => !note.observation);
         if (notes.length === 0)
             return '';
         return notes[notes.length - 1].message;

package/dist/src/utils/logger.js

@@ -189,7 +189,7 @@ class SpanDestination {
         if (!step?.toCode) {
             return;
         }
-        const stepName = step?.name ? `I.${step.name}` : 'I.step';
+        const stepName = step?.title ? `I.${step.title}` : 'I.step';
         const stepInput = typeof step?.toCode === 'function' ? step.toCode() : entry.content;
         const errorFromStep = step?.error;
         const errorMessage = stepError && typeof stepError === 'object' && 'message' in stepError && typeof stepError.message === 'string'

package/dist/src/utils/step-analyzer.d.ts

@@ -9,3 +9,6 @@ export declare function stripComments(code: string): string;
 export declare function isNonReusableCode(code: string): boolean;
 export declare function toReusableSessionStep(step: StepData): SessionStep | null;
 export declare function mergeUniqueStepsByCode(primary: SessionStep[], secondary: SessionStep[]): SessionStep[];
+export declare function isInternalStep(step: {
+    title?: string;
+}): boolean;

package/dist/src/utils/step-analyzer.js

@@ -1,5 +1,6 @@
 import { isDynamicId } from "./xpath.js";
 export const CODECEPT_TOOLS = ['click', 'hover', 'pressKey', 'form'];
+const INTERNAL_STEP_PREFIXES = ['grab', 'save'];
 const CODECEPT_FORM_COMMANDS = ['I.fillField', 'I.type', 'I.selectOption', 'I.attachFile', 'I.checkOption', 'I.uncheckOption'];
 export function isCodeceptToolName(toolName) {
     return CODECEPT_TOOLS.includes(toolName);
@@ -66,3 +67,9 @@ export function mergeUniqueStepsByCode(primary, secondary) {
     }
     return merged;
 }
+export function isInternalStep(step) {
+    const title = step?.title;
+    if (!title)
+        return false;
+    return INTERNAL_STEP_PREFIXES.some((prefix) => title.startsWith(prefix));
+}

package/dist/src/utils/url-matcher.d.ts

@@ -6,3 +6,4 @@ export declare function generalizeUrl(url: string, replaceSegment?: (segment: st
 export declare function matchesUrl(pattern: string, path: string): boolean;
 export declare function extractStatePath(url: string): string;
 export declare function matchesNavigationUrl(expected: string, current: string): boolean;
+export declare function isSameHostFamily(urlA: string, urlB: string): boolean;

package/dist/src/utils/url-matcher.js

@@ -145,3 +145,10 @@ export function matchesNavigationUrl(expected, current) {
         .filter(Boolean);
     return recordSegments.length > 0 && recordSegments.every(isDynamicSegment);
 }
+export function isSameHostFamily(urlA, urlB) {
+    const hostA = URL.parse(urlA)?.host.toLowerCase();
+    const hostB = URL.parse(urlB)?.host.toLowerCase();
+    if (!hostA || !hostB)
+        return false;
+    return hostA === hostB || hostA.endsWith(`.${hostB}`) || hostB.endsWith(`.${hostA}`);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "explorbot",
-  "version": "0.4.9",
+  "version": "0.4.10",
   "description": "CLI app built with React Ink, CodeceptJS, and Playwright",
   "license": "Elastic-2.0",
   "type": "module",

package/src/action.ts

@@ -16,7 +16,9 @@ import { createDebug, setStepSpanParent, tag } from './utils/logger.js';
 import { Overlay, OverlayPage } from './utils/overlay.js';
 import { sleep, waitForPageReadiness } from './utils/page-readiness.ts';
 import type { Region } from './utils/region.js';
+import { isInternalStep } from './utils/step-analyzer.ts';
 import { safeFilename } from './utils/strings.ts';
+import { isSameHostFamily } from './utils/url-matcher.js';
 import { codeceptJSSandbox, hasPlaywrightCommands, playwrightSandbox, sanitizeCodeBlock } from './utils/web-sandbox.ts';
 
 const debugLog = createDebug('explorbot:action');
@@ -317,7 +319,7 @@ class Action {
 
     const url = URL.parse(request.url());
     if (!url) return;
-    if (url.origin !== this.baseOrigin) return;
+    if (!isSameHostFamily(url.href, this.baseOrigin)) return;
 
     const call: NetworkCall = { method: request.method(), path: url.pathname, status };
     if (this.networkRequests.some((r) => r.method === call.method && r.path === call.path && r.status === call.status)) return;
@@ -563,7 +565,7 @@ export const attachStepLogger = (target: ExecutedStep[], assertionsTarget?: Arra
   let batchFailed = false;
   const listener: StepListener = (step, error) => {
     if (!step?.toCode) return;
-    if (step.name?.startsWith('grab')) return;
+    if (isInternalStep(step)) return;
 
     const existing = recorded.get(step);
     if (existing) {

package/src/ai/navigator.ts

@@ -18,7 +18,7 @@ import { createDebug, pluralize, tag } from '../utils/logger.js';
 import { loop, pause } from '../utils/loop.js';
 import { RulesLoader } from '../utils/rules-loader.ts';
 import { normalizeInlineText } from '../utils/strings.ts';
-import { extractStatePath, matchesNavigationUrl } from '../utils/url-matcher.js';
+import { extractStatePath, isSameHostFamily, matchesNavigationUrl } from '../utils/url-matcher.js';
 import type { Agent, AgentDeps } from './agent.js';
 import type { Conversation } from './conversation.js';
 import type { Provider } from './provider.js';
@@ -99,15 +99,6 @@ class Navigator implements Agent {
     return this.config.ai?.agents?.navigator?.verifyTimeout ?? 1500;
   }
 
-  private getBaseOrigin(): string | null {
-    const baseUrl = this.config.playwright.url;
-    try {
-      return new URL(baseUrl).origin;
-    } catch {
-      return null;
-    }
-  }
-
   private getComparableCurrentUrl(stateManager: any, expectedUrl: string): string {
     const currentState = stateManager.getCurrentState();
     if (!currentState) return '';
@@ -126,18 +117,12 @@ class Navigator implements Agent {
     const currentFullUrl = currentState.fullUrl || currentState.url || '';
     if (!currentFullUrl) return false;
 
-    try {
-      const currentOrigin = new URL(currentFullUrl).origin;
-      if (/^https?:\/\//i.test(expectedUrl)) {
-        return currentOrigin === new URL(expectedUrl).origin;
-      }
+    if (!/^https?:\/\//i.test(currentFullUrl)) return !/^https?:\/\//i.test(expectedUrl);
+    if (/^https?:\/\//i.test(expectedUrl)) return isSameHostFamily(currentFullUrl, expectedUrl);
 
-      const baseOrigin = this.getBaseOrigin();
-      if (!baseOrigin) return true;
-      return currentOrigin === baseOrigin;
-    } catch {
-      return !/^https?:\/\//i.test(expectedUrl);
-    }
+    const baseUrl = this.config.playwright.url;
+    if (!baseUrl) return true;
+    return isSameHostFamily(currentFullUrl, baseUrl);
   }
 
   private isOnExpectedPage(expectedUrl: string, stateManager: any): boolean {
@@ -325,8 +310,9 @@ class Navigator implements Agent {
               lastFailure = `Reached ${check.freshState.url} but the page state did not change`;
               tag('warning').log(`Page state did not change at ${check.freshState.url}`);
             } else {
-              lastFailure = `Reached ${check.freshState.url}, expected ${expectedUrl}`;
-              tag('warning').log(`URL verification failed: expected ${expectedUrl}, got ${check.freshState.url}`);
+              const reachedUrl = check.freshState.fullUrl || check.freshState.url;
+              lastFailure = `Reached ${reachedUrl}, expected ${expectedUrl}`;
+              tag('warning').log(`URL verification failed: expected ${expectedUrl}, got ${reachedUrl}`);
             }
             batchFailures.push({
               code: codeBlock,

package/src/ai/pilot.ts

@@ -118,7 +118,9 @@ export class Pilot implements Agent {
     }
 
     const schema = z.object({
-      decision: z.enum(['pass', 'fail', 'continue', 'skipped']).describe('pass = test succeeded, fail = test failed, continue = tester should keep going, skipped = scenario is irrelevant OR systematic execution failures prevented testing'),
+      decision: z
+        .enum(['pass', 'fail', 'continue', 'skipped'])
+        .describe('pass = scenario goal accomplished, fail = the app misbehaved, continue = tester should keep going, skipped = the scenario cannot be judged against this app (its premise does not hold, it is irrelevant, or systematic execution failures prevented testing)'),
       reason: z.string().describe('Concise user-facing reason, maximum 1 short sentence and 120 characters. Do NOT repeat the decision status; explain only the evidence. For continue: explain why rejected and suggest alternatives.'),
       guidance: z.string().nullable().describe('Required for "continue": specific actionable instruction for the tester — what exactly to verify, retry differently, or complete next. Be concrete.'),
       requestVerification: z
@@ -407,9 +409,13 @@ export class Pilot implements Agent {
         DOM assertion can't be made.
         Do not pass when Tester achieved only a related navigation/filter/tab/status outcome instead of the
         requested action, workflow, or entity detail goal.
-      - "fail": goal not achieved and no further step toward it is available on the current page.
-      - "skipped": scenario is irrelevant to the app, OR systematic infrastructure failures (LLM errors,
-        crashes) prevented testing. NOT for "test failed to interact" — that's "fail" or "continue".
+      - "fail": the app misbehaved — the scenario's action ran against the right target and the app
+        produced a wrong, broken, or missing outcome. Not reaching the goal is not by itself a fail.
+      - "skipped": the scenario cannot be judged against this app — the page shows its premise does not
+        hold (the assumed constraint, field, or behaviour is designed differently), the target entity or
+        feature is not the one here, the scenario is irrelevant, OR systematic infrastructure failures
+        (LLM errors, crashes) prevented testing. NOT for "test failed to interact" — that's "fail" or
+        "continue".
       - "continue": goal incomplete but the control for the NEXT step is present on the current page, or a
         concrete missing check would change your verdict. Guidance must name that step.
         If a verify() asserted a state that was ALREADY TRUE before the test, it proves nothing — reject.

package/src/ai/rerunner.ts

@@ -18,6 +18,7 @@ import { formatHeadings } from '../utils/context-formatter.ts';
 import { createDebug, tag } from '../utils/logger.ts';
 import { loop } from '../utils/loop.ts';
 import { RulesLoader } from '../utils/rules-loader.ts';
+import { isInternalStep } from '../utils/step-analyzer.ts';
 import type { Agent, AgentDeps } from './agent.ts';
 import { toolExecutionLabel } from './conversation.ts';
 import type { Navigator } from './navigator.ts';
@@ -85,6 +86,7 @@ export class Rerunner extends TaskAgent implements Agent {
 
     const onStepStarted = (step: any) => {
       if (!step.toCode) return;
+      if (isInternalStep(step)) return;
       const code = highlight(step.toCode(), { language: 'javascript' });
       console.log(chalk.dim(`    ${code}`));
     };
@@ -92,12 +94,14 @@ export class Rerunner extends TaskAgent implements Agent {
     const onStepPassed = (step: any) => {
       const task = this.getCurrentTask(testMap);
       if (!task || !step.toCode) return;
+      if (isInternalStep(step)) return;
       task.addStep(step.toCode(), step.duration, 'passed');
     };
 
     const onStepFailed = (step: any, error: any) => {
       const task = this.getCurrentTask(testMap);
       if (!task || !step.toCode) return;
+      if (isInternalStep(step)) return;
       task.addStep(step.toCode(), step.duration, 'failed', error?.message);
       console.log(chalk.red(`    ${figureSet.cross} ${step.toCode()} — ${error?.message || 'failed'}`));
     };

package/src/api/xhr-capture.ts

@@ -1,3 +1,4 @@
+import { isSameHostFamily } from '../utils/url-matcher.js';
 import { RequestResult, generateRequestId } from './request-result.ts';
 import type { RequestStore } from './request-store.ts';
 
@@ -39,7 +40,7 @@ export class XhrCapture {
 
     const method = request.method();
     const url = request.url();
-    if (!url.startsWith(this.baseOrigin)) return;
+    if (!isSameHostFamily(url, this.baseOrigin)) return;
 
     const status = response.status();
 

package/src/explorer.ts

@@ -21,6 +21,7 @@ import { Test, TestResult } from './test-plan.ts';
 import { BrowserRecoveryError, browserErrorMessage, isFatalBrowserError, isNavigationTransitionError } from './utils/browser-errors.ts';
 import { createDebug, log, tag } from './utils/logger.js';
 import { sleep, waitForPageReadiness } from './utils/page-readiness.ts';
+import { isInternalStep } from './utils/step-analyzer.ts';
 
 declare global {
   namespace NodeJS {
@@ -206,8 +207,7 @@ class Explorer {
 
     const stepHandler = (step: any, status?: string, error?: string, log?: string) => {
       if (!step.toCode) return;
-      if (step?.name?.startsWith('grab')) return;
-      if (step?.name?.startsWith('save')) return;
+      if (isInternalStep(step)) return;
 
       test.addStep(step.toCode(), step.duration, status, error, log);
 

package/src/reporter.ts

@@ -138,6 +138,7 @@ export class Reporter {
 
   protected combineStepsAndNotes(test: Test, lastScreenshotFile?: string): Step[] {
     const noteEntries = Object.entries(test.notes)
+      .filter(([, note]) => !note.observation)
       .map(([timestampKey, note]) => ({
         startTime: note.startTime,
         endTime: note.endTime,
@@ -272,9 +273,12 @@ export class Reporter {
         description: test.description,
         code: test.generatedCode || '',
         steps,
-        logs: Object.values(test.steps)
-          .map((stepData) => stepData.text)
-          .join('\n'),
+        logs: [
+          ...Object.values(test.steps).map((stepData) => stepData.text),
+          ...Object.values(test.notes)
+            .filter((note) => note.observation)
+            .map((note) => note.message),
+        ].join('\n'),
         files: Object.values(test.artifacts) || [],
         message: test.summary || this.extractLastNoteMessage(test) || '',
         meta,
@@ -341,7 +345,7 @@ export class Reporter {
   }
 
   private extractLastNoteMessage(test: Test): string {
-    const notes = Object.values(test.notes);
+    const notes = Object.values(test.notes).filter((note) => !note.observation);
     if (notes.length === 0) return '';
     return notes[notes.length - 1].message;
   }

package/src/utils/logger.ts

@@ -215,7 +215,7 @@ class SpanDestination implements LogDestination {
     if (!step?.toCode) {
       return;
     }
-    const stepName = step?.name ? `I.${step.name}` : 'I.step';
+    const stepName = step?.title ? `I.${step.title}` : 'I.step';
     const stepInput = typeof step?.toCode === 'function' ? step.toCode() : entry.content;
     const errorFromStep = step?.error;
     const errorMessage =

package/src/utils/step-analyzer.ts

@@ -5,6 +5,8 @@ import { isDynamicId } from './xpath.ts';
 export const CODECEPT_TOOLS = ['click', 'hover', 'pressKey', 'form'] as const;
 export type CodeceptToolName = (typeof CODECEPT_TOOLS)[number];
 
+const INTERNAL_STEP_PREFIXES = ['grab', 'save'];
+
 const CODECEPT_FORM_COMMANDS: readonly string[] = ['I.fillField', 'I.type', 'I.selectOption', 'I.attachFile', 'I.checkOption', 'I.uncheckOption'];
 
 export function isCodeceptToolName(toolName: string): toolName is CodeceptToolName {
@@ -71,3 +73,9 @@ export function mergeUniqueStepsByCode(primary: SessionStep[], secondary: Sessio
   }
   return merged;
 }
+
+export function isInternalStep(step: { title?: string }): boolean {
+  const title = step?.title;
+  if (!title) return false;
+  return INTERNAL_STEP_PREFIXES.some((prefix) => title.startsWith(prefix));
+}

package/src/utils/url-matcher.ts

@@ -132,3 +132,10 @@ export function matchesNavigationUrl(expected: string, current: string): boolean
     .filter(Boolean);
   return recordSegments.length > 0 && recordSegments.every(isDynamicSegment);
 }
+
+export function isSameHostFamily(urlA: string, urlB: string): boolean {
+  const hostA = URL.parse(urlA)?.host.toLowerCase();
+  const hostB = URL.parse(urlB)?.host.toLowerCase();
+  if (!hostA || !hostB) return false;
+  return hostA === hostB || hostA.endsWith(`.${hostB}`) || hostB.endsWith(`.${hostA}`);
+}