opencode-sesh 2.1.6 → 2.1.7

package/AGENTS.md

@@ -197,7 +197,16 @@ when opencode exits, the picker reopens with the query preserved.
 - **Picker search:** pages the global session list (`SESSION_PAGE_LIMIT`, capped
   at `SESSION_MAX` and reported when truncated) rather than a fixed-size
   window, then indexes transcript text for **every** session in batches
-  (`TRANSCRIPT_BATCH`) with bounded remote concurrency (`REMOTE_CONCURRENCY`).
+  (`TRANSCRIPT_BATCH`, also capped by `TRANSCRIPT_BATCH_BYTES` of message
+  payload; a single larger session forms its own batch) with bounded remote
+  concurrency (`REMOTE_CONCURRENCY`). Transcript text is cached in memory for
+  the plugin lifetime (never on disk, cleared on dispose); a worker signature
+  query (session timestamp, message count, latest message timestamp, payload
+  size; chunked by `SIGNATURE_CHUNK`) lets reopened pickers reuse unchanged
+  sessions, extract only new or changed ones, and prune vanished ones. Sessions
+  indexed through the remote path are not cached. If the signature query fails
+  (`octet_length` needs SQLite ≥ 3.43; macOS Bun uses the system SQLite),
+  extraction still runs locally, batched by count only, and caches nothing.
   Local extraction is one SQL statement per batch in the DB worker
   (`transcriptTextQuery`, JSON-guarded) that returns only `{sid, text}` rows,
   so tool payloads never reach the UI thread; if the worker rejects, sessions

package/CHANGELOG.md

@@ -5,6 +5,16 @@ All notable changes to this project are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2.1.7] - 2026-11-02
+
+### Changed
+
+- **tui:** Reuse unchanged transcript text and bound index batches by bytes (#37)
+
+### Fixed
+
+- **tui:** Stop the DB worker and sidebar polls on plugin dispose (#36)
+
 ## [2.1.6] - 2026-11-02
 
 ### Changed
@@ -548,6 +558,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fixture-database regression suite, TypeScript typechecking, and ShellCheck in
   CI.
 
+[2.1.7]: https://github.com/ozandogrultan/opencode-sesh/compare/v2.1.6...v2.1.7
 [2.1.6]: https://github.com/ozandogrultan/opencode-sesh/compare/v2.1.5...v2.1.6
 [2.1.5]: https://github.com/ozandogrultan/opencode-sesh/compare/v2.1.4...v2.1.5
 [2.1.4]: https://github.com/ozandogrultan/opencode-sesh/compare/v2.1.3...v2.1.4

package/dist/tui.js

@@ -605,6 +605,8 @@ async function fetchEntries(api, active = () => true) {
 }
 const SESSION_ID_PATTERN = /^ses_[A-Za-z0-9]+$/;
 const TRANSCRIPT_BATCH = 64;
+const TRANSCRIPT_BATCH_BYTES = 8 * 1024 * 1024;
+const SIGNATURE_CHUNK = 500;
 const REMOTE_CONCURRENCY = 8;
 const nextTick = () => new Promise(resolve => setTimeout(resolve, 0));
 let resolvedDbPath;
@@ -663,9 +665,11 @@ function createDbWorker() {
 // Read-only queries against the local store, served by one lazily started
 // worker. Any failure (no worker, no bun:sqlite, query error, timeout) rejects;
 // a timeout or worker error discards the worker so the next call starts afresh.
+// `close` is final: later calls reject and never start a worker.
 function createDbClient(createWorker = createDbWorker, timeoutMs = DB_QUERY_TIMEOUT_MS) {
   let worker;
   let nextId = 0;
+  let closed = false;
   const pending = new Map();
   const reset = error => {
     const dead = worker;
@@ -702,6 +706,10 @@ function createDbClient(createWorker = createDbWorker, timeoutMs = DB_QUERY_TIME
     return created;
   };
   const run = (sql, params = []) => new Promise((resolve, reject) => {
+    if (closed) {
+      reject(new Error("local store closed"));
+      return;
+    }
     try {
       worker ??= start();
     } catch (error) {
@@ -729,10 +737,16 @@ function createDbClient(createWorker = createDbWorker, timeoutMs = DB_QUERY_TIME
   });
   return {
     run,
-    close: () => reset(new Error("local store closed"))
+    close: () => {
+      closed = true;
+      reset(new Error("local store closed"));
+    }
   };
 }
-const dbClient = createDbClient();
+
+// Each plugin setup installs its own client and closes only that one on
+// dispose: a reload reuses this module instance.
+let dbClient = createDbClient();
 function createSessionPoll(api, refreshPins) {
   // Listener -> whether it reads `snapshot.waiting`. The needs-input query
   // costs worker CPU on every poll, so it only runs while such a listener exists.
@@ -948,11 +962,33 @@ function transcriptTextQuery(count) {
 SELECT sid, text FROM parts WHERE text IS NOT NULL AND text <> '' ORDER BY sid, seq, rid, part`;
 }
 
+// Transcript text per session for the plugin lifetime; memory only, so
+// conversation text is never written anywhere. `sig` identifies the extracted
+// state of the session (see transcriptSignatureQuery).
+const transcriptCache = new Map();
+const clearTranscriptCache = () => transcriptCache.clear();
+
+// Cheap change detector mirroring the fzf extraction cache: message count and
+// latest message timestamp, plus the payload size that bounds extraction
+// batches. octet_length() reads the stored size, never the payload; length()
+// would read every payload to count characters.
+function transcriptSignatureQuery(count) {
+  const placeholders = Array.from({
+    length: count
+  }, () => "?").join(",");
+  return `SELECT session_id AS sid, COUNT(*) AS n, MAX(time_updated) AS m, SUM(octet_length(data)) AS bytes
+FROM session_message
+WHERE session_id IN (${placeholders}) AND type IN ('user', 'assistant')
+GROUP BY session_id`;
+}
+
 // Index titles/directories immediately, then add transcript text progressively
 // in batches so the render thread can breathe. Covers every entry, not a fixed
 // newest window; `onProgress` lets the picker show coverage. Local extraction
-// runs through `run` (the worker client) and returns text only.
-async function buildSearchIndex(api, entries, onProgress, active = () => true, run = dbClient.run) {
+// runs through `run` (the worker client) and returns text only; sessions whose
+// signature is unchanged reuse their cached text, and extraction batches are
+// bounded by session count and payload bytes.
+async function buildSearchIndex(api, entries, onProgress, active = () => true, run = dbClient.run, cache = transcriptCache) {
   const index = new Map();
   for (const entry of entries) {
     index.set(entry.id, `${entry.title} ${entry.dir}`);
@@ -971,9 +1007,54 @@ async function buildSearchIndex(api, entries, onProgress, active = () => true, r
   report(false);
   if (!active()) return index;
   try {
-    for (let i = 0; i < ids.length; i += TRANSCRIPT_BATCH) {
+    // `entries` must be the full session list: sessions missing from it are pruned.
+    const idSet = new Set(ids);
+    for (const sid of cache.keys()) if (!idSet.has(sid)) cache.delete(sid);
+    // Undefined when the signature query fails (octet_length needs SQLite
+    // >= 3.43, and macOS Bun uses the system SQLite): extract everything
+    // locally, batched by count only, and cache nothing.
+    let signatures = new Map();
+    try {
+      for (let i = 0; i < ids.length; i += SIGNATURE_CHUNK) {
+        const chunk = ids.slice(i, i + SIGNATURE_CHUNK);
+        const rows = await run(transcriptSignatureQuery(chunk.length), chunk);
+        if (!active()) return index;
+        for (const row of rows) signatures.set(row.sid, {
+          sig: `${row.n}:${row.m}:${row.bytes}`,
+          bytes: row.bytes ?? 0
+        });
+      }
+    } catch {
+      if (!active()) return index;
+      signatures = undefined;
+    }
+    const sigs = signatures && new Map(entries.map(entry => [entry.id, `${entry.updated}:${signatures.get(entry.id)?.sig ?? "0:0:0"}`]));
+    const stale = [];
+    for (const id of ids) {
+      const hit = cache.get(id);
+      if (hit && sigs && hit.sig === sigs.get(id)) {
+        if (hit.text) index.set(id, `${index.get(id) ?? ""} ${hit.text}`.trim());
+        remaining.delete(id);
+        indexed += 1;
+      } else stale.push(id);
+    }
+    if (indexed > 0) report(false);
+    const batches = [];
+    let current = [];
+    let currentBytes = 0;
+    for (const id of stale) {
+      const bytes = signatures?.get(id)?.bytes ?? 0;
+      if (current.length > 0 && (current.length >= TRANSCRIPT_BATCH || currentBytes + bytes > TRANSCRIPT_BATCH_BYTES)) {
+        batches.push(current);
+        current = [];
+        currentBytes = 0;
+      }
+      current.push(id);
+      currentBytes += bytes;
+    }
+    if (current.length > 0) batches.push(current);
+    for (const batch of batches) {
       if (!active()) break;
-      const batch = ids.slice(i, i + TRANSCRIPT_BATCH);
       const rows = await run(transcriptTextQuery(batch.length), batch);
       if (!active()) break;
       const texts = new Map();
@@ -981,8 +1062,15 @@ async function buildSearchIndex(api, entries, onProgress, active = () => true, r
         const list = texts.get(row.sid);
         if (list) list.push(row.text);else texts.set(row.sid, [row.text]);
       }
-      for (const [sid, list] of texts) index.set(sid, [index.get(sid) ?? "", ...list].join(" ").trim());
-      for (const id of batch) remaining.delete(id);
+      for (const id of batch) {
+        const list = texts.get(id) ?? [];
+        if (sigs) cache.set(id, {
+          sig: sigs.get(id),
+          text: list.join(" ").trim()
+        });
+        if (list.length > 0) index.set(id, [index.get(id) ?? "", ...list].join(" ").trim());
+        remaining.delete(id);
+      }
       indexed += batch.length;
       report(false);
       await nextTick();
@@ -4306,7 +4394,11 @@ function createApi(ctx) {
   const [dialogOpen, setDialogOpen] = createSignal(false);
   let sourceTheme;
   let currentTheme;
+  const slotDisposers = [];
   return {
+    dispose() {
+      for (const dispose of slotDisposers.splice(0)) dispose();
+    },
     theme: {
       get current() {
         if (sourceTheme !== ctx.theme) {
@@ -4441,20 +4533,20 @@ function createApi(ctx) {
     slots: {
       register(cfg) {
         if (cfg.slots.sidebar_content && ctx.options?.showSidebar !== false) {
-          ctx.ui.slot({
+          slotDisposers.push(ctx.ui.slot({
             ...(ctx.options?.sidebarPlacement === "prepend" ? {
               prepend: "sidebar.content"
             } : {
               append: "sidebar.content"
             }),
             render: cfg.slots.sidebar_content
-          });
+          }));
         }
         if (cfg.slots.home_bottom) {
-          ctx.ui.slot({
+          slotDisposers.push(ctx.ui.slot({
             prepend: "home.footer",
             render: cfg.slots.home_bottom
-          });
+          }));
         }
       }
     },
@@ -4568,6 +4660,7 @@ function createApi(ctx) {
 export default Plugin.define({
   id: "sesh-panel",
   async setup(ctx) {
+    const client = dbClient = createDbClient();
     const api = createApi(ctx);
     let initialized = false;
     const disposeSlot = ctx.ui.slot({
@@ -4584,8 +4677,10 @@ export default Plugin.define({
     return () => {
       if (api.ui.dialog.open) api.ui.dialog.clear();
       disposeSlot();
+      api.dispose();
       removeRootMouse?.();
-      dbClient.close();
+      clearTranscriptCache();
+      client.close();
     };
   }
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-sesh",
-  "version": "2.1.6",
+  "version": "2.1.7",
   "description": "OpenCode session browser: search, preview, and resume every session across every project from the terminal or TUI.",
   "type": "module",
   "license": "MIT",

package/tui-plugins/sesh-panel/tui.tsx

@@ -503,6 +503,8 @@ async function fetchEntries(api: TuiPluginApi, active = () => true): Promise<Ent
 
 const SESSION_ID_PATTERN = /^ses_[A-Za-z0-9]+$/
 const TRANSCRIPT_BATCH = 64
+const TRANSCRIPT_BATCH_BYTES = 8 * 1024 * 1024
+const SIGNATURE_CHUNK = 500
 const REMOTE_CONCURRENCY = 8
 
 type IndexProgress = { indexed: number; total: number; complete: boolean }
@@ -565,10 +567,12 @@ function createDbWorker(): any {
 // Read-only queries against the local store, served by one lazily started
 // worker. Any failure (no worker, no bun:sqlite, query error, timeout) rejects;
 // a timeout or worker error discards the worker so the next call starts afresh.
+// `close` is final: later calls reject and never start a worker.
 function createDbClient(createWorker: () => any = createDbWorker, timeoutMs = DB_QUERY_TIMEOUT_MS) {
   type Pending = { resolve: (rows: unknown[]) => void; reject: (error: Error) => void; timer: ReturnType<typeof setTimeout> }
   let worker: any
   let nextId = 0
+  let closed = false
   const pending = new Map<number, Pending>()
   const reset = (error: Error) => {
     const dead = worker
@@ -601,6 +605,10 @@ function createDbClient(createWorker: () => any = createDbWorker, timeoutMs = DB
   }
   const run: DbRun = (sql, params = []) =>
     new Promise((resolve, reject) => {
+      if (closed) {
+        reject(new Error("local store closed"))
+        return
+      }
       try {
         worker ??= start()
       } catch (error) {
@@ -617,10 +625,12 @@ function createDbClient(createWorker: () => any = createDbWorker, timeoutMs = DB
         reset(error as Error)
       }
     })
-  return { run, close: () => reset(new Error("local store closed")) }
+  return { run, close: () => { closed = true; reset(new Error("local store closed")) } }
 }
 
-const dbClient = createDbClient()
+// Each plugin setup installs its own client and closes only that one on
+// dispose: a reload reuses this module instance.
+let dbClient = createDbClient()
 
 type SessionSnapshot = { entries: Entry[]; waiting: Map<string, string>; failed: boolean }
 
@@ -830,16 +840,37 @@ function transcriptTextQuery(count: number): string {
 SELECT sid, text FROM parts WHERE text IS NOT NULL AND text <> '' ORDER BY sid, seq, rid, part`
 }
 
+// Transcript text per session for the plugin lifetime; memory only, so
+// conversation text is never written anywhere. `sig` identifies the extracted
+// state of the session (see transcriptSignatureQuery).
+const transcriptCache = new Map<string, { sig: string; text: string }>()
+const clearTranscriptCache = () => transcriptCache.clear()
+
+// Cheap change detector mirroring the fzf extraction cache: message count and
+// latest message timestamp, plus the payload size that bounds extraction
+// batches. octet_length() reads the stored size, never the payload; length()
+// would read every payload to count characters.
+function transcriptSignatureQuery(count: number): string {
+  const placeholders = Array.from({ length: count }, () => "?").join(",")
+  return `SELECT session_id AS sid, COUNT(*) AS n, MAX(time_updated) AS m, SUM(octet_length(data)) AS bytes
+FROM session_message
+WHERE session_id IN (${placeholders}) AND type IN ('user', 'assistant')
+GROUP BY session_id`
+}
+
 // Index titles/directories immediately, then add transcript text progressively
 // in batches so the render thread can breathe. Covers every entry, not a fixed
 // newest window; `onProgress` lets the picker show coverage. Local extraction
-// runs through `run` (the worker client) and returns text only.
+// runs through `run` (the worker client) and returns text only; sessions whose
+// signature is unchanged reuse their cached text, and extraction batches are
+// bounded by session count and payload bytes.
 async function buildSearchIndex(
   api: TuiPluginApi,
   entries: Entry[],
   onProgress?: (progress: IndexProgress, index: Map<string, string>) => void,
   active = () => true,
   run: DbRun = dbClient.run,
+  cache = transcriptCache,
 ): Promise<Map<string, string>> {
   const index = new Map<string, string>()
   for (const entry of entries) {
@@ -854,9 +885,53 @@ async function buildSearchIndex(
 
   if (!active()) return index
   try {
-    for (let i = 0; i < ids.length; i += TRANSCRIPT_BATCH) {
+    // `entries` must be the full session list: sessions missing from it are pruned.
+    const idSet = new Set(ids)
+    for (const sid of cache.keys()) if (!idSet.has(sid)) cache.delete(sid)
+    // Undefined when the signature query fails (octet_length needs SQLite
+    // >= 3.43, and macOS Bun uses the system SQLite): extract everything
+    // locally, batched by count only, and cache nothing.
+    let signatures: Map<string, { sig: string; bytes: number }> | undefined = new Map()
+    try {
+      for (let i = 0; i < ids.length; i += SIGNATURE_CHUNK) {
+        const chunk = ids.slice(i, i + SIGNATURE_CHUNK)
+        const rows = (await run(transcriptSignatureQuery(chunk.length), chunk)) as { sid: string; n: number; m: number; bytes: number }[]
+        if (!active()) return index
+        for (const row of rows) signatures.set(row.sid, { sig: `${row.n}:${row.m}:${row.bytes}`, bytes: row.bytes ?? 0 })
+      }
+    } catch {
+      if (!active()) return index
+      signatures = undefined
+    }
+    const sigs = signatures && new Map(entries.map((entry) => [entry.id, `${entry.updated}:${signatures!.get(entry.id)?.sig ?? "0:0:0"}`]))
+    const stale: string[] = []
+    for (const id of ids) {
+      const hit = cache.get(id)
+      if (hit && sigs && hit.sig === sigs.get(id)) {
+        if (hit.text) index.set(id, `${index.get(id) ?? ""} ${hit.text}`.trim())
+        remaining.delete(id)
+        indexed += 1
+      } else stale.push(id)
+    }
+    if (indexed > 0) report(false)
+
+    const batches: string[][] = []
+    let current: string[] = []
+    let currentBytes = 0
+    for (const id of stale) {
+      const bytes = signatures?.get(id)?.bytes ?? 0
+      if (current.length > 0 && (current.length >= TRANSCRIPT_BATCH || currentBytes + bytes > TRANSCRIPT_BATCH_BYTES)) {
+        batches.push(current)
+        current = []
+        currentBytes = 0
+      }
+      current.push(id)
+      currentBytes += bytes
+    }
+    if (current.length > 0) batches.push(current)
+
+    for (const batch of batches) {
       if (!active()) break
-      const batch = ids.slice(i, i + TRANSCRIPT_BATCH)
       const rows = (await run(transcriptTextQuery(batch.length), batch)) as { sid: string; text: string }[]
       if (!active()) break
       const texts = new Map<string, string[]>()
@@ -865,8 +940,12 @@ async function buildSearchIndex(
         if (list) list.push(row.text)
         else texts.set(row.sid, [row.text])
       }
-      for (const [sid, list] of texts) index.set(sid, [index.get(sid) ?? "", ...list].join(" ").trim())
-      for (const id of batch) remaining.delete(id)
+      for (const id of batch) {
+        const list = texts.get(id) ?? []
+        if (sigs) cache.set(id, { sig: sigs.get(id)!, text: list.join(" ").trim() })
+        if (list.length > 0) index.set(id, [index.get(id) ?? "", ...list].join(" ").trim())
+        remaining.delete(id)
+      }
       indexed += batch.length
       report(false)
       await nextTick()
@@ -3352,7 +3431,11 @@ function createApi(ctx: Plugin.Context) {
   const [dialogOpen, setDialogOpen] = createSignal(false)
   let sourceTheme: Plugin.Context["theme"] | undefined
   let currentTheme: ReturnType<typeof adaptTheme>
+  const slotDisposers: Array<() => void> = []
   return {
+    dispose() {
+      for (const dispose of slotDisposers.splice(0)) dispose()
+    },
     theme: {
       get current() {
         if (sourceTheme !== ctx.theme) {
@@ -3476,18 +3559,18 @@ function createApi(ctx: Plugin.Context) {
     slots: {
       register(cfg: { order?: number; slots: Record<string, () => any> }) {
         if (cfg.slots.sidebar_content && ctx.options?.showSidebar !== false) {
-          ctx.ui.slot({
+          slotDisposers.push(ctx.ui.slot({
             ...(ctx.options?.sidebarPlacement === "prepend"
               ? { prepend: "sidebar.content" as const }
               : { append: "sidebar.content" as const }),
             render: cfg.slots.sidebar_content as any,
-          })
+          }))
         }
         if (cfg.slots.home_bottom) {
-          ctx.ui.slot({
+          slotDisposers.push(ctx.ui.slot({
             prepend: "home.footer",
             render: cfg.slots.home_bottom as any,
-          })
+          }))
         }
       },
     },
@@ -3581,6 +3664,7 @@ function createApi(ctx: Plugin.Context) {
 export default Plugin.define({
   id: "sesh-panel",
   async setup(ctx) {
+    const client = dbClient = createDbClient()
     const api = createApi(ctx)
     let initialized = false
     const disposeSlot = ctx.ui.slot({
@@ -3597,8 +3681,10 @@ export default Plugin.define({
     return () => {
       if (api.ui.dialog.open) api.ui.dialog.clear()
       disposeSlot()
+      api.dispose()
       removeRootMouse?.()
-      dbClient.close()
+      clearTranscriptCache()
+      client.close()
     }
   },
 })