docs: Added documentation for LLMProviders and named instances

apps/gui/src/app/config/page.tsx

@@ -11,7 +11,7 @@ import {
   setProviderMapping,
   updateProviderInstance,
 } from "@/app/play/actions";
-import type { LLMProviderInstance } from "@/lib/provider-manager";
+import type { LLMProviderInstance } from "@omnia/llm";
 
 interface ConfigStatus {
   apiKeySet: boolean;

apps/gui/src/app/play/actions.ts

@@ -4,8 +4,7 @@ import path from "path";
 import fs from "fs";
 import { simulationManager } from "@/lib/simulation";
 import type { SimSnapshot } from "@/lib/simulation";
-import { ProviderManager } from "@/lib/provider-manager";
-import type { LLMProviderInstance } from "@/lib/provider-manager";
+import { ProviderManager, LLMProviderInstance } from "@omnia/llm";
 
 function resolveScenarioPath(relative: string): string {
   const cwd = process.cwd();

apps/gui/src/components/play/PlayView.tsx

@@ -13,7 +13,7 @@ import {
   listProviderInstances,
 } from "@/app/play/actions";
 import type { SimSnapshot } from "@/lib/simulation-types";
-import type { LLMProviderInstance } from "@/lib/provider-manager";
+import type { LLMProviderInstance } from "@omnia/llm";
 
 function IntentTag({
   intent,

apps/gui/src/lib/provider-manager.ts

@@ -1,197 +0,0 @@
-import Database from "better-sqlite3";
-import path from "path";
-import fs from "fs";
-import type { LLMProviderInstance } from "@omnia/llm";
-
-export type { LLMProviderInstance };
-
-function getSettingsDb() {
-  const dbDir = path.resolve(process.cwd(), "data");
-  if (!fs.existsSync(dbDir)) {
-    fs.mkdirSync(dbDir, { recursive: true });
-  }
-  const dbPath = path.join(dbDir, "settings.db");
-  const db = new Database(dbPath);
-  
-  db.prepare(`
-    CREATE TABLE IF NOT EXISTS provider_instances (
-      id TEXT PRIMARY KEY,
-      name TEXT NOT NULL,
-      providerName TEXT NOT NULL,
-      apiKey TEXT NOT NULL,
-      isActive INTEGER NOT NULL DEFAULT 0,
-      modelName TEXT
-    )
-  `).run();
-
-  try {
-    db.prepare(`ALTER TABLE provider_instances ADD COLUMN modelName TEXT`).run();
-  } catch {
-    // ignore
-  }
-  
-  return db;
-}
-
-export class ProviderManager {
-  static list(): LLMProviderInstance[] {
-    const db = getSettingsDb();
-    try {
-      const rows = db.prepare(`SELECT * FROM provider_instances`).all() as {
-        id: string;
-        name: string;
-        providerName: string;
-        apiKey: string;
-        isActive: number;
-        modelName?: string;
-      }[];
-      return rows.map((r) => ({
-        id: r.id,
-        name: r.name,
-        providerName: r.providerName,
-        apiKey: r.apiKey,
-        isActive: r.isActive === 1,
-        modelName: r.modelName || undefined,
-      }));
-    } finally {
-      db.close();
-    }
-  }
-
-  static create(name: string, providerName: string, apiKey: string, modelName?: string): LLMProviderInstance {
-    const db = getSettingsDb();
-    try {
-      const id = "provider-" + Date.now();
-      const activeCount = db.prepare(`SELECT COUNT(*) as count FROM provider_instances WHERE isActive = 1`).get() as { count: number };
-      const isActive = activeCount.count === 0 ? 1 : 0;
-      
-      db.prepare(`
-        INSERT INTO provider_instances (id, name, providerName, apiKey, isActive, modelName)
-        VALUES (?, ?, ?, ?, ?, ?)
-      `).run(id, name, providerName, apiKey, isActive, modelName || null);
-      
-      return { id, name, providerName, apiKey, isActive: isActive === 1, modelName };
-    } finally {
-      db.close();
-    }
-  }
-
-  static delete(id: string): void {
-    const db = getSettingsDb();
-    try {
-      const provider = db.prepare(`SELECT isActive FROM provider_instances WHERE id = ?`).get(id) as { isActive: number } | undefined;
-      db.prepare(`DELETE FROM provider_instances WHERE id = ?`).run(id);
-      
-      if (provider && provider.isActive === 1) {
-        const next = db.prepare(`SELECT id FROM provider_instances LIMIT 1`).get() as { id: string } | undefined;
-        if (next) {
-          db.prepare(`UPDATE provider_instances SET isActive = 1 WHERE id = ?`).run(next.id);
-        }
-      }
-    } finally {
-      db.close();
-    }
-  }
-
-  static setActive(id: string): void {
-    const db = getSettingsDb();
-    try {
-      db.prepare(`UPDATE provider_instances SET isActive = 0`).run();
-      db.prepare(`UPDATE provider_instances SET isActive = 1 WHERE id = ?`).run(id);
-    } finally {
-      db.close();
-    }
-  }
-
-  static update(id: string, name: string, providerName: string, apiKey?: string, modelName?: string): void {
-    const db = getSettingsDb();
-    try {
-      if (apiKey && apiKey.trim()) {
-        db.prepare(`
-          UPDATE provider_instances
-          SET name = ?, providerName = ?, apiKey = ?, modelName = ?
-          WHERE id = ?
-        `).run(name, providerName, apiKey, modelName || null, id);
-      } else {
-        db.prepare(`
-          UPDATE provider_instances
-          SET name = ?, providerName = ?, modelName = ?
-          WHERE id = ?
-        `).run(name, providerName, modelName || null, id);
-      }
-    } finally {
-      db.close();
-    }
-  }
-
-  static getActive(): LLMProviderInstance | null {
-    const db = getSettingsDb();
-    try {
-      const row = db.prepare(`SELECT * FROM provider_instances WHERE isActive = 1`).get() as {
-        id: string;
-        name: string;
-        providerName: string;
-        apiKey: string;
-        isActive: number;
-        modelName?: string;
-      } | undefined;
-      if (!row) return null;
-      return {
-        id: row.id,
-        name: row.name,
-        providerName: row.providerName,
-        apiKey: row.apiKey,
-        isActive: true,
-        modelName: row.modelName || undefined,
-      };
-    } finally {
-      db.close();
-    }
-  }
-
-  static getMappings(): Record<string, string> {
-    const db = getSettingsDb();
-    try {
-      db.prepare(`
-        CREATE TABLE IF NOT EXISTS provider_mappings (
-          task TEXT PRIMARY KEY,
-          providerInstanceId TEXT NOT NULL
-        )
-      `).run();
-      const rows = db.prepare(`SELECT * FROM provider_mappings`).all() as {
-        task: string;
-        providerInstanceId: string;
-      }[];
-      const mappings: Record<string, string> = {};
-      for (const row of rows) {
-        mappings[row.task] = row.providerInstanceId;
-      }
-      return mappings;
-    } finally {
-      db.close();
-    }
-  }
-
-  static setMapping(task: string, providerInstanceId: string): void {
-    const db = getSettingsDb();
-    try {
-      db.prepare(`
-        CREATE TABLE IF NOT EXISTS provider_mappings (
-          task TEXT PRIMARY KEY,
-          providerInstanceId TEXT NOT NULL
-        )
-      `).run();
-      if (!providerInstanceId) {
-        db.prepare(`DELETE FROM provider_mappings WHERE task = ?`).run(task);
-      } else {
-        db.prepare(`
-          INSERT INTO provider_mappings (task, providerInstanceId)
-          VALUES (?, ?)
-          ON CONFLICT(task) DO UPDATE SET providerInstanceId = excluded.providerInstanceId
-        `).run(task, providerInstanceId);
-      }
-    } finally {
-      db.close();
-    }
-  }
-}

apps/gui/src/lib/simulation.ts

@@ -25,9 +25,8 @@ import {
   IActorProseGenerator,
   buildBufferEntryForIntent,
 } from "@omnia/actor";
-import { GeminiProvider, ILLMProvider, MockLLMProvider } from "@omnia/llm";
+import { GeminiProvider, ILLMProvider, MockLLMProvider, ProviderManager } from "@omnia/llm";
 import { ScenarioLoader } from "@omnia/scenario";
-import { ProviderManager } from "./provider-manager";
 
 import type {
   IntentInfo,