@hanphone/dsh-a2a 0.1.0 → 0.3.0

package/src/api.ts

@@ -1,9 +1,10 @@
 /**
  * GUI control API for the A2A settings dashboard: one `GET /a2a/api`
- * snapshot plus `POST /a2a/api` control actions (server toggle, outbound
- * agent management, task view). The browser half talks only to this route;
- * it carries no protocol knowledge. Loopback-only by default so the
- * dashboard cannot be driven from the wire.
+ * snapshot plus `POST /a2a/api` control actions (inbound/outbound server
+ * CRUD + enable/disable, task view/cancel) and `GET /a2a/api/presets` for the
+ * preset roster the pickers need. The browser half talks only to this route;
+ * it carries no protocol knowledge. Loopback-only by default so the dashboard
+ * cannot be driven from the wire.
  *
  * The route body is bound to the same `A2AServiceImpl` facade the `/a2a`
  * command surface uses, so GUI actions and commands cannot disagree.
@@ -15,16 +16,19 @@ import type { A2AServiceImpl } from './service.ts'
 
 /** Control actions the dashboard can issue. */
 export type ApiAction =
-  | { readonly action: 'server.enable' | 'server.disable' }
-  | { readonly action: 'agent.add'; readonly name: string; readonly agentCardUrl: string; readonly bearerTokenEnv?: string }
-  | { readonly action: 'agent.remove' | 'agent.enable' | 'agent.disable' | 'agent.refresh'; readonly id: string }
+  | { readonly action: 'inbound.create'; readonly name: string; readonly description: string; readonly version: string; readonly endpointPath?: string; readonly preset?: string; readonly authTokenEnv?: string; readonly skills?: readonly { id: string; name: string; description?: string }[]; readonly enabled?: boolean }
+  | { readonly action: 'inbound.remove' | 'inbound.enable' | 'inbound.disable' | 'inbound.update'; readonly id: string; readonly name?: string; readonly description?: string; readonly version?: string; readonly endpointPath?: string; readonly preset?: string; readonly authTokenEnv?: string; readonly skills?: readonly { id: string; name: string; description?: string }[] }
+  | { readonly action: 'outbound.create'; readonly name: string; readonly agentCardUrl: string; readonly bearerTokenEnv?: string; readonly preset?: string; readonly enabled?: boolean; readonly timeoutMs?: number }
+  | { readonly action: 'outbound.remove' | 'outbound.enable' | 'outbound.disable' | 'outbound.refresh'; readonly id: string }
   | { readonly action: 'task.cancel'; readonly id: string }
+  | { readonly action: 'inbound.close'; readonly id: string }
 
 /** One snapshot of the whole plugin for the dashboard. */
 export interface ApiSnapshot {
-  readonly server: { readonly enabled: boolean; readonly cardUrl?: string; readonly skills: readonly string[] }
+  readonly inbounds: readonly unknown[]
+  readonly outbounds: readonly unknown[]
   readonly tasks: readonly unknown[]
-  readonly agents: readonly unknown[]
+  readonly peers: readonly unknown[]
 }
 
 function readBody(req: IncomingMessage): Promise<string> {
@@ -58,6 +62,11 @@ export async function handleApiRequest(
     res.end('forbidden')
     return
   }
+  const path = (req.url ?? '').split('?')[0] ?? ''
+  if (req.method === 'GET' && path === '/a2a/api/presets') {
+    json(res, 200, await impl.presets())
+    return
+  }
   if (req.method === 'GET') {
     json(res, 200, snapshotOf(impl))
     return
@@ -79,37 +88,67 @@ export async function handleApiRequest(
 }
 
 function snapshotOf(impl: A2AServiceImpl): ApiSnapshot {
-  const status = impl.status() as {
-    server: { enabled: boolean; cardUrl?: string; skills: readonly string[] }
-    tasks: number
-    agents: readonly unknown[]
-  }
+  const status = impl.status() as { inbounds: readonly unknown[]; outbounds: readonly unknown[]; tasks: number; peers?: readonly unknown[] }
   return {
-    server: status.server,
+    inbounds: status.inbounds ?? [],
+    outbounds: status.outbounds ?? [],
     tasks: impl.listTasks() as readonly unknown[],
-    agents: impl.agents() as readonly unknown[],
+    // `status()` may already carry aggregated peers; fall back to the facade.
+    peers: (status.peers ?? impl.inbounds()) as readonly unknown[],
   }
 }
 
 async function dispatch(payload: ApiAction, impl: A2AServiceImpl): Promise<{ readonly ok: boolean; readonly message: string }> {
   switch (payload.action) {
-    case 'server.enable':
-      return impl.enableServer(true)
-    case 'server.disable':
-      return impl.enableServer(false)
-    case 'agent.add':
-      return impl.addAgent({ name: payload.name, agentCardUrl: payload.agentCardUrl, ...(payload.bearerTokenEnv ? { bearerTokenEnv: payload.bearerTokenEnv } : {}) })
-    case 'agent.remove':
-      return impl.removeAgent(payload.id)
-    case 'agent.enable':
-      return impl.setAgentEnabled(payload.id, true)
-    case 'agent.disable':
-      return impl.setAgentEnabled(payload.id, false)
-    case 'agent.refresh':
-      return impl.refreshAgentCard(payload.id)
+    case 'inbound.create':
+      return impl.createInboundServer({
+        name: payload.name,
+        description: payload.description,
+        version: payload.version,
+        ...(payload.endpointPath !== undefined ? { endpointPath: payload.endpointPath } : {}),
+        ...(payload.preset !== undefined ? { preset: payload.preset } : {}),
+        ...(payload.authTokenEnv !== undefined ? { authTokenEnv: payload.authTokenEnv } : {}),
+        ...(payload.skills !== undefined ? { skills: payload.skills } : {}),
+        ...(payload.enabled !== undefined ? { enabled: payload.enabled } : {}),
+      })
+    case 'inbound.remove':
+      return impl.removeInboundServer(payload.id)
+    case 'inbound.enable':
+      return impl.setInboundServerEnabled(payload.id, true)
+    case 'inbound.disable':
+      return impl.setInboundServerEnabled(payload.id, false)
+    case 'inbound.update':
+      return impl.updateInboundServer(payload.id, {
+        ...(payload.name !== undefined ? { name: payload.name } : {}),
+        ...(payload.description !== undefined ? { description: payload.description } : {}),
+        ...(payload.version !== undefined ? { version: payload.version } : {}),
+        ...(payload.endpointPath !== undefined ? { endpointPath: payload.endpointPath } : {}),
+        ...(payload.preset !== undefined ? { preset: payload.preset } : {}),
+        ...(payload.authTokenEnv !== undefined ? { authTokenEnv: payload.authTokenEnv } : {}),
+        ...(payload.skills !== undefined ? { skills: payload.skills } : {}),
+      })
+    case 'outbound.create':
+      return impl.createOutboundServer({
+        name: payload.name,
+        agentCardUrl: payload.agentCardUrl,
+        ...(payload.bearerTokenEnv !== undefined ? { bearerTokenEnv: payload.bearerTokenEnv } : {}),
+        ...(payload.preset !== undefined ? { preset: payload.preset } : {}),
+        ...(payload.enabled !== undefined ? { enabled: payload.enabled } : {}),
+        ...(payload.timeoutMs !== undefined ? { timeoutMs: payload.timeoutMs } : {}),
+      })
+    case 'outbound.remove':
+      return impl.removeOutboundServer(payload.id)
+    case 'outbound.enable':
+      return impl.setOutboundServerEnabled(payload.id, true)
+    case 'outbound.disable':
+      return impl.setOutboundServerEnabled(payload.id, false)
+    case 'outbound.refresh':
+      return impl.refreshOutboundServer(payload.id)
     case 'task.cancel':
       return impl.cancelTask(payload.id)
+    case 'inbound.close':
+      return impl.closeInbound(payload.id)
     default:
       return { ok: false, message: `unknown action ${String((payload as { action?: unknown }).action)}` }
   }
-}
+}