feat(page): Refactor agent management UI for improved layout and accessibility

This commit is contained in:
Abhimanyu Saharan
2026-02-05 00:35:46 +05:30
parent 81ca340ba8
commit 5135887c3f

View File

@@ -1,6 +1,6 @@
"use client"; "use client";
import { useEffect, useMemo, useState } from "react"; import { useMemo, useState } from "react";
import Link from "next/link"; import Link from "next/link";
import { useRouter } from "next/navigation"; import { useRouter } from "next/navigation";
@@ -18,7 +18,7 @@ import { useQueryClient } from "@tanstack/react-query";
import { StatusPill } from "@/components/atoms/StatusPill"; import { StatusPill } from "@/components/atoms/StatusPill";
import { DashboardSidebar } from "@/components/organisms/DashboardSidebar"; import { DashboardSidebar } from "@/components/organisms/DashboardSidebar";
import { DashboardShell } from "@/components/templates/DashboardShell"; import { DashboardShell } from "@/components/templates/DashboardShell";
import { Button } from "@/components/ui/button"; import { Button, buttonVariants } from "@/components/ui/button";
import { import {
Dialog, Dialog,
DialogContent, DialogContent,
@@ -27,13 +27,6 @@ import {
DialogHeader, DialogHeader,
DialogTitle, DialogTitle,
} from "@/components/ui/dialog"; } from "@/components/ui/dialog";
import {
Select,
SelectContent,
SelectItem,
SelectTrigger,
SelectValue,
} from "@/components/ui/select";
import { apiRequest, useAuthedMutation, useAuthedQuery } from "@/lib/api-query"; import { apiRequest, useAuthedMutation, useAuthedQuery } from "@/lib/api-query";
type Agent = { type Agent = {
@@ -53,14 +46,6 @@ type Board = {
slug: string; slug: string;
}; };
type GatewayStatus = {
connected: boolean;
gateway_url: string;
sessions_count?: number;
sessions?: Record<string, unknown>[];
error?: string;
};
const parseTimestamp = (value?: string | null) => { const parseTimestamp = (value?: string | null) => {
if (!value) return null; if (!value) return null;
const hasTz = /[zZ]|[+-]\d\d:\d\d$/.test(value); const hasTz = /[zZ]|[+-]\d\d:\d\d$/.test(value);
@@ -104,7 +89,6 @@ export default function AgentsPage() {
const queryClient = useQueryClient(); const queryClient = useQueryClient();
const router = useRouter(); const router = useRouter();
const [boardId, setBoardId] = useState("");
const [sorting, setSorting] = useState<SortingState>([ const [sorting, setSorting] = useState<SortingState>([
{ id: "name", desc: false }, { id: "name", desc: false },
]); ]);
@@ -123,27 +107,6 @@ export default function AgentsPage() {
const boards = useMemo(() => boardsQuery.data ?? [], [boardsQuery.data]); const boards = useMemo(() => boardsQuery.data ?? [], [boardsQuery.data]);
const agents = useMemo(() => agentsQuery.data ?? [], [agentsQuery.data]); const agents = useMemo(() => agentsQuery.data ?? [], [agentsQuery.data]);
useEffect(() => {
if (!boardId && boards.length > 0) {
setBoardId(boards[0].id);
}
}, [boardId, boards]);
const statusPath = boardId
? `/api/v1/gateways/status?board_id=${boardId}`
: null;
const gatewayStatusQuery = useAuthedQuery<GatewayStatus>(
["gateway-status", boardId || "all"],
statusPath,
{
enabled: Boolean(statusPath),
refetchInterval: 15_000,
}
);
const gatewayStatus = gatewayStatusQuery.data ?? null;
const deleteMutation = useAuthedMutation<void, Agent, { previous?: Agent[] }>( const deleteMutation = useAuthedMutation<void, Agent, { previous?: Agent[] }>(
async (agent, token) => async (agent, token) =>
apiRequest(`/api/v1/agents/${agent.id}`, { apiRequest(`/api/v1/agents/${agent.id}`, {
@@ -180,14 +143,6 @@ export default function AgentsPage() {
deleteMutation.mutate(deleteTarget); deleteMutation.mutate(deleteTarget);
}; };
const handleRefresh = async () => {
await Promise.all([
boardsQuery.refetch(),
agentsQuery.refetch(),
gatewayStatusQuery.refetch(),
]);
};
const columns = useMemo<ColumnDef<Agent>[]>( const columns = useMemo<ColumnDef<Agent>[]>(
() => { () => {
const resolveBoardName = (agent: Agent) => const resolveBoardName = (agent: Agent) =>
@@ -198,10 +153,12 @@ export default function AgentsPage() {
accessorKey: "name", accessorKey: "name",
header: "Agent", header: "Agent",
cell: ({ row }) => ( cell: ({ row }) => (
<div> <Link href={`/agents/${row.original.id}`} className="group block">
<p className="font-medium text-strong">{row.original.name}</p> <p className="text-sm font-medium text-slate-900 group-hover:text-blue-600">
<p className="text-xs text-quiet">ID {row.original.id}</p> {row.original.name}
</div> </p>
<p className="text-xs text-slate-500">ID {row.original.id}</p>
</Link>
), ),
}, },
{ {
@@ -213,7 +170,7 @@ export default function AgentsPage() {
accessorKey: "openclaw_session_id", accessorKey: "openclaw_session_id",
header: "Session", header: "Session",
cell: ({ row }) => ( cell: ({ row }) => (
<span className="text-xs text-muted"> <span className="text-sm text-slate-700">
{truncate(row.original.openclaw_session_id)} {truncate(row.original.openclaw_session_id)}
</span> </span>
), ),
@@ -222,7 +179,7 @@ export default function AgentsPage() {
accessorKey: "board_id", accessorKey: "board_id",
header: "Board", header: "Board",
cell: ({ row }) => ( cell: ({ row }) => (
<span className="text-xs text-muted"> <span className="text-sm text-slate-700">
{resolveBoardName(row.original)} {resolveBoardName(row.original)}
</span> </span>
), ),
@@ -231,21 +188,16 @@ export default function AgentsPage() {
accessorKey: "last_seen_at", accessorKey: "last_seen_at",
header: "Last seen", header: "Last seen",
cell: ({ row }) => ( cell: ({ row }) => (
<div className="text-xs text-muted"> <span className="text-sm text-slate-700">
<p className="font-medium text-strong"> {formatRelative(row.original.last_seen_at)}
{formatRelative(row.original.last_seen_at)} </span>
</p>
<p className="text-quiet">
{formatTimestamp(row.original.last_seen_at)}
</p>
</div>
), ),
}, },
{ {
accessorKey: "updated_at", accessorKey: "updated_at",
header: "Updated", header: "Updated",
cell: ({ row }) => ( cell: ({ row }) => (
<span className="text-xs text-muted"> <span className="text-sm text-slate-700">
{formatTimestamp(row.original.updated_at)} {formatTimestamp(row.original.updated_at)}
</span> </span>
), ),
@@ -254,24 +206,15 @@ export default function AgentsPage() {
id: "actions", id: "actions",
header: "", header: "",
cell: ({ row }) => ( cell: ({ row }) => (
<div <div className="flex justify-end gap-2">
className="flex items-center justify-end gap-2"
onClick={(event) => event.stopPropagation()}
>
<Link
href={`/agents/${row.original.id}`}
className="inline-flex h-8 items-center justify-center rounded-md border border-slate-200 px-3 text-xs font-medium text-slate-600 transition hover:border-slate-300 hover:text-slate-900"
>
View
</Link>
<Link <Link
href={`/agents/${row.original.id}/edit`} href={`/agents/${row.original.id}/edit`}
className="inline-flex h-8 items-center justify-center rounded-md border border-slate-200 px-3 text-xs font-medium text-slate-600 transition hover:border-slate-300 hover:text-slate-900" className={buttonVariants({ variant: "ghost", size: "sm" })}
> >
Edit Edit
</Link> </Link>
<Button <Button
variant="outline" variant="ghost"
size="sm" size="sm"
onClick={() => setDeleteTarget(row.original)} onClick={() => setDeleteTarget(row.original)}
> >
@@ -314,79 +257,58 @@ export default function AgentsPage() {
<SignedIn> <SignedIn>
<DashboardSidebar /> <DashboardSidebar />
<main className="flex-1 overflow-y-auto bg-slate-50"> <main className="flex-1 overflow-y-auto bg-slate-50">
<div className="border-b border-slate-200 bg-white px-8 py-6"> <div className="sticky top-0 z-30 border-b border-slate-200 bg-white">
<div className="flex flex-wrap items-center justify-between gap-4"> <div className="px-8 py-6">
<div> <div className="flex items-center justify-between">
<h2 className="font-heading text-2xl font-semibold text-slate-900 tracking-tight"> <div>
Agents <h1 className="text-2xl font-semibold tracking-tight text-slate-900">
</h2> Agents
<p className="mt-1 text-sm text-slate-500"> </h1>
{agents.length} agent{agents.length === 1 ? "" : "s"} total. <p className="mt-1 text-sm text-slate-500">
</p> {agents.length} agent{agents.length === 1 ? "" : "s"} total.
</div> </p>
{agents.length > 0 ? ( </div>
<div className="flex items-center gap-2"> {agents.length > 0 ? (
<Button
variant="outline"
onClick={handleRefresh}
disabled={agentsQuery.isLoading}
>
Refresh
</Button>
<Button onClick={() => router.push("/agents/new")}> <Button onClick={() => router.push("/agents/new")}>
New agent New agent
</Button> </Button>
</div> ) : null}
) : null} </div>
</div> </div>
</div> </div>
<div className="p-8"> <div className="p-8">
{agentsQuery.error ? ( <div className="overflow-hidden rounded-xl border border-slate-200 bg-white shadow-sm">
<div className="rounded-lg border border-slate-200 bg-white p-3 text-sm text-slate-600 shadow-sm"> <div className="overflow-x-auto">
{agentsQuery.error.message} <table className="w-full text-left text-sm">
</div> <thead className="sticky top-0 z-10 bg-slate-50 text-xs font-semibold uppercase tracking-wider text-slate-500">
) : null} {table.getHeaderGroups().map((headerGroup) => (
<tr key={headerGroup.id}>
{agents.length === 0 && !agentsQuery.isLoading ? ( {headerGroup.headers.map((header) => (
<div className="flex flex-1 flex-col items-center justify-center gap-4 rounded-xl border border-dashed border-slate-200 bg-white/70 p-10 text-center text-sm text-slate-500"> <th key={header.id} className="px-6 py-3">
<p>No agents yet. Create your first agent to get started.</p> {header.isPlaceholder
<Button onClick={() => router.push("/agents/new")}> ? null
Create your first agent : flexRender(
</Button> header.column.columnDef.header,
</div> header.getContext()
) : ( )}
<div className="overflow-hidden rounded-xl border border-slate-200 bg-white shadow-sm"> </th>
<div className="overflow-x-auto"> ))}
<table className="min-w-full divide-y divide-slate-200 text-sm"> </tr>
<thead className="bg-slate-50"> ))}
{table.getHeaderGroups().map((headerGroup) => ( </thead>
<tr key={headerGroup.id}> <tbody className="divide-y divide-slate-100">
{headerGroup.headers.map((header) => ( {agentsQuery.isLoading ? (
<th <tr>
key={header.id} <td colSpan={columns.length} className="px-6 py-8">
className="px-4 py-3 text-left text-[11px] font-semibold uppercase tracking-wider text-slate-500" <span className="text-sm text-slate-500">Loading</span>
> </td>
{header.isPlaceholder </tr>
? null ) : table.getRowModel().rows.length ? (
: flexRender( table.getRowModel().rows.map((row) => (
header.column.columnDef.header, <tr key={row.id} className="hover:bg-slate-50">
header.getContext()
)}
</th>
))}
</tr>
))}
</thead>
<tbody className="divide-y divide-slate-200 bg-white">
{table.getRowModel().rows.map((row) => (
<tr
key={row.id}
className="cursor-pointer transition hover:bg-slate-50"
onClick={() => router.push(`/agents/${row.original.id}`)}
>
{row.getVisibleCells().map((cell) => ( {row.getVisibleCells().map((cell) => (
<td key={cell.id} className="px-4 py-3 align-top"> <td key={cell.id} className="px-6 py-4">
{flexRender( {flexRender(
cell.column.columnDef.cell, cell.column.columnDef.cell,
cell.getContext() cell.getContext()
@@ -394,55 +316,54 @@ export default function AgentsPage() {
</td> </td>
))} ))}
</tr> </tr>
))} ))
</tbody> ) : (
</table> <tr>
</div> <td colSpan={columns.length} className="px-6 py-16">
<div className="flex flex-col items-center justify-center text-center">
<div className="mb-4 rounded-full bg-slate-50 p-4">
<svg
className="h-16 w-16 text-slate-300"
viewBox="0 0 24 24"
fill="none"
stroke="currentColor"
strokeWidth="1.5"
strokeLinecap="round"
strokeLinejoin="round"
>
<path d="M16 21v-2a4 4 0 0 0-4-4H6a4 4 0 0 0-4 4v2" />
<circle cx="9" cy="7" r="4" />
<path d="M22 21v-2a4 4 0 0 0-3-3.87" />
<path d="M16 3.13a4 4 0 0 1 0 7.75" />
</svg>
</div>
<h3 className="mb-2 text-lg font-semibold text-slate-900">
No agents yet
</h3>
<p className="mb-6 max-w-md text-sm text-slate-500">
Create your first agent to start executing tasks
on this board.
</p>
<Link
href="/agents/new"
className={buttonVariants({ size: "md", variant: "primary" })}
>
Create your first agent
</Link>
</div>
</td>
</tr>
)}
</tbody>
</table>
</div> </div>
)}
<div className="mt-6 rounded-xl border border-slate-200 bg-white p-4 text-sm text-slate-600 shadow-sm">
<div className="flex flex-wrap items-center justify-between gap-3">
<div>
<p className="text-xs font-semibold uppercase tracking-wider text-slate-500">
Gateway status
</p>
<p className="mt-1 text-sm font-semibold text-slate-900">
{gatewayStatus?.gateway_url ?? "Gateway URL not set"}
</p>
</div>
<div className="flex items-center gap-3">
<Select
value={boardId}
onValueChange={(value) => setBoardId(value)}
disabled={boards.length === 0}
>
<SelectTrigger className="h-8 w-[200px]">
<SelectValue placeholder="Select board" />
</SelectTrigger>
<SelectContent>
{boards.map((board) => (
<SelectItem key={board.id} value={board.id}>
{board.name}
</SelectItem>
))}
</SelectContent>
</Select>
<StatusPill status={gatewayStatus?.connected ? "online" : "offline"} />
<span className="text-xs text-slate-500">
{gatewayStatus?.sessions_count ?? 0} sessions
</span>
</div>
</div>
{gatewayStatus?.error ? (
<p className="mt-3 text-xs text-red-600">{gatewayStatus.error}</p>
) : null}
{gatewayStatusQuery.error ? (
<p className="mt-3 text-xs text-red-600">
{gatewayStatusQuery.error.message}
</p>
) : null}
</div> </div>
{agentsQuery.error ? (
<p className="mt-4 text-sm text-red-500">
{agentsQuery.error.message}
</p>
) : null}
</div> </div>
</main> </main>
</SignedIn> </SignedIn>