outlet-orm 9.0.2 → 11.1.0

package/src/AI/Contracts/ImageProviderContract.js

@@ -11,7 +11,7 @@ class ImageProviderContract {
    * @param {Object} [options={}]
    * @returns {Promise<{images: Array, meta?: Object, raw?: Object}>}
    */
-  async generateImage(prompt, options = {}) {
+  async generateImage(prompt, _options = {}) {
     throw new Error('Not implemented: generateImage()');
   }
 }

package/src/AI/Contracts/ModelsProviderContract.js

@@ -18,7 +18,7 @@ class ModelsProviderContract {
    * @param {string} id
    * @returns {Promise<Object>}
    */
-  async getModel(id) {
+  async getModel(_id) {
     throw new Error('Not implemented: getModel()');
   }
 }

package/src/AI/Contracts/ToolContract.js

@@ -19,7 +19,7 @@ class ToolContract {
    * @param {Object} args
    * @returns {Promise<string>|string}
    */
-  execute(args) { throw new Error('Not implemented: execute()'); }
+  execute(_args) { throw new Error('Not implemented: execute()'); }
 }
 
 module.exports = ToolContract;

package/src/AI/Facades/{AiBridge.js → AI.js}

@@ -1,10 +1,10 @@
 'use strict';
 
 /**
- * AiBridge Facade
+ * AI Facade
  *
- * Convenience entry-point mirroring AiBridge\Facades\AiBridge in PHP.
- * Provides static-like helpers that delegate to an AiBridgeManager instance.
+ * Convenience entry-point for AI operations.
+ * Provides static-like helpers that delegate to an AIManager instance.
  */
 
 const ImageNormalizer      = require('../Support/ImageNormalizer');
@@ -13,10 +13,10 @@ const EmbeddingsNormalizer = require('../Support/EmbeddingsNormalizer');
 
 let _manager = null;
 
-const AiBridge = {
+const AI = {
   /**
-   * Bind an AiBridgeManager instance so all helpers delegate to it.
-   * @param {import('../AiBridgeManager')} manager
+   * Bind an AIManager instance so all helpers delegate to it.
+   * @param {import('../AIManager')} manager
    */
   setManager(manager) {
     _manager = manager;
@@ -24,7 +24,7 @@ const AiBridge = {
 
   /**
    * Return the bound manager (or null).
-   * @returns {import('../AiBridgeManager')|null}
+   * @returns {import('../AIManager')|null}
    */
   getManager() {
     return _manager;
@@ -55,7 +55,7 @@ const AiBridge = {
    * @returns {import('../Builders/TextBuilder')}
    */
   text() {
-    if (!_manager) throw new Error('AiBridge facade: no manager bound. Call AiBridge.setManager(manager) first.');
+    if (!_manager) throw new Error('AI facade: no manager bound. Call AI.setManager(manager) first.');
     return _manager.text();
   },
 
@@ -63,7 +63,7 @@ const AiBridge = {
    * Shorthand for manager.chat()
    */
   async chat(messages, opts) {
-    if (!_manager) throw new Error('AiBridge facade: no manager bound.');
+    if (!_manager) throw new Error('AI facade: no manager bound.');
     return _manager.chat(messages, opts);
   },
 
@@ -71,9 +71,9 @@ const AiBridge = {
    * Shorthand for manager.provider()
    */
   provider(name) {
-    if (!_manager) throw new Error('AiBridge facade: no manager bound.');
+    if (!_manager) throw new Error('AI facade: no manager bound.');
     return _manager.provider(name);
   },
 };
 
-module.exports = AiBridge;
+module.exports = AI;

package/src/AI/MCPServer.js

@@ -115,7 +115,7 @@ const TOOL_DEFINITIONS = [
   },
   {
     name: 'ai_query',
-    description: 'Convert a natural language question into SQL and execute it. Requires an AI provider (AiBridge).',
+    description: 'Convert a natural language question into SQL and execute it. Requires an AI provider.',
     inputSchema: {
       type: 'object',
       properties: {
@@ -442,7 +442,7 @@ class MCPServer extends EventEmitter {
 
   // ── migrate_reset ──────────────────────────────────────────────
 
-  async _toolMigrateReset(args) {
+  async _toolMigrateReset(_args) {
     // Consent already validated in _handleToolCall
     const conn = await this._getConnection();
     const { MigrationManager } = require('../../src');
@@ -517,9 +517,9 @@ class MCPServer extends EventEmitter {
       // Get all tables
       let query;
       if (driver === 'sqlite') {
-        query = "SELECT name FROM sqlite_master WHERE type='table' AND name NOT LIKE 'sqlite_%' ORDER BY name";
+        query = 'SELECT name FROM sqlite_master WHERE type=\'table\' AND name NOT LIKE \'sqlite_%\' ORDER BY name';
       } else if (driver === 'postgres') {
-        query = "SELECT tablename AS name FROM pg_tables WHERE schemaname = 'public' ORDER BY tablename";
+        query = 'SELECT tablename AS name FROM pg_tables WHERE schemaname = \'public\' ORDER BY tablename';
       } else {
         query = 'SHOW TABLES';
       }
@@ -644,8 +644,8 @@ class MCPServer extends EventEmitter {
   async _toolAiQuery(args) {
     if (!args.question) throw new Error('A natural language question is required.');
     const conn = await this._getConnection();
-    const manager = this._getAiBridgeManager();
-    if (!manager) throw new Error('AiBridge is not configured. Set OPENAI_API_KEY or configure a provider.');
+    const manager = this._getAIManager();
+    if (!manager) throw new Error('AI is not configured. Set OPENAI_API_KEY or configure a provider.');
 
     const AIQueryBuilder = require('./AIQueryBuilder');
     const builder = new AIQueryBuilder(manager, conn);
@@ -672,8 +672,8 @@ class MCPServer extends EventEmitter {
   async _toolQueryOptimize(args) {
     if (!args.sql) throw new Error('SQL query is required.');
     const conn = await this._getConnection();
-    const manager = this._getAiBridgeManager();
-    if (!manager) throw new Error('AiBridge is not configured. Set OPENAI_API_KEY or configure a provider.');
+    const manager = this._getAIManager();
+    if (!manager) throw new Error('AI is not configured. Set OPENAI_API_KEY or configure a provider.');
 
     const AIQueryOptimizer = require('./AIQueryOptimizer');
     const optimizer = new AIQueryOptimizer(manager, conn);
@@ -692,17 +692,17 @@ class MCPServer extends EventEmitter {
     };
   }
 
-  // ── AiBridge manager helper ────────────────────────────────────
+  // ── AI manager helper ────────────────────────────────────
 
   /**
-   * Lazily creates an AiBridge manager from environment variables.
-   * @returns {import('./Bridge/AiBridgeManager')|null}
+   * Lazily creates an AI manager from environment variables.
+   * @returns {import('./AIManager')|null}
    */
-  _getAiBridgeManager() {
-    if (this._aiBridgeManager) return this._aiBridgeManager;
+  _getAIManager() {
+    if (this._aiManager) return this._aiManager;
 
     try {
-      const AiBridgeManager = require('./AiBridgeManager');
+      const AIManager = require('./AIManager');
       const config = {};
 
       // Auto-detect providers from env
@@ -716,8 +716,8 @@ class MCPServer extends EventEmitter {
 
       if (Object.keys(config).length === 0) return null;
 
-      this._aiBridgeManager = new AiBridgeManager(config);
-      return this._aiBridgeManager;
+      this._aiManager = new AIManager(config);
+      return this._aiManager;
     } catch {
       return null;
     }

package/src/AI/Providers/CustomOpenAIProvider.js

@@ -1,7 +1,5 @@
 'use strict';
 
-const JsonSchemaValidator = require('../Support/JsonSchemaValidator');
-
 /**
  * CustomOpenAIProvider
  * Fully configurable OpenAI-compatible provider.

package/src/AI/Providers/GeminiProvider.js

@@ -18,7 +18,7 @@ class GeminiProvider {
   /** @private */
   _keyQuery() { return `?key=${this.apiKey}`; }
 
-  async chat(messages, options = {}) {
+  async chat(messages, _options = {}) {
     const userTexts = messages
       .filter(m => (m.role || '') !== 'system')
       .map(m => m.content);
@@ -46,7 +46,7 @@ class GeminiProvider {
 
   supportsStreaming() { return true; } // simulated
 
-  async embeddings(inputs, options = {}) {
+  async embeddings(inputs, _options = {}) {
     const vectors = [];
     for (const input of inputs) {
       const payload = {

package/src/AI/Providers/OpenAIProvider.js

@@ -1,10 +1,5 @@
 'use strict';
 
-const ChatProviderContract = require('../Contracts/ChatProviderContract');
-const EmbeddingsProviderContract = require('../Contracts/EmbeddingsProviderContract');
-const ImageProviderContract = require('../Contracts/ImageProviderContract');
-const AudioProviderContract = require('../Contracts/AudioProviderContract');
-const ModelsProviderContract = require('../Contracts/ModelsProviderContract');
 const JsonSchemaValidator = require('../Support/JsonSchemaValidator');
 const DocumentAttachmentMapper = require('../Support/DocumentAttachmentMapper');
 const fs = require('fs');

package/src/AI/Support/DocumentAttachmentMapper.js

@@ -26,46 +26,46 @@ class DocumentAttachmentMapper {
     for (const att of attachments) {
       if (att instanceof Document) {
         switch (att.kind) {
-          case 'text':
-            if (att.text != null) inlineTexts.push(att.text);
-            break;
-          case 'local':
-            if (att.path && sec.validateFile(att.path)) {
-              const mime = att.mime || 'application/octet-stream';
-              const b64 = fs.readFileSync(att.path).toString('base64');
-              if (mime.startsWith(IMAGE_PREFIX)) {
-                images.push(b64);
-              } else {
-                files.push({ name: path.basename(att.path), type: mime, content: b64 });
-              }
+        case 'text':
+          if (att.text != null) inlineTexts.push(att.text);
+          break;
+        case 'local':
+          if (att.path && sec.validateFile(att.path)) {
+            const mime = att.mime || 'application/octet-stream';
+            const b64 = fs.readFileSync(att.path).toString('base64');
+            if (mime.startsWith(IMAGE_PREFIX)) {
+              images.push(b64);
+            } else {
+              files.push({ name: path.basename(att.path), type: mime, content: b64 });
             }
-            break;
-          case 'base64':
-            if (att.base64 && att.mime) {
-              if (att.mime.startsWith(IMAGE_PREFIX)) {
-                images.push(att.base64);
-              } else {
-                files.push({ name: att.title || 'document', type: att.mime, content: att.base64 });
-              }
+          }
+          break;
+        case 'base64':
+          if (att.base64 && att.mime) {
+            if (att.mime.startsWith(IMAGE_PREFIX)) {
+              images.push(att.base64);
+            } else {
+              files.push({ name: att.title || 'document', type: att.mime, content: att.base64 });
             }
-            break;
-          case 'raw':
-            if (att.raw && att.mime) {
-              const b64 = Buffer.from(att.raw).toString('base64');
-              if (att.mime.startsWith(IMAGE_PREFIX)) {
-                images.push(b64);
-              } else {
-                files.push({ name: att.title || 'document', type: att.mime, content: b64 });
-              }
+          }
+          break;
+        case 'raw':
+          if (att.raw && att.mime) {
+            const b64 = Buffer.from(att.raw).toString('base64');
+            if (att.mime.startsWith(IMAGE_PREFIX)) {
+              images.push(b64);
+            } else {
+              files.push({ name: att.title || 'document', type: att.mime, content: b64 });
             }
-            break;
-          case 'chunks':
-            for (const c of att.chunks) inlineTexts.push(String(c));
-            break;
-          case 'url':
-          case 'file_id':
-          default:
-            break;
+          }
+          break;
+        case 'chunks':
+          for (const c of att.chunks) inlineTexts.push(String(c));
+          break;
+        case 'url':
+        case 'file_id':
+        default:
+          break;
         }
       } else if (typeof att === 'string' && sec.validateFile(att)) {
         const b64 = fs.readFileSync(att).toString('base64');

package/src/AI/Support/FileSecurity.js

@@ -31,7 +31,7 @@ class FileSecurity {
    * @param {boolean} [image=false]
    * @returns {boolean}
    */
-  validateFile(filePath, image = false) {
+  validateFile(filePath, _image = false) {
     try {
       if (!fs.existsSync(filePath)) return false;
       const stats = fs.statSync(filePath);

package/src/AI/Support/ToolChatRunner.js

@@ -8,7 +8,7 @@
  */
 class ToolChatRunner {
   /**
-   * @param {import('../AiBridgeManager')} manager
+   * @param {import('../AIManager')} manager
    */
   constructor(manager) {
     this.manager = manager;

package/src/Backup/BackupManager.js

@@ -40,7 +40,7 @@ function quoteValue(val) {
   if (val === null || val === undefined) return 'NULL';
   if (typeof val === 'number' || typeof val === 'boolean') return String(val);
   // Escape single quotes
-  return `'${String(val).replace(/'/g, "''")}'`;
+  return `'${String(val).replace(/'/g, '\'\'')}'`;
 }
 
 /**
@@ -244,7 +244,7 @@ class BackupManager {
 
     for (const table of tables) {
       lines.push(`\n-- Table: ${table}`);
-      lines.push(`-- -----------------------------------------------------------`);
+      lines.push('-- -----------------------------------------------------------');
 
       // Schema
       const createSql = await this._getCreateTable(table);
@@ -302,13 +302,13 @@ class BackupManager {
     case 'postgres':
     case 'postgresql':
       rows = await this.connection.executeRawQuery(
-        "SELECT tablename AS name FROM pg_tables WHERE schemaname = 'public' ORDER BY tablename"
+        'SELECT tablename AS name FROM pg_tables WHERE schemaname = \'public\' ORDER BY tablename'
       );
       return rows.map((r) => r.name);
 
     case 'sqlite':
       rows = await this.connection.executeRawQuery(
-        "SELECT name FROM sqlite_master WHERE type='table' AND name NOT LIKE 'sqlite_%' ORDER BY name"
+        'SELECT name FROM sqlite_master WHERE type=\'table\' AND name NOT LIKE \'sqlite_%\' ORDER BY name'
       );
       return rows.map((r) => r.name);
 
@@ -382,12 +382,12 @@ class BackupManager {
    */
   _sqlHeader(type) {
     return [
-      `-- outlet-orm backup`,
+      '-- outlet-orm backup',
       `-- type      : ${type}`,
       `-- driver    : ${this.connection.driver}`,
       `-- database  : ${this.connection.config?.database || '(unknown)'}`,
       `-- generated : ${new Date().toISOString()}`,
-      `-- ---------------------------------------------------------------`,
+      '-- ---------------------------------------------------------------',
     ].join('\n');
   }
 

package/src/Backup/BackupScheduler.js

@@ -74,7 +74,7 @@ class BackupScheduler {
       throw new Error('BackupScheduler: intervalMs must be >= 1000 (1 second)');
     }
     if (type === 'partial' && (!Array.isArray(config.tables) || config.tables.length === 0)) {
-      throw new Error("BackupScheduler: 'tables' array is required for partial backups");
+      throw new Error('BackupScheduler: \'tables\' array is required for partial backups');
     }
 
     const name = config.name || `${type}_${Date.now()}`;

package/src/Backup/BackupSocketServer.js

@@ -271,7 +271,7 @@ class BackupSocketServer extends events.EventEmitter {
         filePath = await runManager.full(options);
       } else if (type === 'partial') {
         if (!Array.isArray(tables) || tables.length === 0) {
-          throw new Error("'tables' array is required for partial backup");
+          throw new Error('\'tables\' array is required for partial backup');
         }
         filePath = await runManager.partial(tables, options);
       } else if (type === 'journal') {
@@ -284,7 +284,7 @@ class BackupSocketServer extends events.EventEmitter {
 
     case 'restore': {
       if (!msg.filePath) {
-        throw new Error("'filePath' is required for restore");
+        throw new Error('\'filePath\' is required for restore');
       }
       // Build a restore-capable manager (needs encryptionPassword if file is encrypted)
       const restoreOpts = Object.assign({}, this._managerOptions, {

package/src/DatabaseConnection.js

@@ -750,6 +750,57 @@ class DatabaseConnection {
     return result;
   }
 
+  /**
+   * Execute an aggregate function (SUM, AVG, MIN, MAX) on a column
+   * @param {string} table
+   * @param {string} fn - Aggregate function name (SUM, AVG, MIN, MAX)
+   * @param {string} column
+   * @param {Object} query
+   * @returns {Promise<number>}
+   */
+  async aggregate(table, fn, column, query) {
+    await this.connect();
+    const allowedFns = ['SUM', 'AVG', 'MIN', 'MAX'];
+    const safeFn = fn.toUpperCase();
+    if (!allowedFns.includes(safeFn)) {
+      throw new Error(`Invalid aggregate function: ${fn}`);
+    }
+    const safeTable = sanitizeIdentifier(table);
+    const safeColumn = sanitizeIdentifier(column);
+
+    const { whereClause, params } = this.buildWhereClause(query?.wheres || []);
+    const sql = `SELECT ${safeFn}(${safeColumn}) as result FROM ${safeTable}${whereClause}`;
+    const start = Date.now();
+
+    let result;
+    switch (this.driver) {
+    case 'mysql': {
+      const conn = this._getConnection();
+      const [rows] = await conn.execute(this.convertToDriverPlaceholder(sql), params);
+      result = rows[0].result;
+      break;
+    }
+    case 'postgres':
+    case 'postgresql': {
+      const conn = this._getConnection();
+      const pgResult = await conn.query(this.convertToDriverPlaceholder(sql, 'postgres'), params);
+      result = parseFloat(pgResult.rows[0].result);
+      break;
+    }
+    case 'sqlite':
+      result = await new Promise((resolve, reject) => {
+        this.connection.get(sql, params, (err, row) => {
+          if (err) reject(new Error(err.message || String(err)));
+          else resolve(row.result);
+        });
+      });
+      break;
+    }
+
+    logQuery(sql, params, Date.now() - start);
+    return result != null ? Number(result) : 0;
+  }
+
   /**
    * Execute a raw query and return normalized results.
    * ⚠️  SECURITY WARNING: The `sql` parameter is passed to the database driver without