fix: remove dead code

Author: avi-alpert

src/cli/operations/dev/web-ui/README.md

@@ -91,8 +91,6 @@ agent name and error message:
 
 Errors are cleared when the agent is successfully started again via `POST /api/start`.
 
-In invoke mode, all agents appear in `running` with `port: 0` (they're deployed remotely) and `errors` is always empty.
-
 The agent list is kept in sync with `agentcore.json` via `fs.watch` — if you add or remove an agent in another terminal,
 the status endpoint reflects the change without restarting the dev server.
 
@@ -171,8 +169,7 @@ Error:
 ### `POST /invocations`
 
 Proxies a chat invocation to the selected running agent. The `agentName` field routes to the correct agent; falls back
-to the first running agent if omitted. In invoke mode, delegates to the custom invocation handler (calls deployed
-runtime).
+to the first running agent if omitted.
 
 Request:
 
@@ -182,7 +179,7 @@ Request:
 
 ### `GET /api/traces?agentName=xxx[&startTime=ms&endTime=ms]`
 
-Lists recent traces for a deployed agent. Only available in invoke mode.
+Lists recent traces for an agent. Available when the OTEL collector is active.
 
 Query parameters:
 
@@ -198,7 +195,7 @@ Response:
 
 ### `GET /api/traces/:traceId?agentName=xxx[&startTime=ms&endTime=ms]`
 
-Returns full trace data (spans) for a specific trace. Only available in invoke mode.
+Returns full trace data (spans) for a specific trace. Available when the OTEL collector is active.
 
 Query parameters:
 
@@ -214,7 +211,7 @@ Response:
 
 ### `GET /api/memory?memoryName=xxx&namespace=yyy[&strategyId=zzz]`
 
-Lists memory records for a given memory and namespace. Only available in invoke mode.
+Lists memory records for a given memory and namespace. Requires a deployed memory with `onListMemoryRecords` handler.
 
 Response:
 
@@ -224,7 +221,7 @@ Response:
 
 ### `POST /api/memory/search`
 
-Performs semantic search across memory records. Only available in invoke mode.
+Performs semantic search across memory records. Requires a deployed memory with `onRetrieveMemoryRecords` handler.
 
 Request:
 

src/cli/operations/dev/web-ui/api-types.ts

@@ -32,7 +32,7 @@ export interface StatusAgent {
 /** Running agent entry in the status response */
 export interface StatusRunningAgent {
   name: string;
-  /** Port the agent is listening on. 0 in invoke mode (agents are remote). */
+  /** Port the agent is listening on. */
   port: number;
 }
 

src/cli/operations/dev/web-ui/handlers/invocations.ts

@@ -8,7 +8,6 @@ let a2aRequestId = 1;
 /**
  * POST /invocations — proxy to the selected agent.
  * Body must include agentName to route to the correct running agent.
- * When onInvoke is provided, delegates entirely to the custom handler (e.g. for deployed agent invocation).
  */
 export async function handleInvocations(
   ctx: RouteContext,
@@ -18,14 +17,6 @@ export async function handleInvocations(
 ): Promise<void> {
   const body = await ctx.readBody(req);
 
-  // Delegate to custom invocation handler if provided (e.g. invoke mode — calls deployed runtime)
-  if (ctx.options.onInvoke) {
-    const setCors = () => ctx.setCorsHeaders(res, origin);
-    await ctx.options.onInvoke(body, res, setCors);
-    return;
-  }
-
-  // Default dev mode: proxy to local agent server
   let agentPort: number | undefined;
   let agentName: string | undefined;
   let agentProtocol: string | undefined;

src/cli/operations/dev/web-ui/handlers/memory.ts

@@ -4,7 +4,7 @@ import type { IncomingMessage, ServerResponse } from 'node:http';
 
 /**
  * GET /api/memory?memoryName=xxx&namespace=yyy[&strategyId=zzz]
- * Lists memory records. Requires onListMemoryRecords handler (invoke mode only).
+ * Lists memory records. Requires onListMemoryRecords handler.
  */
 export async function handleListMemoryRecords(
   ctx: RouteContext,
@@ -15,7 +15,7 @@ export async function handleListMemoryRecords(
   if (!ctx.options.onListMemoryRecords) {
     ctx.setCorsHeaders(res, origin);
     res.writeHead(404, { 'Content-Type': 'application/json' });
-    res.end(JSON.stringify({ success: false, error: 'Memory browsing is only available in invoke mode' }));
+    res.end(JSON.stringify({ success: false, error: 'Memory browsing is not available' }));
     return;
   }
 
@@ -54,7 +54,7 @@ export async function handleListMemoryRecords(
 /**
  * POST /api/memory/search — semantic search across memory records.
  * Body: { memoryName, namespace, searchQuery, strategyId? }
- * Requires onRetrieveMemoryRecords handler (invoke mode only).
+ * Requires onRetrieveMemoryRecords handler.
  */
 export async function handleRetrieveMemoryRecords(
   ctx: RouteContext,
@@ -65,7 +65,7 @@ export async function handleRetrieveMemoryRecords(
   if (!ctx.options.onRetrieveMemoryRecords) {
     ctx.setCorsHeaders(res, origin);
     res.writeHead(404, { 'Content-Type': 'application/json' });
-    res.end(JSON.stringify({ success: false, error: 'Memory search is only available in invoke mode' }));
+    res.end(JSON.stringify({ success: false, error: 'Memory search is not available' }));
     return;
   }
 

src/cli/operations/dev/web-ui/handlers/start.ts

@@ -29,7 +29,7 @@ export async function handleStart(
     return;
   }
 
-  // Delegate to custom start handler if provided (e.g. invoke mode — agents are already deployed)
+  // Delegate to custom start handler if provided
   if (ctx.options.onStart) {
     const result = await ctx.options.onStart(agentName);
     ctx.setCorsHeaders(res, origin);

src/cli/operations/dev/web-ui/handlers/status.ts

@@ -7,16 +7,8 @@ export function handleStatus(ctx: RouteContext, res: ServerResponse, origin?: st
   const { agents } = ctx.options;
   const running: StatusRunningAgent[] = [];
 
-  if (ctx.options.onInvoke) {
-    // Invoke mode: all agents are always "running" (deployed remotely)
-    for (const agent of agents) {
-      running.push({ name: agent.name, port: 0 });
-    }
-  } else {
-    // Dev mode: only locally started agents are running
-    for (const [name, { port }] of ctx.runningAgents) {
-      running.push({ name, port });
-    }
+  for (const [name, { port }] of ctx.runningAgents) {
+    running.push({ name, port });
   }
 
   // Collect per-agent errors

src/cli/operations/dev/web-ui/handlers/traces.ts

@@ -4,7 +4,7 @@ import type { IncomingMessage, ServerResponse } from 'node:http';
 
 /**
  * GET /api/traces?agentName=xxx — list recent traces.
- * In dev mode this returns local OTEL traces; in invoke mode it returns CloudWatch traces.
+ * Returns local OTEL traces when the collector is active.
  */
 export async function handleListTraces(
   ctx: RouteContext,
@@ -58,7 +58,7 @@ export async function handleListTraces(
 
 /**
  * GET /api/traces/:traceId?agentName=xxx — get full trace data.
- * In dev mode this returns local OTEL traces; in invoke mode it returns CloudWatch traces.
+ * Returns local OTEL trace spans and logs when the collector is active.
  */
 export async function handleGetTrace(
   ctx: RouteContext,

src/cli/operations/dev/web-ui/index.ts

@@ -2,7 +2,6 @@ export {
   WebUIServer,
   type WebUIOptions,
   type StartHandler,
-  type InvocationHandler,
   type ListTracesHandler,
   type GetTraceHandler,
   type ListMemoryRecordsHandler,

src/cli/operations/dev/web-ui/run-web-ui.ts

@@ -7,14 +7,14 @@ import { spawn } from 'child_process';
 export interface RunWebUIOptions {
   /** Options to pass to WebUIServer (minus uiPort, which is resolved automatically) */
   serverOptions: Omit<WebUIOptions, 'uiPort' | 'onReady' | 'onLog'>;
-  /** Logger command label (e.g. 'dev' or 'invoke') */
+  /** Logger command label (e.g. 'dev') */
   logLabel: string;
   /** Optional log handler override. Defaults to console logging errors. */
   onLog?: (level: 'info' | 'warn' | 'error', message: string) => void;
 }
 
 /**
- * Shared entry point for launching the web UI in both dev and invoke modes.
+ * Shared entry point for launching the web UI.
  * Handles port discovery, logger setup, browser launch, SIGINT, and keep-alive.
  */
 export async function runWebUI(opts: RunWebUIOptions): Promise<void> {

src/cli/operations/dev/web-ui/web-server.ts

@@ -56,13 +56,6 @@ export type StartHandler = (
   agentName: string
 ) => Promise<{ success: boolean; name: string; port: number; error?: string }>;
 
-/**
- * Custom handler for POST /invocations.
- * Receives the parsed request body and the raw ServerResponse so it can stream.
- * Must write headers and end the response itself.
- */
-export type InvocationHandler = (body: string, res: ServerResponse, setCors: () => void) => Promise<void>;
-
 /**
  * Custom handler for GET /api/traces.
  * Returns a list of recent traces for the given agent.
@@ -106,8 +99,8 @@ export type RetrieveMemoryRecordsHandler = (
 ) => Promise<{ success: boolean; records?: unknown[]; error?: string }>;
 
 export interface WebUIOptions {
-  /** Whether this server is running in dev or invoke mode */
-  mode: 'dev' | 'invoke';
+  /** Server mode identifier (currently only 'dev' is used) */
+  mode: 'dev';
   /** Port for the web UI server (API proxy) */
   uiPort: number;
   /** Available agents (metadata only — servers are started on demand) */
@@ -126,15 +119,13 @@ export interface WebUIOptions {
   onLog?: (level: 'info' | 'warn' | 'error', message: string) => void;
   /** Custom start handler — overrides the default dev server start logic */
   onStart?: StartHandler;
-  /** Custom invocation handler — overrides the default local proxy logic */
-  onInvoke?: InvocationHandler;
-  /** Custom handler for listing traces (local OTEL in dev mode, CloudWatch in invoke mode) */
+  /** Custom handler for listing traces */
   onListTraces?: ListTracesHandler;
-  /** Custom handler for getting a single trace (local OTEL in dev mode, CloudWatch in invoke mode) */
+  /** Custom handler for getting a single trace */
   onGetTrace?: GetTraceHandler;
-  /** Custom handler for listing memory records — only available in invoke mode */
+  /** Custom handler for listing memory records */
   onListMemoryRecords?: ListMemoryRecordsHandler;
-  /** Custom handler for searching memory records — only available in invoke mode */
+  /** Custom handler for searching memory records */
   onRetrieveMemoryRecords?: RetrieveMemoryRecordsHandler;
   /** Agent to pre-select in the UI dropdown (set when --runtime is specified) */
   selectedAgent?: string;