chore: remove load agent

Author: bryanadenhq

core/frontend/src/pages/workspace.tsx

@@ -182,80 +182,6 @@ function NewTabPopover({ open, onClose, anchorRef, discoverAgents, onFromScratch
   );
 }
 
-// --- LoadAgentPopover ---
-interface LoadAgentPopoverProps {
-  open: boolean;
-  onClose: () => void;
-  anchorRef: React.RefObject<HTMLButtonElement | null>;
-  discoverAgents: DiscoverEntry[];
-  onSelect: (agentPath: string) => void;
-}
-
-function LoadAgentPopover({ open, onClose, anchorRef, discoverAgents, onSelect }: LoadAgentPopoverProps) {
-  const [pos, setPos] = useState<{ top: number; right: number } | null>(null);
-  const ref = useRef<HTMLDivElement>(null);
-
-  useEffect(() => {
-    if (open && anchorRef.current) {
-      const rect = anchorRef.current.getBoundingClientRect();
-      setPos({ top: rect.bottom + 4, right: window.innerWidth - rect.right });
-    }
-  }, [open, anchorRef]);
-
-  useEffect(() => {
-    if (!open) return;
-    const handler = (e: MouseEvent) => {
-      if (
-        ref.current && !ref.current.contains(e.target as Node) &&
-        anchorRef.current && !anchorRef.current.contains(e.target as Node)
-      ) onClose();
-    };
-    document.addEventListener("mousedown", handler);
-    return () => document.removeEventListener("mousedown", handler);
-  }, [open, onClose, anchorRef]);
-
-  useEffect(() => {
-    if (!open) return;
-    const handler = (e: KeyboardEvent) => { if (e.key === "Escape") onClose(); };
-    document.addEventListener("keydown", handler);
-    return () => document.removeEventListener("keydown", handler);
-  }, [open, onClose]);
-
-  if (!open || !pos) return null;
-
-  return ReactDOM.createPortal(
-    <div
-      ref={ref}
-      style={{ position: "fixed", top: pos.top, right: pos.right, zIndex: 9999 }}
-      className="w-60 rounded-xl border border-border/60 bg-card shadow-xl shadow-black/30 overflow-hidden"
-    >
-      <div className="flex items-center gap-2 px-3 py-2.5 border-b border-border/40">
-        <span className="text-xs font-semibold text-muted-foreground uppercase tracking-wider">
-          Load Agent
-        </span>
-      </div>
-      <div className="p-1.5 flex flex-col max-h-64 overflow-y-auto">
-        {discoverAgents.map(agent => (
-          <button
-            key={agent.path}
-            onClick={() => { onSelect(agent.path); onClose(); }}
-            className="flex items-center gap-2.5 w-full px-3 py-2 rounded-lg text-left transition-colors hover:bg-muted/60 text-foreground"
-          >
-            <div className="w-6 h-6 rounded-md bg-muted/80 flex items-center justify-center flex-shrink-0">
-              <Bot className="w-3.5 h-3.5 text-muted-foreground" />
-            </div>
-            <span className="text-sm font-medium">{agent.name}</span>
-          </button>
-        ))}
-        {discoverAgents.length === 0 && (
-          <p className="text-xs text-muted-foreground px-3 py-2">No agents found</p>
-        )}
-      </div>
-    </div>,
-    document.body
-  );
-}
-
 function fmtLogTs(ts: string): string {
   try {
     const d = new Date(ts);
@@ -399,9 +325,6 @@ export default function Workspace() {
   const [selectedNode, setSelectedNode] = useState<GraphNode | null>(null);
   const [newTabOpen, setNewTabOpen] = useState(false);
   const newTabBtnRef = useRef<HTMLButtonElement>(null);
-  const [loadAgentOpen, setLoadAgentOpen] = useState(false);
-  const [loadingWorker, setLoadingWorker] = useState(false);
-  const loadAgentBtnRef = useRef<HTMLButtonElement>(null);
 
   // Ref mirror of sessionsByAgent so SSE callback can read current graph
   // state without adding sessionsByAgent to its dependency array.
@@ -1458,42 +1381,6 @@ export default function Workspace() {
     }
   }, [activeWorker, activeSession, agentStates, updateAgentState]);
 
-  const handleLoadAgent = useCallback(async (agentPath: string) => {
-    const state = agentStates[activeWorker];
-    if (!state?.sessionId) return;
-
-    setLoadingWorker(true);
-    try {
-      await sessionsApi.loadWorker(state.sessionId, agentPath);
-      // Success: worker_loaded SSE event will handle UI updates automatically
-    } catch (err) {
-      // 424 = credentials required — open the credentials modal
-      if (err instanceof ApiError && err.status === 424) {
-        const body = err.body as Record<string, unknown>;
-        setCredentialAgentPath((body.agent_path as string) || null);
-        setCredentialsOpen(true);
-        setLoadingWorker(false);
-        return;
-      }
-
-      const errMsg = err instanceof Error ? err.message : String(err);
-      const activeId = activeSessionRef.current[activeWorker];
-      const errorMsg: ChatMessage = {
-        id: makeId(), agent: "System", agentColor: "",
-        content: `Failed to load agent: ${errMsg}`,
-        timestamp: "", type: "system", thread: activeWorker,
-      };
-      setSessionsByAgent(prev => ({
-        ...prev,
-        [activeWorker]: (prev[activeWorker] || []).map(s =>
-          s.id === activeId ? { ...s, messages: [...s.messages, errorMsg] } : s
-        ),
-      }));
-    } finally {
-      setLoadingWorker(false);
-    }
-  }, [activeWorker, agentStates]);
-
   const closeAgentTab = useCallback((agentType: string) => {
     setSelectedNode(null);
     // Pause worker execution if running (saves checkpoint), then kill the
@@ -1592,30 +1479,6 @@ export default function Workspace() {
           </>
         }
       >
-        {activeWorker === "new-agent" && activeAgentState?.ready && !activeAgentState?.graphId && (
-          <>
-            <button
-              ref={loadAgentBtnRef}
-              onClick={() => setLoadAgentOpen(o => !o)}
-              disabled={loadingWorker}
-              className="flex items-center gap-1.5 px-3 py-1.5 rounded-md text-xs font-medium text-muted-foreground hover:text-foreground hover:bg-muted/50 transition-colors flex-shrink-0"
-            >
-              {loadingWorker ? (
-                <Loader2 className="w-3.5 h-3.5 animate-spin" />
-              ) : (
-                <Bot className="w-3.5 h-3.5" />
-              )}
-              Load Agent
-            </button>
-            <LoadAgentPopover
-              open={loadAgentOpen}
-              onClose={() => setLoadAgentOpen(false)}
-              anchorRef={loadAgentBtnRef}
-              discoverAgents={discoverAgents}
-              onSelect={handleLoadAgent}
-            />
-          </>
-        )}
         <button
           onClick={() => setCredentialsOpen(true)}
           className="flex items-center gap-1.5 px-3 py-1.5 rounded-md text-xs font-medium text-muted-foreground hover:text-foreground hover:bg-muted/50 transition-colors flex-shrink-0"