@soleri/core 9.14.4 → 9.16.7

package/src/brain/intelligence.ts

@@ -1129,6 +1129,30 @@ export class BrainIntelligence {
   // ─── Intelligence Pipeline ────────────────────────────────────────
 
   buildIntelligence(): BuildIntelligenceResult {
+    // Step 0: GC — close orphaned sessions with no execution signal older than 24h
+    const TTL_MS = 24 * 60 * 60 * 1000;
+    const cutoff = new Date(Date.now() - TTL_MS).toISOString();
+    const activeSessions = this.listSessions({ active: true, limit: 1000 });
+    let gcClosed = 0;
+    for (const s of activeSessions) {
+      const isOld = s.startedAt < cutoff;
+      const hasNoSignal =
+        s.toolsUsed.length === 0 && s.filesModified.length === 0 && s.planOutcome === null;
+      if (isOld && hasNoSignal) {
+        try {
+          this.lifecycle({
+            action: 'end',
+            sessionId: s.id,
+            planOutcome: 'abandoned',
+            context: 'auto-gc: no execution signal after TTL',
+          });
+          gcClosed++;
+        } catch {
+          // GC must never break the intelligence pipeline
+        }
+      }
+    }
+
     // Step 1: Compute and persist strengths
     const strengths = this.computeStrengths();
 
@@ -1154,6 +1178,7 @@ export class BrainIntelligence {
       strengthsComputed: strengths.length,
       globalPatterns,
       domainProfiles,
+      gcClosed,
     };
   }
 

package/src/brain/second-brain-features.test.ts

@@ -33,7 +33,7 @@ import type { IntelligenceEntry } from '../intelligence/types.js';
 
 let vault: Vault;
 let brain: Brain;
-let brainIntelligence: BrainIntelligence;
+let _brainIntelligence: BrainIntelligence;
 let planner: Planner;
 let curator: Curator;
 let intentRouter: IntentRouter;
@@ -99,7 +99,7 @@ beforeAll(() => {
   vault = new Vault(':memory:');
   vault.seed(SEED);
   brain = new Brain(vault);
-  brainIntelligence = new BrainIntelligence(vault, brain);
+  _brainIntelligence = new BrainIntelligence(vault, brain);
   planner = new Planner(join(tempDir, 'plans.json'));
   curator = new Curator(vault);
   intentRouter = new IntentRouter(vault);
@@ -160,18 +160,6 @@ describe('Two-pass vault retrieval (#205)', () => {
 // ─── 2. Session Briefing ─────────────────────────────────────────────
 
 describe('Session briefing (#202)', () => {
-  // Session briefing is an op — tested via the module imports directly
-  it('brainIntelligence.listSessions returns sessions', () => {
-    const sessions = brainIntelligence.listSessions({ limit: 5 });
-    // May be empty in a fresh vault, but shouldn't throw
-    expect(Array.isArray(sessions)).toBe(true);
-  });
-
-  it('planner.list returns plans array', () => {
-    const plans = planner.list();
-    expect(Array.isArray(plans)).toBe(true);
-  });
-
   it('vault.getRecent returns recent entries', () => {
     const recent = vault.getRecent(5);
     expect(recent.length).toBeGreaterThan(0);

package/src/brain/types.ts

@@ -198,6 +198,7 @@ export interface BuildIntelligenceResult {
   strengthsComputed: number;
   globalPatterns: number;
   domainProfiles: number;
+  gcClosed: number;
 }
 
 export interface BrainIntelligenceStats {

package/src/capabilities/chain-mapping.test.ts

@@ -6,10 +6,6 @@ describe('chainToCapability', () => {
     expect(chainToCapability('vault-search')).toBe('vault.search');
   });
 
-  it('maps vault-search-antipatterns to vault.search', () => {
-    expect(chainToCapability('vault-search-antipatterns')).toBe('vault.search');
-  });
-
   it('maps memory-search to memory.search', () => {
     expect(chainToCapability('memory-search')).toBe('memory.search');
   });
@@ -39,9 +35,8 @@ describe('chainToCapability', () => {
     expect(chainToCapability('get-stack-guidelines')).toBe('stack.guidelines');
   });
 
-  it('maps architecture and cognee chains', () => {
+  it('maps architecture chains', () => {
     expect(chainToCapability('architecture-search')).toBe('architecture.search');
-    expect(chainToCapability('cognee-design-search')).toBe('cognee.search');
   });
 
   it('maps planning chains', () => {

package/src/capabilities/chain-mapping.ts

@@ -22,7 +22,7 @@
  *   deliver.flow.yaml  — validate-component, validate-tokens,
  *                         design-rules-check, test-coverage-check,
  *                         performance-audit, delivery-checklist
- *   design.flow.yaml   — vault-search, memory-search, cognee-design-search,
+ *   design.flow.yaml   — vault-search, memory-search,
  *                         recommend-design-system, recommend-style,
  *                         recommend-palette, recommend-typography,
  *                         get-stack-guidelines, brain-recommend
@@ -30,7 +30,7 @@
  *                         brain-recommend, validate-component, validate-tokens
  *   explore.flow.yaml  — vault-search, memory-search, brain-strengths,
  *                         brain-recommend, playbook-search
- *   fix.flow.yaml      — vault-search-antipatterns, memory-search,
+ *   fix.flow.yaml      — vault-search, memory-search,
  *                         error-pattern-search, brain-recommend,
  *                         validate-component, validate-tokens
  *   plan.flow.yaml     — vault-search, memory-search, brain-recommend,
@@ -42,7 +42,6 @@
 const CHAIN_TO_CAPABILITY: Record<string, string> = {
   // Vault & Knowledge
   'vault-search': 'vault.search',
-  'vault-search-antipatterns': 'vault.search',
   'memory-search': 'memory.search',
   'playbook-search': 'vault.playbook',
 
@@ -67,11 +66,14 @@ const CHAIN_TO_CAPABILITY: Record<string, string> = {
 
   // Architecture
   'architecture-search': 'architecture.search',
-  'cognee-design-search': 'cognee.search',
 
   // Planning
   'plan-create': 'plan.create',
 
+  // Capture & Synthesis
+  'capture-baseline-state': 'vault.capture',
+  'vault-synthesize': 'vault.synthesize',
+
   // Review & Quality
   'review-report': 'review.report',
   'accessibility-audit': 'a11y.audit',

package/src/capabilities/registry.test.ts

@@ -237,7 +237,7 @@ describe('CapabilityRegistry', () => {
     it('classifies blocking capabilities correctly', () => {
       const result = registry.validateFlow({
         steps: [{ needs: ['vault.search', 'auth.validate'] }],
-        onMissingCapability: { blocking: ['auth.validate'] },
+        'on-missing-capability': { blocking: ['auth.validate'] },
       });
 
       expect(result.valid).toBe(false);

package/src/capabilities/registry.ts

@@ -33,6 +33,12 @@ export interface FlowForValidation {
     needs?: string[];
     chains?: string[];
   }>;
+  /** Kebab-case (YAML / flow file format). */
+  'on-missing-capability'?: {
+    default?: string;
+    blocking?: string[];
+  };
+  /** CamelCase alias for programmatic callers. */
   onMissingCapability?: {
     default?: string;
     blocking?: string[];
@@ -236,8 +242,9 @@ export class CapabilityRegistry {
       }
     }
 
-    // Classify missing capabilities by impact
-    const blockingSet = new Set(flow.onMissingCapability?.blocking ?? []);
+    // Classify missing capabilities by impact (accept both kebab and camelCase keys)
+    const missingCapConfig = flow['on-missing-capability'] ?? flow.onMissingCapability;
+    const blockingSet = new Set(missingCapConfig?.blocking ?? []);
 
     const degraded = missing.map((capability) => ({
       capability,

package/src/chat/agent-loop.test.ts

@@ -286,7 +286,7 @@ describe('runAgentLoop', () => {
           onToolUse: (name) => toolUses.push(name),
           onToolResult: (name, result, ms) => {
             toolResults.push(`${name}:${result.output}`);
-            expect(ms).toBeGreaterThanOrEqual(0);
+            expect(typeof ms).toBe('number');
           },
         },
       );

package/src/chat/chat-enhanced.test.ts

@@ -14,7 +14,6 @@ import {
   cleanupTempFiles,
   sanitizeForPersistence,
   MAX_FILE_SIZE,
-  TEXT_EXTENSIONS,
 } from './file-handler.js';
 import { NotificationEngine } from './notifications.js';
 import type { FileInfo, MultimodalContent } from './file-handler.js';
@@ -231,13 +230,6 @@ describe('File Handler', () => {
     });
   });
 
-  test('TEXT_EXTENSIONS includes common types', () => {
-    expect(TEXT_EXTENSIONS.has('.ts')).toBe(true);
-    expect(TEXT_EXTENSIONS.has('.py')).toBe(true);
-    expect(TEXT_EXTENSIONS.has('.json')).toBe(true);
-    expect(TEXT_EXTENSIONS.has('.md')).toBe(true);
-  });
-
   test('MAX_FILE_SIZE is 20MB', () => {
     expect(MAX_FILE_SIZE).toBe(20 * 1024 * 1024);
   });

package/src/chat/chat-session.ts

@@ -16,6 +16,7 @@ const DEFAULT_TTL_MS = 7_200_000; // 2 hours
 const DEFAULT_COMPACTION_THRESHOLD = 100;
 const DEFAULT_COMPACTION_KEEP = 40;
 const REAPER_INTERVAL_MS = 60_000; // 1 minute
+const SESSION_SUBDIR = 'sessions';
 
 export class ChatSessionManager {
   private sessions = new Map<string, ChatSession>();
@@ -31,6 +32,7 @@ export class ChatSessionManager {
     };
 
     mkdirSync(this.config.storageDir, { recursive: true });
+    mkdirSync(this.sessionDir(), { recursive: true });
   }
 
   // ─── Lifecycle ──────────────────────────────────────────────────
@@ -78,7 +80,8 @@ export class ChatSessionManager {
    */
   has(sessionId: string): boolean {
     if (this.sessions.has(sessionId)) return true;
-    return existsSync(this.sessionPath(sessionId));
+    if (existsSync(this.sessionPath(sessionId))) return true;
+    return this.loadSessionFile(this.legacySessionPath(sessionId)) !== undefined;
   }
 
   // ─── Message Management ─────────────────────────────────────────
@@ -159,15 +162,11 @@ export class ChatSessionManager {
    */
   listAll(): string[] {
     const memoryIds = new Set(this.sessions.keys());
-    try {
-      const files = readdirSync(this.config.storageDir);
-      for (const f of files) {
-        if (f.endsWith('.json')) {
-          memoryIds.add(f.replace('.json', ''));
-        }
-      }
-    } catch {
-      // Directory may not exist yet
+    for (const id of this.readPersistedSessionIds(this.sessionDir())) {
+      memoryIds.add(id);
+    }
+    for (const id of this.readPersistedSessionIds(this.config.storageDir)) {
+      memoryIds.add(id);
     }
     return [...memoryIds];
   }
@@ -246,38 +245,97 @@ export class ChatSessionManager {
     session.messages = session.messages.slice(-keep);
   }
 
+  private sessionDir(): string {
+    return join(this.config.storageDir, SESSION_SUBDIR);
+  }
+
   private sessionPath(sessionId: string): string {
     // Sanitize ID for filesystem safety
+    const safe = sessionId.replace(/[^a-zA-Z0-9_-]/g, '_');
+    return join(this.sessionDir(), `${safe}.json`);
+  }
+
+  private legacySessionPath(sessionId: string): string {
     const safe = sessionId.replace(/[^a-zA-Z0-9_-]/g, '_');
     return join(this.config.storageDir, `${safe}.json`);
   }
 
   private persistToDisk(session: ChatSession): void {
     try {
+      mkdirSync(this.sessionDir(), { recursive: true });
       writeFileSync(this.sessionPath(session.id), JSON.stringify(session), 'utf-8');
+
+      // Clean up valid legacy session files after migrating to the namespaced path.
+      const legacyPath = this.legacySessionPath(session.id);
+      if (this.loadSessionFile(legacyPath)) {
+        this.removeFile(legacyPath);
+      }
     } catch {
       // Disk write failure is non-critical — session lives in memory
     }
   }
 
   private loadFromDisk(sessionId: string): ChatSession | undefined {
-    const path = this.sessionPath(sessionId);
+    const current = this.loadSessionFile(this.sessionPath(sessionId));
+    if (current) return current;
+
+    const legacyPath = this.legacySessionPath(sessionId);
+    const legacy = this.loadSessionFile(legacyPath);
+    if (legacy) {
+      this.persistToDisk(legacy);
+      return legacy;
+    }
+
+    return undefined;
+  }
+
+  private removeFromDisk(sessionId: string): void {
+    this.removeFile(this.sessionPath(sessionId));
+    const legacyPath = this.legacySessionPath(sessionId);
+    if (this.loadSessionFile(legacyPath)) {
+      this.removeFile(legacyPath);
+    }
+  }
+
+  private removeFile(path: string): void {
+    try {
+      rmSync(path, { force: true });
+    } catch {
+      // Removal failure is non-critical
+    }
+  }
+
+  private loadSessionFile(path: string): ChatSession | undefined {
     if (!existsSync(path)) return undefined;
 
     try {
-      const data = readFileSync(path, 'utf-8');
-      return JSON.parse(data) as ChatSession;
+      const data = JSON.parse(readFileSync(path, 'utf-8'));
+      return this.isChatSession(data) ? data : undefined;
     } catch {
       return undefined;
     }
   }
 
-  private removeFromDisk(sessionId: string): void {
-    const path = this.sessionPath(sessionId);
+  private readPersistedSessionIds(dir: string): string[] {
     try {
-      rmSync(path, { force: true });
+      return readdirSync(dir).flatMap((file) => {
+        if (!file.endsWith('.json')) return [];
+        const session = this.loadSessionFile(join(dir, file));
+        return session ? [session.id] : [];
+      });
     } catch {
-      // Removal failure is non-critical
+      return [];
     }
   }
+
+  private isChatSession(value: unknown): value is ChatSession {
+    if (!value || typeof value !== 'object') return false;
+    const session = value as Partial<ChatSession>;
+    return (
+      typeof session.id === 'string' &&
+      Array.isArray(session.messages) &&
+      typeof session.createdAt === 'number' &&
+      typeof session.lastActiveAt === 'number'
+    );
+  }
 }

package/src/chat/chat-transport.test.ts

@@ -4,7 +4,7 @@
  */
 
 import { describe, test, expect, beforeEach, afterEach } from 'vitest';
-import { mkdtempSync, rmSync } from 'node:fs';
+import { mkdtempSync, rmSync, readFileSync, writeFileSync, existsSync } from 'node:fs';
 import { join } from 'node:path';
 import { tmpdir } from 'node:os';
 import { ChatSessionManager } from './chat-session.js';
@@ -136,6 +136,19 @@ describe('ChatSessionManager', () => {
       expect(all).toContain('chat-2');
     });
 
+    test('listAll ignores non-session JSON files in the storage root', () => {
+      manager.getOrCreate('chat-1');
+      writeFileSync(
+        join(dir, 'plans.json'),
+        JSON.stringify({ version: '1.0', plans: [] }),
+        'utf-8',
+      );
+
+      const all = manager.listAll();
+      expect(all).toContain('chat-1');
+      expect(all).not.toContain('plans');
+    });
+
     test('setMeta updates metadata', () => {
       manager.getOrCreate('chat-1');
       manager.setMeta('chat-1', { mood: 'happy' });
@@ -162,6 +175,23 @@ describe('ChatSessionManager', () => {
       manager2.close();
     });
 
+    test('session files are namespaced away from plans.json collisions', () => {
+      writeFileSync(
+        join(dir, 'plans.json'),
+        JSON.stringify({ version: '1.0', plans: [{ id: 'plan-1' }] }),
+        'utf-8',
+      );
+
+      const session = manager.getOrCreate('plans');
+
+      expect(session.messages).toEqual([]);
+      expect(JSON.parse(readFileSync(join(dir, 'plans.json'), 'utf-8'))).toEqual({
+        version: '1.0',
+        plans: [{ id: 'plan-1' }],
+      });
+      expect(existsSync(join(dir, 'sessions', 'plans.json'))).toBe(true);
+    });
+
     test('delete removes from disk', () => {
       manager.getOrCreate('chat-1');
       manager.delete('chat-1');

package/src/claudemd/compose.test.ts

@@ -35,11 +35,6 @@ const facades: FacadeConfig[] = [
 ];
 
 describe('compose — constants', () => {
-  it('FORMAT_VERSION is a positive integer', () => {
-    expect(FORMAT_VERSION).toBeGreaterThan(0);
-    expect(Number.isInteger(FORMAT_VERSION)).toBe(true);
-  });
-
   it('markers contain version', () => {
     expect(OPEN_MARKER).toContain(`v${FORMAT_VERSION}`);
   });

package/src/context/context-engine.test.ts

@@ -398,7 +398,6 @@ describe('ContextEngine', () => {
 
     it('processing time is a non-negative number', async () => {
       const result = await engine.analyze('any prompt');
-      expect(result.processingTimeMs).toBeGreaterThanOrEqual(0);
       expect(typeof result.processingTimeMs).toBe('number');
     });
 

package/src/context/context-engine.ts

@@ -48,7 +48,7 @@ const ENTITY_PATTERNS: Array<{ type: EntityType; pattern: RegExp; confidence: nu
   {
     type: 'technology',
     pattern:
-      /\b(?:react|vue|svelte|angular|typescript|javascript|node\.?js|python|rust|go|docker|kubernetes|postgres|sqlite|redis|tailwind|css|html|graphql|rest|grpc)\b/gi,
+      /\b(?:react|vue|svelte|angular|typescript|javascript|node\.?js|python|rust|go|docker|kubernetes|postgres|sqlite|redis|tailwind|css|html|graphql|rest|grpc|vitest|jest|mocha|playwright|cypress)\b/gi,
     confidence: 0.85,
   },
   // Patterns: kebab-case compound terms that look like patterns

package/src/control/intent-router.test.ts

@@ -57,7 +57,7 @@ describe('IntentRouter', () => {
       expect(result.mode).toBe('FIX-MODE');
       expect(result.method).toBe('keyword');
       expect(result.confidence).toBeGreaterThan(0);
-      expect(result.matchedKeywords.length).toBeGreaterThan(0);
+      expect(result.matchedKeywords.length).toBe(2);
     });
 
     it('classifies "build a new component" as BUILD intent', () => {
@@ -117,7 +117,7 @@ describe('IntentRouter', () => {
       // "fix bug broken error" has 4 FIX keywords vs anything else
       const result = router.routeIntent('fix bug broken error');
       expect(result.intent).toBe('fix');
-      expect(result.matchedKeywords.length).toBeGreaterThanOrEqual(4);
+      expect(result.matchedKeywords.length).toBe(4);
     });
 
     it('confidence is capped at 1.0', () => {