@soleri/core 9.3.1 → 9.5.0

package/src/vault/vault-memories.ts

@@ -299,21 +299,29 @@ export function memoryTopics(
 export function memoriesByProject(
   provider: PersistenceProvider,
 ): Array<{ project: string; count: number; memories: Memory[] }> {
-  const rows = provider.all<{ project: string; count: number }>(
-    'SELECT project_path as project, COUNT(*) as count FROM memories WHERE archived_at IS NULL GROUP BY project_path ORDER BY count DESC',
+  // Single query fetching all non-archived memories, grouped client-side by project
+  const allRows = provider.all<Record<string, unknown>>(
+    'SELECT * FROM memories WHERE archived_at IS NULL ORDER BY project_path, created_at DESC',
   );
 
-  return rows.map((row) => {
-    const mems = provider.all<Record<string, unknown>>(
-      'SELECT * FROM memories WHERE project_path = ? AND archived_at IS NULL ORDER BY created_at DESC',
-      [row.project],
-    );
-    return {
-      project: row.project,
-      count: row.count,
-      memories: mems.map(rowToMemory),
-    };
-  });
+  const projectMap = new Map<string, Memory[]>();
+  for (const row of allRows) {
+    const memory = rowToMemory(row);
+    const project = memory.projectPath;
+    if (!projectMap.has(project)) {
+      projectMap.set(project, []);
+    }
+    projectMap.get(project)!.push(memory);
+  }
+
+  // Sort by count descending (matching original behavior)
+  return [...projectMap.entries()]
+    .map(([project, memories]) => ({
+      project,
+      count: memories.length,
+      memories,
+    }))
+    .sort((a, b) => b.count - a.count);
 }
 
 // ── Helper ──────────────────────────────────────────────────────────────

package/src/vault/vault-schema.ts

@@ -27,6 +27,7 @@ export function initializeSchema(provider: PersistenceProvider): void {
   migrateOriginColumn(provider);
   migrateContentHash(provider);
   migrateTierColumn(provider);
+  migratePerformanceIndexes(provider);
 }
 
 function createCoreTables(provider: PersistenceProvider): void {
@@ -236,3 +237,23 @@ function migrateTierColumn(provider: PersistenceProvider): void {
     'CREATE INDEX IF NOT EXISTS idx_entries_tier ON entries(tier) WHERE tier IS NOT NULL',
   );
 }
+
+export function migratePerformanceIndexes(provider: PersistenceProvider): void {
+  provider.execSql(`
+    CREATE INDEX IF NOT EXISTS idx_memories_archived_at ON memories(archived_at);
+    CREATE INDEX IF NOT EXISTS idx_entries_updated_at ON entries(updated_at);
+    CREATE INDEX IF NOT EXISTS idx_brain_feedback_entry_id ON brain_feedback(entry_id);
+    CREATE INDEX IF NOT EXISTS idx_entries_valid_until ON entries(valid_until) WHERE valid_until IS NOT NULL;
+    CREATE INDEX IF NOT EXISTS idx_entries_valid_from ON entries(valid_from) WHERE valid_from IS NOT NULL;
+  `);
+
+  // brain_sessions may not exist yet if intelligence module hasn't initialized
+  try {
+    provider.execSql(`
+      CREATE INDEX IF NOT EXISTS idx_brain_sessions_plan_id ON brain_sessions(plan_id) WHERE plan_id IS NOT NULL;
+      CREATE INDEX IF NOT EXISTS idx_brain_sessions_started_at ON brain_sessions(started_at);
+    `);
+  } catch {
+    /* brain_sessions table doesn't exist yet — indexes will be created on next init */
+  }
+}

package/src/vault/vault.ts

@@ -53,6 +53,13 @@ export interface MemoryStats {
   byProject: Record<string, number>;
 }
 
+/** Apply critical PRAGMAs that every vault database must have. */
+function applyVaultPragmas(provider: PersistenceProvider): void {
+  provider.run('PRAGMA journal_mode = WAL');
+  provider.run('PRAGMA foreign_keys = ON');
+  provider.run('PRAGMA synchronous = NORMAL');
+}
+
 export class Vault {
   private provider: PersistenceProvider;
   private sqliteProvider: SQLitePersistenceProvider | null;
@@ -65,14 +72,12 @@ export class Vault {
       const sqlite = new SQLitePersistenceProvider(providerOrPath);
       this.provider = sqlite;
       this.sqliteProvider = sqlite;
-      this.provider.run('PRAGMA journal_mode = WAL');
-      this.provider.run('PRAGMA foreign_keys = ON');
-      this.provider.run('PRAGMA synchronous = NORMAL');
     } else {
       this.provider = providerOrPath;
       this.sqliteProvider =
         providerOrPath instanceof SQLitePersistenceProvider ? providerOrPath : null;
     }
+    applyVaultPragmas(this.provider);
     initializeSchema(this.provider);
     checkFormatVersion(this.provider);
   }

package/vitest.config.ts

@@ -6,6 +6,7 @@ export default defineConfig({
     pool: 'forks',
     poolOptions: { forks: { singleFork: true } },
     testTimeout: 30_000,
+    exclude: ['**/node_modules/**', '**/.claude/worktrees/**'],
     coverage: {
       provider: 'v8',
       include: ['src/**/*.ts'],