Add errorType field to InteractionSummary to handle errors based on error type and not content

Author: smorelle-abstracta

browser-extension/src/background.ts

@@ -5,7 +5,7 @@ import { AgentSession } from "./scripts/agent-session";
 import { findAgentSession, saveAgentSession, removeAgentSession, } from "./scripts/agent-session-repository";
 import { saveAgentPrompts } from "./scripts/prompt-repository";
 import { BrowserMessage, ToggleSidebar, ActiveTabListener, ActivateAgent, AgentActivation, InteractionSummary, } from "./scripts/browser-message";
-import { HttpServiceError, NetworkError } from "./scripts/http";
+import { BaseError } from "./scripts/http";
 import { isActiveTabListener, setTabListenerActive, removeTabListenerStatus, } from "./scripts/tab-listener-status-repository";
 import { removeTabState } from "./scripts/tab-state-repository";
 
@@ -175,7 +175,8 @@ async function asyncProcessRequest(req: RequestEvent) {
         tabId,
         new InteractionSummary(
           false, 
-          (e instanceof HttpServiceError || e instanceof NetworkError) ? e.detail : undefined
+          e instanceof BaseError ? e.detail : undefined,
+          e instanceof BaseError ? e.getType() : undefined
         )
       );
     }

browser-extension/src/pages/Index.vue

@@ -165,9 +165,14 @@ const onAgentActivation = (msg: AgentActivation) => {
 }
 
 const onInteractionSummary = (msg: InteractionSummary) => {
-  const text = msg.text 
-    ? (msg.text === "Failed to fetch" ? t('networkError') : msg.text) 
-    : t('interactionSummaryError', { contactEmail: agent.value!.manifest.contactEmail })
+  let text = msg.text ?? ''
+
+  if (!msg.success) {
+      text = msg.errorType === 'NetworkError' ? t('networkError') : t('interactionSummaryError', {
+          contactEmail: agent.value!.manifest.contactEmail
+      })
+  }
+
   const lastMessage = messages.value[messages.value.length - 1]
   const messagePosition = lastMessage.isComplete ? messages.value.length : messages.value.length - 1
   messages.value.splice(messagePosition, 0, msg.success ? ChatMessage.agentMessage(text) : ChatMessage.agentErrorMessage(text))

browser-extension/src/scripts/browser-message.ts

@@ -102,15 +102,17 @@ export class AgentActivation extends BrowserMessage {
 export class InteractionSummary extends BrowserMessage {
   text?: string
   success: boolean
+  errorType?: string
 
-  constructor(success: boolean, text?: string) {
+  constructor(success: boolean, text?: string, errorType?: string) {
     super("interactionSummary")
     this.text = text
     this.success = success
+    this.errorType = errorType
   }
 
   public static fromJsonObject(obj: any): InteractionSummary {
-    return new InteractionSummary(obj.success, obj.text)
+    return new InteractionSummary(obj.success, obj.text, obj.errorType)
   }
 }
 

browser-extension/src/scripts/http.ts

@@ -33,18 +33,27 @@ const fetchResponse = async (url: string, options?: RequestInit) => {
   return ret
 }
 
-class BaseError extends Error {
+export class BaseError extends Error {
   detail?: string
+  readonly type: string = 'BaseError'
 
   constructor(detail?: string) {
     super()
     this.detail = detail
   }
+
+  getType(): string {
+    return this.type
+  }
 }
 
-export class HttpServiceError extends BaseError {}
+export class HttpServiceError extends BaseError {
+  readonly type: string = 'HttpServiceError'
+}
 
-export class NetworkError extends BaseError {}
+export class NetworkError extends BaseError {
+  readonly type: string = 'NetworkError'
+}
 
 export async function* fetchStreamJson(url: string, options?: RequestInit): AsyncIterable<any> {
   let resp = await fetchResponse(url, options)