@soleri/core 9.2.0 → 9.3.0

package/src/vault/linking.test.ts

@@ -59,9 +59,7 @@ class LinkingMockDB implements PersistenceProvider {
     }
     if (sql.includes('DELETE FROM vault_links')) {
       const before = this.links.length;
-      this.links = this.links.filter(
-        (l) => !(l.source_id === p[0] && l.target_id === p[1]),
-      );
+      this.links = this.links.filter((l) => !(l.source_id === p[0] && l.target_id === p[1]));
       return { changes: before - this.links.length, lastInsertRowid: 0 };
     }
     return { changes: 0, lastInsertRowid: 0 };
@@ -71,9 +69,7 @@ class LinkingMockDB implements PersistenceProvider {
     const p = params ?? [];
     if (sql.includes('COUNT(*)')) {
       const id = p[0] as string;
-      const count = this.links.filter(
-        (l) => l.source_id === id || l.target_id === id,
-      ).length;
+      const count = this.links.filter((l) => l.source_id === id || l.target_id === id).length;
       return { count } as T;
     }
     if (sql.includes('FROM entries WHERE id')) {
@@ -83,7 +79,12 @@ class LinkingMockDB implements PersistenceProvider {
     if (sql.includes('title, description, type, tags')) {
       const entry = this.entries.find((e) => e.id === p[0]);
       if (!entry) return undefined;
-      return { title: entry.title, description: entry.description, type: entry.type, tags: entry.tags } as T;
+      return {
+        title: entry.title,
+        description: entry.description,
+        type: entry.type,
+        tags: entry.tags,
+      } as T;
     }
     return undefined;
   }
@@ -99,9 +100,7 @@ class LinkingMockDB implements PersistenceProvider {
     if (sql.includes('source_id IN')) {
       const half = p.length / 2;
       const ids = new Set(p.slice(0, half) as string[]);
-      return this.links.filter(
-        (l) => ids.has(l.source_id) || ids.has(l.target_id),
-      ) as T[];
+      return this.links.filter((l) => ids.has(l.source_id) || ids.has(l.target_id)) as T[];
     }
     if (sql.includes('NOT IN')) {
       const limit = p[0] as number;
@@ -125,8 +124,12 @@ class LinkingMockDB implements PersistenceProvider {
     return [];
   }
 
-  transaction<T>(fn: () => T): T { return fn(); }
-  ftsSearch<T>(): T[] { return []; }
+  transaction<T>(fn: () => T): T {
+    return fn();
+  }
+  ftsSearch<T>(): T[] {
+    return [];
+  }
   ftsRebuild(): void {}
   close(): void {}
 }
@@ -200,7 +203,9 @@ describe('LinkManager', () => {
   // ── traverse ────────────────────────────────────────────────────────
 
   it('returns empty for entry with no links', () => {
-    db.seedEntries([{ id: 'solo', title: 'Solo', type: 'pattern', domain: 'd', description: '', tags: '' }]);
+    db.seedEntries([
+      { id: 'solo', title: 'Solo', type: 'pattern', domain: 'd', description: '', tags: '' },
+    ]);
     expect(mgr.traverse('solo')).toEqual([]);
   });
 
@@ -293,8 +298,22 @@ describe('LinkManager', () => {
 
   it('suggests links based on FTS matches', () => {
     db.seedEntries([
-      { id: 'e1', title: 'Accessibility Pattern', type: 'pattern', domain: 'a11y', description: 'Screen reader support', tags: '' },
-      { id: 'e2', title: 'ARIA Rules', type: 'rule', domain: 'a11y', description: 'Always use ARIA labels', tags: '' },
+      {
+        id: 'e1',
+        title: 'Accessibility Pattern',
+        type: 'pattern',
+        domain: 'a11y',
+        description: 'Screen reader support',
+        tags: '',
+      },
+      {
+        id: 'e2',
+        title: 'ARIA Rules',
+        type: 'rule',
+        domain: 'a11y',
+        description: 'Always use ARIA labels',
+        tags: '',
+      },
     ]);
     const suggestions = mgr.suggestLinks('e1', 5);
     // e2 should be suggested (e1 is filtered as self)
@@ -329,8 +348,22 @@ describe('LinkManager', () => {
 
   it('dry run populates preview array', () => {
     db.seedEntries([
-      { id: 'orphan', title: 'Orphan', type: 'pattern', domain: 'd', description: 'test content', tags: '' },
-      { id: 'target', title: 'Target', type: 'rule', domain: 'd', description: 'test content', tags: '' },
+      {
+        id: 'orphan',
+        title: 'Orphan',
+        type: 'pattern',
+        domain: 'd',
+        description: 'test content',
+        tags: '',
+      },
+      {
+        id: 'target',
+        title: 'Target',
+        type: 'rule',
+        domain: 'd',
+        description: 'test content',
+        tags: '',
+      },
     ]);
     // Link target so orphan is the only orphan
     mgr.addLink('target', 'target', 'supports'); // self-link to exclude from orphans

package/src/vault/linking.ts

@@ -21,9 +21,35 @@ import type {
 
 // ── Stop words for keyword extraction ─────────────────────────────────
 const STOP_WORDS = new Set([
-  'the', 'and', 'for', 'with', 'from', 'this', 'that', 'are', 'was', 'not',
-  'but', 'have', 'has', 'use', 'can', 'will', 'all', 'each', 'than', 'its',
-  'more', 'when', 'into', 'also', 'any', 'may', 'only', 'should', 'which',
+  'the',
+  'and',
+  'for',
+  'with',
+  'from',
+  'this',
+  'that',
+  'are',
+  'was',
+  'not',
+  'but',
+  'have',
+  'has',
+  'use',
+  'can',
+  'will',
+  'all',
+  'each',
+  'than',
+  'its',
+  'more',
+  'when',
+  'into',
+  'also',
+  'any',
+  'may',
+  'only',
+  'should',
+  'which',
 ]);
 
 export class LinkManager {
@@ -165,7 +191,12 @@ export class LinkManager {
     nextFrontier.push(neighborId);
     const entry = this.getEntryMeta(neighborId);
     if (!entry) return;
-    result.push({ ...entry, linkType: link.linkType, linkDirection: direction, linkNote: link.note });
+    result.push({
+      ...entry,
+      linkType: link.linkType,
+      linkDirection: direction,
+      linkNote: link.note,
+    });
   }
 
   // ── Bulk Queries ────────────────────────────────────────────────────
@@ -239,7 +270,11 @@ export class LinkManager {
   ): Array<{ id: string; title: string; type: string; domain: string; rank: number }> {
     const queryTerms = keywords.join(' OR ');
     return this.provider.all<{
-      id: string; title: string; type: string; domain: string; rank: number;
+      id: string;
+      title: string;
+      type: string;
+      domain: string;
+      rank: number;
     }>(
       `SELECT e.id, e.title, e.type, e.domain, rank
        FROM entries_fts fts
@@ -285,7 +320,8 @@ export class LinkManager {
     const orphans = this.getOrphans(10000);
     let processed = 0;
     let linksCreated = 0;
-    const preview: Array<{ sourceId: string; targetId: string; linkType: string; score: number }> = [];
+    const preview: Array<{ sourceId: string; targetId: string; linkType: string; score: number }> =
+      [];
 
     for (let i = 0; i < orphans.length; i += batchSize) {
       const batch = orphans.slice(i, i + batchSize);
@@ -317,7 +353,12 @@ export class LinkManager {
     const qualifying = suggestions.filter((s) => s.score >= threshold).slice(0, maxLinks);
     for (const s of qualifying) {
       if (dryRun) {
-        preview.push({ sourceId: entryId, targetId: s.entryId, linkType: s.suggestedType, score: s.score });
+        preview.push({
+          sourceId: entryId,
+          targetId: s.entryId,
+          linkType: s.suggestedType,
+          score: s.score,
+        });
       } else {
         this.addLink(entryId, s.entryId, s.suggestedType);
       }

package/src/vault/obsidian-sync.test.ts

@@ -278,9 +278,12 @@ describe('ObsidianSync', () => {
     const sync = new ObsidianSync({ vault: mockVault });
     const result = sync.import(obsidianDir);
     expect(result.updated).toBe(1);
-    expect(mockVault.update).toHaveBeenCalledWith('existing', expect.objectContaining({
-      title: 'Updated Title',
-    }));
+    expect(mockVault.update).toHaveBeenCalledWith(
+      'existing',
+      expect.objectContaining({
+        title: 'Updated Title',
+      }),
+    );
   });
 
   it('skips files without title or description', () => {

package/src/vault/scope-detector.test.ts

@@ -90,9 +90,7 @@ describe('detectScope', () => {
   });
 
   it('detects agent tier for home directory paths', () => {
-    const result = detectScope(
-      makeInput({ description: 'Config lives in ~/dotfiles/zshrc' }),
-    );
+    const result = detectScope(makeInput({ description: 'Config lives in ~/dotfiles/zshrc' }));
     expect(result.tier).toBe('agent');
   });
 

package/src/vault/vault-branching.test.ts

@@ -98,7 +98,9 @@ class BranchingMockDB implements PersistenceProvider {
     return fn();
   }
 
-  ftsSearch<T>(): T[] { return []; }
+  ftsSearch<T>(): T[] {
+    return [];
+  }
   ftsRebuild(): void {}
   close(): void {}
 
@@ -135,7 +137,7 @@ describe('VaultBranching', () => {
 
   it('throws when creating duplicate active branch', () => {
     branching.branch('feature-x');
-    expect(() => branching.branch('feature-x')).toThrow("already exists");
+    expect(() => branching.branch('feature-x')).toThrow('already exists');
   });
 
   // ── addOperation ────────────────────────────────────────────────────
@@ -159,14 +161,14 @@ describe('VaultBranching', () => {
 
   it('throws when adding to non-existent branch', () => {
     expect(() => branching.addOperation('missing', 'e1', 'add', makeEntry('e1'))).toThrow(
-      "does not exist",
+      'does not exist',
     );
   });
 
   it('throws when add/modify has no entry data', () => {
     branching.branch('b1');
-    expect(() => branching.addOperation('b1', 'e1', 'add')).toThrow("Entry data required");
-    expect(() => branching.addOperation('b1', 'e1', 'modify')).toThrow("Entry data required");
+    expect(() => branching.addOperation('b1', 'e1', 'add')).toThrow('Entry data required');
+    expect(() => branching.addOperation('b1', 'e1', 'modify')).toThrow('Entry data required');
   });
 
   // ── listEntries ─────────────────────────────────────────────────────
@@ -226,13 +228,13 @@ describe('VaultBranching', () => {
   });
 
   it('throws when merging non-existent branch', () => {
-    expect(() => branching.merge('missing')).toThrow("does not exist");
+    expect(() => branching.merge('missing')).toThrow('does not exist');
   });
 
   it('throws when merging already-merged branch', () => {
     branching.branch('b1');
     branching.merge('b1');
-    expect(() => branching.merge('b1')).toThrow("already merged");
+    expect(() => branching.merge('b1')).toThrow('already merged');
   });
 
   // ── deleteBranch ────────────────────────────────────────────────────