@parhelia/core 0.1.12663 → 0.1.12694

package/dist/editor/ai/AgentTerminal.js

@@ -2,7 +2,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import React, { useEffect, useState, useRef, useCallback, useLayoutEffect, useMemo, } from "react";
 import { flushSync } from "react-dom";
 import { Send, AlertCircle, Loader2, User, Wand2, Square, Mic, MicOff, ChevronDown, ChevronUp, ListTodo, ArrowLeft, DollarSign, ExternalLink, Settings2, Target, X, Plus, } from "lucide-react";
-import { getAgent, startAgent, claimAgentBrowser, cancelAgentDialog, assignAgentSkill, emitAgentDocumentsChanged, persistDraftAgent, updateAgentSettings, updateAgentCostLimit, updateAgentContext, getAgentSkillCatalog, getAgentAvailableTools, getAgentOperationAllowances, getAgentTriggerSubscriptions, cancelAgent, canonicalizeAgentMetadata, getPendingPrompts, releaseAgentBrowser, revokeAgentSkill, } from "../services/agentService";
+import { getAgent, startAgent, claimAgentBrowser, cancelAgentDialog, assignAgentSkill, persistDraftAgent, updateAgentSettings, updateAgentCostLimit, updateAgentContext, getAgentSkillCatalog, getAgentAvailableTools, getAgentOperationAllowances, getAgentTriggerSubscriptions, cancelAgent, canonicalizeAgentMetadata, getPendingPrompts, releaseAgentBrowser, revokeAgentSkill, } from "../services/agentService";
 import { parseAgentStatus } from "../services/agentStatus";
 import { useEditContext, useFieldsEditContext } from "../client/editContext";
 import { localStorageService } from "../services/localStorageService";
@@ -33,52 +33,6 @@ import { Splitter } from "../ui/Splitter";
 import { ScrollingContentTree } from "../ScrollingContentTree";
 import { MarkdownDisplay, } from "../../components/MarkdownDisplay";
 const AGENT_HISTORY_LIMIT = 1000;
-function parseToolResultPayload(value) {
-    if (!value) {
-        return null;
-    }
-    if (typeof value === "object") {
-        return value;
-    }
-    if (typeof value !== "string") {
-        return null;
-    }
-    try {
-        const parsed = JSON.parse(value);
-        return parsed && typeof parsed === "object" ? parsed : null;
-    }
-    catch {
-        return null;
-    }
-}
-function shouldRefreshDocumentsForToolResult(functionName, data) {
-    const normalizedName = (functionName || "").trim().toLowerCase();
-    const hasError = !!(data?.functionError || data?.error);
-    if (!normalizedName || hasError) {
-        return false;
-    }
-    if (normalizedName === "attach-media-item") {
-        return true;
-    }
-    if (normalizedName === "detach-document") {
-        const parsed = parseToolResultPayload(data?.functionResult || data?.result);
-        return parsed?.Removed === true || parsed?.removed === true;
-    }
-    if (normalizedName === "extract-pdf-images") {
-        const parsed = parseToolResultPayload(data?.functionResult || data?.result);
-        if (!parsed) {
-            return false;
-        }
-        const destination = String(parsed.Destination ?? parsed.destination ?? "");
-        const discoveryOnly = Boolean(parsed.DiscoveryOnly ?? parsed.discoveryOnly);
-        const createdImageCount = Number(parsed.CreatedImageCount ?? parsed.createdImageCount ?? 0);
-        return (!discoveryOnly &&
-            destination === "agentDocumentStore" &&
-            Number.isFinite(createdImageCount) &&
-            createdImageCount > 0);
-    }
-    return false;
-}
 function mergeAgentOperationHistory(existing, incoming, limit = AGENT_HISTORY_LIMIT) {
     const merged = new Map(existing.map((operation) => [operation.id, operation]));
     for (const operation of incoming) {
@@ -2762,14 +2716,9 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
             appendToolUiEvent("ui:tool-result-applied", `${extractedToolCall.functionName || "unknown"} toolCallId=${resultToolCallId} targetMessageId=${resultMessageId} completed=${matchingToolCall?.isCompleted ? "yes" : "no"} messageToolCalls=${messageWithToolResult?.toolCalls?.length || 0}`);
             return updated;
         });
-        const currentAgentId = agentData?.id || agent?.id || agentStub.id;
-        if (currentAgentId &&
-            shouldRefreshDocumentsForToolResult(extractedToolCall.functionName || "", message.data)) {
-            emitAgentDocumentsChanged({
-                agentId: currentAgentId,
-                reason: extractedToolCall.functionName || "tool-result",
-            });
-        }
+        // Document-store refresh is now triggered by the backend agent:documents:changed
+        // WebSocket message (fired from AgentDocumentRepository), bridged to the in-process
+        // emitAgentDocumentsChanged event in EditorShell. No per-tool allow-list here.
     }, [agent?.id, agentStub.id, appendToolUiEvent]);
     // Listen for local approval resolution to update UI
     useEffect(() => {
@@ -3616,6 +3565,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 if (normalizedStatus === "idle") {
                     clearStopGuard();
                     setLastRunStatusReason(null);
+                    setError(null);
                     // Stop indicators and mark any in-progress streaming messages as completed
                     clearHeartbeatMessages();
                     setAgent((prev) => (prev ? { ...prev, status: "idle" } : prev));
@@ -3759,6 +3709,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 if (normalizedStatus === "waitingForApproval") {
                     clearStopGuard();
                     setLastRunStatusReason(null);
+                    setError(null);
                     setPendingBrowserCaptureDialogType(null);
                     setPendingBrowserCaptureCallbackId(null);
                     setAgent((prev) => prev ? { ...prev, status: "waitingForApproval" } : prev);
@@ -3771,6 +3722,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 if (normalizedStatus === "waitingForInput") {
                     clearStopGuard();
                     setLastRunStatusReason(null);
+                    setError(null);
                     const dialogType = typeof statusData?.dialogType === "string"
                         ? statusData.dialogType
                         : null;
@@ -3801,6 +3753,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 }
                 if (normalizedStatus === "costLimitReached") {
                     clearStopGuard();
+                    setError(null);
                     setPendingBrowserCaptureDialogType(null);
                     setPendingBrowserCaptureCallbackId(null);
                     const totalCost = Number(statusData.totalCost) || 0;
@@ -3851,6 +3804,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 // Handle "completed" state (fallback for legacy code paths that send status instead of lifecycle event)
                 if (normalizedStatus === "completed") {
                     clearStopGuard();
+                    setError(null);
                     // Reset deduplication for the next run
                     lastSeqRef.current = 0;
                     clearHeartbeatMessages();
@@ -3899,6 +3853,7 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                         return;
                     }
                     setLastRunStatusReason(null);
+                    setError(null);
                     // Update agent status to running and clear any previous error statusMessage
                     setAgent((prev) => prev
                         ? { ...prev, status: "running", statusMessage: undefined }
@@ -3912,8 +3867,11 @@ export function AgentTerminal({ agentStub, initialMetadata, profiles, isActive =
                 if (normalizedStatus === "error") {
                     clearStopGuard();
                     setLastRunStatusReason(null);
-                    const errorMsg = toUserFacingAgentErrorMessage(statusData?.statusMessage ?? statusData?.error) || "Unknown error";
+                    const errorMsg = toUserFacingAgentErrorMessage(statusData?.statusMessage ??
+                        statusData?.message ??
+                        statusData?.error) || "Unknown error";
                     clearHeartbeatMessages();
+                    setError(errorMsg);
                     setAgent((prev) => prev
                         ? { ...prev, status: "error", statusMessage: errorMsg }
                         : prev);