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

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

@@ -16,7 +16,11 @@ import type {
   IExecutionDetailRecord,
   IExecutionWorkspaceSnapshot,
 } from '@robota-sdk/agent-interface-execution';
-import type { IWireAgentDefinitionSummary, TServerMessage } from '@robota-sdk/agent-transport';
+import type {
+  IWireAgentDefinitionSummary,
+  TServerMessage,
+  TSessionSurfaceRole,
+} from '@robota-sdk/agent-transport';
 
 export type { TAppToolArguments, TAppToolOutcome, TAppView } from '../app-view/types.js';
 
@@ -34,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<
@@ -55,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'];
@@ -66,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;
 }
 
@@ -97,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';
@@ -113,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`.
  */
@@ -148,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.
    */
@@ -161,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.
    */
@@ -191,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).
    */
@@ -204,12 +208,14 @@ export interface IWsSessionState<TStatus extends string = TConnectionStatus> {
   sessionName: string | null;
   /** This connection's own driver id, learned from the server's first frame; null until then. */
   ownDriverId: TDriverId | null;
+  /** The role the host gave this connection, learned when it connects; null until the host answers. */
+  role: TSessionSurfaceRole | null;
   /** Null until the session has answered; refreshed after every command. */
   commandCatalog: TCommandCatalog | null;
   /** 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;
@@ -233,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;
@@ -257,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[];
@@ -294,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;
@@ -325,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'>[];
@@ -334,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-app-view-requests.ts

@@ -20,6 +20,7 @@ function nextRequestId(prefix: string): string {
 }
 
 const CONNECTION_CLOSED = 'The connection to the session closed.';
+const READ_ONLY = 'This session is read-only.';
 
 /**
  * The key a finished call's app view is read and called by, taken from the tool row's `appView`
@@ -53,6 +54,7 @@ function callAppToolMessage(
 export function useAppViewRequests(
   send: (msg: TClientMessage) => void,
   isConnected: () => boolean,
+  isObserving: () => boolean,
 ): {
   readAppView: (viewKey: string, signal?: AbortSignal) => Promise<TAppView>;
   callAppTool: (
@@ -92,6 +94,11 @@ export function useAppViewRequests(
           reject(new Error(signal?.aborted ? 'The app view closed.' : CONNECTION_CLOSED));
           return;
         }
+        // An observer's request is never sent, so no reply would come.
+        if (isObserving()) {
+          reject(new Error(READ_ONLY));
+          return;
+        }
         const requestId = nextRequestId(`${requestOwner}_${prefix}`);
         const dispose = (): void => signal?.removeEventListener('abort', abort);
         const abort = (): void => {
@@ -109,7 +116,7 @@ export function useAppViewRequests(
           reject(error instanceof Error ? error : new Error(String(error)));
         }
       }),
-    [isConnected, requestOwner, send],
+    [isConnected, isObserving, requestOwner, send],
   );
 
   const readAppView = useCallback(

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
       }

package/src/hooks/use-session-directory.ts

@@ -30,7 +30,7 @@ function keepsSessionsLive(listing: TSessionListing): boolean {
 }
 
 /**
- * Issue #3280 §5: a desktop Reconnect reloads the page, which loses every bit of JS state — so the
+ * A desktop Reconnect reloads the page, which loses every bit of JS state — so the
  * session to go back to cannot live in a ref like `lastSwitchedIdRef` below. `sessionStorage` survives
  * the reload (and only that tab), so it carries the id across it. Namespaced so a page hosting other
  * state under the same origin does not collide.
@@ -74,14 +74,17 @@ function initialSidebarOpen(): boolean {
 }
 
 /**
- * #3189 — the host's session list beside the conversation: requests correlated by id (the latest
+ * The host's session list beside the conversation: requests correlated by id (the latest
  * wins), the start/switch commands, and whether the sidebar is shown.
  *
  * A host that keeps sessions live binds each connection to its own session and puts a new connection
  * on its primary one. After a reconnect to such a host this surface goes back to the session it was
  * on; an older host has one session for everyone, so there is nothing to go back to.
  */
-export function useSessionDirectoryState(send: (msg: TClientMessage) => void): TSessionDirectoryState & {
+export function useSessionDirectoryState(
+  send: (msg: TClientMessage) => void,
+  isObserving: () => boolean,
+): TSessionDirectoryState & {
   handleSessionsMessage: (msg: TServerMessage) => boolean;
   /** Whether the host can list sessions, as far as its last answer says. */
   canListSessions: () => boolean;
@@ -117,7 +120,7 @@ export function useSessionDirectoryState(send: (msg: TClientMessage) => void): T
     (): void => send({ type: 'new-session', requestId: nextRequestId('session_change') }),
     [send, browserNamespace],
   );
-  // #3289 §1: rename or delete a row in the list — current or not. Both answer on their own
+  // Rename or delete a row in the list — current or not. Both answer on their own
   // correlated reply (`session_renamed_in_list`/`session_deleted`, or a `_failed` counterpart); the
   // reducer in `useSessionClient` refreshes the listing on success and raises a notice on failure.
   const renameSessionInList = useCallback(
@@ -135,7 +138,7 @@ export function useSessionDirectoryState(send: (msg: TClientMessage) => void): T
   const lastSwitchedIdRef = useRef<string | null>(null);
   const restoreTargetRef = useRef<string | null>(null);
   const armRestore = useCallback((): void => {
-    // A remembered id (issue #3280 §5, a desktop Reconnect's reload) wins over the same-page target:
+    // A remembered id (a desktop Reconnect's reload) wins over the same-page target:
     // on a fresh page, `lastSwitchedIdRef` is always null anyway, so this only ever adds a source.
     restoreTargetRef.current = readStoredRestoreId(browserNamespace) ?? lastSwitchedIdRef.current;
   }, [browserNamespace]);
@@ -148,12 +151,14 @@ export function useSessionDirectoryState(send: (msg: TClientMessage) => void): T
       // launch with no remembered id never reaches here (target stays null), so nothing switches.
       clearStoredRestoreId(browserNamespace);
       if (target === listing.currentSessionId) return;
+      // An observer never moves the session, so there is nothing to go back to.
+      if (isObserving()) return;
       if (!keepsSessionsLive(listing)) return;
       // A session the host no longer lists (an empty one it never saved) is nothing to go back to.
       if (!listing.sessions.some((row) => row.id === target)) return;
       send({ type: 'switch-session', sessionId: target, requestId: nextRequestId('session_change') });
     },
-    [send, browserNamespace],
+    [send, browserNamespace, isObserving],
   );
 
   const handleSessionsMessage = useCallback(

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

@@ -24,12 +24,12 @@ function nextRequestId(prefix: string): string {
 }
 
 /**
- * #3282 §4a: the Settings screen's own correlated request/response state, independent from chat
+ * The Settings screen's own correlated request/response state, independent from chat
  * state — the same shape `usePersonalUsageState` already uses for `get-personal-usage-report`.
  *
  * `openSettings` re-fetches every time (opening from the gear, `/settings`, or reopening after a
  * close): the simplest correct way for a change made elsewhere — another typed command, another
- * client — to reach an open-then-reopened screen, per #3282 §4's "pick the simplest correct option."
+ * client — to reach an open-then-reopened screen.
  *
  * A failed UPDATE never touches `settingsSnapshot`, so a control bound to it shows the value it had
  * before the write — "a failed write leaves the control at its previous value" — while
@@ -39,7 +39,7 @@ function nextRequestId(prefix: string): string {
 export function useSettingsState(send: (msg: TClientMessage) => void): TSettingsState & {
   handleSettingsMessage: (msg: TServerMessage) => boolean;
   /**
-   * #3282 §4b: re-fetch without touching `settingsOpen`/`settingsInitialSectionId` — for a change
+   * Re-fetch without touching `settingsOpen`/`settingsInitialSectionId` — for a change
    * made elsewhere while the screen is already open (e.g. "Add provider…"'s ask flow finishing).
    * Unlike `openSettings`, this never resets which section is showing.
    */
@@ -49,7 +49,7 @@ export function useSettingsState(send: (msg: TClientMessage) => void): TSettings
   const [settingsStatus, setSettingsStatus] = useState<IWsSessionState['settingsStatus']>('idle');
   const [settingsSnapshot, setSettingsSnapshot] = useState<ISettingsSnapshot | null>(null);
   const [settingsError, setSettingsError] = useState<string | null>(null);
-  // #3282 §4 part b-2 / §4b: which section to land on THIS open — `/plugin` opens straight to
+  // Which section to land on THIS open — `/plugin` opens straight to
   // Plugins, "Manage providers…" straight to Providers & Models; everything else (the gear,
   // `/settings`) opens on the screen's own default (General).
   const [settingsInitialSectionId, setSettingsInitialSectionId] = useState<string | null>(null);