@robota-sdk/agent-ui-web 3.2.1 → 3.3.1

package/docs/SPEC.md

@@ -68,6 +68,11 @@ does not own session lifecycle, conversation history, or agent runtime state.
   a schedule or a goal alone (no running background task or loop) still opens the rail, and a
   schedule's Pause/Resume and Delete, and a goal's Cancel, run the same command/control paths their
   slash commands already use — the panel adds no write this package did not already expose.
+- **The surface follows the host's role.** The session client learns its effective role from the
+  host when it connects and, while it observes, sends nothing an observer may not send; the surface
+  then offers only what an observer can do: the conversation, its work and the project's reads, with
+  no composer, answers, session switching or control that changes the session or its work. The host
+  remains the authority; hiding a control is presentation, not enforcement.
 - The title bar and the page title both name the workspace folder the session works in, once the
   host has said one; the desktop window follows the page title.
 - A pending permission or ask prompt is visible whatever view is open, because a gated turn waits
@@ -90,7 +95,7 @@ does not own session lifecycle, conversation history, or agent runtime state.
   representation, when it has one, is only its disposable wake timer — cancelling that would leave
   the loop itself running. Every other entry sends `cancel-background-task`.
 - The Project panel (git status, one file's diff, project memory) correlates its own requests the
-  same way, and every file status it shows is a plain word, never a raw git XY code (#3277 "usable by
+  same way, and every file status it shows is a plain word, never a raw git XY code ("usable by
   anyone"); a workspace that is not a git repository, or a diff whose path the host refused, is one
   plain sentence, never left as a stuck loading state.
 - A session error finalizes any partial streamed text and clears thinking/running-tool state

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@robota-sdk/agent-ui-web",
-  "version": "3.2.1",
+  "version": "3.3.1",
   "description": "Browser-safe React session presentation, wire reducer, WebSocket client and runtime-host contract for consumer-owned agent renderers.",
   "license": "AGPL-3.0-only OR LicenseRef-Commercial",
   "type": "module",
@@ -60,11 +60,11 @@
     "pretendard": "^1.3.9",
     "react-markdown": "^10.1.0",
     "remark-gfm": "^4.0.1",
-    "@robota-sdk/agent-interface-execution": "3.2.1",
-    "@robota-sdk/agent-interface-session": "3.2.1",
-    "@robota-sdk/agent-transport": "3.2.1",
-    "@robota-sdk/agent-interface-command": "3.2.1",
-    "@robota-sdk/agent-interface-transport": "3.2.1"
+    "@robota-sdk/agent-interface-command": "^3.2.3",
+    "@robota-sdk/agent-interface-session": "^3.2.3",
+    "@robota-sdk/agent-transport": "^3.3.1",
+    "@robota-sdk/agent-interface-transport": "^3.2.3",
+    "@robota-sdk/agent-interface-execution": "^3.2.3"
   },
   "peerDependencies": {
     "react": ">=18"

package/src/client/ws-session-client.ts

@@ -72,7 +72,7 @@ export function createWsSessionClient(
     ws.onmessage = (event: MessageEvent): void => {
       const data = event.data;
       if (typeof data !== 'string') return;
-      // Issue #2045: `raw → owner decoder → typed`. A malformed frame (invalid JSON or an invalid
+      // `raw → owner decoder → typed`. A malformed frame (invalid JSON or an invalid
       // shape) must not throw inside the handler (it would break the socket's message pump and
       // freeze the UI) and must not be cast onward; surface it via the normal path.
       const decoded = decodeFrame(data, decodeServerMessage);

package/src/clipboard.ts

@@ -1,7 +1,7 @@
 import { useEffect, useState } from 'react';
 
 /**
- * Copies `text` to the clipboard for the "Copy code" / "Copy message" buttons (#3289 §2): the async
+ * Copies `text` to the clipboard for the "Copy code" / "Copy message" buttons: the async
  * Clipboard API first, and — for a page that denies it (no permission granted yet, a browser that
  * never implements it, or a context that disallows it outright) — a same-tick `execCommand('copy')`
  * fallback through a throwaway, invisible textarea. Rejects only once both have failed, so a caller
@@ -59,7 +59,7 @@ const FEEDBACK_MS = 2000;
 /**
  * The "Copy code" / "Copy message" buttons' shared feedback state: `copied` or `error` for a couple of
  * seconds, then back to `idle` — drives both the button's own label swap and an `aria-live` region a
- * screen reader announces without moving focus (#3289 §2).
+ * screen reader announces without moving focus.
  */
 export function useCopyFeedback(): { status: TCopyStatus; copy: (text: string) => void } {
   const [status, setStatus] = useState<TCopyStatus>('idle');

package/src/components/AgentActivityPanel.tsx

@@ -31,22 +31,22 @@ import type { IGoalState } from '@robota-sdk/agent-interface-session';
 
 interface IAgentActivityPanelProps {
   tasks: readonly IExecutionWorkspaceEntry[];
-  /** #3282 §4 part b-3: the "Scheduled" group — every `kind: 'scheduled'` background task. */
+  /** The "Scheduled" group — every `kind: 'scheduled'` background task. */
   schedules?: readonly IBackgroundTaskState<'scheduled'>[];
-  /** #3282 §4 part b-3: the current goal (`/goal`), or null/absent when none is set. */
+  /** The current goal (`/goal`), or null/absent when none is set. */
   goal?: IGoalState | null;
   onPauseSchedule?: (taskId: string) => void;
   onResumeSchedule?: (taskId: string) => void;
   onDeleteSchedule?: (taskId: string) => void;
   onCancelGoal?: () => void;
   className?: string;
-  /** #3288 §1: open a non-main-thread entry's detail sheet. */
+  /** Open a non-main-thread entry's detail sheet. */
   onSelect?: (entry: IExecutionWorkspaceEntry) => void;
-  /** #3288 §1: the main thread entry was clicked — the conversation beside this panel is it. */
+  /** The main thread entry was clicked — the conversation beside this panel is it. */
   onReturnToConversation?: () => void;
-  /** #3288 §1: Stop was clicked on a running/queued entry. */
+  /** Stop was clicked on a running/queued entry. */
   onStop?: (entry: IExecutionWorkspaceEntry) => void;
-  /** #3288 §1: the entry whose detail sheet is currently open, for a highlighted row. */
+  /** The entry whose detail sheet is currently open, for a highlighted row. */
   selectedEntryId?: string;
 }
 
@@ -115,7 +115,7 @@ export function AgentActivityPanel({
                   task={task}
                   onPause={onPauseSchedule}
                   onResume={onResumeSchedule}
-                  onRequestDelete={() => setPendingDeleteId(task.id)}
+                  onRequestDelete={onDeleteSchedule ? () => setPendingDeleteId(task.id) : undefined}
                 />
               ))}
             </div>
@@ -221,7 +221,7 @@ function AgentCard({
   );
 }
 
-/** #3282 §4 part b-3: one row of the "Scheduled" group — what it runs, when next, status, controls. */
+/** One row of the "Scheduled" group — what it runs, when next, status, controls. */
 function ScheduleCard({
   task,
   onPause,
@@ -231,7 +231,8 @@ function ScheduleCard({
   task: IBackgroundTaskState<'scheduled'>;
   onPause?: (taskId: string) => void;
   onResume?: (taskId: string) => void;
-  onRequestDelete: () => void;
+  /** Absent when the schedule is shown without controls (an observer). */
+  onRequestDelete?: () => void;
 }): React.ReactElement {
   const isPaused = task.status === 'paused';
   return (
@@ -242,39 +243,45 @@ function ScheduleCard({
       <p className="mt-1 text-[13px] text-muted-foreground">
         {describeScheduleTiming(task)} · {describeScheduleStatus(task)}
       </p>
-      <div className="mt-2 flex items-center gap-3">
-        {isPaused ? (
-          <button
-            type="button"
-            onClick={() => onResume?.(task.id)}
-            className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
-          >
-            <Play size={11} fill="currentColor" aria-hidden="true" />
-            Resume
-          </button>
-        ) : (
-          <button
-            type="button"
-            onClick={() => onPause?.(task.id)}
-            className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
-          >
-            <Pause size={11} fill="currentColor" aria-hidden="true" />
-            Pause
-          </button>
-        )}
-        <button
-          type="button"
-          onClick={onRequestDelete}
-          className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-destructive hover:bg-destructive/10"
-        >
-          Delete…
-        </button>
-      </div>
+      {onResume || onPause || onRequestDelete ? (
+        <div className="mt-2 flex items-center gap-3">
+          {isPaused
+            ? onResume && (
+                <button
+                  type="button"
+                  onClick={() => onResume(task.id)}
+                  className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
+                >
+                  <Play size={11} fill="currentColor" aria-hidden="true" />
+                  Resume
+                </button>
+              )
+            : onPause && (
+                <button
+                  type="button"
+                  onClick={() => onPause(task.id)}
+                  className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
+                >
+                  <Pause size={11} fill="currentColor" aria-hidden="true" />
+                  Pause
+                </button>
+              )}
+          {onRequestDelete && (
+            <button
+              type="button"
+              onClick={onRequestDelete}
+              className="flex items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-destructive hover:bg-destructive/10"
+            >
+              Delete…
+            </button>
+          )}
+        </div>
+      ) : null}
     </div>
   );
 }
 
-/** #3282 §4 part b-3: the current goal — objective, plain-word status, and Cancel. */
+/** The current goal — objective, plain-word status, and Cancel. */
 function GoalCard({
   goal,
   onCancel,
@@ -293,7 +300,7 @@ function GoalCard({
       <p className="mt-1.5 pl-[26px] text-[13px] text-muted-foreground">
         {describeGoalStatus(goal)}
       </p>
-      {goal.status === 'active' && (
+      {goal.status === 'active' && onCancel && (
         <div className="mt-2 pl-[26px]">
           <button
             type="button"

package/src/components/AgentSwitcherSheet.tsx

@@ -8,12 +8,12 @@ import { Dialog } from './Dialog.js';
 import type { IWsSessionState } from '../hooks/useSessionClient.js';
 
 /**
- * #3282 §4 part b-3 — the agent switcher: `show-agent-switcher` (`/agent` with no args) opens this
+ * The agent switcher: `show-agent-switcher` (`/agent` with no args) opens this
  * sheet instead of the surface's old "not available" line. It lists every agent this session can
  * run — name, one-line description, and where each is defined in plain words — with the current one
  * checked. Choosing a row runs the SAME path as typing `/agent <name>` (a plain `command`, wired in
  * `useAgentSwitcherState`); the reply is this sheet's own confirmation line, never a conversation
- * card (issue #3282 §4's decided design).
+ * card.
  */
 export function AgentSwitcherSheet({ state }: { state: IWsSessionState }): React.ReactElement | null {
   if (!state.agentSwitcherOpen) return null;

package/src/components/Composer.tsx

@@ -28,7 +28,7 @@ export interface IComposerHandle {
   focus: () => void;
 }
 
-/** The draft as persisted: the typed text plus any attachment chips (#3282 §4d). */
+/** The draft as persisted: the typed text plus any attachment chips. */
 interface IStoredDraft {
   readonly text: string;
   readonly attachments: readonly IDraftAttachment[];
@@ -37,7 +37,7 @@ interface IStoredDraft {
 const EMPTY_DRAFT: IStoredDraft = { text: '', attachments: [] };
 
 /**
- * #3282 §4e: a command whose whole job is opening a GUI screen — choosing it from the `/` menu runs
+ * A command whose whole job is opening a GUI screen — choosing it from the `/` menu runs
  * it at once (its `ui_intent` opens the screen the normal way) instead of filling the draft and
  * waiting for Enter, since there is nothing useful to type after it. `help` never reaches the
  * session at all (`useSessionClient.ts`'s `send` opens the Help sheet locally). Every other command
@@ -55,14 +55,14 @@ function menuGroupLabel(item: ICommandMenuItem): 'Commands' | 'Skills' {
 }
 
 /**
- * #3280 §4: the unsent draft survives a Chat → Usage → Chat switch (the composer unmounts), a page
+ * The unsent draft survives a Chat → Usage → Chat switch (the composer unmounts), a page
  * reload, and a desktop relaunch — `localStorage`, not `sessionStorage`, since only `localStorage`
  * survives a closed window/tab being reopened. Namespaced (the configured draft prefix, matching
  * the configured restore key in `use-session-directory.ts`) so a page hosting other state under the
  * same origin does not collide. Per session id when one is known; a single fallback key before the
  * first status arrives (the gap is brief and is reconciled once it does — see the effect below).
  *
- * #3282 §4d: the stored value is now JSON (`IStoredDraft`), not the bare text string it used to be —
+ * The stored value is now JSON (`IStoredDraft`), not the bare text string it used to be —
  * `parseStoredDraft` treats anything that does not parse as that shape (including a draft saved
  * before this change shipped) as plain text with no attachments, so an old stored draft still loads.
  */
@@ -141,38 +141,38 @@ const ComposerInstance = forwardRef<
     catalog: TCommandCatalog | null;
     status: TSessionStatus | null;
     /**
-     * False while the transport is not `connected` (issue #3280 §5): Enter and Send refuse to submit,
+     * False while the transport is not `connected`: Enter and Send refuse to submit,
      * and nothing typed is cleared or lost — the composer never sends into a socket that is not there.
-     * #3282 §2 (part 2): the status chips and the `/` command menu are disabled the same way.
+     * The status chips and the `/` command menu are disabled the same way.
      */
     connected?: boolean;
-    /** #3280 §2: a turn (or a blocking command) is running — Send becomes Stop and Esc stops it too. */
+    /** A turn (or a blocking command) is running — Send becomes Stop and Esc stops it too. */
     running: boolean;
     onStop: () => void;
-    /** #3280 §2: the prompt queued behind the running turn, or null when none is queued. */
+    /** The prompt queued behind the running turn, or null when none is queued. */
     queued: IQueuedPrompt | null;
     onCancelQueue: () => void;
     /**
      * Opens the host's native multi-file dialog with real filesystem paths — present only on the
-     * desktop app (#3282 §4d). Its absence means the attach button falls back to a plain HTML file
+     * desktop app. Its absence means the attach button falls back to a plain HTML file
      * picker, whose picks a browser can never resolve to a path (rule 3: shown plainly, nothing
      * attached).
      */
     pickFiles?: () => Promise<readonly IPickedFile[]>;
     /**
      * Resolves a dropped or picked `File` to its real filesystem path — present only on the desktop
-     * app, via Electron's `webUtils.getPathForFile` (#3282 §4d). Its absence means a drop can never
+     * app, via Electron's `webUtils.getPathForFile`. Its absence means a drop can never
      * become an `@`-reference either.
      */
     getPathForFile?: (file: File) => string;
-    /** #3282 §2 (part 2): the model control's pop-up menu — null until requested. */
+    /** The model control's pop-up menu — null until requested. */
     modelList?: TModelListSnapshot | null;
     onRequestModelList?: () => void;
     /** Applies a model/mode/effort choice without a conversation card (the control's label confirms it). */
     onSilentCommand?: (name: string, args?: string) => void;
     /**
-     * #3282 §4b: "Manage providers…" opens Settings at the Providers & Models section. Defaults to
-     * dispatching the bare `/provider` command (the pre-#3282-§4b behavior) for a caller that has not
+     * "Manage providers…" opens Settings at the Providers & Models section. Defaults to
+     * dispatching the bare `/provider` command (the legacy behavior) for a caller that has not
      * wired Settings — every shipped caller passes its own `state.openSettings('providers')`.
      */
     onManageProviders?: () => void;
@@ -238,25 +238,25 @@ const ComposerInstance = forwardRef<
   const [attachmentNotice, setAttachmentNotice] = useState<string | null>(null);
   const [selected, setSelected] = useState(0);
   const [dismissed, setDismissed] = useState(false);
-  // #3282 §2 (part 2): the slash menu is disabled while disconnected — none of its commands could run.
+  // The slash menu is disabled while disconnected — none of its commands could run.
   const menu = dismissed || !connected ? null : commandMenuFor(catalog, draft);
   useEffect(() => {
     setSelected(0);
     setDismissed(false);
   }, [draft]);
-  // #3282 §4e: the menu can hold more rows than fit in its scroll area — keyboard navigation (below)
+  // The menu can hold more rows than fit in its scroll area — keyboard navigation (below)
   // must keep the highlighted row in view, not just move the highlight off-screen.
   const menuOptionRefs = useRef<(HTMLButtonElement | null)[]>([]);
   useEffect(() => {
     menuOptionRefs.current[selected]?.scrollIntoView?.({ block: 'nearest' });
   }, [selected, menu]);
-  /** #3280 §4: every draft change is persisted at once, so a reload or relaunch loses nothing. */
+  /** Every draft change is persisted at once, so a reload or relaunch loses nothing. */
   const setDraft = (value: string): void => {
     stateRef.current = { ...stateRef.current, text: value };
     setDraftState(value);
     if (draftMode === 'persistent') writeDraft(browserNamespace, sessionIdRef.current, stateRef.current);
   };
-  /** #3282 §4d: attachment chips persist alongside the text, under the same per-session draft key. */
+  /** Attachment chips persist alongside the text, under the same per-session draft key. */
   const setAttachments = (
     updater: readonly IDraftAttachment[] | ((current: readonly IDraftAttachment[]) => readonly IDraftAttachment[]),
   ): void => {
@@ -265,7 +265,7 @@ const ComposerInstance = forwardRef<
     setAttachmentsState(next);
     if (draftMode === 'persistent') writeDraft(browserNamespace, sessionIdRef.current, stateRef.current);
   };
-  // #3280 §4: a session switch shows THAT session's own saved draft, never what was typed for
+  // A session switch shows THAT session's own saved draft, never what was typed for
   // another one. The session id becoming known for the very FIRST time (the fallback key was in use
   // until now) instead carries over what is already typed, rather than discarding it. A transient
   // `undefined` mid-switch, once a real id has already been seen, is not a change at all: stay bound
@@ -413,7 +413,7 @@ const ComposerInstance = forwardRef<
     chooseMenuItem(item);
     return true;
   };
-  /** #3280 §2: Edit puts the queued text back in the draft and cancels the queue behind it (the wire
+  /** Edit puts the queued text back in the draft and cancels the queue behind it (the wire
    *  has no per-message cancel — only a whole-queue clear). Offered only when exactly one prompt is
    *  queued: with more than one, `cancel-queue` would still drop every one of them, but only the
    *  shown prompt's text is known here, so "Edit" would silently lose the rest — the row offers only
@@ -432,7 +432,7 @@ const ComposerInstance = forwardRef<
       onDragLeave={onDragLeave}
       onDrop={onDrop}
     >
-      {/* #3282 §4d rule 7: the drop target is announced even to someone who cannot drag a file —
+      {/* The drop target is announced even to someone who cannot drag a file —
           the attach button beside the textarea is how they reach the same result. */}
       <p id="composer-attach-hint" className="sr-only">
         Drag files here, or use Attach files, to add them to your message.
@@ -493,7 +493,7 @@ const ComposerInstance = forwardRef<
           className="gui-rise absolute bottom-full left-0 right-0 mb-2 max-h-[320px] overflow-y-auto rounded-2xl bg-popover p-1.5 shadow-2xl shadow-black/35"
         >
           {menu.map((item, index) => {
-            // #3282 §4e: "Commands" then "Skills" — a header appears once, above the first row of
+            // "Commands" then "Skills" — a header appears once, above the first row of
             // its group (the menu always lists every command before every skill, so a group's rows
             // are contiguous).
             const showGroupHeader = index === 0 || menuGroupLabel(menu[index - 1]!) !== menuGroupLabel(item);
@@ -590,10 +590,10 @@ const ComposerInstance = forwardRef<
           value={draft}
           onChange={(e) => setDraft(e.target.value)}
           onKeyDown={(e) => {
-            // #3280 §4: an Enter that only finishes an IME (Korean/Japanese/Chinese) composition must
+            // An Enter that only finishes an IME (Korean/Japanese/Chinese) composition must
             // not submit or accept the menu — `isComposing` (or `keyCode` 229, on browsers that
             // predate it) marks it; the keystroke is left alone so the browser commits the composition
-            // normally, matching the guard the free-text prompt field uses (#3280 §3).
+            // normally, matching the guard the free-text prompt field uses.
             if (e.key === 'Enter' && (e.nativeEvent.isComposing || e.keyCode === 229)) return;
             if (menu) {
               if (e.key === 'ArrowDown' || e.key === 'ArrowUp') {
@@ -612,9 +612,9 @@ const ComposerInstance = forwardRef<
                 return;
               }
             }
-            // #3280 §2: Esc stops a running turn — but only once the menu (handled above) is out of
+            // Esc stops a running turn — but only once the menu (handled above) is out of
             // the way, so dismissing the `/` menu never doubles as an abort. Not while disconnected:
-            // an abort could not reach the host either (issue #3280 §5).
+            // an abort could not reach the host either.
             if (e.key === 'Escape' && running && connected) {
               e.preventDefault();
               onStop();
@@ -632,7 +632,7 @@ const ComposerInstance = forwardRef<
           <button
             type="button"
             aria-label="Attach files"
-            // #3282 §2 (part 2): the attach button follows the same disconnected rule as the status
+            // The attach button follows the same disconnected rule as the status
             // chips and the `/` menu — a short "Reconnecting…" tooltip in place of its usual one.
             title={connected ? 'Attach files' : 'Reconnecting…'}
             disabled={!connected}
@@ -690,7 +690,8 @@ export function GoalBar({
   onStop,
 }: {
   status: TSessionStatus | null;
-  onStop: () => void;
+  /** Absent for an observer: the goal is shown without a Stop. */
+  onStop?: () => void;
 }): React.ReactElement | null {
   const goal = status?.goal;
   if (!goal || goal.status !== 'active') return null;
@@ -707,15 +708,17 @@ export function GoalBar({
       <span className="flex-shrink-0 text-[13px] tabular-nums text-subtle">
         {goal.iterations}/{goal.maxIterations}
       </span>
-      <button
-        type="button"
-        aria-label="Stop goal"
-        onClick={onStop}
-        className="flex flex-shrink-0 items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
-      >
-        <Square size={11} fill="currentColor" aria-hidden="true" />
-        Stop
-      </button>
+      {onStop && (
+        <button
+          type="button"
+          aria-label="Stop goal"
+          onClick={onStop}
+          className="flex flex-shrink-0 items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground"
+        >
+          <Square size={11} fill="currentColor" aria-hidden="true" />
+          Stop
+        </button>
+      )}
       <span
         aria-hidden="true"
         className="absolute bottom-0 left-0 h-[2px] bg-accent/70 transition-[width]"
@@ -726,4 +729,4 @@ export function GoalBar({
 }
 
 // `StatusRow` (model/mode/effort pop-up menus) and its `ContextRing` moved to `StatusControls.tsx`
-// (#3282 §2 part 2) — imported above.
+// — imported above.

package/src/components/ConversationView.tsx

@@ -44,7 +44,7 @@ function nodeText(node: React.ReactNode): string {
 }
 
 /**
- * "Copy code" / "Copy message" (#3289 §2): shows on hover and keyboard focus, always on touch (no
+ * "Copy code" / "Copy message": shows on hover and keyboard focus, always on touch (no
  * `hover` capability to gate on there). The transient "Copied" / "Couldn't copy" feedback lives in a
  * visually hidden `aria-live` region that is a SIBLING of the button, not a descendant of it — ARIA
  * name computation only ever looks at an element's own subtree, so a live region placed anywhere
@@ -96,7 +96,7 @@ function CopyButton({
 /**
  * A fenced code block: unwrapped and horizontally scrollable (a visible scrollbar via
  * `.gui-code-scroll`, `tabIndex=0` and a name so a keyboard user without a mouse wheel can still reach
- * the rest of a long line), with its own "Copy code" button revealed on hover/focus (#3289 §2).
+ * the rest of a long line), with its own "Copy code" button revealed on hover/focus.
  */
 function CodeBlockRenderer({ children }: { children?: React.ReactNode }): React.ReactElement {
   const text = nodeText(children).replace(/\n$/, '');
@@ -123,7 +123,7 @@ interface IConversationViewProps {
   activeTools: IActiveTool[];
   streamingText: string;
   isThinking: boolean;
-  /** This connection's own driver id (§3289 §3), threaded to every message it renders. */
+  /** This connection's own driver id, threaded to every message it renders. */
   ownDriverId: TDriverId | null;
 }
 
@@ -207,11 +207,11 @@ function UserBlock({
 }: {
   content: string;
   author?: string;
-  /** This connection's own driver id (§3289 §3): its own turns never carry a "from" label. */
+  /** This connection's own driver id: its own turns never carry a "from" label. */
   ownDriverId: TDriverId | null;
 }): React.ReactElement {
-  // REMOTE-014 E5 (display-only, OWNER PRINCIPLE): show WHO drove this turn when it wasn't this
-  // connection's own — in plain words, never the raw server-assigned id. #3288 §1: AGENT_DRIVER_ID
+  // Display-only: show WHO drove this turn when it wasn't this
+  // connection's own — in plain words, never the raw server-assigned id. AGENT_DRIVER_ID
   // (never "the same surface" — see isSameSurface) reads "Automatic — loop": today it is only ever
   // a loop's own injected turn (requestWakeup in interactive-session.ts).
   const coDriver =
@@ -239,7 +239,7 @@ function AgentBlock({
       {isStreaming && (
         <span className="ml-1 inline-block h-2.5 w-2.5 animate-pulse rounded-full bg-foreground/70 align-middle" />
       )}
-      {/* #3289 §2: every assistant message can copy its own Markdown source; a still-streaming one
+      {/* Every assistant message can copy its own Markdown source; a still-streaming one
           has nothing settled yet to copy, so the button waits for it to finish. */}
       {!isStreaming && (
         <div className="flex opacity-0 transition-opacity focus-within:opacity-100 group-hover/message:opacity-100 group-focus-within/message:opacity-100 [@media(hover:none)]:opacity-100">
@@ -264,7 +264,7 @@ function ToolIcon({ name, className }: { name: string; className?: string }): Re
 }
 
 /**
- * #3288: a long path's directory is shortened FROM THE LEFT so the filename — the part someone
+ * A long path's directory is shortened FROM THE LEFT so the filename — the part someone
  * actually needs to recognise — is never the part that gets cut.
  */
 const PATH_DISPLAY_MAX_CHARS = 52;
@@ -405,7 +405,7 @@ function ToolCard({ tool }: { tool: IActiveTool }): React.ReactElement {
     Boolean(parsedResult?.structured);
   const hasObservations = Boolean(tool.toolResultParts?.length);
   const expandable = hasDiff || hasOutput || hasObservations;
-  // #3288: a projected `/command` tool shows its command, never the internal provider tool name.
+  // A projected `/command` tool shows its command, never the internal provider tool name.
   const label = tool.commandName ? `Ran /${tool.commandName}` : tool.name;
 
   return (
@@ -431,7 +431,7 @@ function ToolCard({ tool }: { tool: IActiveTool }): React.ReactElement {
           {label}
         </span>
         {tool.displayPath ? (
-          // #3288: server-computed, workspace-relative, and never middle/end-truncated — only the
+          // Server-computed, workspace-relative, and never middle/end-truncated — only the
           // directory portion is shortened, from the left, so the filename always reads in full.
           <span
             className="min-w-0 flex-1 break-all font-mono text-[12.5px] text-subtle"
@@ -493,7 +493,7 @@ const COMMAND_TONE: Record<ICommandOutputEntry['tone'], { dot: string; text: str
 };
 
 /**
- * A slash command's outcome, where it was typed: line breaks kept, long output folded. #3289 §2 — the
+ * A slash command's outcome, where it was typed: line breaks kept, long output folded. The
  * card itself still marks the command (the `/name` header stays monospace, a fixed-width label rather
  * than prose), but its BODY reads as normal text — proportional, normal size — unless it is actually a
  * long listing (the same measure that decides whether to fold it: many raw lines, or tall once
@@ -554,7 +554,7 @@ function CommandCard({ entry }: { entry: ICommandOutputEntry }): React.ReactElem
 /** One line per finished turn's tool calls; the calls themselves open on demand. */
 function ToolGroup({ tools }: { tools: readonly IActiveTool[] }): React.ReactElement | null {
   const [open, setOpen] = useState(false);
-  // #3288: an internal signal tool (e.g. the goal-status tool) is never shown as a call — the goal
+  // An internal signal tool (e.g. the goal-status tool) is never shown as a call — the goal
   // bar shows progress instead. A group made up ENTIRELY of internal tools renders nothing at all.
   const visible = tools.filter((tool) => !tool.internal);
   const failed = visible.filter((tool) => tool.status === 'error').length;
@@ -605,7 +605,7 @@ function ToolGroup({ tools }: { tools: readonly IActiveTool[] }): React.ReactEle
   );
 }
 
-/** #3288: a turn that changed files ends with this compact row — a click opens that file's diff. */
+/** A turn that changed files ends with this compact row — a click opens that file's diff. */
 function ChangedFilesRow({ files }: { files: readonly IChangedFileSummary[] }): React.ReactElement {
   const [openPath, setOpenPath] = useState<string | null>(null);
   return (
@@ -671,7 +671,7 @@ export function ConversationView({
   };
 
   /**
-   * #3289 §2 — follows new content only while the person is already at the bottom: streaming text, a
+   * Follows new content only while the person is already at the bottom: streaming text, a
    * finished message, a tool starting or finishing. Expanding something already on screen (a tool row,
    * a folded command card) touches none of these, so it never causes a jump — the effect simply does
    * not run for it. An instant jump rather than a smooth one, so it can never itself be mistaken by

package/src/components/Dialog.tsx

@@ -3,9 +3,9 @@
 import React, { useEffect, useId, useRef } from 'react';
 
 /**
- * The shared modal-shell primitive (#3282 §4a, coordinated with #3282 §2's model/mode controls,
- * which also need a confirmation dialog). Whichever of the two lands first defines this file; the
- * other rebases onto it rather than shipping a second copy.
+ * The shared modal-shell primitive (the model/mode controls and the
+ * session-delete confirmation both need a confirmation dialog). Both
+ * consumers share it rather than shipping a second copy.
  *
  * `role="dialog"` (or `"alertdialog"`, see `role` below), `aria-modal`, a labelled title, focus moved
  * inside on open, a focus trap while open, Esc to close, and focus restored to whatever had it before
@@ -26,7 +26,7 @@ export interface IDialogProps {
   /** Id of an element describing the dialog, wired to `aria-describedby`. */
   describedById?: string;
   /**
-   * Overrides what focus returns to when the dialog closes (#3289 §1). Default behavior captures
+   * Overrides what focus returns to when the dialog closes. Default behavior captures
    * `document.activeElement` when the dialog opens — which fails for a caller whose opener element
    * is removed from the DOM in the SAME update that opens this dialog (e.g. a menu item whose menu
    * closes as this dialog opens): by the time this dialog's open-effect runs, the browser has already
@@ -34,7 +34,7 @@ export interface IDialogProps {
    */
   restoreFocusTo?: HTMLElement | null;
   /**
-   * #3282 §4e: `'alertdialog'` for a dialog that interrupts to demand a decision before anything else
+   * `'alertdialog'` for a dialog that interrupts to demand a decision before anything else
    * can happen (a destructive confirmation) — WAI-ARIA's distinction from a dialog the user can set
    * aside. Default `'dialog'`.
    */
@@ -182,7 +182,7 @@ export interface IConfirmDialogProps {
 
 /**
  * A title, a body, Cancel focused by default, and a confirm button that can be styled destructive.
- * #3282 §4e: `destructive` also picks the dialog's ARIA role — `"alertdialog"` when it is (an action
+ * `destructive` also picks the dialog's ARIA role — `"alertdialog"` when it is (an action
  * that demands a decision before anything else, e.g. deleting a session), `"dialog"` otherwise.
  */
 export function ConfirmDialog({

package/src/components/DiffLines.tsx

@@ -5,7 +5,7 @@ import React, { useState } from 'react';
 import type { IDiffLine } from '@robota-sdk/agent-interface-session';
 
 /**
- * #3288: renders a unified diff already built server-side (the ONE diff builder, in
+ * Renders a unified diff already built server-side (the ONE diff builder, in
  * `interactive-session-streaming.ts`) — a tool row's expanded result and the Edit/Write permission
  * prompt both show the SAME diff through this, so approving a change and reviewing it afterwards
  * look identical. This component never computes a diff itself, only displays one.
@@ -24,7 +24,7 @@ const DIFF_LINE_PREFIX: Record<IDiffLine['type'], string> = {
   hunk: '',
 };
 
-// #3288 review SHOULD 3: the server already caps each diff side at 500 lines — this is a second,
+// The server already caps each diff side at 500 lines — this is a second,
 // independent fold so a diff that size never renders into the DOM at once either.
 const DIFF_FOLD_LINES = 40;
 const DIFF_EXPANDED_CAP_LINES = 500;

package/src/components/ExecutionDetailSheet.tsx

@@ -28,7 +28,7 @@ const RECORD_KIND_LABEL: Record<IExecutionDetailRecord['kind'], string> = {
 };
 
 /**
- * #3288 §1: opening an Agents panel entry — status, what it was asked (one line, from its
+ * Opening an Agents panel entry — status, what it was asked (one line, from its
  * headline), its transcript so far (paginated via `onLoadMore`) and, once it has one, its result.
  * `entry === null` renders nothing; the caller owns showing/hiding this over the panel.
  */