pi-sessions 0.6.0 → 0.7.0

package/extensions/shared/session-index/sqlite.ts

@@ -48,22 +48,29 @@ function loadDatabaseConstructor(): SqliteConstructor {
 
 export function openSqlite(
   dbPath: string,
-  options: { create: boolean; timeoutMs?: number | undefined },
+  options: { create: boolean; readonly?: boolean | undefined; timeoutMs?: number | undefined },
 ): SqliteDatabase {
   const Database = loadDatabaseConstructor();
+  const readonly = options.readonly ?? false;
   const db = isBun
-    ? new Database(dbPath, { create: options.create, readwrite: true })
-    : new Database(dbPath, { fileMustExist: !options.create });
+    ? new Database(dbPath, {
+        create: readonly ? false : options.create,
+        readonly,
+        readwrite: !readonly,
+      })
+    : new Database(dbPath, { fileMustExist: readonly || !options.create, readonly });
 
-  // busy_timeout must be set before any other statement: the journal_mode
-  // pragma below is the connection's first lock acquisition (and may run WAL
-  // recovery), and immediate transactions rely on this timeout to queue behind
-  // concurrent writers instead of failing.
   db.exec(`PRAGMA busy_timeout = ${options.timeoutMs ?? DEFAULT_BUSY_TIMEOUT_MS}`);
-  db.exec("PRAGMA journal_mode = WAL");
-  db.exec("PRAGMA synchronous = NORMAL");
   db.exec("PRAGMA foreign_keys = ON");
 
+  if (!readonly) {
+    // journal_mode is the connection's first write-capable lock acquisition (and
+    // may run WAL recovery), and immediate transactions rely on busy_timeout to
+    // queue behind concurrent writers instead of failing.
+    db.exec("PRAGMA journal_mode = WAL");
+    db.exec("PRAGMA synchronous = NORMAL");
+  }
+
   return withStatementCache(db);
 }
 

package/extensions/shared/session-index/store.ts

@@ -1,7 +1,7 @@
+import path from "node:path";
 import { type Static, Type } from "typebox";
 import { parseTypeBoxValue } from "../typebox.ts";
 import {
-  compactSessionId,
   INDEX_SCHEMA_VERSION,
   NULLABLE_STRING_SCHEMA,
   parseRepoRoots,
@@ -76,7 +76,7 @@ export function insertSession(
       ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
     `,
   ).run(...sessionRowBindings(row, indexSource));
-  insertSessionIdChunk(db, row);
+  syncSessionRepoRoots(db, row);
 }
 
 export function upsertSession(
@@ -117,7 +117,7 @@ export function upsertSession(
         index_source = excluded.index_source
     `,
   ).run(...sessionRowBindings(row, indexSource));
-  syncSessionIdChunk(db, row);
+  syncSessionRepoRoots(db, row);
 }
 
 export function getSessionRowByPath(
@@ -186,24 +186,15 @@ function buildSessionRow(row: Static<typeof SESSION_ROW_QUERY_SCHEMA>): SessionR
   };
 }
 
-function insertSessionIdChunk(db: SessionIndexDatabase, row: SessionRow): void {
-  insertTextChunk(db, {
-    sessionId: row.sessionId,
-    entryType: "session_info",
-    ts: row.modifiedAt,
-    sourceKind: "session_id",
-    text: buildSessionIdSearchText(row.sessionId),
-  });
-}
-
-function syncSessionIdChunk(db: SessionIndexDatabase, row: SessionRow): void {
-  clearSessionChunksBySourceKind(db, row.sessionId, "session_id");
-  insertSessionIdChunk(db, row);
-}
+function syncSessionRepoRoots(db: SessionIndexDatabase, row: SessionRow): void {
+  db.prepare(`DELETE FROM session_repo_roots WHERE session_id = ?`).run(row.sessionId);
 
-function buildSessionIdSearchText(sessionId: string): string {
-  const compact = compactSessionId(sessionId);
-  return compact === sessionId ? sessionId : `${sessionId} ${compact}`;
+  const insert = db.prepare(
+    `INSERT INTO session_repo_roots(session_id, repo_root, repo_basename) VALUES (?, ?, ?)`,
+  );
+  for (const repoRoot of [...new Set(row.repoRoots)]) {
+    insert.run(row.sessionId, repoRoot, path.basename(repoRoot));
+  }
 }
 
 export function clearSessionChunksBySourceKind(
@@ -231,7 +222,7 @@ export function insertTextChunk(db: SessionIndexDatabase, row: SessionTextChunkR
     `,
   ).run(
     row.sessionId,
-    row.entryId ?? null,
+    row.entryId,
     row.entryType,
     row.role ?? null,
     row.ts,

package/extensions/shared/settings.ts

@@ -1,5 +1,6 @@
 import os from "node:os";
 import path from "node:path";
+import type { ThinkingLevel } from "@earendil-works/pi-agent-core";
 import { SettingsManager } from "@earendil-works/pi-coding-agent";
 import type { KeyId } from "@earendil-works/pi-tui";
 import { type Static, Type } from "typebox";
@@ -22,9 +23,17 @@ const SESSION_FILE_SETTINGS_SCHEMA = Type.Object({
     Type.Object({
       refreshTurns: Type.Optional(Type.Integer({ minimum: 1 })),
       model: Type.Optional(Type.String()),
+      thinkingLevel: Type.Optional(Type.String()),
       prompt: Type.Optional(Type.String()),
     }),
   ),
+  ask: Type.Optional(
+    Type.Object({
+      model: Type.Optional(Type.String()),
+      thinkingLevel: Type.Optional(Type.String()),
+      persistRuns: Type.Optional(Type.Boolean()),
+    }),
+  ),
 });
 const ROOT_SETTINGS_SCHEMA = Type.Object({
   sessions: Type.Optional(SESSION_FILE_SETTINGS_SCHEMA),
@@ -41,12 +50,20 @@ export class ModelReference {
   }
 }
 
-export interface AutoTitleSettings {
+export interface AgentModelSettings {
+  model?: ModelReference | undefined;
+  thinkingLevel?: ThinkingLevel | undefined;
+}
+
+export interface AutoTitleSettings extends AgentModelSettings {
   refreshTurns: number;
-  model: ModelReference | undefined;
   prompt: string;
 }
 
+export interface AskSettings extends AgentModelSettings {
+  persistRuns: boolean;
+}
+
 export interface SessionSettings {
   handoff: {
     pickerShortcut: KeyId;
@@ -55,6 +72,7 @@ export interface SessionSettings {
     path: string;
   };
   autoTitle: AutoTitleSettings;
+  ask: AskSettings;
 }
 
 type SessionFileSettings = Static<typeof SESSION_FILE_SETTINGS_SCHEMA>;
@@ -67,6 +85,10 @@ export function getDefaultIndexPath(): string {
   return path.join(getDefaultIndexDir(), "index.sqlite");
 }
 
+export function getDefaultSessionAskRunsDir(): string {
+  return path.join(getDefaultIndexDir(), "session-ask");
+}
+
 function expandHome(rawPath: string): string {
   if (rawPath === "~") {
     return os.homedir();
@@ -102,7 +124,7 @@ function normalizePickerShortcut(value: string | undefined): KeyId {
   return (trimmed ? trimmed : "alt+o") as KeyId;
 }
 
-function parseModelReference(value: string | undefined): ModelReference | undefined {
+export function parseModelReference(value: string | undefined): ModelReference | undefined {
   const trimmed = value?.trim();
   if (!trimmed) {
     return undefined;
@@ -116,6 +138,37 @@ function parseModelReference(value: string | undefined): ModelReference | undefi
   return new ModelReference(trimmed.slice(0, slashIndex), trimmed.slice(slashIndex + 1));
 }
 
+function parseThinkingLevel(value: string | undefined): ThinkingLevel | undefined {
+  const trimmed = value?.trim();
+  switch (trimmed) {
+    case "off":
+    case "minimal":
+    case "low":
+    case "medium":
+    case "high":
+    case "xhigh":
+      return trimmed;
+    default:
+      return undefined;
+  }
+}
+
+function resolveAgentModelSettings(
+  value:
+    | {
+        model?: string | undefined;
+        thinkingLevel?: string | undefined;
+      }
+    | undefined,
+): AgentModelSettings {
+  const model = parseModelReference(value?.model);
+  const thinkingLevel = parseThinkingLevel(value?.thinkingLevel);
+  return {
+    ...(model ? { model } : {}),
+    ...(thinkingLevel ? { thinkingLevel } : {}),
+  };
+}
+
 function normalizeAutoTitlePrompt(value: string | undefined): string {
   const trimmed = value?.trim();
   return trimmed ? trimmed : DEFAULT_AUTO_TITLE_PROMPT;
@@ -138,10 +191,14 @@ function resolveSessionSettings(fileSettings: SessionFileSettings): SessionSetti
       path: path.join(indexDir, "index.sqlite"),
     },
     autoTitle: {
+      ...resolveAgentModelSettings(fileSettings.autoTitle),
       refreshTurns: fileSettings.autoTitle?.refreshTurns ?? DEFAULT_AUTO_TITLE_REFRESH_TURNS,
-      model: parseModelReference(fileSettings.autoTitle?.model),
       prompt: normalizeAutoTitlePrompt(fileSettings.autoTitle?.prompt),
     },
+    ask: {
+      ...resolveAgentModelSettings(fileSettings.ask),
+      persistRuns: fileSettings.ask?.persistRuns ?? false,
+    },
   };
 }
 

package/extensions/shared/text.ts

@@ -0,0 +1,50 @@
+export function contentToText(content: unknown): string {
+  if (typeof content === "string") {
+    return content.trim();
+  }
+
+  if (!Array.isArray(content)) {
+    return "";
+  }
+
+  return content
+    .filter(isTextBlock)
+    .map((part) => part.text)
+    .join("\n")
+    .trim();
+}
+
+export function isRecord(value: unknown): value is Record<string, unknown> {
+  return typeof value === "object" && value !== null;
+}
+
+export function truncateInline(value: string, maxChars: number): string {
+  const cleaned = value.replace(/\s+/g, " ").trim();
+  if (cleaned.length <= maxChars) {
+    return cleaned;
+  }
+  return `${cleaned.slice(0, maxChars)}…`;
+}
+
+export function truncateBlock(
+  value: string,
+  maxChars: number,
+): { text: string; truncated: boolean } {
+  if (value.length <= maxChars) {
+    return { text: value, truncated: false };
+  }
+
+  return {
+    text: `${value.slice(0, maxChars)}\n\n[truncated to ${maxChars} characters]`,
+    truncated: true,
+  };
+}
+
+interface TextBlock {
+  type: "text";
+  text: string;
+}
+
+function isTextBlock(part: unknown): part is TextBlock {
+  return isRecord(part) && part.type === "text" && typeof part.text === "string";
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-sessions",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "type": "module",
   "description": "Pi session search, ask, handoff, messaging, auto-titling, and indexing tools",
   "license": "MIT",