explorbot 0.1.27 → 0.1.29

package/src/utils/browser-errors.ts

@@ -0,0 +1,25 @@
+// Playwright and CodeceptJS surface browser/page disposal as plain Error objects,
+// not typed exceptions. Keep those external message markers in one adapter so
+// recovery decisions are not duplicated across agents/actions.
+const FATAL_BROWSER_ERROR_MARKERS = ['Frame was detached', 'Target closed', 'Target page, context or browser has been closed', 'Execution context was destroyed', 'Protocol error', 'Session closed'];
+
+export class BrowserRecoveryError extends Error {
+  constructor(
+    label: string,
+    public originalError: unknown,
+    public recovered: boolean
+  ) {
+    super(`${label} failed ${recovered ? 'after browser recovery' : 'because browser could not be recovered'}: ${browserErrorMessage(originalError)}`);
+    this.name = 'BrowserRecoveryError';
+  }
+}
+
+export function isFatalBrowserError(error: unknown): boolean {
+  if (error instanceof BrowserRecoveryError) return true;
+  const message = (error instanceof Error ? error.message : String(error)).toLowerCase();
+  return FATAL_BROWSER_ERROR_MARKERS.some((marker) => message.includes(marker.toLowerCase()));
+}
+
+export function browserErrorMessage(error: unknown): string {
+  return error instanceof Error ? error.message : String(error);
+}

package/src/utils/error-page.ts

@@ -1,4 +1,5 @@
-import type { ActionResult } from '../action-result.js';
+import { ActionResult } from '../action-result.js';
+import type { WebPageState } from '../state-manager.js';
 import { isBodyEmpty } from './html.js';
 
 const HTTP_ERRORS = ['400 Bad Request', '401 Unauthorized', '403 Forbidden', '404 Not Found', '405 Method Not Allowed', '408 Request Timeout', '500 Internal Server Error', '502 Bad Gateway', '503 Service Unavailable', '504 Gateway Timeout'];
@@ -9,6 +10,8 @@ const LOADING_WORD = /\bloading\b/i;
 export type PageCondition = 'ok' | 'loading' | 'error';
 
 export function detectPageCondition(actionResult: ActionResult): PageCondition {
+  if (actionResult.httpStatus && actionResult.httpStatus >= 400) return 'error';
+
   const headingFields = [actionResult.title, actionResult.h1, actionResult.h2].filter(Boolean) as string[];
 
   for (const field of headingFields) {
@@ -37,12 +40,22 @@ export function isErrorPage(actionResult: ActionResult): boolean {
   return detectPageCondition(actionResult) === 'error';
 }
 
+export function getStateErrorPageError(state: WebPageState | null | undefined): ErrorPageError | null {
+  if (!state) return null;
+  const actionResult = ActionResult.fromState(state);
+  if (!isErrorPage(actionResult)) return null;
+  return new ErrorPageError(actionResult.url, actionResult.title, actionResult.httpStatus);
+}
+
 export class ErrorPageError extends Error {
   constructor(
     public readonly url: string,
-    public readonly title?: string
+    public readonly title?: string,
+    public readonly httpStatus?: number
   ) {
-    super(`Error page detected at ${url}${title ? ` (${title})` : ''}`);
+    const status = httpStatus ? `HTTP ${httpStatus}` : '';
+    const details = [status, title].filter(Boolean).join(', ');
+    super(`Error page detected at ${url}${details ? ` (${details})` : ''}`);
     this.name = 'ErrorPageError';
   }
 }

package/src/utils/logger.ts

@@ -11,7 +11,7 @@ import { Observability } from '../observability.ts';
 import { RecentStepFilter } from './log-filters.ts';
 import { parseMarkdownToTerminal } from './markdown-terminal.ts';
 
-export type LogType = 'info' | 'success' | 'error' | 'warning' | 'debug' | 'substep' | 'operation' | 'step' | 'multiline' | 'html' | 'input';
+export type LogType = 'info' | 'success' | 'error' | 'warning' | 'debug' | 'substep' | 'operation' | 'step' | 'multiline' | 'details' | 'html' | 'input';
 
 export interface TaggedLogEntry {
   type: LogType;
@@ -99,7 +99,7 @@ class ConsoleDestination implements LogDestination {
     if (entry.type === 'operation' && !this.verboseMode) return;
     if (entry.type === 'step' && !this.verboseMode && this.recentSteps.shouldSuppress(entry.content)) return;
     let content = entry.content;
-    if (entry.type === 'multiline') {
+    if (entry.type === 'multiline' || entry.type === 'details') {
       const cleaned = stripAnsi(dedent(entry.content));
       const parsed = parseMarkdownToTerminal(cleaned);
       content = parsed;
@@ -314,7 +314,7 @@ class CaptainDestination implements LogDestination {
 
   stopCapture(): string[] {
     this.capturing = false;
-    const logs = this.entries.filter((e) => e.type !== 'debug' && e.type !== 'html' && e.type !== 'multiline' && e.type !== 'operation').map((e) => `[${e.type}] ${e.content}`);
+    const logs = this.entries.filter((e) => e.type !== 'debug' && e.type !== 'html' && e.type !== 'multiline' && e.type !== 'details' && e.type !== 'operation').map((e) => `[${e.type}] ${e.content}`);
     this.entries = [];
     return logs;
   }