nudge-ui 0.2.4 → 0.2.6

package/dist/inspector/agent/McpConnectionDialog.d.ts

@@ -18,10 +18,5 @@ export interface McpConnectionDialogProps extends McpConnectionContentProps {
 }
 /** Renders the MCP setup and pairing flow without owning a dialog surface. */
 export declare function McpConnectionContent({ projectId, origin, snapshot, onConnect, onTakeOver, onDisconnect, onCheckAgain, }: McpConnectionContentProps): ReactElement;
-/**
- * Explains and controls the project-scoped MCP companion connection.
- *
- * This compatibility wrapper keeps the existing standalone MCP surface while
- * the shared content is also embedded in the general settings modal.
- */
+/** Compatibility wrapper: the shared content is also embedded in the general settings modal. */
 export declare function McpConnectionDialog({ open, projectId, origin, snapshot, onOpenChange, onConnect, onTakeOver, onDisconnect, onCheckAgain, }: McpConnectionDialogProps): ReactElement;

package/dist/inspector/agent/McpConnectionDialog.js

@@ -2,7 +2,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import { useMemo, useRef, useState } from "react";
 import { Dialog } from "@base-ui/react/dialog";
 import { IconCheck, IconCopy, IconPlugConnected, IconRefresh, IconUnlink, IconX, } from "@tabler/icons-react";
-import { getAgentConnectionStatus } from "./connectionStatus.js";
+import { useAgentConnectionStatus } from "./connectionStatus.js";
 import { copyToClipboard } from "../prompt/copyToClipboard.js";
 import { Button } from "../ui/Button.js";
 import { Disclosure } from "../ui/Disclosure.js";
@@ -35,20 +35,24 @@ export function McpConnectionContent({ projectId, origin, snapshot, onConnect, o
         "Start the application normally. Reload the agent if needed, then listen to Nudge.",
     ].join("\n"), [setupCommand]);
     const doctorCommand = "nudge-ui agent doctor";
-    const status = getAgentConnectionStatus(snapshot);
+    const status = useAgentConnectionStatus(snapshot);
     const showConnectionStatus = status.kind !== "not-found"
         || Boolean(snapshot.error || checkError || snapshot.request?.summary || snapshot.request?.error);
     const canConnect = snapshot.companionReachable
         && !snapshot.paired
         && !snapshot.pairedElsewhere
         && snapshot.state !== "pairing"
+        && snapshot.state !== "delivered"
         && snapshot.state !== "working"
+        && snapshot.request?.status !== "delivered"
         && snapshot.request?.status !== "working";
-    const canDisconnect = snapshot.paired || snapshot.request?.status === "working";
+    const canDisconnect = snapshot.paired || snapshot.request?.status === "working" || snapshot.request?.status === "delivered";
     const canTakeOver = snapshot.companionReachable
         && snapshot.pairedElsewhere
         && snapshot.state !== "pairing"
+        && snapshot.state !== "delivered"
         && snapshot.state !== "working"
+        && snapshot.request?.status !== "delivered"
         && snapshot.request?.status !== "working";
     async function handleCheckAgain() {
         if (checking)
@@ -100,12 +104,7 @@ export function McpConnectionContent({ projectId, origin, snapshot, onConnect, o
     }
     return (_jsxs(_Fragment, { children: [showConnectionStatus ? (_jsxs(StatusCallout, { className: "mcp-connection__status", tone: status.tone, "data-test": "mcp-connection-status", children: [_jsx("span", { className: "mcp-connection__status-label", role: "status", children: status.label }), snapshot.error ? (_jsx("span", { className: "mcp-connection__status-detail", children: snapshot.error })) : null, checkError ? _jsx("span", { role: "alert", children: checkError }) : null, snapshot.request?.summary ? (_jsx("span", { className: "mcp-connection__status-detail", children: snapshot.request.summary })) : null, snapshot.request?.error ? (_jsx("span", { className: "mcp-connection__status-detail", children: snapshot.request.error })) : null] })) : null, _jsxs("ol", { className: "mcp-connection__timeline", "data-test": "mcp-connection-timeline", children: [_jsxs(TimelineStep, { number: 1, complete: snapshot.companionReachable || snapshot.listenerActive || snapshot.request !== null, title: "Set up agent", children: [_jsx("p", { className: "mcp-connection__copy", children: "Agent prompt" }), _jsxs("div", { className: "mcp-connection__command-block", children: [_jsx(IconButton, { className: "mcp-connection__copy-button", variant: "quiet", size: "compact", label: copiedPrompt ? "Copied" : "Copy setup instructions", "data-test": "mcp-copy-setup-prompt", onClick: () => void handleCopyPrompt(), children: copiedPrompt ? _jsx(IconCheck, { size: 16, "aria-hidden": "true" }) : _jsx(IconCopy, { size: 16, "aria-hidden": "true" }) }), _jsx("pre", { className: "mcp-connection__command", children: _jsx("code", { "data-test": "mcp-setup-prompt", children: setupPrompt }) })] }), _jsx("p", { className: "mcp-connection__copy", children: "Or, terminal command" }), _jsxs("div", { className: "mcp-connection__command-block", children: [_jsx(IconButton, { className: "mcp-connection__copy-button", variant: "quiet", size: "compact", label: copiedCommand ? "Copied" : "Copy command", "data-test": "mcp-copy-command", onClick: () => void handleCopyCommand(), children: copiedCommand ? _jsx(IconCheck, { size: 16, "aria-hidden": "true" }) : _jsx(IconCopy, { size: 16, "aria-hidden": "true" }) }), _jsx("pre", { className: "mcp-connection__command", children: _jsx("code", { "data-test": "mcp-setup-command", children: setupCommand }) })] })] }), _jsx(TimelineStep, { number: 2, complete: snapshot.listenerActive, title: "Tell agent to listen for instructions", children: _jsx("p", { className: "mcp-connection__copy", children: "Tell your coding agent: \u201CListen to Nudge.\u201D" }) })] }), _jsxs(Disclosure, { className: "mcp-connection__details", title: "Troubleshooting", defaultOpen: true, children: [_jsx("p", { className: "mcp-connection__copy", children: "To diagnose setup" }), _jsxs("div", { className: "mcp-connection__command-block", children: [_jsx(IconButton, { className: "mcp-connection__copy-button", variant: "quiet", size: "compact", label: copiedDoctor ? "Copied" : "Copy diagnostic command", "data-test": "mcp-copy-doctor", onClick: () => void handleCopyDoctor(), children: copiedDoctor ? _jsx(IconCheck, { size: 16, "aria-hidden": "true" }) : _jsx(IconCopy, { size: 16, "aria-hidden": "true" }) }), _jsx("pre", { className: "mcp-connection__command", children: _jsx("code", { children: doctorCommand }) })] }), _jsxs("dl", { className: "mcp-connection__diagnostics", "data-test": "mcp-connection-diagnostics", children: [_jsxs("div", { className: "mcp-connection__diagnostic", children: [_jsx("dt", { children: "Project ID" }), _jsx("dd", { "data-test": "mcp-project-id", children: projectId })] }), _jsxs("div", { className: "mcp-connection__diagnostic", children: [_jsx("dt", { children: "Origin" }), _jsx("dd", { "data-test": "mcp-origin", children: origin })] })] }), _jsxs("div", { className: "mcp-connection__actions", children: [canTakeOver ? (_jsxs(Button, { variant: "primary", "data-test": "mcp-takeover", type: "button", onClick: () => void onTakeOver(), children: [_jsx(IconPlugConnected, { size: "var(--icon-size-small)", stroke: 1.8, "aria-hidden": "true" }), "Take over"] })) : null, canConnect ? (_jsxs(Button, { variant: "primary", "data-test": "mcp-connect", type: "button", onClick: () => void onConnect(), children: [_jsx(IconPlugConnected, { size: "var(--icon-size-small)", stroke: 1.8, "aria-hidden": "true" }), "Connect"] })) : null, _jsxs(Button, { variant: "quiet", "data-test": "mcp-check-again", type: "button", disabled: checking || snapshot.state === "disabled" || snapshot.state === "pairing", onClick: () => void handleCheckAgain(), children: [_jsx(IconRefresh, { size: "var(--icon-size-small)", stroke: 1.8, "aria-hidden": "true" }), checking ? "Checking…" : "Refresh connection"] }), canDisconnect ? (_jsxs(Button, { variant: "danger", "data-test": "mcp-disconnect", type: "button", onClick: onDisconnect, children: [_jsx(IconUnlink, { size: "var(--icon-size-small)", stroke: 1.8, "aria-hidden": "true" }), snapshot.paired ? "Disconnect" : "Forget connection"] })) : null] })] }), copyError ? _jsx("p", { className: "mcp-connection__error", role: "alert", children: copyError }) : null] }));
 }
-/**
- * Explains and controls the project-scoped MCP companion connection.
- *
- * This compatibility wrapper keeps the existing standalone MCP surface while
- * the shared content is also embedded in the general settings modal.
- */
+/** Compatibility wrapper: the shared content is also embedded in the general settings modal. */
 export function McpConnectionDialog({ open, projectId, origin, snapshot, onOpenChange, onConnect, onTakeOver, onDisconnect, onCheckAgain, }) {
     const closeRef = useRef(null);
     return (_jsx(Dialog.Root, { open: open, onOpenChange: onOpenChange, children: _jsxs(Dialog.Portal, { container: portalContainer(), children: [_jsx(Dialog.Backdrop, { className: "mcp-connection__backdrop", "data-test": "mcp-connection-backdrop" }), _jsxs(Dialog.Popup, { className: "mcp-connection__popup", "data-test": "mcp-connection-dialog", initialFocus: closeRef, children: [_jsxs("div", { className: "mcp-connection__header", children: [_jsxs("div", { children: [_jsx(Dialog.Title, { className: "mcp-connection__title", children: "Connect MCP" }), _jsx(Dialog.Description, { className: "mcp-connection__description", children: "Set up your coding agent once, then ask it to listen to Nudge in this worktree." })] }), _jsx(Dialog.Close, { ref: closeRef, className: "icon-button icon-button--quiet mcp-connection__close", "aria-label": "Close MCP connection", "data-test": "mcp-connection-close", type: "button", children: _jsx(IconX, { size: "var(--icon-size-small)", stroke: 1.8, "aria-hidden": "true" }) })] }), _jsx(McpConnectionContent, { projectId: projectId, origin: origin, snapshot: snapshot, onConnect: onConnect, onTakeOver: onTakeOver, onDisconnect: onDisconnect, onCheckAgain: onCheckAgain })] })] }) }));

package/dist/inspector/agent/activity.d.ts

@@ -9,10 +9,10 @@ export interface AgentActivitySnapshot {
     readonly target?: DraftTarget;
     readonly files: readonly ActivityFile[];
 }
-/** Pins submitted frames before dispatch. Activity never selects or edits a draft. */
+/** Pins submitted frames before dispatch; activity never selects or edits a draft. */
 export declare function beginActivityDispatch(projectId: string, dispatchId: string, targets: readonly string[], target?: DraftTarget): void;
 /** Dispatch and status are the only authority for the layer's lifetime. */
-export declare function synchronizeAgentActivity(projectId: string, requestId: string | undefined, working: boolean, dispatchId?: string): void;
+export declare function synchronizeAgentActivity(projectId: string, requestId: string | undefined, working: boolean, dispatchId?: string, awaitingAcknowledgment?: boolean): void;
 export declare function reportAgentActivity(projectId: string, activity: AgentActivity): void;
 export declare function getAgentActivity(projectId: string): AgentActivitySnapshot;
 export declare function useAgentActivity(projectId: string): AgentActivitySnapshot;

package/dist/inspector/agent/activity.js

@@ -14,18 +14,19 @@ function publish(state, snapshot) {
     for (const listener of state.listeners)
         listener();
 }
-/** Pins submitted frames before dispatch. Activity never selects or edits a draft. */
+/** Pins submitted frames before dispatch; activity never selects or edits a draft. */
 export function beginActivityDispatch(projectId, dispatchId, targets, target) {
     project(projectId).dispatch = { id: dispatchId, targets: [...targets], target };
 }
 /** Dispatch and status are the only authority for the layer's lifetime. */
-export function synchronizeAgentActivity(projectId, requestId, working, dispatchId) {
+export function synchronizeAgentActivity(projectId, requestId, working, dispatchId, awaitingAcknowledgment = false) {
     const state = project(projectId);
     if (!working || !requestId) {
         if (state.timer)
             clearTimeout(state.timer);
         state.timer = undefined;
-        state.dispatch = undefined;
+        if (!awaitingAcknowledgment)
+            state.dispatch = undefined;
         if (state.snapshot !== EMPTY)
             publish(state, EMPTY);
         return;

package/dist/inspector/agent/client.d.ts

@@ -1,5 +1,5 @@
 import { type AgentBridgeTransport, type AgentClientOptions, type AgentSketchAttachment, type AgentSketchMetadata, type AgentRequestStatus, type AgentStatusSnapshot, type PromptDispatchResponse } from "./protocol.js";
-export type AgentClientState = "disabled" | "disconnected" | "available" | "pairing" | "connected" | "working" | "completed" | "failed" | "interrupted";
+export type AgentClientState = "disabled" | "disconnected" | "available" | "pairing" | "connected" | "delivered" | "working" | "completed" | "failed" | "interrupted";
 export interface AgentRequestSnapshot {
     readonly requestId: string;
     readonly changeRevision?: number;
@@ -16,23 +16,14 @@ export interface AgentClientSnapshot {
     readonly connection: AgentStatusSnapshot["connection"];
     readonly listenerActive: boolean;
     readonly agentClientName?: string;
-    /** Whether the last valid discovery response came from the local companion. */
     readonly companionReachable: boolean;
     readonly paired: boolean;
-    /** Whether another browser currently owns this project's pairing. */
     readonly pairedElsewhere: boolean;
     readonly request: AgentRequestSnapshot | null;
     readonly error?: string;
 }
-/** Creates a stable numeric revision for one prompt snapshot. */
 export declare function createPromptRevision(changes: readonly unknown[], structuralChanges?: readonly unknown[], sketches?: readonly unknown[]): number;
-/**
- * Browser-side companion client.
- *
- * Discovery is deliberately started by `start()`, which is guarded by the
- * dev-only flag. Constructing this class never opens a socket or performs a
- * network request, so importing the inspector remains inert in production.
- */
+/** Construction never touches the network; discovery starts only from the dev-flag-guarded start(). */
 export declare class AgentClient {
     readonly projectId: string;
     readonly origin: string;
@@ -60,35 +51,23 @@ export declare class AgentClient {
     private readonly autoConnect;
     private autoConnectSuppressed;
     constructor(options: AgentClientOptions);
-    /** Installs the controller callback used to acknowledge Canvas commands. */
     setCanvasCommandHandler(handler: AgentClientOptions["canvasCommandHandler"]): void;
     getSnapshot: () => AgentClientSnapshot;
     subscribe: (listener: () => void) => (() => void);
-    /** Starts loopback discovery. Calling it repeatedly is idempotent. */
     start(): void;
-    /** Stops discovery and closes the browser event stream. */
     stop(): void;
     private restoreOrDiscover;
-    /** Runs an immediate authenticated restore or companion discovery probe. */
     checkConnection(): Promise<void>;
-    /** Triggers a new discovery probe after a connection loss. */
     reconnect(): void;
-    /** Pairs the browser with a reachable companion, including an idle agent. */
     connect(): Promise<boolean>;
-    /** Replaces an existing browser pairing after an explicit user action. */
     takeOver(): Promise<boolean>;
     private pairWith;
     private openEvents;
-    /**
-     * Dispatches one immutable prompt revision. The request enters `working`
-     * synchronously, before the transport promise resolves, so the UI cannot
-     * issue a second prompt while the first one is in flight.
-     */
+    /** Enters `delivered` synchronously so the UI cannot dispatch a second prompt while the first is in flight. */
     dispatchPrompt(prompt: string, changeRevision?: number, options?: {
         readonly clientDispatchId?: string;
         readonly attachments?: readonly AgentSketchAttachment[];
     }): Promise<PromptDispatchResponse | null>;
-    /** Interrupts local request state and asks the companion to revoke pairing. */
     disconnect(reason?: string): void;
     private discover;
     private scheduleDiscovery;
@@ -103,11 +82,8 @@ export declare class AgentClient {
     private deriveState;
     private publish;
 }
-/** Sets a host/test transport for subsequently created project clients. */
 export declare function configureAgentBridgeTransport(transport: AgentBridgeTransport | undefined): void;
-/** Clears cached clients and their active streams. */
 export declare function resetAgentClients(): void;
 export declare function createAgentClient(options: AgentClientOptions): AgentClient;
 export declare function getAgentClient(projectId: string, options?: Omit<AgentClientOptions, "projectId">): AgentClient;
-/** React hook used by the inspector handoff control. */
 export declare function useAgentClient(projectId: string, suppliedClient?: AgentClient): AgentClientSnapshot;

package/dist/inspector/agent/client.js

@@ -3,7 +3,7 @@ import { useEffect, useMemo, useSyncExternalStore } from "react";
 import { isNudgeUiDev } from "../runtime/devFlag.js";
 import { isDemoRuntime } from "../runtime/runtimeConfig.js";
 import { AgentBridgeHttpError, HttpAgentBridgeTransport, shouldAutoConnectAgentBridge, } from "./httpTransport.js";
-import { AGENT_PROTOCOL_VERSION, } from "./protocol.js";
+import { AGENT_PROTOCOL_VERSION, deriveConnectionState, } from "./protocol.js";
 const DISABLED_SNAPSHOT = Object.freeze({
     state: "disabled",
     connection: "offline",
@@ -57,8 +57,7 @@ function writeStoredSession(session) {
         localStorage.setItem(sessionStorageKey(session.projectId), JSON.stringify(session));
     }
     catch {
-        // Storage can be disabled by browser privacy settings. The in-memory
-        // pairing remains valid for this document in that case.
+        // Storage can be disabled by browser privacy settings; the in-memory pairing stays valid.
     }
 }
 function clearStoredSession(projectId) {
@@ -92,24 +91,11 @@ function setAutoConnectDisabled(projectId, disabled) {
             localStorage.removeItem(key);
     }
     catch {
-        // Privacy settings can disable storage. The current document still keeps
-        // the explicit disconnect preference in memory.
+        // Storage can be disabled; the disconnect preference stays in memory for this document.
     }
 }
-function statusForConnection(status) {
-    if (status.connection === "working" || status.request?.status === "working")
-        return "working";
-    if (status.paired)
-        return "paired";
-    if (status.listenerActive)
-        return "listening";
-    return "offline";
-}
 function hashRevision(value) {
-    // FNV-1a is sufficient here: the revision is an opaque, bounded identity
-    // used to prevent a later browser edit from being mistaken for the sent
-    // canonical change set. Keep it in the safe-integer range accepted by the
-    // companion protocol.
+    // Opaque bounded identity, kept in the safe-integer range the companion protocol accepts.
     let hash = 2_166_136_261;
     for (let index = 0; index < value.length; index += 1) {
         hash ^= value.charCodeAt(index);
@@ -117,7 +103,6 @@ function hashRevision(value) {
     }
     return (hash >>> 0) & 0x7fff_ffff;
 }
-/** Creates a stable numeric revision for one prompt snapshot. */
 export function createPromptRevision(changes, structuralChanges = [], sketches = []) {
     let serialized;
     try {
@@ -128,13 +113,7 @@ export function createPromptRevision(changes, structuralChanges = [], sketches =
     }
     return hashRevision(serialized);
 }
-/**
- * Browser-side companion client.
- *
- * Discovery is deliberately started by `start()`, which is guarded by the
- * dev-only flag. Constructing this class never opens a socket or performs a
- * network request, so importing the inspector remains inert in production.
- */
+/** Construction never touches the network; discovery starts only from the dev-flag-guarded start(). */
 export class AgentClient {
     projectId;
     origin;
@@ -169,12 +148,10 @@ export class AgentClient {
         this.canvasCommandHandler = options.canvasCommandHandler;
         this.autoConnect = options.autoConnect ?? shouldAutoConnectAgentBridge();
         this.autoConnectSuppressed = this.autoConnect && autoConnectDisabled(this.projectId);
-        // The public demo renders the real inspector but intentionally keeps the
-        // local MCP/agent bridge inert.
+        // The public demo renders the real inspector but intentionally keeps the local MCP/agent bridge inert.
         this.enabled = isNudgeUiDev() && !isDemoRuntime();
         this.snapshot = this.enabled ? this.makeSnapshot() : DISABLED_SNAPSHOT;
     }
-    /** Installs the controller callback used to acknowledge Canvas commands. */
     setCanvasCommandHandler(handler) {
         this.canvasCommandHandler = handler;
     }
@@ -183,14 +160,12 @@ export class AgentClient {
         this.listeners.add(listener);
         return () => this.listeners.delete(listener);
     };
-    /** Starts loopback discovery. Calling it repeatedly is idempotent. */
     start() {
         if (!this.enabled || this.started)
             return;
         this.started = true;
         void this.restoreOrDiscover();
     }
-    /** Stops discovery and closes the browser event stream. */
     stop() {
         if (!this.started && !this.eventSubscription)
             return;
@@ -207,10 +182,7 @@ export class AgentClient {
         this.dispatchAbort = null;
         this.eventSubscription?.close();
         this.eventSubscription = null;
-        // `stop` is used by React effect cleanup and page remounts. Keep the
-        // request and ephemeral token in memory/storage so a still-running
-        // companion can be resumed; explicit `disconnect()` is the revocation
-        // and interruption path.
+        // Keep the request and token so a remount can resume; disconnect() is the revocation path.
         this.connection = "offline";
         this.listenerActive = false;
         this.companionReachable = false;
@@ -249,8 +221,7 @@ export class AgentClient {
             catch (error) {
                 if (abortController.signal.aborted || !this.started)
                     return;
-                // A stale or revoked token is not a connection error. Clearing it
-                // lets the normal listener discovery path offer a fresh pairing.
+                // A stale or revoked token is not a connection error; clearing it lets discovery offer a fresh pairing.
                 clearStoredSession(this.projectId);
                 this.sessionToken = null;
                 this.resetForDiscovery();
@@ -263,7 +234,6 @@ export class AgentClient {
         }
         await this.discover();
     }
-    /** Runs an immediate authenticated restore or companion discovery probe. */
     async checkConnection() {
         if (!this.enabled)
             return;
@@ -279,7 +249,6 @@ export class AgentClient {
         this.publish();
         await this.restoreOrDiscover();
     }
-    /** Triggers a new discovery probe after a connection loss. */
     reconnect() {
         if (!this.enabled)
             return;
@@ -301,11 +270,9 @@ export class AgentClient {
         this.publish();
         void this.restoreOrDiscover();
     }
-    /** Pairs the browser with a reachable companion, including an idle agent. */
     async connect() {
         return this.pairWith((request, signal) => this.transport.pair(request, signal));
     }
-    /** Replaces an existing browser pairing after an explicit user action. */
     async takeOver() {
         if (!this.transport.takeOver)
             return false;
@@ -316,7 +283,7 @@ export class AgentClient {
             return false;
         this.autoConnectSuppressed = false;
         setAutoConnectDisabled(this.projectId, false);
-        if (this.connection === "working" || this.activeRequest?.status === "working")
+        if (this.connection === "working" || this.connection === "delivered" || this.activeRequest?.status === "working" || this.activeRequest?.status === "delivered")
             return false;
         const idleListenerActive = this.listenerActive;
         const idleCompanionReachable = this.companionReachable;
@@ -397,15 +364,11 @@ export class AgentClient {
             this.handleDisconnect(errorMessage(error, "The agent event stream could not be opened."));
         }
     }
-    /**
-     * Dispatches one immutable prompt revision. The request enters `working`
-     * synchronously, before the transport promise resolves, so the UI cannot
-     * issue a second prompt while the first one is in flight.
-     */
+    /** Enters `delivered` synchronously so the UI cannot dispatch a second prompt while the first is in flight. */
     async dispatchPrompt(prompt, changeRevision, options = {}) {
         if (!this.enabled || !this.started || !this.paired || !this.listenerActive || !this.sessionToken)
             return null;
-        if (!prompt || this.activeRequest?.status === "working")
+        if (!prompt || this.activeRequest?.status === "working" || this.activeRequest?.status === "delivered")
             return null;
         const localRequestId = randomId("request");
         const request = {
@@ -424,9 +387,9 @@ export class AgentClient {
             ...(changeRevision === undefined ? {} : { changeRevision }),
             ...(options.clientDispatchId === undefined ? {} : { clientDispatchId: options.clientDispatchId }),
             ...(sketches === undefined ? {} : { sketches }),
-            status: "working",
+            status: "delivered",
         };
-        this.connection = "working";
+        this.connection = "delivered";
         this.lastError = undefined;
         this.publish();
         const abortController = new AbortController();
@@ -445,8 +408,9 @@ export class AgentClient {
                     }
                     : this.activeRequest;
             }
-            if (response?.status && response.status !== "working") {
+            if (response?.status) {
                 this.applyRequestStatus(response.status, response.request.requestId);
+                this.publish();
             }
             return response ?? null;
         }
@@ -464,14 +428,13 @@ export class AgentClient {
                 this.dispatchAbort = null;
         }
     }
-    /** Interrupts local request state and asks the companion to revoke pairing. */
     disconnect(reason = "browser_disconnected") {
         this.autoConnectSuppressed = true;
         if (this.autoConnect)
             setAutoConnectDisabled(this.projectId, true);
         const token = this.sessionToken;
         const active = this.activeRequest;
-        if (active?.status === "working") {
+        if (active?.status === "working" || active?.status === "delivered") {
             this.activeRequest = { ...active, status: "interrupted", error: reason };
         }
         this.dispatchAbort?.abort();
@@ -518,11 +481,8 @@ export class AgentClient {
                 }
                 else {
                     this.companionReachable = true;
-                    if (this.connection !== "paired" && this.connection !== "working" && !this.pairingAbort) {
-                        // Discovery is unauthenticated. Another browser tab can own the
-                        // pairing, so do not adopt `paired` until this tab restores the
-                        // shared token. The next poll runs restore first in case a sibling
-                        // tab has just stored that token.
+                    if (this.connection !== "paired" && this.connection !== "working" && this.connection !== "delivered" && !this.pairingAbort) {
+                        // Discovery is unauthenticated; do not adopt `paired` until this tab restores the shared token.
                         const pairedElsewhere = status.paired && !this.sessionToken;
                         this.applyStatus(status.paired ? {
                             ...status,
@@ -538,7 +498,7 @@ export class AgentClient {
                     }
                 }
             }
-            else if (this.connection !== "paired" && this.connection !== "working" && !this.pairingAbort) {
+            else if (this.connection !== "paired" && this.connection !== "working" && this.connection !== "delivered" && !this.pairingAbort) {
                 this.connection = "offline";
                 this.listenerActive = false;
                 this.companionReachable = false;
@@ -549,14 +509,13 @@ export class AgentClient {
             }
         }
         catch (error) {
-            if (!abortController.signal.aborted && this.started && this.connection !== "paired" && this.connection !== "working" && !this.pairingAbort) {
+            if (!abortController.signal.aborted && this.started && this.connection !== "paired" && this.connection !== "working" && this.connection !== "delivered" && !this.pairingAbort) {
                 this.connection = "offline";
                 this.listenerActive = false;
                 this.companionReachable = false;
                 this.paired = false;
                 this.pairedElsewhere = false;
-                // Discovery failures are expected while the companion is not running;
-                // retain a quiet disconnected state rather than flashing an error.
+                // Discovery failures are expected while the companion is down; stay quietly disconnected.
                 this.lastError = undefined;
                 this.publish();
             }
@@ -624,10 +583,7 @@ export class AgentClient {
                 },
             };
         }
-        // Every tab with the shared browser session receives the SSE command, but
-        // only the Canvas lease owner may answer it. A read-only tab reports this
-        // local sentinel; leave the command pending for the owning tab instead of
-        // racing its acknowledgement with a false failure.
+        // Every tab receives the SSE command but only the Canvas lease owner may answer; leave it pending for that tab.
         if (!acknowledgement.ok && acknowledgement.error?.code === "workspace-locked")
             return;
         if (acknowledgement.commandId !== command.commandId) {
@@ -647,8 +603,7 @@ export class AgentClient {
             await this.transport.acknowledgeCanvasCommand(request);
         }
         catch {
-            // A command acknowledgement is best effort. The companion owns the
-            // bounded timeout and reports any expired command to its agent host.
+            // Acknowledgement is best effort; the companion owns the timeout.
         }
     }
     applyStatus(status, pairedElsewhere = false) {
@@ -658,7 +613,7 @@ export class AgentClient {
         this.agentClientName = status.agentClientName;
         this.paired = status.paired;
         this.pairedElsewhere = pairedElsewhere;
-        this.connection = statusForConnection(status);
+        this.connection = deriveConnectionState(status);
         if (status.paired && this.discoveryTimer !== null) {
             clearTimeout(this.discoveryTimer);
             this.discoveryTimer = null;
@@ -666,16 +621,13 @@ export class AgentClient {
         if (status.request) {
             this.applyRequestStatus(status.request.status, status.request.requestId, status.request.summary, status.request.error, status.request.changeRevision, status.request.clientDispatchId, status.request.sketches);
         }
-        else if (status.listenerActive && status.paired && this.activeRequest?.status !== "working") {
-            // A re-armed MCP listen call is the lifecycle boundary for the previous
-            // terminal request. Drop only that terminal status; newer edits remain
-            // in the inspector's canonical change log.
+        else if (status.listenerActive && status.paired && this.activeRequest?.status !== "working" && this.activeRequest?.status !== "delivered") {
+            // A re-armed listen call ends the previous terminal request; newer edits stay in the canonical change log.
             this.activeRequest = null;
         }
-        else if (this.activeRequest?.status === "working" && status.connection !== "working") {
-            // The bridge may omit a request during a brief re-arm transition. Do
-            // not mark a local request interrupted until the stream disconnects.
-            this.connection = "working";
+        else if ((this.activeRequest?.status === "working" || this.activeRequest?.status === "delivered") && status.connection !== this.activeRequest.status) {
+            // A brief re-arm can omit the request; do not mark it interrupted until the stream disconnects.
+            this.connection = this.activeRequest.status;
         }
         if (!status.paired && this.sessionToken) {
             this.handleDisconnect("The agent bridge pairing expired.");
@@ -695,10 +647,14 @@ export class AgentClient {
     }
     applyRequestStatus(status, requestId, summary, error, changeRevision, clientDispatchId, sketches) {
         const active = this.activeRequest;
+        // A dispatch response can arrive after an SSE acknowledgment or outcome.
+        if (status === "delivered" && active && active.status !== "delivered") {
+            this.connection = active.status === "working" ? "working" : this.paired ? "paired" : "offline";
+            return;
+        }
         if (active && active.requestId !== requestId && active.localRequestId !== requestId) {
-            // A fresh server id can arrive before the POST response. Associate it
-            // with the one local request that was just marked working.
-            if (active.status !== "working")
+            // A fresh server id can arrive before the POST response; associate it with the local request.
+            if (active.status !== "working" && active.status !== "delivered")
                 return;
             this.activeRequest = { ...active, requestId };
         }
@@ -727,17 +683,15 @@ export class AgentClient {
                 ...(error === undefined ? {} : { error }),
             };
         }
-        if (status === "working")
-            this.connection = "working";
+        if (status === "working" || status === "delivered")
+            this.connection = status;
         else if (this.paired)
             this.connection = "paired";
     }
     handleDisconnect(reason = "The agent connection was lost.") {
         this.eventSubscription?.close();
         this.eventSubscription = null;
-        // An SSE disconnect is not proof that the coding request stopped. Keep
-        // its last known state until an authenticated status restore says
-        // otherwise. Explicit browser disconnect remains the interruption path.
+        // An SSE disconnect is not proof the request stopped; keep its last known state until an authenticated restore says otherwise.
         this.connection = "offline";
         this.listenerActive = false;
         this.companionReachable = false;
@@ -745,8 +699,7 @@ export class AgentClient {
         this.pairedElsewhere = false;
         this.lastError = reason;
         if (reason === "browser_disconnected") {
-            // A remote takeover deliberately invalidates this browser's token. Do
-            // not immediately race the new browser by auto-pairing again.
+            // A remote takeover invalidates this token; do not race the new browser by auto-pairing.
             this.autoConnectSuppressed = true;
             if (this.autoConnect)
                 setAutoConnectDisabled(this.projectId, true);
@@ -784,6 +737,8 @@ export class AgentClient {
             return "disconnected";
         if (this.connection === "listening")
             return "available";
+        if (this.connection === "delivered" || this.activeRequest?.status === "delivered")
+            return "delivered";
         if (this.connection === "working" || this.activeRequest?.status === "working")
             return "working";
         if (this.activeRequest?.status === "completed")
@@ -796,7 +751,7 @@ export class AgentClient {
     }
     publish(stateOverride) {
         this.snapshot = Object.freeze(this.makeSnapshot(stateOverride));
-        synchronizeAgentActivity(this.projectId, this.snapshot.request?.requestId, this.snapshot.paired && this.snapshot.request?.status === "working", this.snapshot.request?.clientDispatchId);
+        synchronizeAgentActivity(this.projectId, this.snapshot.request?.requestId, this.snapshot.paired && this.snapshot.request?.status === "working", this.snapshot.request?.clientDispatchId, this.snapshot.paired && this.snapshot.request?.status === "delivered");
         for (const listener of this.listeners)
             listener();
     }
@@ -804,7 +759,6 @@ export class AgentClient {
 let configuredTransport;
 const clients = new Map();
 const consumerCounts = new Map();
-/** Sets a host/test transport for subsequently created project clients. */
 export function configureAgentBridgeTransport(transport) {
     for (const client of clients.values())
         client.stop();
@@ -812,7 +766,6 @@ export function configureAgentBridgeTransport(transport) {
     consumerCounts.clear();
     configuredTransport = transport;
 }
-/** Clears cached clients and their active streams. */
 export function resetAgentClients() {
     for (const client of clients.values())
         client.stop();
@@ -847,7 +800,6 @@ export function getAgentClient(projectId, options = {}) {
     }
     return client;
 }
-/** React hook used by the inspector handoff control. */
 export function useAgentClient(projectId, suppliedClient) {
     const client = useMemo(() => suppliedClient ?? getAgentClient(projectId), [projectId, suppliedClient]);
     useEffect(() => {

package/dist/inspector/agent/connectionStatus.d.ts

@@ -1,5 +1,5 @@
 import type { AgentClientSnapshot } from "./client.js";
-export type AgentConnectionStatusKind = "unavailable" | "connecting" | "working-disconnected" | "working" | "paired-elsewhere" | "not-found" | "ready-to-connect" | "connected-not-listening" | "listening";
+export type AgentConnectionStatusKind = "unavailable" | "connecting" | "delivered" | "delivered-disconnected" | "working-disconnected" | "working" | "paired-elsewhere" | "not-found" | "ready-to-connect" | "connected" | "connected-not-listening" | "listening";
 export interface AgentConnectionStatus {
     readonly kind: AgentConnectionStatusKind;
     readonly label: string;
@@ -9,5 +9,6 @@ export interface AgentConnectionStatus {
         readonly label: "Connect" | "Take over";
     };
 }
-/** Returns the user-facing connection state shared by the prompt UI and MCP panel. */
 export declare function getAgentConnectionStatus(snapshot: AgentClientSnapshot): AgentConnectionStatus;
+/** Gives the agent time to rearm after completing a request before showing a reminder. */
+export declare function useAgentConnectionStatus(snapshot: AgentClientSnapshot): AgentConnectionStatus;

package/dist/inspector/agent/connectionStatus.js

@@ -1,4 +1,4 @@
-/** Returns the user-facing connection state shared by the prompt UI and MCP panel. */
+import { useEffect, useState } from "react";
 export function getAgentConnectionStatus(snapshot) {
     if (snapshot.state === "disabled") {
         return { kind: "unavailable", label: "Unavailable in this build", tone: "neutral" };
@@ -14,6 +14,11 @@ export function getAgentConnectionStatus(snapshot) {
             action: { kind: "takeover", label: "Take over" },
         };
     }
+    if (snapshot.state === "delivered" || snapshot.request?.status === "delivered") {
+        return snapshot.paired
+            ? { kind: "delivered", label: "Connected · Awaiting agent acknowledgment", tone: "accent" }
+            : { kind: "delivered-disconnected", label: "Disconnected · Agent acknowledgment unknown", tone: "warning" };
+    }
     if (!snapshot.paired && (snapshot.state === "working" || snapshot.request?.status === "working")) {
         return { kind: "working-disconnected", label: "Disconnected · Last work status unknown", tone: "warning" };
     }
@@ -40,3 +45,22 @@ export function getAgentConnectionStatus(snapshot) {
         action: { kind: "connect", label: "Connect" },
     };
 }
+/** Gives the agent time to rearm after completing a request before showing a reminder. */
+export function useAgentConnectionStatus(snapshot) {
+    const status = getAgentConnectionStatus(snapshot);
+    const completedRequestId = status.kind === "connected-not-listening" && snapshot.request?.status === "completed"
+        ? snapshot.request.requestId
+        : undefined;
+    const [reminderRequestId, setReminderRequestId] = useState();
+    useEffect(() => {
+        setReminderRequestId(undefined);
+        if (!completedRequestId)
+            return;
+        const timeout = setTimeout(() => setReminderRequestId(completedRequestId), 5_000);
+        return () => clearTimeout(timeout);
+    }, [completedRequestId]);
+    if (completedRequestId && reminderRequestId !== completedRequestId) {
+        return { kind: "connected", label: "MCP connected", tone: "accent" };
+    }
+    return status;
+}