@hanphone/dsh-a2a 0.2.0 → 0.3.1

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,27 +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 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 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: 'identity.update'; readonly name?: string; readonly description?: string; readonly version?: 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 name?: string
-    readonly description?: string
-    readonly version?: string
-    readonly configured: boolean
-  }
-  readonly tasks: readonly unknown[]
-  readonly agents: readonly unknown[]
   readonly inbounds: readonly unknown[]
+  readonly outbounds: readonly unknown[]
+  readonly tasks: readonly unknown[]
+  readonly peers: readonly unknown[]
 }
 
 function readBody(req: IncomingMessage): Promise<string> {
@@ -69,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
@@ -90,51 +88,65 @@ export async function handleApiRequest(
 }
 
 function snapshotOf(impl: A2AServiceImpl): ApiSnapshot {
-  const status = impl.status() as {
-    server: {
-      enabled: boolean
-      cardUrl?: string
-      skills: readonly string[]
-      name?: string
-      description?: string
-      version?: string
-      configured: boolean
-    }
-    tasks: number
-    agents: readonly unknown[]
-    inbounds: readonly unknown[]
-  }
+  const status = impl.status() as { inbounds: readonly unknown[]; outbounds: readonly unknown[]; tasks: number; peers?: readonly unknown[] }
   return {
-    server: status.server,
-    tasks: impl.listTasks() as readonly unknown[],
-    agents: impl.agents() as readonly unknown[],
     inbounds: status.inbounds ?? [],
+    outbounds: status.outbounds ?? [],
+    tasks: impl.listTasks() 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.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 } : {}),
+      })
+    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 'identity.update':
-      return impl.updateIdentity({ ...(payload.name !== undefined ? { name: payload.name } : {}), ...(payload.description !== undefined ? { description: payload.description } : {}), ...(payload.version !== undefined ? { version: payload.version } : {}) })
     case 'inbound.close':
       return impl.closeInbound(payload.id)
     default:
       return { ok: false, message: `unknown action ${String((payload as { action?: unknown }).action)}` }
   }
-}
+}