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

package/src/components/StatusControls.tsx

@@ -10,7 +10,7 @@ import type { IPopupMenuItem, IPopupMenuSection } from './PopupMenu.js';
 import type { TCommandCatalog, TModelListSnapshot, TSessionStatus } from '../hooks/session-client-types.js';
 
 /**
- * #3282 §2 (part 2) — the plain label for each effort selection. Kept in step with
+ * The plain label for each effort selection. Kept in step with
  * `EFFORT_LEVEL_LABELS` in `packages/agent-framework/src/effort/effort-resolution.ts` (which
  * `agent-command`'s `/effort` re-exports and reports as its own outcome text) — this package cannot
  * import that one directly (it is a Node-side package this browser bundle does not depend on), so the
@@ -28,7 +28,7 @@ const EFFORT_LEVEL_LABELS: Readonly<Record<TSessionStatus['effort'], string>> =
   max: 'Maximum',
 };
 
-/** The primary row (#3282 §2 design point 4); the rest sit under "More". */
+/** The primary row; the rest sit under "More". */
 const EFFORT_PRIMARY: readonly TSessionStatus['effort'][] = ['auto', 'low', 'medium', 'high'];
 const EFFORT_MORE: readonly TSessionStatus['effort'][] = ['none', 'minimal', 'xhigh', 'max'];
 
@@ -48,7 +48,7 @@ const CHIP_CLASS =
   'flex min-w-0 items-center gap-1.5 rounded-lg px-2 py-1 text-[13px] text-muted-foreground hover:bg-hover hover:text-foreground disabled:cursor-not-allowed disabled:opacity-60 disabled:hover:bg-transparent';
 const CHIP_WARNING_CLASS = 'text-warning hover:text-warning';
 
-/** One status chip that opens a `PopupMenu` HIG pop-up beneath it (#3282 §2). */
+/** One status chip that opens a `PopupMenu` HIG pop-up beneath it. */
 function StatusChip({
   chipLabel,
   value,
@@ -79,7 +79,7 @@ function StatusChip({
         aria-label={`${chipLabel}: ${value}`}
         aria-haspopup="menu"
         aria-expanded={open}
-        // #3289 §2: below `md` the value text is hidden (icon-only) — the tooltip is where it stays
+        // Below `md` the value text is hidden (icon-only) — the tooltip is where it stays
         // reachable on hover and keyboard focus, so it always carries the value, not just "Change X".
         title={connected ? `${chipLabel}: ${value}` : 'Reconnecting…'}
         disabled={!connected}
@@ -90,7 +90,7 @@ function StatusChip({
         className={`${CHIP_CLASS} ${warning ? CHIP_WARNING_CLASS : ''}`}
       >
         {icon}
-        {/* #3289 §2: below `md` (the same breakpoint the session-list sheet uses) the chip collapses
+        {/* Below `md` (the same breakpoint the session-list sheet uses) the chip collapses
             to its icon alone — never a truncated fragment ("d…", "claude-…") — the value stays
             reachable via the tooltip above and the accessible name either way. */}
         <span className="hidden truncate md:inline">{value}</span>
@@ -110,7 +110,7 @@ function StatusChip({
 const ICON_PROPS = { size: 14, strokeWidth: 1.75, 'aria-hidden': true } as const;
 
 /**
- * Model · mode · effort, each a pop-up menu (#3282 §2), and the context the conversation fills.
+ * Model · mode · effort, each a pop-up menu, and the context the conversation fills.
  * Every menu applies its choice through the same command wire path the equivalent typed slash
  * command uses — `onSilentCommand` — so `/model <id>`, `/mode <name>` and `/effort <value>` and a
  * click in these menus can never drift apart; the control's own chip label (fed by the session's next
@@ -129,14 +129,14 @@ export function StatusRow({
   catalog: TCommandCatalog | null;
   modelList: TModelListSnapshot | null;
   onRequestModelList: () => void;
-  /** #3282 §4b: "Manage providers…" opens Settings at the Providers & Models section. */
+  /** "Manage providers…" opens Settings at the Providers & Models section. */
   onManageProviders: () => void;
   /** Applies a model/mode/effort choice without a conversation card (see the module doc). */
   onSilentCommand: (name: string, args?: string) => void;
   connected: boolean;
 }): React.ReactElement {
   const used = status ? Math.round(status.context.usedPercentage) : null;
-  // #3282 §2: "Skip all checks" asks for confirmation on the shared `ConfirmDialog` (#3331) before
+  // "Skip all checks" asks for confirmation on the shared `ConfirmDialog` before
   // applying — every other mode applies at once. `true` while that confirmation is open.
   const [pendingBypassConfirm, setPendingBypassConfirm] = useState(false);
 
@@ -158,7 +158,7 @@ export function StatusRow({
           key: 'manage-providers',
           label: 'Manage providers…',
           kind: 'action',
-          // #3282 §4b: opens Settings at the "Providers & Models" section — the same list, Use/Model/
+          // Opens Settings at the "Providers & Models" section — the same list, Use/Model/
           // Edit/Test/Duplicate/Delete actions and "Add provider…" flow now live there instead of in
           // this menu's own server-driven picker.
           onSelect: onManageProviders,
@@ -175,7 +175,7 @@ export function StatusRow({
           label: sub.displayName ?? sub.name,
           description: sub.description,
           checked: sub.name === status?.permissionMode,
-          // #3282 §2: every mode but "Skip all checks" applies at once; that one opens a
+          // Every mode but "Skip all checks" applies at once; that one opens a
           // confirmation first (rendered below) instead of applying here directly.
           onSelect:
             sub.name === 'bypassPermissions'
@@ -253,8 +253,8 @@ export function StatusRow({
         <ContextRing percent={used ?? 0} />
         {used === null ? '' : `${used}%`}
       </span>
-      {/* #3282 §2: matches the wording the Settings screen's own permission-mode picker already
-          ships for the identical action (#3331's `SettingsScreen.tsx`), so switching to "Skip all
+      {/* Matches the wording the Settings screen's own permission-mode picker already
+          ships for the identical action (`SettingsScreen.tsx`), so switching to "Skip all
           checks" reads the same warning regardless of which control was used. */}
       <ConfirmDialog
         open={pendingBypassConfirm}

package/src/components/composer-attachments.ts

@@ -1,5 +1,5 @@
 /**
- * #3282 §4d — pure logic behind the composer's attach button and drag-and-drop, kept apart from
+ * Pure logic behind the composer's attach button and drag-and-drop, kept apart from
  * `Composer.tsx` so it is testable without React or a DOM.
  *
  * The runtime resolves `@relative/path` references already (`prompt-file-reference-resolver.ts` in
@@ -26,7 +26,7 @@ export const MAX_ATTACHMENT_TOTAL_BYTES = 256 * 1024;
  */
 export const MAX_ATTACHMENT_COUNT = 8;
 
-/** Shown whenever a file cannot be resolved to a path inside the workspace — rule 3 (#3282 §4). */
+/** Shown whenever a file cannot be resolved to a path inside the workspace — rule 3. */
 export const ATTACHMENT_NO_PATH_NOTICE = 'Only files inside this project folder can be attached.';
 
 /** Shown once the running count of attachments would pass `MAX_ATTACHMENT_COUNT`. */
@@ -50,7 +50,7 @@ export interface IDraftAttachment {
 }
 
 /**
- * One file the desktop host's native "Attach files" dialog returned (#3282 §4d). Kept in structural
+ * One file the desktop host's native "Attach files" dialog returned. Kept in structural
  * sync with `IPickedFile` in `apps/agent-gui-web/src/gui-host.ts` and
  * `apps/agent-app/electron/sidecar.ts` — the three are not imported across the Electron/browser/
  * presentation package boundaries, same as this codebase already does for the trust question.
@@ -228,11 +228,11 @@ export function evaluateCandidateFile(
 }
 
 /**
- * Inserts each chip as the runtime's existing `@relative/path` reference syntax — rule 2 (#3282 §4).
+ * Inserts each chip as the runtime's existing `@relative/path` reference syntax — rule 2.
  * Always prefixed with `./`: the parser's `isPathLikeReference` only treats a token as a file
  * reference when it has a `.` somewhere or an explicit relative-path prefix, so a dotless name like
  * `Makefile`, `Dockerfile` or `LICENSE` — common, useful, and never refused for lacking one — would
- * otherwise be sent as plain text with no file and no diagnostic (follow-up to #3282 §4d). `./` makes
+ * otherwise be sent as plain text with no file and no diagnostic. `./` makes
  * every path path-like regardless of its name, and the resolver collapses it back to the plain
  * relative path before use.
  */

package/src/driver-labels.ts

@@ -1,5 +1,5 @@
 /**
- * #3289 §3 — plain, human labels for a REMOTE-014 E5 driver id, and whether one names the SAME KIND
+ * Plain, human labels for a driver id, and whether one names the SAME KIND
  * of surface as this connection's own. Shared by every place the GUI shows who drove a turn (a
  * message, a permission/ask prompt): the person's own input never carries a label, and a turn from a
  * different kind of surface does — in words, never a raw id.
@@ -16,7 +16,7 @@ import { AGENT_DRIVER_ID, OWNER_DRIVER_ID } from '@robota-sdk/agent-interface-se
 import type { TDriverId } from '@robota-sdk/agent-interface-session';
 
 /**
- * `remote` (a plain WS remote connection's own id, `'remote:ws'`) and `peer` (an unrelated REMOTE-014
+ * `remote` (a plain WS remote connection's own id, `'remote:ws'`) and `peer` (an unrelated
  * mesh peer's turn, `'peer:<sessionId>'`) are kept as DIFFERENT kinds even though both render the same
  * label text: a plain WS viewer's own id is `'remote:ws'`, and folding a peer's turn into that same
  * kind would make `isSameSurface` treat a co-driver's turn as this connection's own and hide it — the
@@ -76,7 +76,7 @@ export function humanDriverLabel(driverId: TDriverId): string {
 
 /**
  * The ready-to-render attribution phrase: "from <label>" for a co-driver, or "Automatic — loop"
- * for the agent's own wake-ups — "from automatic" reads as a typo, not a source. #3288 §1: every
+ * for the agent's own wake-ups — "from automatic" reads as a typo, not a source. Every
  * `AGENT_DRIVER_ID` turn is, today, a loop's own injected iteration (fixed or self-paced —
  * `requestWakeup` in `interactive-session.ts` is the one place that stamps this id); if a
  * differently-sourced agent-wakeup turn is ever added, this label needs to name that source

package/src/execution-entry-status.ts

@@ -1,5 +1,5 @@
 /**
- * #3288 §1: one true-status label for an execution-workspace entry, shared by the Agents panel row
+ * One true-status label for an execution-workspace entry, shared by the Agents panel row
  * and its detail sheet so the two never say something different about the same entry.
  *
  * A completed task whose result still reports a refused tool call is not "Done" — it is "Needs

package/src/goal-view.ts

@@ -1,5 +1,5 @@
 /**
- * #3282 §4 part b-3 — the Agents panel's Goal row: the objective, and its status in plain words.
+ * The Agents panel's Goal row: the objective, and its status in plain words.
  */
 
 import type { IGoalState } from '@robota-sdk/agent-interface-session';

package/src/hooks/command-menu.ts

@@ -1,8 +1,8 @@
 /**
- * #3186 — what the composer's `/` menu offers for a draft. Pure, so the matching is testable without
+ * What the composer's `/` menu offers for a draft. Pure, so the matching is testable without
  * rendering: the TUI's autocomplete offers commands and skills the same way.
  *
- * #3282 §4e: a command the GUI cannot run at all (`isExcludedCommand`) never appears here — there is
+ * A command the GUI cannot run at all (`isExcludedCommand`) never appears here — there is
  * nothing useful for choosing it to do, and typing it by hand is answered separately (see
  * `excluded-commands.ts`, `useSessionClient.ts`). Commands are listed before skills, each in its own
  * group, so the menu can head them "Commands" and "Skills".

package/src/hooks/excluded-commands.ts

@@ -1,9 +1,9 @@
 /**
- * #3282 §4e — built-in commands left out of the GUI on purpose (the design decisions in
+ * Built-in commands left out of the GUI on purpose (the design decisions in
  * `packages/agent-gui-web/docs/SPEC.md` group them by reason). The `/` menu never lists one of these
  * (`command-menu.ts`), and typing one by hand anyway is caught before it reaches the session
  * (`useSessionClient.ts`'s `send`) and answered with the plain sentence here — never the session's own
- * refusal, and never the generic "not available on this surface" line CMD-004 uses for a screen this
+ * refusal, and never the generic "not available on this surface" line used for a screen this
  * surface simply has not built yet (see `ui-intent-state.ts`).
  */
 

package/src/hooks/prompt-state.ts

@@ -1,5 +1,5 @@
 /**
- * Pure prompt-state derivation for the web UI (REMOTE-009 Step 4 / REMOTE-007 render+answer).
+ * Pure prompt-state derivation for the web UI (render + answer).
  *
  * The paired browser is the session OWNER (local == remote), so it must render and ANSWER the owner's own
  * permission/ask prompts. The server carries `permission_request`/`ask_request` (a prompt appears) and
@@ -32,9 +32,9 @@ export function applyPromptEvent(
 ): readonly TPendingPrompt[] {
   switch (msg.type) {
     case 'permission_request': {
-      // REMOTE-014 E5: keep requesterDriverId so the surface can show WHICH driver's turn raised the
-      // prompt. Issue #3288 §1: keep `requester` too, so it can show WHICH background agent did.
-      // #3288 §2: diffLines/diffFile (Edit/Write) and cwd (Shell, only when it differs) are the
+      // Keep requesterDriverId so the surface can show WHICH driver's turn raised the
+      // prompt. Keep `requester` too, so it can show WHICH background agent did.
+      // diffLines/diffFile (Edit/Write) and cwd (Shell, only when it differs) are the
       // server's diff/command preview — carried through unchanged so the prompt can render them
       // instead of raw args.
       const { id, toolName, toolArgs, requesterDriverId, requester, diffLines, diffFile, cwd } =

package/src/hooks/server-message-handling.ts

@@ -4,7 +4,7 @@ export type TServerMessageHandling =
   'reducer-state' | 'visible-notice' | 'intentionally-not-rendered' | 'transport-control';
 
 /**
- * ARCH-2164: exhaustive ownership for every decoded server variant on the shared GUI surface.
+ * Exhaustive ownership for every decoded server variant on the shared GUI surface.
  * Adding a wire variant without deciding its GUI disposition is a compile error instead of a silent
  * reducer omission. `intentionally-not-rendered` is an explicit disposition for server-side query
  * snapshots and domain events that this session/usage surface does not currently visualize.
@@ -28,10 +28,10 @@ export const SERVER_MESSAGE_HANDLING = {
   turn_source: 'intentionally-not-rendered',
   commands: 'reducer-state',
   session_status: 'reducer-state',
-  // #3282 §2 part 2: the reply to `list-models`, fed into `modelList` state by the model control's
+  // The reply to `list-models`, fed into `modelList` state by the model control's
   // pop-up menu (`useModelListState`).
   model_list: 'reducer-state',
-  // #3282 §4 part b-3: the agent switcher sheet's own correlated request/response state (`useAgentSwitcherState`).
+  // The agent switcher sheet's own correlated request/response state (`useAgentSwitcherState`).
   agent_definitions: 'reducer-state',
   sessions: 'reducer-state',
   sessions_error: 'reducer-state',
@@ -47,10 +47,10 @@ export const SERVER_MESSAGE_HANDLING = {
   stored_session_usage_report: 'reducer-state',
   stored_session_usage_report_error: 'visible-notice',
   executing: 'intentionally-not-rendered',
-  // #3280 §2: the prompt queued behind a running turn — shown above the composer with Edit/Remove.
+  // The prompt queued behind a running turn — shown above the composer with Edit/Remove.
   pending: 'reducer-state',
   execution_workspace_event: 'reducer-state',
-  // #3288 §1: the Agents panel's detail sheet reads a page at a time.
+  // The Agents panel's detail sheet reads a page at a time.
   execution_detail: 'reducer-state',
   execution_detail_error: 'reducer-state',
   // Esc's loop stop is a TUI-only shortcut; this surface's loops stop through the Agents panel
@@ -58,7 +58,7 @@ export const SERVER_MESSAGE_HANDLING = {
   waiting_loop_stop: 'intentionally-not-rendered',
   // The Agents panel already reflects a task's cancel/close/send through `execution_workspace_event`;
   // nothing here renders this narrower, task-only event on its own.
-  // #3282 §4c: the Project panel's reads — reduced into state by `useProjectPanelState`.
+  // The Project panel's reads — reduced into state by `useProjectPanelState`.
   project_status: 'reducer-state',
   project_diff: 'reducer-state',
   project_memory: 'reducer-state',
@@ -70,7 +70,7 @@ export const SERVER_MESSAGE_HANDLING = {
   plan_event: 'intentionally-not-rendered',
   context_file_refreshed: 'intentionally-not-rendered',
   branch_event: 'intentionally-not-rendered',
-  // #3282 §4 part b-3: the Agents panel's "Scheduled" group's own roster (`useSchedulesState`).
+  // The Agents panel's "Scheduled" group's own roster (`useSchedulesState`).
   background_tasks: 'reducer-state',
   background_task: 'intentionally-not-rendered',
   background_job_groups: 'intentionally-not-rendered',
@@ -82,16 +82,16 @@ export const SERVER_MESSAGE_HANDLING = {
   ui_intent: 'reducer-state',
   session_renamed: 'reducer-state',
   history_cleared: 'reducer-state',
-  // #3288 §1: a failed Stop (e.g. a task that finished a moment before it arrived) is worth telling
-  // the operator; a success shows itself through the workspace snapshot's own update. #3282 §4 part
-  // b-3: a schedule's Delete uses this same `cancel-background-task` write — `useSchedulesState`
+  // A failed Stop (e.g. a task that finished a moment before it arrived) is worth telling
+  // the operator; a success shows itself through the workspace snapshot's own update. A schedule's
+  // Delete uses this same `cancel-background-task` write — `useSchedulesState`
   // refreshes the roster on a successful cancel, and a failure gets the same visible notice as Stop.
   background_task_control_result: 'visible-notice',
   protocol_error: 'visible-notice',
   resume_gap: 'transport-control',
   // The connection's effective role: the session client holds it and stops sending what an observer may not.
   surface_role: 'reducer-state',
-  // #3282 §4a: the Settings screen's own correlated request/response state (`useSettingsState`).
+  // The Settings screen's own correlated request/response state (`useSettingsState`).
   settings: 'reducer-state',
   settings_error: 'reducer-state',
 } as const satisfies Readonly<Record<TServerMessage['type'], TServerMessageHandling>>;

package/src/hooks/session-client-types.ts

@@ -38,7 +38,7 @@ export type TCommandCatalog = Omit<Extract<TServerMessage, { type: 'commands' }>
 export type TSessionStatus = Extract<TServerMessage, { type: 'session_status' }>['status'];
 
 /**
- * #3282 §2 (part 2): the models a person may switch to, grouped by configured provider profile —
+ * The models a person may switch to, grouped by configured provider profile —
  * what the model control's pop-up menu is built from. `null` until requested (`list-models`).
  */
 export type TModelListSnapshot = Omit<
@@ -59,7 +59,7 @@ export type TCurrentSessionUsageReport = Extract<
   { type: 'usage_report' }
 >['report'];
 
-/** #3282 §4c — the Project panel's reads. Each result crosses the wire unchanged from the session
+/** The Project panel's reads. Each result crosses the wire unchanged from the session
  *  method's own return value (see `wire-messages.ts`'s doc comment on `project_status`). */
 export type TProjectStatusRead = Extract<TServerMessage, { type: 'project_status' }>['result'];
 export type TProjectDiffRead = Extract<TServerMessage, { type: 'project_diff' }>['result'];
@@ -70,7 +70,7 @@ export interface IConversationMessage {
   role: 'user' | 'assistant';
   content: string;
   isStreaming?: boolean;
-  /** REMOTE-014 E5: the co-driving author of a user turn. */
+  /** The co-driving author of a user turn. */
   author?: string;
 }
 
@@ -101,7 +101,7 @@ export interface IChangedFileSummary {
   diffLines: readonly IDiffLine[];
 }
 
-/** #3288: a turn that changed files ends with this compact summary row. */
+/** A turn that changed files ends with this compact summary row. */
 export interface IChangedFilesEntry {
   id: string;
   role: 'changed-files';
@@ -117,28 +117,28 @@ export interface IActiveTool {
   status: 'running' | 'done' | 'error';
   input?: string;
   result?: IToolState['result'];
-  /** #3288: correlates this call across `tool_start`/`tool_end` — how a finished call is matched. */
+  /** Correlates this call across `tool_start`/`tool_end` — how a finished call is matched. */
   executionId?: string;
-  /** #3288: a unified diff (Edit/Write), built server-side — the same one an approval prompt showed. */
+  /** A unified diff (Edit/Write), built server-side — the same one an approval prompt showed. */
   diffLines?: readonly IDiffLine[];
-  /** #3288: the file `diffLines` concerns, workspace-relative when the server could make it so. */
+  /** The file `diffLines` concerns, workspace-relative when the server could make it so. */
   diffFile?: string;
-  /** #3288: the tool's raw result payload (Shell output, Read content, …), already capped server-side. */
+  /** The tool's raw result payload (Shell output, Read content, …), already capped server-side. */
   toolResultData?: string;
   /** Admitted observations; opaque resource URIs do not authorize a client-side fetch. */
   toolResultParts?: IToolState['toolResultParts'];
-  /** #3288: a workspace-relative display form of a path argument, additive to `input`/`firstArg`. */
+  /** A workspace-relative display form of a path argument, additive to `input`/`firstArg`. */
   displayPath?: string;
-  /** #3288: set when this call is a projected `/command` tool — render "Ran /<commandName>". */
+  /** Set when this call is a projected `/command` tool — render "Ran /<commandName>". */
   commandName?: string;
-  /** #3288: true for an internal signal tool (e.g. goal-status) — never rendered as a call. */
+  /** True for an internal signal tool (e.g. goal-status) — never rendered as a call. */
   internal?: boolean;
 
   appView?: IToolAppViewRef;
 }
 
 /**
- * #3280 §2: a prompt the host queued behind the running turn — the server has already taken it, so it
+ * A prompt the host queued behind the running turn — the server has already taken it, so it
  * is not lost, but it will not run until the turn ends. `count` is the TOTAL number waiting (the shown
  * `text` plus any behind it), matching the wire's `pendingCount`.
  */
@@ -152,7 +152,7 @@ export interface ISessionNotice {
   /**
    * `session-change-refused`: the host refused a new or switch this surface asked for.
    * `background-task-control-failed`: a Stop in the Agents panel did not take effect.
-   * `command-failed` (#3282 §2 part 2): a change made through a status control (model/mode/effort)
+   * `command-failed`: a change made through a status control (model/mode/effort)
    * failed — a plain notice rather than a conversation card, since the control's own label is where
    * a successful change confirms itself and the label stays unchanged when it fails.
    */
@@ -165,7 +165,7 @@ export interface ISessionNotice {
     | 'context-compacted';
   message: string;
   /**
-   * A `session-error` classified at the wire boundary (#3289 §3) — present only when the host could
+   * A `session-error` classified at the wire boundary — present only when the host could
    * tell what kind of failure this was. `SessionNotices` maps it to one plain sentence and keeps
    * `message` behind a "Details" disclosure; absent, it shows `message` as before.
    */
@@ -195,7 +195,7 @@ export type TMakeSessionClient<TStatus extends string = TConnectionStatus> = (ca
 export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   status: TStatus;
   /**
-   * Set by `useWsSession` once its reconnect retries give up (issue #3280 §5): the runtime is not
+   * Set by `useWsSession` once its reconnect retries give up: the runtime is not
    * coming back by itself. Cleared again once `status` reaches `connected`. Undefined for a reducer
    * with no such concept (e.g. the WebRTC surface, which has its own `failed`/`refused` statuses).
    */
@@ -215,7 +215,7 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   /** Null until the session has answered; refreshed after every command and turn. */
   sessionStatus: TSessionStatus | null;
   /**
-   * #3282 §2 (part 2): the models the model control's pop-up menu offers. Null until requested — the
+   * The models the model control's pop-up menu offers. Null until requested — the
    * menu asks for it when it opens (or refreshes it) rather than every surface fetching it eagerly.
    */
   modelList: TModelListSnapshot | null;
@@ -239,13 +239,13 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   setSessionSidebarOpen: (open: boolean) => void;
   send: (msg: TClientMessage) => void;
   /**
-   * #3282 §2 (part 2): run a command the SAME way `send({type:'command', ...})` does, but mark its
+   * Run a command the SAME way `send({type:'command', ...})` does, but mark its
    * outcome as controls-triggered — `useSessionClient` skips the conversation card for a change made
    * this way (the control's own label confirms it instead), while a typed command keeps its card.
    */
   sendCommandSilently: (name: string, args?: string) => void;
   pendingPrompts: readonly TPendingPrompt[];
-  /** #3280 §2: the prompt queued behind a running turn, or null when none is queued. */
+  /** The prompt queued behind a running turn, or null when none is queued. */
   queuedPrompt: IQueuedPrompt | null;
   answerPermission: (id: string, result: TPermissionResultValue) => void;
   answerAsk: (id: string, response: TActionResponse) => void;
@@ -263,29 +263,29 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   requestCurrentSessionUsage: () => void;
   sessionNotices: readonly ISessionNotice[];
   dismissSessionNotice: (id: string) => void;
-  /** #3282 §4a: whether the Settings modal is shown. `/settings` and the gear button both open it. */
+  /** Whether the Settings modal is shown. `/settings` and the gear button both open it. */
   settingsOpen: boolean;
   settingsStatus: 'idle' | 'loading' | 'ready' | 'error';
   /** Null until the first `get-settings` reply arrives. */
   settingsSnapshot: ISettingsSnapshot | null;
   /** The plain message from the most recent failed read or write, if any. */
   settingsError: string | null;
-  /** #3282 §4 part b-2: which section the screen should land on for the open in progress. */
+  /** Which section the screen should land on for the open in progress. */
   settingsInitialSectionId: string | null;
   /**
    * Opens the screen and (re-)fetches its snapshot — the "reopen and see the new value" path.
    * `sectionId` jumps straight to that section (e.g. `/plugin` → `'plugins'`, "Manage providers…"
-   * (#3282 §2/§4b) → `'providers'`); omitted opens on the screen's own default (General).
+   * → `'providers'`); omitted opens on the screen's own default (General).
    */
   openSettings: (sectionId?: string) => void;
   closeSettings: () => void;
   /** A failed write leaves `settingsSnapshot` at its previous value; `settingsError` names why. */
   updateSettings: (patch: TSettingsPatch) => void;
-  /** #3282 §4e: whether the Help sheet is shown. `/help` opens it instead of reaching the session. */
+  /** Whether the Help sheet is shown. `/help` opens it instead of reaching the session. */
   helpOpen: boolean;
   openHelp: () => void;
   closeHelp: () => void;
-  /** #3288 §1: the id of the execution-workspace entry open in the detail sheet, or null when closed. */
+  /** The id of the execution-workspace entry open in the detail sheet, or null when closed. */
   openEntryId: string | null;
   executionDetailStatus: 'idle' | 'loading' | 'ready' | 'error';
   executionDetailRecords: readonly IExecutionDetailRecord[];
@@ -300,21 +300,21 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   openExecutionDetail: (entryId: string) => void;
   loadMoreExecutionDetail: () => void;
   closeExecutionDetail: () => void;
-  /** #3282 §4c — the Project panel's "Changes" section (git status). */
+  /** The Project panel's "Changes" section (git status). */
   projectStatusState: 'idle' | 'loading' | 'ready' | 'error';
   projectStatus: TProjectStatusRead | null;
   requestProjectStatus: () => void;
-  /** #3282 §4c — the Project panel's "File diff" section, for the file `projectDiffPath` names. */
+  /** The Project panel's "File diff" section, for the file `projectDiffPath` names. */
   projectDiffState: 'idle' | 'loading' | 'ready' | 'error';
   projectDiffPath: string | null;
   projectDiff: TProjectDiffRead | null;
   requestProjectDiff: (path: string) => void;
-  /** #3282 §4c — the Project panel's "Memory" section. */
+  /** The Project panel's "Memory" section. */
   projectMemoryState: 'idle' | 'loading' | 'ready' | 'error';
   projectMemory: TProjectMemoryRead | null;
   requestProjectMemory: () => void;
   /**
-   * #3282 §4 part b-3: whether the agent switcher sheet is shown. `/agent` (bare) and this surface's
+   * Whether the agent switcher sheet is shown. `/agent` (bare) and this surface's
    * own control both open it.
    */
   agentSwitcherOpen: boolean;
@@ -331,7 +331,7 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   /** Runs the same path as `/agent <name>`. */
   selectAgent: (agentType: string) => void;
   /**
-   * #3282 §4 part b-3: the schedules shown in the Agents panel's "Scheduled" group — every
+   * The schedules shown in the Agents panel's "Scheduled" group — every
    * `IBackgroundTaskState` of `kind: 'scheduled'`, refreshed on connect and after a write.
    */
   scheduledTasks: readonly IBackgroundTaskState<'scheduled'>[];
@@ -340,7 +340,7 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   resumeSchedule: (taskId: string) => void;
   /**
    * Permanently stops a schedule — the same `cancel-background-task` write Stop already uses. A
-   * failure surfaces the same way a failed Stop does (#3288 §1's `background-task-control-failed`
+   * failure surfaces the same way a failed Stop does (the `background-task-control-failed`
    * session notice) — no separate error state here.
    */
   deleteSchedule: (taskId: string) => void;

package/src/hooks/ui-intent-state.ts

@@ -1,10 +1,10 @@
 /**
- * CMD-004 Stage D — how the GUI surface answers a `ui_intent`.
+ * How the GUI surface answers a `ui_intent`.
  *
  * The server requester-routes `ui_intent`, so everything that arrives here was asked for by THIS
  * surface and must be answered visibly. An intent with a GUI screen opens it (the session picker is
- * the session sidebar, #3189; settings and the plugin manager both open the Settings modal, #3282
- * §4a/§4 part b-2; the agent switcher is a sheet, #3282 §4 part b-3). #3282 §4e: an intent whose
+ * the session sidebar; settings and the plugin manager both open the Settings modal; the agent
+ * switcher is a sheet). An intent whose
  * command is on the GUI's exclusion list never reaches here at all (`excluded-commands.ts` catches it
  * before the session is asked) — this module only answers a screen this HOST cannot back even though
  * the GUI knows how to render it (the session picker, when the host cannot list sessions), or a kind
@@ -40,8 +40,8 @@ export function guiScreenForUiIntent(
 ): 'session-sidebar' | 'settings' | 'settings-plugins' | 'agent-switcher' | null {
   if (intent.type === 'show-session-picker') return 'session-sidebar';
   if (intent.type === 'show-settings') return 'settings';
-  // #3282 §4 part b-2: `/plugin` opens the Settings screen's Plugins section, the way #3331 mapped
-  // `show-settings` — replacing the "not available on this surface" line this intent used to get.
+  // `/plugin` opens the Settings screen's Plugins section, the way `show-settings`
+  // does.
   if (intent.type === 'show-plugin-manager') return 'settings-plugins';
   if (intent.type === 'show-agent-switcher') return 'agent-switcher';
   return null;
@@ -51,7 +51,7 @@ export function guiScreenForUiIntent(
  * Describe one intent for this surface: a plain, honest sentence for a screen this HOST cannot back
  * right now (a session picker with no session list), the `/theme` exclusion this intent defensively
  * reuses, or an unrecognized future kind. Never called for `show-settings`, `show-plugin-manager` or
- * `show-agent-switcher` — all three always have a screen now (#3282 §4a, §4 part b-2, §4 part b-3) —
+ * `show-agent-switcher` — all three always have a screen now —
  * but their cases are left out of this switch rather than kept as dead branches that would claim
  * otherwise; an unknown wire-level kind (a newer host) still yields an explicit notice naming the raw
  * kind, so the "never a silent no-op" floor holds for future intents too.

package/src/hooks/use-agent-switcher-state.ts

@@ -24,10 +24,10 @@ function nextRequestId(prefix: string): string {
 }
 
 /**
- * #3282 §4 part b-3: the agent switcher sheet's own correlated request/response state, the same
+ * The agent switcher sheet's own correlated request/response state, the same
  * shape `useSettingsState` already uses for `get-settings`/`update-settings` — independent from the
  * chat conversation, because choosing an agent is answered by the sheet's own confirmation line
- * ("Default agent: Explore"), never a conversation card (issue #3282 §4's decided design).
+ * ("Default agent: Explore"), never a conversation card.
  *
  * `openAgentSwitcher` re-fetches every time (`/agent` bare, or reopening after a close), so a change
  * made elsewhere reaches an open-then-reopened sheet, matching `useSettingsState`'s own reasoning.

package/src/hooks/use-media-query.ts

@@ -1,7 +1,7 @@
 import { useEffect, useState } from 'react';
 
 /**
- * Tracks whether `query` currently matches, reactively (#3289 §2 — the narrow-window session sheet
+ * Tracks whether `query` currently matches, reactively (the narrow-window session sheet
  * below `md`). A host without `window.matchMedia` (an older embedding, or a non-browser test) reports
  * `false` and never subscribes, rather than throwing.
  */

package/src/hooks/use-model-list.ts

@@ -13,7 +13,7 @@ function nextRequestId(): string {
 }
 
 /**
- * #3282 §2 (part 2) — the model list the model control's pop-up menu is built from, requested on
+ * The model list the model control's pop-up menu is built from, requested on
  * demand (`list-models`) and correlated by `requestId` (mirrors `usePersonalUsageState`'s pattern) so
  * a reply to an older request — one already superseded by a newer one, e.g. across a reconnect — is
  * never applied over the list a still-open menu is currently showing.

package/src/hooks/use-project-panel.ts

@@ -25,7 +25,7 @@ function nextRequestId(prefix: string): string {
 }
 
 /**
- * #3282 §4c — the Project panel's correlated request/response state (status, one file's diff, project
+ * The Project panel's correlated request/response state (status, one file's diff, project
  * memory), independent from chat state. Mirrors `usePersonalUsageState`'s shape: a status per read, a
  * `requestId` ref that drops a stale reply (a diff request superseded by clicking another file before
  * the first answered), and a request function per read.

package/src/hooks/use-schedules-state.ts

@@ -10,15 +10,15 @@ type TSchedulesState = Pick<
 >;
 
 /**
- * #3282 §4 part b-3: the Agents panel's "Scheduled" group. No `requestId` correlates
+ * The Agents panel's "Scheduled" group. No `requestId` correlates
  * `get-background-tasks`/`background_tasks` (the message carries none — see `wire-messages.ts`), so
  * this owns that whole message type for the GUI: every reply is a full refresh of the roster,
  * filtered server-side to `kind: 'scheduled'` (the same tasks `/schedule list` shows).
  *
  * Pause/resume send the SAME `command` text `/schedule pause <id>` / `/schedule resume <id>` runs.
  * Delete sends the existing `cancel-background-task` write — the same one Stop already uses for any
- * other background-task kind (#3288/#3332) — since a schedule IS a background task, and cancelling
- * one is permanent (distinct from the reversible pause), matching #3282 §4's "Delete…" requirement.
+ * other background-task kind — since a schedule IS a background task, and cancelling
+ * one is permanent (distinct from the reversible pause).
  */
 export function useSchedulesState(send: (msg: TClientMessage) => void): TSchedulesState & {
   /** Handles `background_tasks` and a `cancel` `background_task_control_result` — both refresh. */
@@ -46,7 +46,7 @@ export function useSchedulesState(send: (msg: TClientMessage) => void): TSchedul
       if (msg.type === 'background_task_control_result' && msg.action === 'cancel') {
         // A schedule's Delete — refresh so a successful cancel drops it from the group at once; a
         // refused one leaves the row exactly as it was (the central reducer's own case already
-        // turns a failure into a plain-message session notice — #3288 §1 — so this never duplicates it).
+        // turns a failure into a plain-message session notice — so this never duplicates it).
         if (msg.success) requestSchedules();
         return false; // not exclusively ours — other future consumers may still want this event
       }