mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-11-04 02:32:07 +01:00
## Thinking Path > - Paperclip is the open source app people use to manage AI agents for work. > - The agent page is where an operator reads and changes one agent. > - The page grew one tab at a time. Ten tabs sit in one flat list, the header holds many buttons, and the audit links leave the agent. > - A design pass, called Refine-v2, made a calmer layout for this page. It ran in a prototype on fixture data only. > - This pull request moves the Refine-v2 layout and behavior into the live agent page. Where the prototype and the live page differ, the live behavior stays. For example, the floating save bar stays on screen. > - The benefit is a shorter navigation, edits in place, and audit data that stays inside the agent. ## Linked Issues or Issue Description Refs #13716 Refs #9166 Both pull requests mount plugin tabs or slots on the agent page. They touch the same files, so one of the three needs a rebase after the others merge. **Subsystem affected** The agent detail page in `ui/` (`ui/src/pages/AgentDetail.tsx`, its sidebar, header actions, and the shared agent config form). **Problem or motivation** The agent page is hard to scan. The navigation shows ten items in one list. The header shows Run now, Pause, Clear error, and Assign task as separate buttons. The name, title, and manager are form fields deep in the Harness tab. Activity, Runs, Costs, and Budgets open company-wide pages, so the operator loses the agent context. Each tab uses a different card and table style. **Proposed solution** Use one layout system for every agent tab. Group the navigation into Agent and Audit. Edit the name and title in the header. Move the work actions into the actions menu. Show audit views inside the agent. Keep the persistent save bar and add a status dot to it. **Alternatives considered** We considered a flag that lets operators opt in to the new layout. We did not use a flag, because two layouts on one page double the test surface. We also considered moving the save button to the end of the scrolling content, as the prototype did. We kept the floating save bar, because it is always visible. **Roadmap alignment** `ROADMAP.md` has no item for the agent page. This change is a UI refinement of an existing page. It adds one optional field to an existing API request and no database schema change. ## What Changed - **Navigation.** The agent sidebar shows two groups. Agent has Overview, Identity, Harness / Runtime, Secrets & variables, Permissions / Trust, and Advanced. Audit has Activity, Runs, Costs, and Budgets. Skills, Tools, Channels, and API Keys leave the navigation. Their URLs still open. The Instructions tab is now Identity, at `/identity`. The old `/instructions` URL redirects. - **Header.** The agent character shows at a larger size. The name and title edit in place. Enter or blur saves. Escape cancels. An error restores the old value. A rename updates the URL. While a tab has unsaved changes, the header refuses the edit and asks the operator to save or discard first. The third line shows the title and a link to the manager. A crown button to the right of the name replaces the "Set as my primary" link and the separate primary indicator. It is outlined and dimmed until the agent is your primary, then fills with a short pop. Replacing an existing primary asks for confirmation first, and the pop plays after the dialog closes. Reduced motion skips the pop. Run now, Pause or Resume, Clear error, and Agent history move into the actions menu. Assign task is the primary button. For an agent that still needs setup, Finish setup is the primary button and Assign task moves into the menu. - **Agent history.** A new dialog in the actions menu replaces the Revisions entry. Each row shows a plain-language summary, the person or agent who made the change, the date, and Restore. Restore asks for confirmation. A failed restore shows its reason inside the confirmation. - **Overview.** Four metric tiles show activity this week, runs this week, costs this month, and budget used. Each tile links to its audit view. When the list behind a count is capped, the tile shows a lower bound, for example 200+. The activity tile follows live updates. Profile, Skills, and Recent tasks panels follow. The Profile panel lists direct reports under their label in at most two rows, then a "+N more" button that expands the list. The Latest run card is removed. - **Identity.** The file preview shows the raw text. A pencil button switches to a plain text editor. Copy and Delete are icon buttons. A chip next to the file name says "Managed by you" or "Managed externally". Per-file history and unsaved edits stay. - **External instructions.** "Advanced settings" opens Root path and Entry file fields and a Load profile button. Load profile asks the server to switch only if the root directory and the entry file already exist. If either is missing, the server refuses and the panel shows the reason. The entry file can have any name inside the root. - **Instructions bundle API.** `PATCH /agents/:id/instructions-bundle` accepts an optional `requireExistingEntry` flag (`packages/shared` validator, `server` service, `ui` client). With the flag, the server checks the root directory and entry file before it creates or copies anything. Without the flag, the behavior does not change. - **Harness / Runtime.** The config form gets a `panels` layout: Harness and Execution side by side, then the runtime test, then environment variables. The identity and run-policy sections are hidden from this tab. - **Secrets & variables and Advanced.** These tabs use the same filled panels and table headers. - **Permissions / Trust.** The trust presets show as selectable cards. An Allowed connectors panel shows up to three connector logos and links to Apps, filtered to installed connectors. Capabilities sit in their own panel. Changes on this tab save at once, with no save bar. - **Audit inside the agent.** Activity, Runs, Costs, and Budgets render inside the agent page. Runs is locked to the current agent. A failed budget incident action shows its reason under the incident. - **Save bar.** The floating save bar stays. It shows an amber dot for unsaved changes and a green dot with "Changes saved" after a save. Discard stays. - **Leave guard.** If the form has unsaved changes and the operator clicks a link, a dialog asks to keep editing or discard the changes. Browser back and the header actions keep the native confirm. - **Apps.** `/apps?source=installed` opens the Apps page on the Installed filter. - **Tokens.** New `--agent-detail-*` tokens in `ui/src/index.css` hold every spacing and fill value. Each fill is a mix of foreground into background, so one value works in light and dark themes. - **Tests and maps.** Tests cover the navigation model, sidebar, header menu, Overview layout, Identity editor, and helpers. The agent-configuration recipe in `maps/feature-map/agents.md` describes the new page. ## Verification - `cd ui && npx vitest run`: 849 test files and 10,081 tests pass. - `cd server && npx vitest run src/__tests__/agent-instructions-routes.test.ts src/__tests__/agent-instructions-service.test.ts`: 40 tests pass. - `npx playwright test --config tests/e2e/playwright.config.ts tests/e2e/onboarding.spec.ts tests/e2e/chat-adapters-ui-providers.spec.ts`: 26 tests pass. - `cd ui && pnpm typecheck` passes. This includes the Storybook catalog check. The `server` and `packages/shared` typechecks pass. - `cd ui && pnpm build` passes. - `pnpm run check:token-gates` reports all gates clean. - `node scripts/check-maps.mjs` and `node scripts/storybook-catalog.mjs --check` pass. - Manual check on a local instance, in light and dark themes: 1. Open an agent. Confirm the sidebar shows the Agent and Audit groups. 2. Click the agent name in the header. Type a new name and press Enter. Confirm the URL changes. Rename it back. 3. Open the actions menu. Open Agent history. Confirm the rows and the Restore confirmation. 4. Open Identity. Click the pencil. Change a line. Confirm the save bar shows the amber dot. Click a sidebar link. Confirm the discard dialog. Click Keep editing, then save. Confirm the green dot. 5. Open Harness / Runtime, Secrets & variables, and Permissions / Trust. Confirm the panels and the trust cards. 6. Open Runs under Audit. Confirm the list shows only this agent and the page stays inside the agent. ## Risks - This changes a high-traffic page for every operator at once. There is no flag to switch back. - Run policy and Reports to are no longer editable on the agent page. The header shows the manager as a link only. - Skills, Tools, Channels, and API Keys are reachable by URL only. Operators who use them from the sidebar must use the URL. - The pull request conflicts with #13716 and #9166 in `AgentDetail.tsx` and the agent sidebar. - Storybook stories for the agent page and `DESIGN.md` are not updated in this pull request. - The new `requireExistingEntry` flag is opt-in. Existing API callers keep the old behavior, which creates a missing external directory and entry file. - I ran two e2e spec files locally, not the full `pnpm test:e2e` or `pnpm test:release-smoke`. CI runs the full e2e suite. I did not run the repo-wide `pnpm -r typecheck`, because the local toolchain has no Rust. > For core feature work, check [`ROADMAP.md`](ROADMAP.md) first and discuss it in `#dev` before opening the PR. Feature PRs that overlap with planned core work may need to be redirected — check the roadmap first. See `CONTRIBUTING.md`. ## Model Used - Anthropic Claude Opus 5.5 (`claude-opus-5-5`), in Claude Code, with reasoning and tool use (shell, file edits, and a browser for visual checks). A person directed every design decision. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work - [x] I have searched GitHub for duplicate or related PRs and linked them above - [x] I have either (a) linked existing issues with `Fixes: #` / `Closes #` / `Refs #` OR (b) described the issue in-PR following the relevant issue template - [x] I have not referenced internal/instance-local Paperclip issues or links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip` URLs) - [x] My branch name describes the change (e.g. `docs/...`, `fix/...`) and contains no internal Paperclip ticket id or instance-derived details - [x] I have run tests locally and they pass - [x] I have added or updated tests where applicable - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [x] All Paperclip CI gates are green - [x] Greptile is 5/5 with no open P2s, recommendations, or follow-ups - [x] I will address all Greptile and reviewer comments before requesting merge 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
684 lines
26 KiB
TypeScript
684 lines
26 KiB
TypeScript
import { useCallback, useRef, useState, type ReactNode } from "react";
|
|
import { useNavigate } from "@/lib/router";
|
|
import { useMutation, useQueryClient } from "@tanstack/react-query";
|
|
import {
|
|
Pause,
|
|
Play,
|
|
Plus,
|
|
MoreHorizontal,
|
|
Loader2,
|
|
Copy,
|
|
RotateCcw,
|
|
Trash2,
|
|
CheckCircle2,
|
|
Bug,
|
|
} from "lucide-react";
|
|
import { Button } from "@/components/ui/button";
|
|
import {
|
|
Popover,
|
|
PopoverContent,
|
|
PopoverTrigger,
|
|
} from "@/components/ui/popover";
|
|
import {
|
|
AlertDialog,
|
|
AlertDialogAction,
|
|
AlertDialogCancel,
|
|
AlertDialogContent,
|
|
AlertDialogDescription,
|
|
AlertDialogFooter,
|
|
AlertDialogHeader,
|
|
AlertDialogTitle,
|
|
} from "@/components/ui/alert-dialog";
|
|
import { AgentStatusBadge } from "./StatusBadge";
|
|
import { agentsApi } from "../api/agents";
|
|
import { ApiError } from "../api/client";
|
|
import { queryKeys } from "../lib/queryKeys";
|
|
import { agentRouteRef, cn } from "../lib/utils";
|
|
import { pendingAgentSetupChoice } from "../lib/agent-setup-choice";
|
|
import { copyTextToClipboard } from "../lib/clipboard";
|
|
import { useDialogActions } from "../context/DialogContext";
|
|
import { useToastActions } from "../context/ToastContext";
|
|
import {
|
|
buildDuplicateAgentPayload,
|
|
duplicateAgentName,
|
|
type DuplicateInstructionsBundle,
|
|
} from "../lib/duplicate-agent-payload";
|
|
import type {
|
|
Agent,
|
|
AgentInstructionsBundle,
|
|
AgentInstructionsFileSummary,
|
|
HeartbeatRun,
|
|
} from "@paperclipai/shared";
|
|
|
|
export function RunButton({
|
|
onClick,
|
|
disabled,
|
|
label = "Run now",
|
|
size = "sm",
|
|
}: {
|
|
onClick: () => void;
|
|
disabled?: boolean;
|
|
label?: string;
|
|
size?: "sm" | "default";
|
|
}) {
|
|
return (
|
|
<Button variant="outline" size={size} onClick={onClick} disabled={disabled}>
|
|
<Play className="h-3.5 w-3.5 sm:mr-1" />
|
|
<span className="hidden sm:inline">{label}</span>
|
|
</Button>
|
|
);
|
|
}
|
|
|
|
export function PauseResumeButton({
|
|
isPaused,
|
|
onPause,
|
|
onResume,
|
|
disabled,
|
|
size = "sm",
|
|
}: {
|
|
isPaused: boolean;
|
|
onPause: () => void;
|
|
onResume: () => void;
|
|
disabled?: boolean;
|
|
size?: "sm" | "default";
|
|
}) {
|
|
if (isPaused) {
|
|
return (
|
|
<Button variant="outline" size={size} onClick={onResume} disabled={disabled}>
|
|
<Play className="h-3.5 w-3.5 sm:mr-1" />
|
|
<span className="hidden sm:inline">Resume</span>
|
|
</Button>
|
|
);
|
|
}
|
|
|
|
return (
|
|
<Button variant="outline" size={size} onClick={onPause} disabled={disabled}>
|
|
<Pause className="h-3.5 w-3.5 sm:mr-1" />
|
|
<span className="hidden sm:inline">Pause</span>
|
|
</Button>
|
|
);
|
|
}
|
|
|
|
export function ClearErrorButton({
|
|
onClick,
|
|
disabled,
|
|
size = "sm",
|
|
}: {
|
|
onClick: () => void;
|
|
disabled?: boolean;
|
|
size?: "sm" | "default";
|
|
}) {
|
|
return (
|
|
<Button
|
|
variant="outline"
|
|
size={size}
|
|
onClick={onClick}
|
|
disabled={disabled}
|
|
className="border-destructive/60 text-destructive hover:bg-destructive/10 hover:text-destructive dark:border-destructive/50"
|
|
aria-label="Clear error and return agent to idle"
|
|
>
|
|
<CheckCircle2 className="h-3.5 w-3.5 sm:mr-1" />
|
|
<span className="hidden sm:inline">Clear error</span>
|
|
</Button>
|
|
);
|
|
}
|
|
|
|
function duplicateInstructionFilePath(
|
|
_bundle: AgentInstructionsBundle,
|
|
summary: AgentInstructionsFileSummary,
|
|
): string | null {
|
|
if (summary.deprecated || summary.virtual) return null;
|
|
return summary.path;
|
|
}
|
|
|
|
export async function loadDuplicateInstructionsBundle(
|
|
agentId: string,
|
|
companyId?: string,
|
|
): Promise<DuplicateInstructionsBundle | null> {
|
|
const bundle = await agentsApi.instructionsBundle(agentId, companyId);
|
|
const files: Record<string, string> = {};
|
|
|
|
for (const summary of bundle.files) {
|
|
const path = duplicateInstructionFilePath(bundle, summary);
|
|
if (!path) continue;
|
|
const file = await agentsApi.instructionsFile(agentId, summary.path, companyId);
|
|
files[path] = file.content;
|
|
}
|
|
|
|
const entryFile = Object.prototype.hasOwnProperty.call(files, bundle.entryFile)
|
|
? bundle.entryFile
|
|
: Object.keys(files)[0] ?? "AGENTS.md";
|
|
return Object.keys(files).length > 0 ? { entryFile, files } : null;
|
|
}
|
|
|
|
/**
|
|
* Shared agent action cluster used by both the agent detail header and the
|
|
* agents list rows. Encapsulates the invoke / pause / resume / terminate /
|
|
* duplicate / reset-session mutations so callers do not diverge in behavior.
|
|
*/
|
|
export function AgentActionButtons({
|
|
agent,
|
|
companyId,
|
|
size = "sm",
|
|
assignLabel = "Assign Task",
|
|
runLabel = "Run now",
|
|
showStatus = true,
|
|
showRun = true,
|
|
actionsDisabled = false,
|
|
workActionsDisabled = false,
|
|
workActionsDisabledReason,
|
|
navigateToRunOnInvoke = true,
|
|
hasPendingNavigationChanges = false,
|
|
onBeforeNavigate,
|
|
onActionError,
|
|
onTerminateSuccess,
|
|
pauseConfirm,
|
|
hideTerminate = false,
|
|
workActionsInMenu = false,
|
|
menuItems = [],
|
|
children,
|
|
className,
|
|
}: {
|
|
agent: Agent;
|
|
companyId?: string | null;
|
|
size?: "sm" | "default";
|
|
assignLabel?: string;
|
|
runLabel?: string;
|
|
showStatus?: boolean;
|
|
showRun?: boolean;
|
|
actionsDisabled?: boolean;
|
|
workActionsDisabled?: boolean;
|
|
workActionsDisabledReason?: string;
|
|
navigateToRunOnInvoke?: boolean;
|
|
/** Whether the caller currently has an unsaved draft that navigation would discard. */
|
|
hasPendingNavigationChanges?: boolean;
|
|
/** Return false to stop an action whose success would navigate away. */
|
|
onBeforeNavigate?: () => boolean;
|
|
/**
|
|
* When set, pausing prompts a confirmation dialog first (e.g. for built-in
|
|
* agents that power a feature). Omit for the immediate-pause default.
|
|
*/
|
|
pauseConfirm?: { title: string; description: ReactNode };
|
|
/** Hide the Terminate action (e.g. built-in agents are undeletable). */
|
|
hideTerminate?: boolean;
|
|
/** Put Run now, Pause/Resume and Clear error in the overflow menu, leaving Assign task (or Finish setup, while setup is pending) as the one button. */
|
|
workActionsInMenu?: boolean;
|
|
/** Extra overflow items shown after the work actions when `workActionsInMenu` is on. */
|
|
menuItems?: ReadonlyArray<{ label: string; icon: typeof Plus; onSelect: () => void; disabled?: boolean }>;
|
|
/**
|
|
* Optional inline error reporter. When provided it is used instead of a toast
|
|
* for action failures (preserves the detail page's inline error banner). When
|
|
* omitted, failures surface as toasts (used by the list view).
|
|
*/
|
|
onActionError?: (message: string | null) => void;
|
|
/** Called after termination succeeds so callers can leave now-hidden detail routes. */
|
|
onTerminateSuccess?: (agent: Agent) => void;
|
|
/** Extra content rendered just before the overflow menu (e.g. live-run link). */
|
|
children?: React.ReactNode;
|
|
className?: string;
|
|
}) {
|
|
const navigate = useNavigate();
|
|
const queryClient = useQueryClient();
|
|
const { openNewIssue } = useDialogActions();
|
|
const { pushToast } = useToastActions();
|
|
const [moreOpen, setMoreOpen] = useState(false);
|
|
const [pauseConfirmOpen, setPauseConfirmOpen] = useState(false);
|
|
const pendingNavigationChangesRef = useRef(hasPendingNavigationChanges);
|
|
const beforeNavigateRef = useRef(onBeforeNavigate);
|
|
const agentActionStartedDirtyRef = useRef(false);
|
|
const duplicateStartedDirtyRef = useRef(false);
|
|
pendingNavigationChangesRef.current = hasPendingNavigationChanges;
|
|
beforeNavigateRef.current = onBeforeNavigate;
|
|
|
|
function confirmNavigationStart(startedDirtyRef: React.MutableRefObject<boolean>) {
|
|
startedDirtyRef.current = pendingNavigationChangesRef.current;
|
|
return beforeNavigateRef.current?.() !== false;
|
|
}
|
|
|
|
function confirmLateNavigationChanges(startedDirtyRef: React.MutableRefObject<boolean>) {
|
|
return (
|
|
!pendingNavigationChangesRef.current ||
|
|
startedDirtyRef.current ||
|
|
beforeNavigateRef.current?.() !== false
|
|
);
|
|
}
|
|
|
|
const resolvedCompanyId = companyId ?? agent.companyId;
|
|
const canonicalAgentRef = agentRouteRef(agent);
|
|
const isPaused = agent.lifecycleState ? agent.lifecycleState === "paused" : agent.status === "paused";
|
|
const isError = agent.status === "error";
|
|
const needsSetup = agent.pauseReason === "setup" && agent.status !== "terminated"
|
|
&& !["terminating", "cleaning_up", "terminated", "rejected"].includes(agent.lifecycleState ?? "");
|
|
|
|
const reportError = useCallback(
|
|
(message: string) => {
|
|
if (onActionError) {
|
|
onActionError(message);
|
|
} else {
|
|
pushToast({ title: "Action failed", body: message, tone: "error" });
|
|
}
|
|
},
|
|
[onActionError, pushToast],
|
|
);
|
|
|
|
const invalidateAgent = useCallback(() => {
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.detail(agent.id) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.detail(canonicalAgentRef) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.runtimeState(agent.id) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.taskSessions(agent.id) });
|
|
if (resolvedCompanyId) {
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.list(resolvedCompanyId) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.liveRuns(resolvedCompanyId) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.heartbeats(resolvedCompanyId, agent.id) });
|
|
}
|
|
}, [agent.id, canonicalAgentRef, queryClient, resolvedCompanyId]);
|
|
|
|
const agentAction = useMutation({
|
|
mutationFn: async (action: "invoke" | "pause" | "resume" | "clear_error" | "approve" | "terminate") => {
|
|
switch (action) {
|
|
case "invoke": return agentsApi.invoke(agent.id, resolvedCompanyId ?? undefined);
|
|
case "pause": return agentsApi.pause(agent.id, resolvedCompanyId ?? undefined);
|
|
case "resume": return agentsApi.resume(agent.id, resolvedCompanyId ?? undefined);
|
|
case "clear_error": return agentsApi.clearError(agent.id, resolvedCompanyId ?? undefined);
|
|
case "approve": return agentsApi.approve(agent.id, resolvedCompanyId ?? undefined);
|
|
case "terminate": return agentsApi.terminate(agent.id, resolvedCompanyId ?? undefined);
|
|
}
|
|
},
|
|
onSuccess: (data, action) => {
|
|
onActionError?.(null);
|
|
invalidateAgent();
|
|
if (action === "terminate") {
|
|
const state = (data as Agent).lifecycleState;
|
|
if (state === "terminating" || state === "cleaning_up") return;
|
|
if (!confirmLateNavigationChanges(agentActionStartedDirtyRef)) return;
|
|
onTerminateSuccess?.(data as Agent);
|
|
}
|
|
if (action === "invoke" && navigateToRunOnInvoke && data && typeof data === "object" && "id" in data) {
|
|
if (!confirmLateNavigationChanges(agentActionStartedDirtyRef)) return;
|
|
navigate(`/agents/${canonicalAgentRef}/runs/${(data as HeartbeatRun).id}`);
|
|
}
|
|
},
|
|
onError: (err) => {
|
|
reportError(err instanceof Error ? err.message : "Action failed");
|
|
},
|
|
});
|
|
|
|
const duplicateAgent = useMutation({
|
|
mutationFn: async () => {
|
|
if (!resolvedCompanyId) {
|
|
throw new Error("Agent is not ready to duplicate");
|
|
}
|
|
const instructionsBundle = await loadDuplicateInstructionsBundle(agent.id, resolvedCompanyId);
|
|
const payload = buildDuplicateAgentPayload(agent, instructionsBundle);
|
|
try {
|
|
return await agentsApi.create(resolvedCompanyId, payload);
|
|
} catch (error) {
|
|
if (error instanceof ApiError && error.status === 409 && error.message.includes("requires board approval")) {
|
|
const hire = await agentsApi.hire(resolvedCompanyId, payload);
|
|
return hire.agent;
|
|
}
|
|
throw error;
|
|
}
|
|
},
|
|
onSuccess: async (createdAgent) => {
|
|
onActionError?.(null);
|
|
if (resolvedCompanyId) {
|
|
await queryClient.invalidateQueries({ queryKey: queryKeys.agents.list(resolvedCompanyId) });
|
|
}
|
|
pushToast({ title: "Agent duplicated", body: createdAgent.name, tone: "success" });
|
|
if (!confirmLateNavigationChanges(duplicateStartedDirtyRef)) return;
|
|
navigate(`/agents/${agentRouteRef(createdAgent)}/dashboard`);
|
|
},
|
|
onError: (err) => {
|
|
const message = err instanceof Error ? err.message : "Failed to duplicate agent";
|
|
onActionError?.(message);
|
|
pushToast({ title: "Could not duplicate agent", body: message, tone: "error" });
|
|
},
|
|
});
|
|
|
|
const handleDuplicateAgent = useCallback(() => {
|
|
if (duplicateAgent.isPending) return;
|
|
const nextName = duplicateAgentName(agent.name);
|
|
const confirmed = window.confirm(`Duplicate ${agent.name} as ${nextName}?`);
|
|
setMoreOpen(false);
|
|
if (!confirmed || !confirmNavigationStart(duplicateStartedDirtyRef)) return;
|
|
duplicateAgent.mutate();
|
|
}, [agent.name, duplicateAgent]);
|
|
|
|
const resetTaskSession = useMutation({
|
|
mutationFn: () => agentsApi.resetSession(agent.id, null, resolvedCompanyId ?? undefined),
|
|
onSuccess: () => {
|
|
onActionError?.(null);
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.runtimeState(agent.id) });
|
|
queryClient.invalidateQueries({ queryKey: queryKeys.agents.taskSessions(agent.id) });
|
|
},
|
|
onError: (err) => {
|
|
reportError(err instanceof Error ? err.message : "Failed to reset session");
|
|
},
|
|
});
|
|
|
|
const isPendingApproval = agent.status === "pending_approval";
|
|
const disabled = actionsDisabled || agentAction.isPending;
|
|
const assignAndRunDisabled = disabled || isPendingApproval || workActionsDisabled || agent.status === "terminated";
|
|
const pauseResumeDisabled = disabled || isPendingApproval || (isPaused && workActionsDisabled)
|
|
|| Boolean(agent.lifecycleState && ["pausing", "terminating", "cleaning_up", "terminated", "rejected"].includes(agent.lifecycleState));
|
|
const clearErrorDisabled = disabled;
|
|
const runtimeConfig = agent.runtimeConfig as Record<string, unknown> | null;
|
|
const runtimeDebug =
|
|
runtimeConfig && typeof runtimeConfig.debug === "object" && runtimeConfig.debug !== null
|
|
? (runtimeConfig.debug as Record<string, unknown>)
|
|
: null;
|
|
const persistentProviderTrace = runtimeDebug?.providerTrace === "raw";
|
|
|
|
const finishSetup = () => {
|
|
if (!confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
navigate(`/agents/new?${new URLSearchParams({
|
|
name: agent.name, adapterType: agent.adapterType, setupAgentId: agent.id,
|
|
runnerProvider: String(agent.adapterConfig.provider === "acpx" ? agent.adapterConfig.acpxAgent : agent.adapterConfig.provider ?? "codex"),
|
|
...(agent.adapterType === "paperclip_runner" ? { runner: "paperclip" } : ["codex_local", "claude_local", "opencode_local", "pi_local"].includes(agent.adapterType) ? { runner: "legacy" } : {}),
|
|
...pendingAgentSetupChoice(agent),
|
|
})}`);
|
|
};
|
|
const pauseDialog = pauseConfirm ? (
|
|
<AlertDialog open={pauseConfirmOpen} onOpenChange={setPauseConfirmOpen}>
|
|
<AlertDialogContent>
|
|
<AlertDialogHeader>
|
|
<AlertDialogTitle>{pauseConfirm.title}</AlertDialogTitle>
|
|
<AlertDialogDescription asChild>
|
|
<div>{pauseConfirm.description}</div>
|
|
</AlertDialogDescription>
|
|
</AlertDialogHeader>
|
|
<AlertDialogFooter>
|
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
|
<AlertDialogAction onClick={() => agentAction.mutate("pause")}>
|
|
Pause anyway
|
|
</AlertDialogAction>
|
|
</AlertDialogFooter>
|
|
</AlertDialogContent>
|
|
</AlertDialog>
|
|
) : null;
|
|
|
|
if (workActionsInMenu) {
|
|
const item = "flex w-full items-center gap-2 rounded-sm px-2 py-1.5 text-left text-sm transition-colors hover:bg-accent disabled:pointer-events-none disabled:opacity-50";
|
|
const icon = "size-4 shrink-0 text-muted-foreground";
|
|
const select = (action: () => void) => () => {
|
|
setMoreOpen(false);
|
|
action();
|
|
};
|
|
return (
|
|
<div className={className ?? "flex shrink-0 items-center gap-2"}>
|
|
{persistentProviderTrace ? (
|
|
<span
|
|
className="hidden items-center gap-1 rounded-md border border-primary/30 bg-primary/10 px-2 py-1 text-xs font-medium text-primary lg:inline-flex"
|
|
title="Exact provider traffic will be captured for future runs and retained for up to 24 hours."
|
|
>
|
|
<Bug className="h-3.5 w-3.5" />
|
|
Raw tracing on
|
|
</span>
|
|
) : null}
|
|
{children}
|
|
<Popover open={moreOpen} onOpenChange={setMoreOpen}>
|
|
<PopoverTrigger asChild>
|
|
<Button variant="ghost" size="icon" aria-label={`Open actions for ${agent.name}`}>
|
|
<MoreHorizontal className="size-4" />
|
|
</Button>
|
|
</PopoverTrigger>
|
|
<PopoverContent className="w-52 p-1" align="end">
|
|
{needsSetup ? (
|
|
<button
|
|
type="button"
|
|
className={item}
|
|
disabled={assignAndRunDisabled}
|
|
title={workActionsDisabled ? workActionsDisabledReason : undefined}
|
|
onClick={select(() => openNewIssue({ assigneeAgentId: agent.id }))}
|
|
>
|
|
<Plus className={icon} />
|
|
{assignLabel}
|
|
</button>
|
|
) : showRun ? (
|
|
<button
|
|
type="button"
|
|
className={item}
|
|
disabled={assignAndRunDisabled}
|
|
title={workActionsDisabled ? workActionsDisabledReason : undefined}
|
|
onClick={select(() => {
|
|
if (navigateToRunOnInvoke && !confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
agentAction.mutate("invoke");
|
|
})}
|
|
>
|
|
<Play className={icon} />
|
|
{runLabel}
|
|
</button>
|
|
) : null}
|
|
{needsSetup ? null : isError ? (
|
|
<button
|
|
type="button"
|
|
className={cn(item, "text-destructive")}
|
|
disabled={clearErrorDisabled}
|
|
aria-label="Clear error and return agent to idle"
|
|
onClick={select(() => agentAction.mutate("clear_error"))}
|
|
>
|
|
<CheckCircle2 className={cn(icon, "text-destructive")} />
|
|
Clear error
|
|
</button>
|
|
) : isPaused ? (
|
|
<button type="button" className={item} disabled={pauseResumeDisabled} onClick={select(() => agentAction.mutate("resume"))}>
|
|
<Play className={icon} />
|
|
Resume
|
|
</button>
|
|
) : (
|
|
<button
|
|
type="button"
|
|
className={item}
|
|
disabled={pauseResumeDisabled}
|
|
onClick={select(() => (pauseConfirm ? setPauseConfirmOpen(true) : agentAction.mutate("pause")))}
|
|
>
|
|
<Pause className={icon} />
|
|
Pause
|
|
</button>
|
|
)}
|
|
<div role="separator" className="-mx-1 my-1 h-px bg-border" />
|
|
{menuItems.map(({ label, icon: Icon, onSelect, disabled: itemDisabled }) => (
|
|
<button key={label} type="button" className={item} disabled={itemDisabled} onClick={select(onSelect)}>
|
|
<Icon className={icon} />
|
|
{label}
|
|
</button>
|
|
))}
|
|
<button type="button" className={item} disabled={duplicateAgent.isPending} onClick={handleDuplicateAgent}>
|
|
{duplicateAgent.isPending ? <Loader2 className={cn(icon, "animate-spin")} /> : <Copy className={icon} />}
|
|
Duplicate agent
|
|
</button>
|
|
<button
|
|
type="button"
|
|
className={item}
|
|
onClick={select(() => {
|
|
void copyTextToClipboard(agent.id).catch(() => {
|
|
pushToast({ title: "Copy failed", body: "Clipboard access is unavailable.", tone: "error" });
|
|
});
|
|
})}
|
|
>
|
|
<Copy className={icon} />
|
|
Copy agent ID
|
|
</button>
|
|
<button type="button" className={item} onClick={select(() => resetTaskSession.mutate())}>
|
|
<RotateCcw className={icon} />
|
|
Reset sessions
|
|
</button>
|
|
{hideTerminate ? null : (
|
|
<>
|
|
<div role="separator" className="-mx-1 my-1 h-px bg-border" />
|
|
<button
|
|
type="button"
|
|
className={cn(item, "text-destructive")}
|
|
onClick={select(() => {
|
|
if (onTerminateSuccess && !confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
agentAction.mutate("terminate");
|
|
})}
|
|
>
|
|
<Trash2 className={cn(icon, "text-destructive")} />
|
|
Terminate
|
|
</button>
|
|
</>
|
|
)}
|
|
</PopoverContent>
|
|
</Popover>
|
|
{needsSetup ? (
|
|
<Button size={size} disabled={disabled} onClick={finishSetup}>
|
|
<Play className="size-4" />
|
|
Finish setup
|
|
</Button>
|
|
) : (
|
|
<Button
|
|
size={size}
|
|
onClick={() => openNewIssue({ assigneeAgentId: agent.id })}
|
|
disabled={assignAndRunDisabled}
|
|
title={workActionsDisabled ? workActionsDisabledReason : undefined}
|
|
>
|
|
<Plus className="size-4" />
|
|
{assignLabel}
|
|
</Button>
|
|
)}
|
|
{pauseDialog}
|
|
</div>
|
|
);
|
|
}
|
|
|
|
return (
|
|
<div className={className ?? "flex items-center gap-1 sm:gap-2 shrink-0"}>
|
|
{persistentProviderTrace ? (
|
|
<span
|
|
className="hidden items-center gap-1 rounded-md border border-primary/30 bg-primary/10 px-2 py-1 text-xs font-medium text-primary lg:inline-flex"
|
|
title="Exact provider traffic will be captured for future runs and retained for up to 24 hours."
|
|
>
|
|
<Bug className="h-3.5 w-3.5" />
|
|
Raw tracing on
|
|
</span>
|
|
) : null}
|
|
<Button
|
|
variant="outline"
|
|
size={size}
|
|
onClick={() => openNewIssue({ assigneeAgentId: agent.id })}
|
|
disabled={assignAndRunDisabled}
|
|
title={workActionsDisabled ? workActionsDisabledReason : undefined}
|
|
>
|
|
<Plus className="h-3.5 w-3.5 sm:mr-1" />
|
|
<span className="hidden sm:inline">{assignLabel}</span>
|
|
</Button>
|
|
{needsSetup ? (
|
|
<Button variant="outline" size={size} disabled={disabled} onClick={() => {
|
|
if (!confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
navigate(`/agents/new?${new URLSearchParams({
|
|
name: agent.name, adapterType: agent.adapterType, setupAgentId: agent.id,
|
|
runnerProvider: String(agent.adapterConfig.provider === "acpx" ? agent.adapterConfig.acpxAgent : agent.adapterConfig.provider ?? "codex"),
|
|
...(agent.adapterType === "paperclip_runner" ? { runner: "paperclip" } : ["codex_local", "claude_local", "opencode_local", "pi_local"].includes(agent.adapterType) ? { runner: "legacy" } : {}),
|
|
...pendingAgentSetupChoice(agent),
|
|
})}`);
|
|
}}>Finish setup</Button>
|
|
) : showRun && <RunButton
|
|
onClick={() => {
|
|
if (navigateToRunOnInvoke && !confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
agentAction.mutate("invoke");
|
|
}}
|
|
disabled={assignAndRunDisabled}
|
|
label={runLabel}
|
|
size={size}
|
|
/>}
|
|
{!needsSetup && (isError ? (
|
|
<ClearErrorButton
|
|
onClick={() => agentAction.mutate("clear_error")}
|
|
disabled={clearErrorDisabled}
|
|
size={size}
|
|
/>
|
|
) : (
|
|
<PauseResumeButton
|
|
isPaused={isPaused}
|
|
onPause={() => (pauseConfirm ? setPauseConfirmOpen(true) : agentAction.mutate("pause"))}
|
|
onResume={() => agentAction.mutate("resume")}
|
|
disabled={pauseResumeDisabled}
|
|
size={size}
|
|
/>
|
|
))}
|
|
{pauseConfirm && (
|
|
<AlertDialog open={pauseConfirmOpen} onOpenChange={setPauseConfirmOpen}>
|
|
<AlertDialogContent>
|
|
<AlertDialogHeader>
|
|
<AlertDialogTitle>{pauseConfirm.title}</AlertDialogTitle>
|
|
<AlertDialogDescription asChild>
|
|
<div>{pauseConfirm.description}</div>
|
|
</AlertDialogDescription>
|
|
</AlertDialogHeader>
|
|
<AlertDialogFooter>
|
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
|
<AlertDialogAction onClick={() => agentAction.mutate("pause")}>
|
|
Pause anyway
|
|
</AlertDialogAction>
|
|
</AlertDialogFooter>
|
|
</AlertDialogContent>
|
|
</AlertDialog>
|
|
)}
|
|
{showStatus && (
|
|
<span className="hidden sm:inline">
|
|
<AgentStatusBadge status={agent.status} />
|
|
</span>
|
|
)}
|
|
{children}
|
|
<Popover open={moreOpen} onOpenChange={setMoreOpen}>
|
|
<PopoverTrigger asChild>
|
|
<Button variant="ghost" size="icon-xs" aria-label={`Open actions for ${agent.name}`}>
|
|
<MoreHorizontal className="h-4 w-4" />
|
|
</Button>
|
|
</PopoverTrigger>
|
|
<PopoverContent className="w-44 p-1" align="end">
|
|
<button
|
|
className="flex items-center gap-2 w-full px-2 py-1.5 text-xs rounded hover:bg-accent/50"
|
|
disabled={duplicateAgent.isPending}
|
|
onClick={handleDuplicateAgent}
|
|
>
|
|
{duplicateAgent.isPending ? (
|
|
<Loader2 className="h-3 w-3 animate-spin" />
|
|
) : (
|
|
<Copy className="h-3 w-3" />
|
|
)}
|
|
Duplicate Agent
|
|
</button>
|
|
<button
|
|
className="flex items-center gap-2 w-full px-2 py-1.5 text-xs rounded hover:bg-accent/50"
|
|
onClick={() => {
|
|
void copyTextToClipboard(agent.id).catch(() => {
|
|
pushToast({ title: "Copy failed", body: "Clipboard access is unavailable.", tone: "error" });
|
|
});
|
|
setMoreOpen(false);
|
|
}}
|
|
>
|
|
<Copy className="h-3 w-3" />
|
|
Copy Agent ID
|
|
</button>
|
|
<button
|
|
className="flex items-center gap-2 w-full px-2 py-1.5 text-xs rounded hover:bg-accent/50"
|
|
onClick={() => {
|
|
resetTaskSession.mutate();
|
|
setMoreOpen(false);
|
|
}}
|
|
>
|
|
<RotateCcw className="h-3 w-3" />
|
|
Reset Sessions
|
|
</button>
|
|
{!hideTerminate && (
|
|
<button
|
|
className="flex items-center gap-2 w-full px-2 py-1.5 text-xs rounded hover:bg-accent/50 text-destructive"
|
|
onClick={() => {
|
|
setMoreOpen(false);
|
|
if (onTerminateSuccess && !confirmNavigationStart(agentActionStartedDirtyRef)) return;
|
|
agentAction.mutate("terminate");
|
|
}}
|
|
>
|
|
<Trash2 className="h-3 w-3" />
|
|
Terminate
|
|
</button>
|
|
)}
|
|
</PopoverContent>
|
|
</Popover>
|
|
</div>
|
|
);
|
|
}
|