@openchambery/web 1.18.2-beta.9 → 1.18.3-beta.2

package/server/lib/notifications/runtime.test.js

@@ -0,0 +1,176 @@
+import { afterEach, describe, expect, it, vi } from 'vitest';
+
+import { createNotificationTriggerRuntime } from './runtime.js';
+
+const defaultSettings = {
+  notifyOnCompletion: true,
+  notifyOnQuestion: true,
+  notificationMode: 'always',
+  nativeNotificationsEnabled: true,
+};
+
+const createRuntime = (overrides = {}) => {
+  const emitDesktopNotification = vi.fn(() => true);
+  const broadcastUiNotification = vi.fn();
+  const sendPushToAllUiSessions = vi.fn(async () => {});
+  const sendApnsToAllUiSessions = vi.fn(async () => {});
+  const runtime = createNotificationTriggerRuntime({
+    readSettingsFromDisk: vi.fn(async () => defaultSettings),
+    prepareNotificationLastMessage: vi.fn(async ({ message }) => message || ''),
+    buildTemplateVariables: vi.fn(async () => ({ session_name: 'Session' })),
+    extractLastMessageText: vi.fn(() => ''),
+    fetchLastAssistantMessageText: vi.fn(async () => ''),
+    resolveNotificationTemplate: vi.fn((template) => template),
+    shouldApplyResolvedTemplateMessage: vi.fn(() => true),
+    emitDesktopNotification,
+    broadcastUiNotification,
+    sendPushToAllUiSessions,
+    sendApnsToAllUiSessions,
+    isAnyInteractiveClientVisible: () => false,
+    buildOpenCodeUrl: (path) => `http://opencode.test${path}`,
+    getOpenCodeAuthHeaders: () => ({}),
+    getIsWindowFocused: () => false,
+    ...overrides,
+  });
+  return {
+    runtime,
+    emitDesktopNotification,
+    broadcastUiNotification,
+    sendPushToAllUiSessions,
+    sendApnsToAllUiSessions,
+  };
+};
+
+const jsonResponse = (body, status = 200) => new Response(JSON.stringify(body), {
+  status,
+  headers: { 'Content-Type': 'application/json' },
+});
+
+afterEach(() => {
+  vi.unstubAllGlobals();
+  vi.useRealTimers();
+});
+
+describe('notification trigger runtime smallModel suppression', () => {
+  it('skips ready notifications for smallModel sessions', async () => {
+    vi.stubGlobal('fetch', vi.fn(async () => jsonResponse({
+      id: 'ses_small',
+      parentID: null,
+      metadata: { openchamber: { smallModel: { purpose: 'session-title' } } },
+    })));
+
+    const { runtime, emitDesktopNotification, sendPushToAllUiSessions } = createRuntime();
+    await runtime.maybeSendPushForTrigger({
+      type: 'message.updated',
+      properties: {
+        directory: '/repo',
+        info: {
+          id: 'msg_1',
+          sessionID: 'ses_small',
+          role: 'assistant',
+          finish: 'stop',
+          mode: 'build',
+          modelID: 'small',
+        },
+      },
+    });
+
+    expect(emitDesktopNotification).not.toHaveBeenCalled();
+    expect(sendPushToAllUiSessions).not.toHaveBeenCalled();
+  });
+
+  it('skips ready notifications synthesized from session.idle for smallModel sessions', async () => {
+    vi.stubGlobal('fetch', vi.fn(async () => jsonResponse({
+      id: 'ses_small',
+      parentID: null,
+      metadata: { openchamber: { smallModel: { purpose: 'commit' } } },
+    })));
+
+    const { runtime, emitDesktopNotification, sendPushToAllUiSessions } = createRuntime();
+    await runtime.maybeSendPushForTrigger({
+      type: 'session.idle',
+      properties: {
+        directory: '/repo',
+        sessionID: 'ses_small',
+      },
+    });
+
+    expect(emitDesktopNotification).not.toHaveBeenCalled();
+    expect(sendPushToAllUiSessions).not.toHaveBeenCalled();
+  });
+
+  it('skips question notifications for smallModel sessions', async () => {
+    vi.useFakeTimers();
+    vi.stubGlobal('fetch', vi.fn(async () => jsonResponse({
+      id: 'ses_small',
+      parentID: null,
+      metadata: { openchamber: { smallModel: { purpose: 'session-title' } } },
+    })));
+
+    const { runtime, emitDesktopNotification, sendPushToAllUiSessions } = createRuntime();
+    await runtime.maybeSendPushForTrigger({
+      type: 'question.asked',
+      properties: {
+        directory: '/repo',
+        sessionID: 'ses_small',
+        questions: [{ header: 'Input needed', question: 'Continue?' }],
+      },
+    });
+    await vi.runAllTimersAsync();
+
+    expect(emitDesktopNotification).not.toHaveBeenCalled();
+    expect(sendPushToAllUiSessions).not.toHaveBeenCalled();
+  });
+
+  it('skips permission notifications for smallModel sessions', async () => {
+    vi.useFakeTimers();
+    vi.stubGlobal('fetch', vi.fn(async () => jsonResponse({
+      id: 'ses_small',
+      parentID: null,
+      metadata: { openchamber: { smallModel: { purpose: 'session-title' } } },
+    })));
+
+    const { runtime, emitDesktopNotification, sendPushToAllUiSessions } = createRuntime();
+    await runtime.maybeSendPushForTrigger({
+      type: 'permission.asked',
+      properties: {
+        directory: '/repo',
+        sessionID: 'ses_small',
+        id: 'perm_1',
+        permission: 'edit',
+      },
+    });
+    await vi.runAllTimersAsync();
+
+    expect(emitDesktopNotification).not.toHaveBeenCalled();
+    expect(sendPushToAllUiSessions).not.toHaveBeenCalled();
+  });
+
+  it('still notifies ordinary root sessions', async () => {
+    vi.stubGlobal('fetch', vi.fn(async () => jsonResponse({
+      id: 'ses_root',
+      parentID: null,
+      title: 'Ordinary',
+      metadata: {},
+    })));
+
+    const { runtime, emitDesktopNotification, sendPushToAllUiSessions } = createRuntime();
+    await runtime.maybeSendPushForTrigger({
+      type: 'message.updated',
+      properties: {
+        directory: '/repo',
+        info: {
+          id: 'msg_2',
+          sessionID: 'ses_root',
+          role: 'assistant',
+          finish: 'stop',
+          mode: 'build',
+          modelID: 'gpt',
+        },
+      },
+    });
+
+    expect(emitDesktopNotification).toHaveBeenCalledTimes(1);
+    expect(sendPushToAllUiSessions).toHaveBeenCalledTimes(1);
+  });
+});

package/server/lib/opencode/DOCUMENTATION.md

@@ -28,7 +28,7 @@ This module provides OpenCode server integration utilities for the web server ru
 - `packages/web/server/lib/opencode/static-routes-runtime.js`: static asset/SPA fallback route registration and manifest route wiring.
 - `packages/web/server/lib/opencode/feature-routes-runtime.js`: feature route composition runtime for dynamic import-backed config/skill/provider route registration.
 - `packages/web/server/lib/conversations/`: combined create-session-and-prompt orchestration (OpenChamber-owned, registered before generic proxy).
-- `packages/web/server/lib/session-turn-pages/`: turn-window session message pagination (`GET /api/openchamber/sessions/:sessionID/messages`), anchor reconcile (`GET .../messages/reconcile`), Changes L2/L3 (`GET .../changes`), and exact message L1 projection (`GET /api/session/:sessionID/message/:messageID`); loops official OpenCode `session.messages` for turn pages and head→anchor gap recovery with continuation/budgets/`resetRequired`; all registered before generic proxy. L1 first-packet / SSE / exact-message paths project `summary.diffs` to `diffCount`/`hasDiffs` only. See `session-turn-pages/DOCUMENTATION.md`.
+- `packages/web/server/lib/session-turn-pages/`: turn-window session message pagination (`GET /api/openchamber/sessions/:sessionID/messages`), anchor reconcile (`GET .../messages/reconcile`), Changes L2/L3 (`GET .../changes`), and exact message L1 projection (`GET /api/session/:sessionID/message/:messageID`); loops official OpenCode `session.messages` for turn pages and head→anchor gap recovery with continuation/budgets/`resetRequired`; all registered before generic proxy. L1 first-packet / SSE / exact-message paths keep `summary.diffs` as a slim `{ file, status?, additions, deletions }` list plus additive `diffCount`/`hasDiffs` (patch bodies stay on L3). See `session-turn-pages/DOCUMENTATION.md`.
 - `packages/web/server/lib/opencode/opencode-resolution-runtime.js`: OpenCode binary resolution snapshot runtime for settings routes and diagnostics.
 - `packages/web/server/lib/opencode/tunnel-wiring-runtime.js`: active loopback-port wiring shared by pairing/LAN URLs and the private relay host.
 - `packages/web/server/lib/opencode/startup-pipeline-runtime.js`: server startup tail orchestration runtime for terminal/proxy/static/start-listen flow.
@@ -214,7 +214,7 @@ This module provides OpenCode server integration utilities for the web server ru
   - `isUnsafeSkillRelativePath(value)`
   - `sanitizeTypographySizesPartial(input)`
   - `normalizeStringArray(input)`
-  - `sanitizeModelRefs(input, limit)`
+  - `sanitizeModelRefs(input, limit, options?)` — optional `{ preserveVariant: true }` keeps a non-empty trimmed `variant` on favorite/recent model refs; hidden models omit it
   - `sanitizeSkillCatalogs(input)`
   - `sanitizeProjects(input)`
 

package/server/lib/opencode/settings-helpers.js

@@ -616,12 +616,12 @@ export const createSettingsHelpers = (dependencies) => {
       result.shortcutOverrides = shortcutOverrides;
     }
 
-    const favoriteModels = sanitizeModelRefs(candidate.favoriteModels, 64);
+    const favoriteModels = sanitizeModelRefs(candidate.favoriteModels, 64, { preserveVariant: true });
     if (favoriteModels) {
       result.favoriteModels = favoriteModels;
     }
 
-    const recentModels = sanitizeModelRefs(candidate.recentModels, 16);
+    const recentModels = sanitizeModelRefs(candidate.recentModels, 16, { preserveVariant: true });
     if (recentModels) {
       result.recentModels = recentModels;
     }

package/server/lib/opencode/settings-helpers.test.js

@@ -439,5 +439,47 @@ describe('settings helpers', () => {
       expect(sanitized.favoriteModels).toEqual(payload.favoriteModels);
       expect(sanitized.recentModels).toEqual(payload.recentModels);
     });
+
+    it('preserves trimmed thinking variants on favoriteModels and recentModels', () => {
+      const helpers = createTestHelpersWithRealSanitizers();
+
+      expect(
+        helpers.sanitizeSettingsUpdate({
+          favoriteModels: [{ providerID: 'anthropic', modelID: 'claude-haiku-4', variant: '  high  ' }],
+          recentModels: [
+            { providerID: 'openai', modelID: 'gpt-5', variant: 'medium' },
+            { providerID: 'google', modelID: 'gemini-pro' },
+          ],
+        }),
+      ).toEqual({
+        favoriteModels: [{ providerID: 'anthropic', modelID: 'claude-haiku-4', variant: 'high' }],
+        recentModels: [
+          { providerID: 'openai', modelID: 'gpt-5', variant: 'medium' },
+          { providerID: 'google', modelID: 'gemini-pro' },
+        ],
+      });
+    });
+
+    it('strips invalid favorite/recent variants and never preserves variant on hiddenModels', () => {
+      const helpers = createTestHelpersWithRealSanitizers();
+
+      expect(
+        helpers.sanitizeSettingsUpdate({
+          favoriteModels: [
+            { providerID: 'anthropic', modelID: 'claude-haiku-4', variant: '   ' },
+            { providerID: 'openai', modelID: 'gpt-5', variant: 12 },
+          ],
+          recentModels: [{ providerID: 'google', modelID: 'gemini-pro', variant: null }],
+          hiddenModels: [{ providerID: 'anthropic', modelID: 'claude-opus-4', variant: 'high' }],
+        }),
+      ).toEqual({
+        favoriteModels: [
+          { providerID: 'anthropic', modelID: 'claude-haiku-4' },
+          { providerID: 'openai', modelID: 'gpt-5' },
+        ],
+        recentModels: [{ providerID: 'google', modelID: 'gemini-pro' }],
+        hiddenModels: [{ providerID: 'anthropic', modelID: 'claude-opus-4' }],
+      });
+    });
   });
 });

package/server/lib/opencode/settings-normalization-runtime.js

@@ -309,13 +309,14 @@ export const createSettingsNormalizationRuntime = (dependencies) => {
     return populated ? result : undefined;
   };
 
-  const sanitizeModelRefs = (input, limit) => {
+  const sanitizeModelRefs = (input, limit, options) => {
     if (!Array.isArray(input)) {
       return undefined;
     }
 
     const result = [];
     const seen = new Set();
+    const preserveVariant = options?.preserveVariant === true;
 
     for (const entry of input) {
       if (!entry || typeof entry !== 'object') continue;
@@ -325,7 +326,11 @@ export const createSettingsNormalizationRuntime = (dependencies) => {
       const key = `${providerID}/${modelID}`;
       if (seen.has(key)) continue;
       seen.add(key);
-      result.push({ providerID, modelID });
+      if (preserveVariant && typeof entry.variant === 'string' && entry.variant.trim().length > 0) {
+        result.push({ providerID, modelID, variant: entry.variant.trim() });
+      } else {
+        result.push({ providerID, modelID });
+      }
       if (result.length >= limit) break;
     }
 

package/server/lib/session-index/DOCUMENTATION.md

@@ -13,14 +13,20 @@ newest 20 root-session summaries per runtime and directory. It never stores
 messages, attachments, permissions, provider data, or model metadata.
 Sessions titled `smartfetch-secondary` are temporary SmartFetch model calls;
 the index excludes them from every snapshot and clears prior summaries when a
-matching session update arrives. System-owned sessions are excluded by
-authoritative metadata only: a non-empty
-`metadata.openchamber.assistant.assistantID` or
-`metadata.openchamber.scheduledTask.taskID`. Title prefixes are human labels and
-never participate in this filter. Metadata is the ownership/isolation signal;
-`time.archived` is archive state; titles are for recognition. The index does
+  matching session update arrives. System-owned sessions are excluded by
+  authoritative metadata only: a non-empty
+  `metadata.openchamber.assistant.assistantID`,
+  `metadata.openchamber.scheduledTask.taskID`, or
+  `metadata.openchamber.smallModel.purpose`. Title prefixes are human labels and
+  never participate in this filter. Metadata is the ownership/isolation signal;
+`time.archived` is archive state; `time.pinned` is OpenChamber-owned pin state
+(persisted as `pinned_at`, rescued across directory rebuilds, never overwritten
+by live OpenCode upserts); titles are for recognition. The index does
 not cache system sessions, so cold-start and live upserts stay aligned with
 ordinary sidebar lists.
+Pin and unpin use `POST` / `DELETE`
+`/api/openchamber/session-index/session/:id/pin`, update `pinned_at`, then
+`publishChange()` so revision tips broadcast `openchamber:session-index-changed`.
 
 The server-side global OpenCode event subscriber writes session summary events
 directly into this index. User `message.updated` events and `session.idle`

package/server/lib/session-index/routes.js

@@ -62,4 +62,38 @@ export const registerSessionIndexRoutes = (app, { sessionIndexService, sessionIn
     }
     res.json({ available: true, session: hit });
   });
+
+  app.post('/api/openchamber/session-index/session/:id/pin', (req, res) => {
+    if (!sessionIndexService) return unsupported(res);
+    if (typeof sessionIndexService.setPinned !== 'function') {
+      return res.status(501).json({ error: 'Session pin is unavailable' });
+    }
+    const sessionId = req.params.id;
+    if (typeof sessionId !== 'string' || !sessionId.trim()) {
+      return res.status(400).json({ error: 'sessionId is required' });
+    }
+    const changed = sessionIndexService.setPinned(sessionId.trim());
+    if (!changed) {
+      return res.status(404).json({ error: 'session_not_found', sessionId: sessionId.trim() });
+    }
+    sessionIndexSyncRuntime?.publishChange?.();
+    res.status(204).end();
+  });
+
+  app.delete('/api/openchamber/session-index/session/:id/pin', (req, res) => {
+    if (!sessionIndexService) return unsupported(res);
+    if (typeof sessionIndexService.clearPinned !== 'function') {
+      return res.status(501).json({ error: 'Session unpin is unavailable' });
+    }
+    const sessionId = req.params.id;
+    if (typeof sessionId !== 'string' || !sessionId.trim()) {
+      return res.status(400).json({ error: 'sessionId is required' });
+    }
+    const changed = sessionIndexService.clearPinned(sessionId.trim());
+    if (!changed) {
+      return res.status(404).json({ error: 'session_not_found', sessionId: sessionId.trim() });
+    }
+    sessionIndexSyncRuntime?.publishChange?.();
+    res.status(204).end();
+  });
 };

package/server/lib/session-index/routes.test.js

@@ -137,4 +137,80 @@ describe('session index routes', () => {
     expect(res.statusCode).toBe(404);
     expect(res.body).toMatchObject({ error: 'session_not_found' });
   });
+
+  it('pins a session and publishes a session-index change', () => {
+    const { app, route } = registry();
+    const setPinned = vi.fn(() => true);
+    const publishChange = vi.fn();
+    registerSessionIndexRoutes(app, {
+      sessionIndexService: {
+        snapshot: () => ({ directories: [] }),
+        setPinned,
+      },
+      sessionIndexSyncRuntime: { publishChange },
+    });
+    const res = response();
+
+    route('POST', '/api/openchamber/session-index/session/:id/pin')(
+      { params: { id: 'ses_abc' } },
+      res,
+    );
+
+    expect(setPinned).toHaveBeenCalledWith('ses_abc');
+    expect(publishChange).toHaveBeenCalledTimes(1);
+    expect(res.statusCode).toBe(204);
+  });
+
+  it('unpins a session and publishes a session-index change', () => {
+    const { app, route } = registry();
+    const clearPinned = vi.fn(() => true);
+    const publishChange = vi.fn();
+    registerSessionIndexRoutes(app, {
+      sessionIndexService: {
+        snapshot: () => ({ directories: [] }),
+        clearPinned,
+      },
+      sessionIndexSyncRuntime: { publishChange },
+    });
+    const res = response();
+
+    route('DELETE', '/api/openchamber/session-index/session/:id/pin')(
+      { params: { id: 'ses_abc' } },
+      res,
+    );
+
+    expect(clearPinned).toHaveBeenCalledWith('ses_abc');
+    expect(publishChange).toHaveBeenCalledTimes(1);
+    expect(res.statusCode).toBe(204);
+  });
+
+  it('returns 404 for pin and unpin when the session is missing', () => {
+    const { app, route } = registry();
+    const publishChange = vi.fn();
+    registerSessionIndexRoutes(app, {
+      sessionIndexService: {
+        snapshot: () => ({ directories: [] }),
+        setPinned: () => false,
+        clearPinned: () => false,
+      },
+      sessionIndexSyncRuntime: { publishChange },
+    });
+
+    const pinRes = response();
+    route('POST', '/api/openchamber/session-index/session/:id/pin')(
+      { params: { id: 'ses_missing' } },
+      pinRes,
+    );
+    expect(pinRes.statusCode).toBe(404);
+    expect(pinRes.body).toMatchObject({ error: 'session_not_found' });
+
+    const unpinRes = response();
+    route('DELETE', '/api/openchamber/session-index/session/:id/pin')(
+      { params: { id: 'ses_missing' } },
+      unpinRes,
+    );
+    expect(unpinRes.statusCode).toBe(404);
+    expect(unpinRes.body).toMatchObject({ error: 'session_not_found' });
+    expect(publishChange).not.toHaveBeenCalled();
+  });
 });

package/server/lib/session-index/service.js

@@ -15,6 +15,7 @@ const isSystemSession = (session) => {
   if (!openchamber || typeof openchamber !== 'object') return false;
   if (nonEmptySystemID(openchamber.assistant?.assistantID)) return true;
   if (nonEmptySystemID(openchamber.scheduledTask?.taskID)) return true;
+  if (nonEmptySystemID(openchamber.smallModel?.purpose)) return true;
   return false;
 };
 
@@ -116,6 +117,7 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
       updated_at INTEGER NOT NULL,
       activity_updated_at INTEGER NOT NULL,
       archived_at INTEGER NOT NULL DEFAULT 0,
+      pinned_at INTEGER,
       status TEXT,
       status_changed_at INTEGER NOT NULL DEFAULT 0,
       parent_id TEXT,
@@ -139,6 +141,12 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
     CREATE INDEX IF NOT EXISTS session_child_parent
       ON session_child(runtime_key, directory, parent_id);
   `);
+  const sessionSummaryColumns = new Set(
+    db.prepare("SELECT name FROM pragma_table_info('session_summary')").all().map((column) => column.name),
+  );
+  if (!sessionSummaryColumns.has('pinned_at')) {
+    db.exec('ALTER TABLE session_summary ADD COLUMN pinned_at INTEGER');
+  }
   db.prepare('INSERT OR REPLACE INTO session_index_meta(key, value) VALUES (?, ?)')
     .run('schema_version', String(SCHEMA_VERSION));
 
@@ -165,7 +173,7 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
   const deleteDirectoryRows = db.prepare('DELETE FROM session_summary WHERE runtime_key = ? AND directory = ?');
   const existingSummaryState = db.prepare(`
     SELECT session_id, has_children AS hasChildren, activity_updated_at AS activityUpdatedAt,
-      status, status_changed_at AS statusChangedAt
+      status, status_changed_at AS statusChangedAt, pinned_at AS pinnedAt
     FROM session_summary WHERE runtime_key = ? AND directory = ?
   `);
   const readSummary = db.prepare(`
@@ -183,8 +191,8 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
     LIMIT 1 OFFSET ?
   `);
   const insertSummary = db.prepare(`
-    INSERT INTO session_summary(runtime_key, directory, session_id, title, created_at, updated_at, activity_updated_at, archived_at, status, status_changed_at, parent_id, has_children, assistant_id, assistant_name)
-    VALUES (@runtimeKey, @directory, @id, @title, @createdAt, @updatedAt, @activityUpdatedAt, @archivedAt, @status, @statusChangedAt, @parentID, @hasChildren, @assistantID, @assistantName)
+    INSERT INTO session_summary(runtime_key, directory, session_id, title, created_at, updated_at, activity_updated_at, archived_at, pinned_at, status, status_changed_at, parent_id, has_children, assistant_id, assistant_name)
+    VALUES (@runtimeKey, @directory, @id, @title, @createdAt, @updatedAt, @activityUpdatedAt, @archivedAt, @pinnedAt, @status, @statusChangedAt, @parentID, @hasChildren, @assistantID, @assistantName)
   `);
   const upsertSummary = db.prepare(`
     INSERT INTO session_summary(runtime_key, directory, session_id, title, created_at, updated_at, activity_updated_at, archived_at, status, status_changed_at, parent_id, has_children, assistant_id, assistant_name)
@@ -259,6 +267,7 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
       insertSummary.run({
         ...summary,
         activityUpdatedAt: Math.max(summary.activityUpdatedAt, toTimestamp(previous?.activityUpdatedAt)),
+        pinnedAt: previous?.pinnedAt != null ? toTimestamp(previous.pinnedAt) : null,
         status: typeof previous?.status === 'string' ? previous.status : null,
         statusChangedAt: toTimestamp(previous?.statusChangedAt),
         hasChildren: toBooleanInt(summary.hasChildren ?? Boolean(previous?.hasChildren)),
@@ -364,6 +373,7 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
       sessions: db.prepare(`
         SELECT session_id AS id, title, created_at AS createdAt, updated_at AS updatedAt,
           activity_updated_at AS activityUpdatedAt, archived_at AS archivedAt,
+          pinned_at AS pinnedAt,
           status, status_changed_at AS statusChangedAt, parent_id AS parentID,
           has_children AS hasChildren, assistant_id AS assistantID, assistant_name AS assistantName
         FROM session_summary WHERE runtime_key = ? AND directory = ?
@@ -373,7 +383,12 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
           id: session.id,
           title: session.title,
           directory: directory.directory,
-          time: { created: session.createdAt, updated: session.updatedAt, ...(session.archivedAt ? { archived: session.archivedAt } : {}) },
+          time: {
+            created: session.createdAt,
+            updated: session.updatedAt,
+            ...(session.archivedAt ? { archived: session.archivedAt } : {}),
+            ...(session.pinnedAt ? { pinned: session.pinnedAt } : {}),
+          },
           metadata: {
             openchamber: {
               titleRefresh: { activityUpdatedAt: session.activityUpdatedAt },
@@ -465,6 +480,38 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
     return result.changes > 0;
   };
 
+  const summaryExists = db.prepare(`
+    SELECT 1 AS present FROM session_summary
+    WHERE runtime_key = ? AND session_id = ?
+    LIMIT 1
+  `);
+
+  const setPinned = (sessionID, pinnedAt = Date.now()) => {
+    if (typeof sessionID !== 'string' || !sessionID) return false;
+    const timestamp = toTimestamp(pinnedAt);
+    if (!timestamp) return false;
+    const key = runtimeKey();
+    if (!summaryExists.get(key, sessionID)) return false;
+    db.prepare(`
+      UPDATE session_summary
+      SET pinned_at = ?
+      WHERE runtime_key = ? AND session_id = ?
+    `).run(timestamp, key, sessionID);
+    return true;
+  };
+
+  const clearPinned = (sessionID) => {
+    if (typeof sessionID !== 'string' || !sessionID) return false;
+    const key = runtimeKey();
+    if (!summaryExists.get(key, sessionID)) return false;
+    db.prepare(`
+      UPDATE session_summary
+      SET pinned_at = NULL
+      WHERE runtime_key = ? AND session_id = ?
+    `).run(key, sessionID);
+    return true;
+  };
+
   const replaceChildSessions = db.transaction((directory, parentSessionID, children) => {
     const normalizedDirectory = normalizeDirectory(directory);
     if (!normalizedDirectory || typeof parentSessionID !== 'string' || !parentSessionID) return false;
@@ -487,6 +534,8 @@ export const createSessionIndexService = ({ dbPath, getRuntimeConfig = () => nul
     upsertAndReportChange,
     touchActivity,
     updateStatus,
+    setPinned,
+    clearPinned,
     remove,
     findBySessionId,
     replaceChildSessions,

package/server/lib/session-index/service.test.js

@@ -68,7 +68,7 @@ describe('Electron session index', () => {
     service.close();
   });
 
-  it('excludes Assistant and Scheduled system sessions from ordinary sidebar summaries', () => {
+  it('excludes Assistant, Scheduled, and smallModel system sessions from ordinary sidebar summaries', () => {
     const runtimeRef = { value: 'http://runtime-a.test' };
     const service = createService(runtimeRef);
     const assistantSession = {
@@ -79,11 +79,15 @@ describe('Electron session index', () => {
       ...session('ses_scheduled', 99),
       metadata: { openchamber: { scheduledTask: { taskID: 'task_1' } } },
     };
+    const smallModelSession = {
+      ...session('ses_small_model', 97),
+      metadata: { openchamber: { smallModel: { purpose: 'session-title' } } },
+    };
     const ordinary = session('ses_ordinary', 98);
 
     service.replaceDirectory({
       directory: '/repo',
-      sessions: [assistantSession, scheduledSession, ordinary],
+      sessions: [assistantSession, scheduledSession, smallModelSession, ordinary],
       cursor: null,
       hasMore: false,
     });
@@ -91,6 +95,7 @@ describe('Electron session index', () => {
 
     service.upsert(assistantSession);
     service.upsert(scheduledSession);
+    service.upsert(smallModelSession);
     expect(service.snapshot().directories[0].sessions.map((item) => item.id)).toEqual(['ses_ordinary']);
     service.close();
   });
@@ -318,4 +323,69 @@ describe('Electron session index', () => {
     expect(rebuilt.snapshot().directories).toEqual([]);
     rebuilt.close();
   });
+
+  it('exposes time.pinned in snapshots and clears it on unpin', () => {
+    const runtimeRef = { value: 'http://runtime-a.test' };
+    const service = createService(runtimeRef);
+    service.replaceDirectory({ directory: '/repo', sessions: [session('ses_a', 10)], cursor: null, hasMore: false });
+
+    expect(service.setPinned('ses_a', 55)).toBe(true);
+    expect(service.snapshot().directories[0].sessions[0].time).toMatchObject({
+      created: 9,
+      updated: 10,
+      pinned: 55,
+    });
+
+    expect(service.clearPinned('ses_a')).toBe(true);
+    expect(service.snapshot().directories[0].sessions[0].time).toEqual({
+      created: 9,
+      updated: 10,
+    });
+    service.close();
+  });
+
+  it('rescues pinned_at across replaceDirectory rebuilds', () => {
+    const runtimeRef = { value: 'http://runtime-a.test' };
+    const service = createService(runtimeRef);
+    service.replaceDirectory({ directory: '/repo', sessions: [session('ses_a', 10)], cursor: null, hasMore: false });
+    expect(service.setPinned('ses_a', 77)).toBe(true);
+
+    service.replaceDirectory({
+      directory: '/repo',
+      sessions: [session('ses_a', 20)],
+      cursor: null,
+      hasMore: false,
+      now: 2000,
+    });
+
+    expect(service.snapshot().directories[0].sessions[0].time).toMatchObject({
+      updated: 20,
+      pinned: 77,
+    });
+    service.close();
+  });
+
+  it('does not let live upsert overwrite pinned_at', () => {
+    const runtimeRef = { value: 'http://runtime-a.test' };
+    const service = createService(runtimeRef);
+    service.replaceDirectory({ directory: '/repo', sessions: [session('ses_a', 10)], cursor: null, hasMore: false });
+    expect(service.setPinned('ses_a', 88)).toBe(true);
+
+    service.upsert(session('ses_a', 30));
+
+    expect(service.snapshot().directories[0].sessions[0].time.pinned).toBe(88);
+    service.close();
+  });
+
+  it('removes pinned rows when an archived upsert deletes the session', () => {
+    const runtimeRef = { value: 'http://runtime-a.test' };
+    const service = createService(runtimeRef);
+    service.replaceDirectory({ directory: '/repo', sessions: [session('ses_a', 10)], cursor: null, hasMore: false });
+    expect(service.setPinned('ses_a', 99)).toBe(true);
+
+    expect(service.upsert({ ...session('ses_a', 11), time: { created: 9, updated: 11, archived: 12 } })).toBe(true);
+    expect(service.snapshot().directories[0].sessions).toEqual([]);
+    expect(service.clearPinned('ses_a')).toBe(false);
+    service.close();
+  });
 });