ompchamber 3.1.0 → 3.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ompchamber",
-  "version": "3.1.0",
+  "version": "3.2.0",
   "description": "Developer web console and diagnostic chamber for AI Oh-My-Pi — Elysia + Preact on Bun.",
   "license": "UNLICENSED",
   "author": "Wijanarko Putra Rajeb <wijanarko.rajeb@gmail.com>",

package/src/cli/lib/args.js

@@ -188,7 +188,7 @@ USAGE:
 COMMANDS:
   serve          Start the OMPChamber web server (default when no command is given)
   update         Update OMPChamber to the latest GitHub release
-  stop           Stop running OMPChamber instances (every one, or --port <port>)
+  stop           Stop the instances the CLI started (--port/--all to stop any)
   restart        Stop and start again (every CLI-started instance, or --port <port>)
   status         Show running instances (every one, or --port <port>)
   logs           Print or follow server logs (every instance, or --port <port>)
@@ -201,7 +201,7 @@ OPTIONS:
   --prod                  Run the production build instead of the dev server
   --foreground            Run the server in the foreground (no daemon)
   --no-daemon             Alias for --foreground
-  --all                   Explicit form of the default for stop/restart (every instance)
+  --all                   Every instance, including ones started outside the CLI
   -c, --check             Report whether a newer release exists without installing it
   --force                 Reinstall even when already up to date
   --no-restart            Do not restart a running instance after updating
@@ -226,7 +226,8 @@ EXAMPLES:
   ompchamber serve --prod          # Serve the production build
   ompchamber update --check        # Is a newer release available?
   ompchamber update                # Update, then restart a running instance
-  ompchamber stop                  # Stop the running instance
+  ompchamber stop                  # Stop the instances the CLI started
+  ompchamber stop --port 3000      # Stop the instance on port 3000, whoever started it
   ompchamber restart               # Restart the server
   ompchamber status                # Show whether the server is running
   ompchamber logs -f -n 200        # Follow the last 200 log lines

package/src/cli/lib/commands/restart.js

@@ -1,19 +1,20 @@
 // `ompchamber restart` — stop live instances, then serve again.
 //
 // Scoping matches `status` and `stop`: without `--port` every live instance is
-// restarted on its own recorded port/host/mode, and `--port <port>` restricts
-// the restart to that one. With nothing running, the command is simply `serve`.
+// considered, and `--port <port>` restricts the restart to that one. With
+// nothing running, the command is simply `serve`.
 //
 // Instances the CLI did not start are left alone, exactly as `ompchamber update`
 // leaves them: `bun run dev` and `bun run start` own their own process, so
-// replacing it would break a loop OMPChamber does not manage. `ompchamber stop`
-// is still the deliberate way to end one.
+// replacing it would break a loop OMPChamber does not manage. `ompchamber stop
+// --port <port>` (or `--all`) is the deliberate way to end one.
 
 import { findLiveInstance, listLiveInstances, stopInstance } from '@/cli/lib/runtime.js';
 import { STOP_TIMEOUT_MS } from '@/cli/lib/process-lifecycle.js';
 import { log, isJson, isQuiet } from '@/cli/lib/output.js';
 import { run as runServe } from '@/cli/lib/commands/serve.js';
-import { isAutoRestartable, skipRestartNote } from '@/server/lib/lifecycle/restart';
+import { skipRestartNote } from '@/server/lib/lifecycle/restart';
+import { isCliManaged } from '@/server/lib/lifecycle/launch-mode';
 
 function explicitPort(options) {
   const raw = options?.port;
@@ -34,7 +35,7 @@ export async function run(options, ctx) {
   if (targets.length === 0) return runServe(options, ctx);
 
   for (const live of targets) {
-    if (!isAutoRestartable(live.launchMode)) {
+    if (!isCliManaged(live.launchMode)) {
       if (!json && !quiet) log(`Skipping OMPChamber on port ${live.port}: ${skipRestartNote(live.launchMode)}.`);
       continue;
     }

package/src/cli/lib/commands/stop.js

@@ -1,22 +1,19 @@
-// `ompchamber stop` — stop one or every live OMPChamber instance.
+// `ompchamber stop` — stop OMPChamber instances.
+//
+// Scope follows OWNERSHIP, the same rule `restart` and `update` apply. By
+// default only the instances the CLI started are stopped, because a server
+// running from source is supervised by a launcher the CLI does not own and
+// that launcher exits with the child it spawned: `bun run dev` is a process
+// whose whole job is to mirror the script under it, so signalling the server
+// takes the user's dev loop down with it. `--port <port>` and `--all` are the
+// explicit forms that do end one — naming an instance is a deliberate call,
+// not a default.
 
 import { listLiveInstances, findLiveInstance, stopInstance } from '@/cli/lib/runtime.js';
 import { STOP_TIMEOUT_MS } from '@/cli/lib/process-lifecycle.js';
-import { ok, warn, printJson, isJson } from '@/cli/lib/output.js';
-
-function resolvePort(options) {
-  const raw = options?.port;
-  if (raw !== null && raw !== undefined) {
-    const parsed = Number(raw);
-    if (Number.isFinite(parsed)) return parsed;
-  }
-  const env = Bun.env.OMPCHAMBER_PORT;
-  if (typeof env === 'string' && env.trim().length > 0) {
-    const parsed = Number(env.trim());
-    if (Number.isFinite(parsed)) return parsed;
-  }
-  return 3000;
-}
+import { ok, warn, log, printJson, isJson, isQuiet } from '@/cli/lib/output.js';
+import { unmanagedReason } from '@/server/lib/lifecycle/launch-mode';
+import { partitionStopTargets } from '@/cli/lib/stop-scope.js';
 
 function explicitPort(options) {
   const raw = options?.port;
@@ -25,30 +22,40 @@ function explicitPort(options) {
   return Number.isFinite(parsed) ? parsed : null;
 }
 
+/** How to end an instance the default scope leaves alone. */
+function stopHint(port) {
+  return `stop it with \`ompchamber stop --port ${port}\` or \`ompchamber stop --all\``;
+}
+
 export async function run(options) {
   const json = isJson();
+  const quiet = isQuiet();
   const requested = explicitPort(options);
+  // `--all` is the explicit spelling of "every instance, whoever started it".
+  const explicitScope = requested !== null || Boolean(options?.all);
+
+  const live = requested !== null
+    ? [await findLiveInstance(requested)].filter(Boolean)
+    : await listLiveInstances();
+  const { targets, skipped } = partitionStopTargets(live, explicitScope);
+  const skippedReport = skipped.map((entry) => ({ port: entry.port, pid: entry.pid, launchMode: entry.launchMode }));
 
-  // No `--port` means every live instance: a CLI server, a dev run and a
-  // production server are three processes, and `stop` without a port is the
-  // command that clears all of them. `--all` is the explicit spelling of the
-  // same default, kept because it reads well in scripts.
-  let targets;
-  if (requested !== null) {
-    const live = await findLiveInstance(requested);
-    targets = live ? [live] : [];
-  } else {
-    targets = await listLiveInstances();
+  if (!json && !quiet) {
+    for (const entry of skipped) {
+      log(`Skipping OMPChamber on port ${entry.port} (pid ${entry.pid}): it ${unmanagedReason(entry.launchMode)} — ${stopHint(entry.port)}.`);
+    }
   }
 
   if (targets.length === 0) {
     if (json) {
-      printJson({ stopped: [] });
+      printJson({ stopped: [], skipped: skippedReport });
       return;
     }
-    warn(requested === null
-      ? 'No running OMPChamber instances found.'
-      : `No running OMPChamber instance on port ${requested}.`);
+    if (skipped.length === 0) {
+      warn(requested === null
+        ? 'No running OMPChamber instances found.'
+        : `No running OMPChamber instance on port ${requested}.`);
+    }
     return;
   }
 
@@ -64,5 +71,5 @@ export async function run(options) {
     }
   }
 
-  if (json) printJson({ stopped });
+  if (json) printJson({ stopped, skipped: skippedReport });
 }

package/src/cli/lib/commands/update.js

@@ -10,7 +10,8 @@ import { color, configure, error, log, ok, warn, printJson, isJson, isQuiet } fr
 import { listLiveInstances, findLiveInstance, stopInstance } from '@/cli/lib/runtime.js';
 import { STOP_TIMEOUT_MS } from '@/cli/lib/process-lifecycle.js';
 import { run as runServe } from '@/cli/lib/commands/serve.js';
-import { isAutoRestartable, skipRestartNote } from '@/server/lib/lifecycle/restart';
+import { skipRestartNote } from '@/server/lib/lifecycle/restart';
+import { isCliManaged } from '@/server/lib/lifecycle/launch-mode';
 import { resolveInstallContext, resolveOmpChamberVersion, updateOmpChamber } from '@/server/lib/updates/install';
 
 /**
@@ -30,7 +31,7 @@ async function restartInstances(options, ctx) {
   const restarted = [];
   const skipped = [];
   for (const entry of await liveInstances(options)) {
-    if (!isAutoRestartable(entry.launchMode)) {
+    if (!isCliManaged(entry.launchMode)) {
       skipped.push({ port: entry.port, pid: entry.pid, launchMode: entry.launchMode });
       log(`Skipping OMPChamber on port ${entry.port}: ${skipRestartNote(entry.launchMode)}.`);
       continue;

package/src/cli/lib/output.js

@@ -95,6 +95,12 @@ export function fail(message, code = 1) {
 
 /**
  * Render a simple padded plain-text table for human output.
+ *
+ * Widths are measured with `Bun.stringWidth`, not `String.length`: `.length`
+ * counts UTF-16 code units, so a CJK session title or an emoji reported a width
+ * it does not render at and the whole table misaligned (verified: a 5-column
+ * overcount on a Japanese title). `stringWidth` also ignores ANSI escapes, so a
+ * coloured cell does not pad against its own escape bytes.
  */
 export function formatTable(rows, headers = []) {
   const body = Array.isArray(rows) ? rows.map((row) => (Array.isArray(row) ? row : [row])) : [];
@@ -105,21 +111,22 @@ export function formatTable(rows, headers = []) {
     return '';
   }
 
+  const cellText = (row, i) => (row[i] === undefined || row[i] === null ? '' : String(row[i]));
   const widths = [];
   for (let i = 0; i < columns; i += 1) {
-    const headerWidth = head[i] === undefined ? 0 : String(head[i]).length;
-    const cellWidth = body.reduce((max, row) => {
-      const cell = row[i] === undefined || row[i] === null ? '' : String(row[i]);
-      return Math.max(max, cell.length);
-    }, 0);
-    widths[i] = Math.max(headerWidth, cellWidth);
+    let width = head[i] === undefined ? 0 : Bun.stringWidth(String(head[i]));
+    for (const row of body) {
+      const cellWidth = Bun.stringWidth(cellText(row, i));
+      if (cellWidth > width) width = cellWidth;
+    }
+    widths[i] = width;
   }
 
   const renderRow = (row) =>
     widths
       .map((width, i) => {
-        const cell = row[i] === undefined || row[i] === null ? '' : String(row[i]);
-        return cell.padEnd(width);
+        const cell = cellText(row, i);
+        return cell + ' '.repeat(Math.max(0, width - Bun.stringWidth(cell)));
       })
       .join('  ')
       .trimEnd();

package/src/cli/lib/stop-scope.js

@@ -0,0 +1,23 @@
+// Ownership scope for `ompchamber stop`.
+//
+// Kept separate from the command so the rule is testable without a live
+// server: the default scope is the instances the CLI started, and only an
+// explicit `--port <port>` / `--all` widens it to the ones it did not. That
+// distinction is the whole reason this module exists — a server running from
+// source sits under a launcher that exits with its child, so stopping it by
+// default takes a `bun run dev` loop down with it.
+
+import { isCliManaged } from '@/server/lib/lifecycle/launch-mode';
+
+/**
+ * Split the live instances into the ones `stop` acts on and the ones it leaves
+ * alone. `explicit` is true when the user named a port or passed `--all`:
+ * naming an instance is a deliberate call, so ownership no longer gates it.
+ */
+export function partitionStopTargets(instances, explicit) {
+  if (explicit) return { targets: instances, skipped: [] };
+  return {
+    targets: instances.filter((entry) => isCliManaged(entry.launchMode)),
+    skipped: instances.filter((entry) => !isCliManaged(entry.launchMode)),
+  };
+}

package/src/cli/lib/stop-scope.test.js

@@ -0,0 +1,41 @@
+/**
+ * @license
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import { describe, expect, test } from 'bun:test';
+
+import { partitionStopTargets } from '@/cli/lib/stop-scope.js';
+
+const daemon = { port: 3000, pid: 11, launchMode: 'daemon' };
+const foreground = { port: 3001, pid: 12, launchMode: 'foreground' };
+const dev = { port: 3002, pid: 13, launchMode: 'direct' };
+const unknown = { port: 3003, pid: 14, launchMode: 'unknown' };
+
+describe('partitionStopTargets', () => {
+  // The default scope is ownership, not "everything that answers /api/health".
+  // `bun run dev` is a supervisor whose only job is to mirror the script under
+  // it, so stopping that server exits the user's dev loop too — the bug this
+  // rule exists to prevent.
+  test('stops only the instances the CLI started when the scope is implicit', () => {
+    const { targets, skipped } = partitionStopTargets([daemon, dev, foreground, unknown], false);
+    expect(targets).toEqual([daemon, foreground]);
+    expect(skipped).toEqual([dev, unknown]);
+  });
+
+  // Naming an instance — a port or `--all` — is a deliberate call, so the
+  // ownership rule no longer gates it.
+  test('widens to every instance when the scope is explicit', () => {
+    const { targets, skipped } = partitionStopTargets([daemon, dev, unknown], true);
+    expect(targets).toEqual([daemon, dev, unknown]);
+    expect(skipped).toEqual([]);
+  });
+
+  test('an unreadable launch mode is never stopped implicitly', () => {
+    // A record whose launchMode could not be read must not authorise a kill:
+    // the safe reading is "not ours".
+    const { targets, skipped } = partitionStopTargets([{ port: 4000, pid: 99, launchMode: undefined }], false);
+    expect(targets).toEqual([]);
+    expect(skipped.length).toBe(1);
+  });
+});

package/src/client/components/common/Modal.tsx

@@ -45,13 +45,19 @@ export function Modal({
   return (
     <div
       className={`fixed inset-0 bg-ink/40 backdrop-blur-[2px] ${zClass} flex items-center justify-center p-4`}
+      style={{
+        paddingTop: 'max(1rem, env(safe-area-inset-top, 0px))',
+        paddingBottom: 'max(1rem, env(safe-area-inset-bottom, 0px))',
+        paddingLeft: 'max(1rem, env(safe-area-inset-left, 0px))',
+        paddingRight: 'max(1rem, env(safe-area-inset-right, 0px))',
+      }}
       onClick={onClose}
     >
       <div
-        className={`bg-paper border border-ink/15 rounded-xl shadow-2xl w-full ${maxWidthClass} overflow-hidden text-ink`}
+        className={`bg-paper border border-ink/15 rounded-xl shadow-2xl w-full ${maxWidthClass} max-h-full flex flex-col overflow-hidden text-ink`}
         onClick={(e) => e.stopPropagation()}
       >
-        <div className="px-5 py-4 border-b border-ink/10 flex items-center justify-between">
+        <div className="px-5 py-4 border-b border-ink/10 flex items-center justify-between flex-shrink-0">
           {header}
           <button
             type="button"
@@ -63,11 +69,15 @@ export function Modal({
         </div>
 
         {form ? (
-          <form onSubmit={form.onSubmit} className={form.className}>
+          // The body scrolls, not the panel: a long form (the add-provider
+          // dialog is 832px of fields) is taller than a phone viewport, and
+          // without a bounded, scrollable body its footer sat below the screen
+          // with no way to reach it — the dialog could not be submitted.
+          <form onSubmit={form.onSubmit} className={`${form.className} min-h-0 overflow-y-auto scrollbar-overlay-container scrollbar-overlay-static`}>
             {body}
           </form>
         ) : (
-          body
+          <div className="min-h-0 overflow-y-auto scrollbar-overlay-container scrollbar-overlay-static">{body}</div>
         )}
       </div>
     </div>

package/src/client/components/{layout/session-sidebar/SubagentList.tsx → common/subagent-list/index.tsx}

@@ -4,7 +4,7 @@ import { SubagentStatusIcon } from '@/client/components/common/SubagentStatusIco
 import { isRecord } from '@/shared/lib/util/guards';
 import { fetchSubagentHistory, historyEntryToSubagentInfo } from '@/shared/lib/omp/subagent/history/client';
 import { subagentRowLabel } from '@/shared/lib/omp/subagent/label';
-import { mergeSubagentRoster, parseSubagentLifecycle, parseSubagentProgress, parseSubagentRosterResponse } from '@/shared/lib/omp/subagent/parse';
+import { mergeSubagentRoster, parseSubagentLifecycle, parseSubagentRosterResponse, readSubagentProgressFrame } from '@/shared/lib/omp/subagent/parse';
 import type { SubagentInfo, SubagentProgress } from '@/shared/types';
 
 type SubagentListProps = {
@@ -18,14 +18,6 @@ const PROGRESS_STATUS: Record<NonNullable<SubagentProgress['status']>, SubagentI
   pending: 'started', running: 'started', completed: 'completed', failed: 'failed', aborted: 'aborted',
 };
 
-/** Progress frames arrive AgentProgress-shaped; some wrappers nest the snapshot. */
-function readProgress(payload: unknown): SubagentProgress | undefined {
-  const direct = parseSubagentProgress(payload);
-  if (!isRecord(payload) || !isRecord(payload.progress)) return direct;
-  const nested = parseSubagentProgress(payload.progress);
-  return nested ? { ...nested, id: nested.id ?? direct?.id, index: nested.index ?? direct?.index } : direct;
-}
-
 /** Fold one progress frame into the roster entry it names (id, else index). */
 function applyProgress(roster: SubagentInfo[], progress: SubagentProgress): SubagentInfo[] {
   return roster.map((entry) => {
@@ -44,6 +36,8 @@ function applyProgress(roster: SubagentInfo[], progress: SubagentProgress): Suba
 /**
  * Subagent roster nested under a session row: renders clean, readable subagent
  * items matching the minimalist sidebar layout while preserving interactive inspection.
+ * Shared by the desktop sidebar's `CategoryItem` and the mobile drawer's
+ * `MobileSessionCategory`, so both list the same roster.
  */
 export function SubagentList({ sessionId, isActiveSession }: SubagentListProps) {
   const [searchParams] = useSearchParams();
@@ -85,13 +79,17 @@ export function SubagentList({ sessionId, isActiveSession }: SubagentListProps)
       }
     };
     const onProgress = (event: Event) => {
-      const progress = readProgress(detailOf(event)?.payload);
+      const progress = readSubagentProgressFrame(detailOf(event)?.payload);
       if (progress) setSubagents((prev) => applyProgress(prev, progress));
     };
 
     window.addEventListener('subagent_lifecycle', onLifecycle);
     window.addEventListener('subagent_progress', onProgress);
 
+    // Snapshot the LIVE registry. This POST is the spawn path, so the server
+    // refuses it (409) for a session it does not manage — an observer read must
+    // never boot an omp child. The history pass above already covers a finished
+    // session from disk, so nothing is missing when this comes back empty.
     fetch(`/api/agent/${encodeURIComponent(sid)}`, {
       method: 'POST',
       headers: { 'Content-Type': 'application/json' },

package/src/client/components/layout/session-sidebar/CategoryItem.tsx

@@ -6,9 +6,9 @@ import { useOnClickOutside } from '@/client/hooks/ui/on-click-outside';
 import { useShowMore } from '@/client/hooks/ui/show-more';
 import { useWorkspaceFolderActions } from '@/client/hooks/workspace/workspace-folder-actions';
 import { SessionItem } from '@/client/components/layout/session-sidebar/SessionItem';
-import { SubagentList } from '@/client/components/layout/session-sidebar/SubagentList';
+import { SubagentList } from '@/client/components/common/subagent-list';
 import { WorkspaceOptionsMenu } from '@/client/components/common/workspace-options-menu';
-import { loadExpandedSessionIds, saveExpandedSessionIds } from '@/shared/lib/workspace/sidebar-expanded';
+import { useExpandedSessions } from '@/client/hooks/workspace/expanded-sessions';
 import { getProjectIcon } from '@/shared/lib/workspace/project-icon';
 import { useSidebarData } from '@/client/hooks/chat/omp/session-list';
 
@@ -37,32 +37,14 @@ export function Category({
   const [searchParams] = useSearchParams();
   const urlSubagentId = searchParams.get('subagent');
   const urlSessionId = searchParams.get('sessionId');
-  
-  // Sidebar-level expanded session set
-  const [expandedSessionIds, setExpandedSessionIds] = useState<Set<string>>(() => new Set());
 
-  useEffect(() => {
-    const saved = loadExpandedSessionIds();
-    if (saved && saved.size > 0) {
-      setExpandedSessionIds(saved);
-    }
-  }, []);
+  // Roster expansion is shared with the mobile drawer: one persisted set, so a
+  // row expanded on the phone is expanded here too.
+  const { expandedSessionIds, toggleSession } = useExpandedSessions();
 
   useEffect(() => {
     if (typeof folder.isExpanded === 'boolean') setIsOpen(folder.isExpanded);
   }, [folder.isExpanded]);
-
-  // A deep-linked transcript (?sessionId=…&subagent=…) auto-expands its
-  // session row so the viewed roster entry is visible after a reload.
-  useEffect(() => {
-    if (!urlSubagentId || !urlSessionId) return;
-    setExpandedSessionIds(prev => {
-      if (prev.has(urlSessionId)) return prev;
-      const next = new Set(prev);
-      next.add(urlSessionId);
-      return next;
-    });
-  }, [urlSubagentId, urlSessionId]);
   
   const menuRef = useRef<HTMLDivElement>(null);
   const { refresh } = useSidebarData();
@@ -106,13 +88,7 @@ export function Category({
   };
 
   const handleToggleSessionExpand = (sessionId: string) => {
-    setExpandedSessionIds((prev) => {
-      const next = new Set(prev);
-      if (next.has(sessionId)) next.delete(sessionId);
-      else next.add(sessionId);
-      saveExpandedSessionIds(next);
-      return next;
-    });
+    toggleSession(sessionId);
   };
 
   const onPin = () => {

package/src/client/components/layout/session-sidebar/Toolbar.tsx

@@ -18,6 +18,8 @@ interface SessionSidebarToolbarProps {
   onNewWorkspace: () => void;
   onScheduler: () => void;
   onRefresh: () => void;
+  /** True while the user-initiated refresh is in flight — spins the icon. */
+  refreshing: boolean;
   onClose?: () => void;
 }
 
@@ -37,6 +39,7 @@ export function SessionSidebarToolbar({
   onNewWorkspace,
   onScheduler,
   onRefresh,
+  refreshing,
   onClose,
 }: SessionSidebarToolbarProps) {
   return (
@@ -45,16 +48,19 @@ export function SessionSidebarToolbar({
         <div className="flex space-x-3 items-center">
           <FolderPlus size={14} className="hover:text-ink cursor-pointer"  onClick={onNewWorkspace} />
           <Calendar size={14} className="hover:text-ink cursor-pointer" onClick={onScheduler} />
-          <span 
-            title="Refresh sessions" 
+          <button
+            type="button"
+            title="Refresh sessions"
             onClick={onRefresh}
-            className="inline-flex cursor-pointer"
+            className="inline-flex cursor-pointer bg-transparent border-0 p-0 text-ink/60"
+            aria-label="Refresh sessions"
+            aria-busy={refreshing}
           >
             <RefreshCw 
               size={14} 
-              className="hover:text-ink" 
+              className={`hover:text-ink ${refreshing ? 'animate-spin' : ''}`} 
             />
-          </span>
+          </button>
         </div>
         <div className="flex space-x-3 items-center relative">
           <Search 

package/src/client/components/layout/session-sidebar/index.tsx

@@ -17,7 +17,8 @@ export function SessionSidebar({ className = '', onClose, appSettings = {} }: {
   const {
     folders,
     initializing,
-    refresh,
+    refreshNow,
+    refreshing,
     activeSessionId,
     sessionParam,
     searchQuery,
@@ -105,7 +106,8 @@ export function SessionSidebar({ className = '', onClose, appSettings = {} }: {
           onSortChange={handleSortChange}
           onNewWorkspace={() => setNewWorkspaceOpen(true)}
           onScheduler={() => setSchedulerOpen(true)}
-          onRefresh={refresh}
+          onRefresh={refreshNow}
+          refreshing={refreshing}
           onClose={onClose}
         />
 

package/src/client/components/mobile/LayoutWrapper.tsx

@@ -74,6 +74,12 @@ export function MobileLayoutWrapper({ onDesktopToggle, appSettings = {} }: Mobil
     });
   });
 
+  // A roster row opens a subagent transcript in the chat timeline (the
+  // `omp:view-subagent` listener there owns the URL). On the phone the chat is
+  // a separate screen, so the tap must also bring it forward — otherwise the
+  // transcript opens behind the drawer and the tap reads as a no-op.
+  useChamberEvent('omp:view-subagent', () => setCurrentScreen('main'));
+
   // Settings requests (e.g. the model dropdown asking for Settings → Providers
   // when no provider is configured) arrive on the same global channel the
   // desktop layout listens on. SettingsModal itself only reads the payload and

package/src/client/components/mobile/SessionSidebar.tsx

@@ -36,6 +36,8 @@ export function MobileSessionSidebar({
   const {
     folders,
     initializing,
+    refreshNow,
+    refreshing,
     searchQuery,
     setSearchQuery,
     showArchived,
@@ -124,6 +126,8 @@ export function MobileSessionSidebar({
         onNewSession={handleNewSessionAndClose}
         onNewWorkspace={() => setNewWorkspaceOpen(true)}
         onScheduler={() => setSchedulerOpen(true)}
+        onRefresh={refreshNow}
+        refreshing={refreshing}
         onToggleOptions={() => setOptionsOpen(!optionsOpen)}
         onSortChange={handleSortChange}
         onResetSort={() => handleSortChange('A-Z')}