@objectql/driver-excel 4.0.1 → 4.0.3

package/dist/index.js

@@ -44,34 +44,43 @@ exports.ExcelDriver = void 0;
 /**
  * Excel Driver for ObjectQL (Production-Ready)
  *
- * A driver for ObjectQL that reads and writes data from Excel (.xlsx) files.
- * Each worksheet in the Excel file represents an object type, with the first row
- * containing column headers (field names) and subsequent rows containing data.
+ * A persistent Excel-based driver for ObjectQL that stores data in Excel (.xlsx) files.
+ * Extends MemoryDriver with Excel file persistence using ExcelJS.
  *
- * ✅ Features:
- * - Read data from Excel files
- * - Write data back to Excel files
- * - Multiple sheets support (one sheet per object type)
- * - Full CRUD operations
- * - Query support (filters, sorting, pagination)
- * - Automatic data type handling
- * - Secure: Uses ExcelJS (no known vulnerabilities)
+ * Implements both the legacy Driver interface from @objectql/types and
+ * the standard DriverInterface from @objectstack/spec for compatibility
+ * with the new kernel-based plugin system.
+ *
+ * ✅ Production-ready features:
+ * - Persistent storage with Excel files
+ * - Support for both single-file and file-per-object modes
+ * - One worksheet per table/object (e.g., users sheet, projects sheet)
+ * - Atomic write operations
+ * - Full query support (filters, sorting, pagination) inherited from MemoryDriver
+ * - Human-readable Excel format
  *
  * Use Cases:
  * - Import/export data from Excel spreadsheets
  * - Use Excel as a simple database for prototyping
  * - Data migration from Excel to other databases
  * - Generate reports in Excel format
+ * - Small to medium datasets (< 10k records per object)
  */
-const types_1 = require("@objectql/types");
-const ExcelJS = __importStar(require("exceljs"));
 const fs = __importStar(require("fs"));
 const path = __importStar(require("path"));
+const ExcelJS = __importStar(require("exceljs"));
+const types_1 = require("@objectql/types");
+const driver_memory_1 = require("@objectql/driver-memory");
 /**
  * Excel Driver Implementation
  *
- * Stores ObjectQL documents in Excel worksheets. Each object type is stored
- * in a separate worksheet, with the first row containing column headers.
+ * Extends MemoryDriver with Excel file persistence.
+ * All query and aggregation logic is inherited from MemoryDriver.
+ * Only the persistence layer (load/save) is overridden.
+ *
+ * Stores ObjectQL documents in Excel worksheets with format:
+ * - Single-file mode: All objects as worksheets in one .xlsx file
+ * - File-per-object mode: Each object in a separate .xlsx file
  *
  * Uses ExcelJS library for secure Excel file operations.
  *
@@ -79,42 +88,27 @@ const path = __importStar(require("path"));
  * the standard DriverInterface from @objectstack/spec for compatibility
  * with the new kernel-based plugin system.
  */
-class ExcelDriver {
+class ExcelDriver extends driver_memory_1.MemoryDriver {
     constructor(config) {
-        // Driver metadata (ObjectStack-compatible)
+        // Initialize parent with inherited config properties
+        super({
+            strictMode: config.strictMode,
+            initialData: config.initialData,
+            indexes: config.indexes
+        });
+        // Override driver name and version
         this.name = 'ExcelDriver';
         this.version = '4.0.0';
-        this.supports = {
-            transactions: false,
-            joins: false,
-            fullTextSearch: false,
-            jsonFields: true,
-            arrayFields: true,
-            queryFilters: true,
-            queryAggregations: false,
-            querySorting: true,
-            queryPagination: true,
-            queryWindowFunctions: false,
-            querySubqueries: false
-        };
-        this.config = {
-            autoSave: true,
-            createIfMissing: true,
-            strictMode: false,
-            fileStorageMode: 'single-file',
-            ...config
-        };
         this.filePath = path.resolve(config.filePath);
-        this.fileStorageMode = this.config.fileStorageMode;
-        this.data = new Map();
-        this.idCounters = new Map();
+        this.fileStorageMode = config.fileStorageMode || 'single-file';
+        this.autoSave = config.autoSave !== false;
+        this.createIfMissing = config.createIfMissing !== false;
         this.workbooks = new Map();
+        this.fileLocks = new Map();
         // Initialize workbook for single-file mode
         if (this.fileStorageMode === 'single-file') {
             this.workbook = new ExcelJS.Workbook();
         }
-        // Note: Actual file loading happens in init()
-        // Call init() after construction or use the async create() factory method
     }
     /**
      * Initialize the driver by loading the workbook from file.
@@ -125,55 +119,40 @@ class ExcelDriver {
     }
     /**
      * Connect to the database (for DriverInterface compatibility)
-     * This calls init() to load the workbook.
      */
     async connect() {
         await this.init();
     }
+    /**
+     * Factory method to create and initialize the driver.
+     */
+    static async create(config) {
+        const driver = new ExcelDriver(config);
+        await driver.init();
+        return driver;
+    }
     /**
      * Check database connection health
      */
     async checkHealth() {
         try {
             if (this.fileStorageMode === 'single-file') {
-                // Check if file exists or can be created
-                if (!fs.existsSync(this.filePath)) {
-                    if (!this.config.createIfMissing) {
-                        return false;
-                    }
-                    // Check if directory is writable
-                    const dir = path.dirname(this.filePath);
-                    if (!fs.existsSync(dir)) {
-                        return false;
-                    }
+                if (!fs.existsSync(this.filePath) && !this.createIfMissing) {
+                    return false;
                 }
-                return true;
+                const dir = path.dirname(this.filePath);
+                return fs.existsSync(dir) || this.createIfMissing;
             }
             else {
-                // Check if directory exists or can be created
-                if (!fs.existsSync(this.filePath)) {
-                    if (!this.config.createIfMissing) {
-                        return false;
-                    }
-                }
-                return true;
+                return fs.existsSync(this.filePath) || this.createIfMissing;
             }
         }
-        catch (error) {
+        catch {
             return false;
         }
     }
-    /**
-     * Factory method to create and initialize the driver.
-     */
-    static async create(config) {
-        const driver = new ExcelDriver(config);
-        await driver.init();
-        return driver;
-    }
     /**
      * Load workbook from file or create a new one.
-     * Handles both single-file and file-per-object modes.
      */
     async loadWorkbook() {
         if (this.fileStorageMode === 'single-file') {
@@ -184,40 +163,38 @@ class ExcelDriver {
         }
     }
     /**
-     * Load workbook in single-file mode (all objects in one Excel file).
+     * Load workbook in single-file mode.
      */
     async loadSingleFileWorkbook() {
         this.workbook = new ExcelJS.Workbook();
         if (fs.existsSync(this.filePath)) {
+            await this.acquireLock(this.filePath);
             try {
                 await this.workbook.xlsx.readFile(this.filePath);
                 this.loadDataFromWorkbook();
             }
             catch (error) {
+                this.releaseLock(this.filePath);
                 const errorMessage = error.message;
-                // Provide helpful error messages for common issues
                 let detailedMessage = `Failed to read Excel file: ${this.filePath}`;
                 if (errorMessage.includes('corrupted') || errorMessage.includes('invalid')) {
                     detailedMessage += ' - File may be corrupted or not a valid .xlsx file';
                 }
                 else if (errorMessage.includes('permission') || errorMessage.includes('EACCES')) {
-                    detailedMessage += ' - Permission denied. Check file permissions.';
+                    detailedMessage += ' - Permission denied';
                 }
                 else if (errorMessage.includes('EBUSY')) {
-                    detailedMessage += ' - File is locked by another process. Close it and try again.';
+                    detailedMessage += ' - File is locked by another process';
                 }
                 throw new types_1.ObjectQLError({
                     code: 'FILE_READ_ERROR',
                     message: detailedMessage,
-                    details: {
-                        filePath: this.filePath,
-                        error: errorMessage
-                    }
+                    details: { filePath: this.filePath, error: errorMessage }
                 });
             }
+            this.releaseLock(this.filePath);
         }
-        else if (this.config.createIfMissing) {
-            // Create new empty workbook
+        else if (this.createIfMissing) {
             await this.saveWorkbook();
         }
         else {
@@ -229,12 +206,11 @@ class ExcelDriver {
         }
     }
     /**
-     * Load workbooks in file-per-object mode (each object in separate Excel file).
+     * Load workbooks in file-per-object mode.
      */
     async loadFilePerObjectWorkbooks() {
-        // Ensure directory exists
         if (!fs.existsSync(this.filePath)) {
-            if (this.config.createIfMissing) {
+            if (this.createIfMissing) {
                 fs.mkdirSync(this.filePath, { recursive: true });
             }
             else {
@@ -245,7 +221,6 @@ class ExcelDriver {
                 });
             }
         }
-        // Check if it's actually a directory
         const stats = fs.statSync(this.filePath);
         if (!stats.isDirectory()) {
             throw new types_1.ObjectQLError({
@@ -254,129 +229,82 @@ class ExcelDriver {
                 details: { filePath: this.filePath }
             });
         }
-        // Load all existing .xlsx files in the directory
         const files = fs.readdirSync(this.filePath);
         for (const file of files) {
-            if (file.endsWith('.xlsx') && !file.startsWith('~$')) {
+            if (file.endsWith('.xlsx')) {
                 const objectName = file.replace('.xlsx', '');
                 const filePath = path.join(this.filePath, file);
+                await this.acquireLock(filePath);
                 try {
                     const workbook = new ExcelJS.Workbook();
                     await workbook.xlsx.readFile(filePath);
                     this.workbooks.set(objectName, workbook);
-                    // Load data from first worksheet
-                    const worksheet = workbook.worksheets[0];
-                    if (worksheet) {
-                        this.loadDataFromSingleWorksheet(worksheet, objectName);
-                    }
+                    this.loadDataFromWorkbookForObject(workbook, objectName);
                 }
                 catch (error) {
-                    console.warn(`[ExcelDriver] Warning: Failed to load file ${file}:`, error.message);
+                    console.warn(`[ExcelDriver] Failed to load ${file}:`, error);
                 }
+                this.releaseLock(filePath);
             }
         }
     }
     /**
-     * Load data from workbook into memory.
-     *
-     * Expected Excel format:
-     * - First row contains column headers (field names)
-     * - Subsequent rows contain data records
-     * - Each worksheet represents one object type
+     * Load all data from single-file workbook into memory.
      */
     loadDataFromWorkbook() {
-        this.workbook.eachSheet((worksheet) => {
-            this.loadDataFromSingleWorksheet(worksheet, worksheet.name);
+        this.workbook.worksheets.forEach(worksheet => {
+            const objectName = worksheet.name;
+            const records = this.parseWorksheet(worksheet);
+            for (const record of records) {
+                const id = record.id || record._id;
+                const key = `${objectName}:${id}`;
+                this.store.set(key, record);
+            }
         });
     }
     /**
-     * Load data from a single worksheet into memory.
+     * Load data from workbook for a specific object.
      */
-    loadDataFromSingleWorksheet(worksheet, objectName) {
-        const records = [];
-        // Get headers from first row
-        const headerRow = worksheet.getRow(1);
-        const headers = [];
-        headerRow.eachCell((cell, colNumber) => {
-            const headerValue = cell.value;
-            if (headerValue) {
-                headers[colNumber - 1] = String(headerValue);
-            }
-        });
-        // Warn if worksheet has no headers (might be corrupted or wrong format)
-        if (headers.length === 0 && worksheet.rowCount > 0) {
-            console.warn(`[ExcelDriver] Warning: Worksheet "${objectName}" has no headers in first row. Skipping.`);
-            return;
-        }
-        // Skip first row (headers) and read data rows
-        let rowsProcessed = 0;
-        let rowsSkipped = 0;
-        worksheet.eachRow((row, rowNumber) => {
-            if (rowNumber === 1)
-                return; // Skip header row
-            const record = {};
-            let hasData = false;
-            row.eachCell((cell, colNumber) => {
-                const header = headers[colNumber - 1];
-                if (header) {
-                    record[header] = cell.value;
-                    hasData = true;
-                }
-            });
-            // Skip completely empty rows
-            if (!hasData) {
-                rowsSkipped++;
-                return;
+    loadDataFromWorkbookForObject(workbook, objectName) {
+        workbook.worksheets.forEach(worksheet => {
+            const records = this.parseWorksheet(worksheet);
+            for (const record of records) {
+                const id = record.id || record._id;
+                const key = `${objectName}:${id}`;
+                this.store.set(key, record);
             }
-            // Ensure ID exists
-            if (!record.id) {
-                record.id = this.generateId(objectName);
-            }
-            records.push(record);
-            rowsProcessed++;
         });
-        // Log summary for debugging
-        if (rowsSkipped > 0) {
-            console.warn(`[ExcelDriver] Worksheet "${objectName}": Processed ${rowsProcessed} rows, skipped ${rowsSkipped} empty rows`);
-        }
-        this.data.set(objectName, records);
-        this.updateIdCounter(objectName, records);
     }
     /**
-     * Update ID counter based on existing records.
-     *
-     * Attempts to extract counter from auto-generated IDs (format: objectName-timestamp-counter).
-     * If IDs don't follow this format, counter starts from existing record count.
+     * Parse worksheet into array of records.
      */
-    updateIdCounter(objectName, records) {
-        let maxCounter = 0;
-        for (const record of records) {
-            if (record.id) {
-                // Try to extract counter from generated IDs (format: objectName-timestamp-counter)
-                const idStr = String(record.id);
-                const parts = idStr.split('-');
-                // Only parse if it matches the expected auto-generated format
-                if (parts.length === 3 && parts[0] === objectName && !isNaN(Number(parts[2]))) {
-                    const counter = Number(parts[2]);
-                    if (counter > maxCounter) {
-                        maxCounter = counter;
+    parseWorksheet(worksheet) {
+        const records = [];
+        const headers = [];
+        worksheet.eachRow((row, rowNumber) => {
+            if (rowNumber === 1) {
+                row.eachCell((cell) => {
+                    headers.push(String(cell.value || ''));
+                });
+            }
+            else {
+                const record = {};
+                row.eachCell((cell, colNumber) => {
+                    const header = headers[colNumber - 1];
+                    if (header) {
+                        record[header] = cell.value;
                     }
+                });
+                if (Object.keys(record).length > 0) {
+                    records.push(record);
                 }
             }
-        }
-        // If no auto-generated IDs found, start from record count to avoid collisions
-        if (maxCounter === 0 && records.length > 0) {
-            maxCounter = records.length;
-        }
-        this.idCounters.set(objectName, maxCounter);
+        });
+        return records;
     }
     /**
      * Save workbook to file.
      */
-    /**
-     * Save workbook to file.
-     * Handles both single-file and file-per-object modes.
-     */
     async saveWorkbook(objectName) {
         if (this.fileStorageMode === 'single-file') {
             await this.saveSingleFileWorkbook();
@@ -389,13 +317,12 @@ class ExcelDriver {
      * Save workbook in single-file mode.
      */
     async saveSingleFileWorkbook() {
+        const dir = path.dirname(this.filePath);
+        if (!fs.existsSync(dir)) {
+            fs.mkdirSync(dir, { recursive: true });
+        }
+        await this.acquireLock(this.filePath);
         try {
-            // Ensure directory exists
-            const dir = path.dirname(this.filePath);
-            if (!fs.existsSync(dir)) {
-                fs.mkdirSync(dir, { recursive: true });
-            }
-            // Write workbook to file
             await this.workbook.xlsx.writeFile(this.filePath);
         }
         catch (error) {
@@ -405,6 +332,9 @@ class ExcelDriver {
                 details: { error: error.message }
             });
         }
+        finally {
+            this.releaseLock(this.filePath);
+        }
     }
     /**
      * Save workbook in file-per-object mode.
@@ -418,8 +348,9 @@ class ExcelDriver {
                 details: { objectName }
             });
         }
+        const filePath = path.join(this.filePath, `${objectName}.xlsx`);
+        await this.acquireLock(filePath);
         try {
-            const filePath = path.join(this.filePath, `${objectName}.xlsx`);
             await workbook.xlsx.writeFile(filePath);
         }
         catch (error) {
@@ -429,11 +360,12 @@ class ExcelDriver {
                 details: { objectName, error: error.message }
             });
         }
+        finally {
+            this.releaseLock(filePath);
+        }
     }
     /**
-     * Sync in-memory data to workbook.
-     *
-     * Creates or updates a worksheet for the given object type.
+     * Sync in-memory data to workbook and save.
      */
     async syncToWorkbook(objectName) {
         if (this.fileStorageMode === 'single-file') {
@@ -447,7 +379,7 @@ class ExcelDriver {
      * Sync data to worksheet in single-file mode.
      */
     async syncToSingleFileWorkbook(objectName) {
-        const records = this.data.get(objectName) || [];
+        const records = this.getObjectRecords(objectName);
         // Remove existing worksheet if it exists
         const existingSheet = this.workbook.getWorksheet(objectName);
         if (existingSheet) {
@@ -455,29 +387,8 @@ class ExcelDriver {
         }
         // Create new worksheet
         const worksheet = this.workbook.addWorksheet(objectName);
-        if (records.length > 0) {
-            // Get all unique keys from records to create headers
-            const headers = new Set();
-            records.forEach(record => {
-                Object.keys(record).forEach(key => headers.add(key));
-            });
-            const headerArray = Array.from(headers);
-            // Add header row
-            worksheet.addRow(headerArray);
-            // Add data rows
-            records.forEach(record => {
-                const row = headerArray.map(header => record[header]);
-                worksheet.addRow(row);
-            });
-            // Auto-fit columns
-            worksheet.columns.forEach((column) => {
-                if (column.header) {
-                    column.width = Math.max(10, String(column.header).length + 2);
-                }
-            });
-        }
-        // Auto-save if enabled
-        if (this.config.autoSave) {
+        this.populateWorksheet(worksheet, records);
+        if (this.autoSave) {
             await this.saveWorkbook();
         }
     }
@@ -485,8 +396,7 @@ class ExcelDriver {
      * Sync data to separate file in file-per-object mode.
      */
     async syncToFilePerObjectWorkbook(objectName) {
-        const records = this.data.get(objectName) || [];
-        // Get or create workbook for this object
+        const records = this.getObjectRecords(objectName);
         let workbook = this.workbooks.get(objectName);
         if (!workbook) {
             workbook = new ExcelJS.Workbook();
@@ -496,648 +406,130 @@ class ExcelDriver {
         workbook.worksheets.forEach(ws => workbook.removeWorksheet(ws.id));
         // Create new worksheet
         const worksheet = workbook.addWorksheet(objectName);
-        if (records.length > 0) {
-            // Get all unique keys from records to create headers
-            const headers = new Set();
-            records.forEach(record => {
-                Object.keys(record).forEach(key => headers.add(key));
-            });
-            const headerArray = Array.from(headers);
-            // Add header row
-            worksheet.addRow(headerArray);
-            // Add data rows
-            records.forEach(record => {
-                const row = headerArray.map(header => record[header]);
-                worksheet.addRow(row);
-            });
-            // Auto-fit columns
-            worksheet.columns.forEach((column) => {
-                if (column.header) {
-                    column.width = Math.max(10, String(column.header).length + 2);
-                }
-            });
-        }
-        // Auto-save if enabled
-        if (this.config.autoSave) {
+        this.populateWorksheet(worksheet, records);
+        if (this.autoSave) {
             await this.saveWorkbook(objectName);
         }
     }
     /**
-     * Find multiple records matching the query criteria.
-     */
-    async find(objectName, query = {}, options) {
-        // Normalize query to support both legacy and QueryAST formats
-        const normalizedQuery = this.normalizeQuery(query);
-        let results = this.data.get(objectName) || [];
-        // Return empty array if no data
-        if (results.length === 0) {
-            return [];
-        }
-        // Deep copy to avoid mutations
-        results = results.map(r => ({ ...r }));
-        // Apply filters
-        if (normalizedQuery.filters) {
-            results = this.applyFilters(results, normalizedQuery.filters);
-        }
-        // Apply sorting
-        if (normalizedQuery.sort && Array.isArray(normalizedQuery.sort)) {
-            results = this.applySort(results, normalizedQuery.sort);
-        }
-        // Apply pagination
-        if (normalizedQuery.skip) {
-            results = results.slice(normalizedQuery.skip);
-        }
-        if (normalizedQuery.limit) {
-            results = results.slice(0, normalizedQuery.limit);
-        }
-        // Apply field projection
-        if (normalizedQuery.fields && Array.isArray(normalizedQuery.fields)) {
-            results = results.map(doc => this.projectFields(doc, normalizedQuery.fields));
-        }
-        return results;
-    }
-    /**
-     * Find a single record by ID or query.
+     * Get all records for an object from memory store.
      */
-    async findOne(objectName, id, query, options) {
-        const records = this.data.get(objectName) || [];
-        // If ID is provided, fetch directly
-        if (id) {
-            const record = records.find(r => r.id === String(id));
-            return record ? { ...record } : null;
-        }
-        // If query is provided, use find and return first result
-        if (query) {
-            const results = await this.find(objectName, { ...query, limit: 1 }, options);
-            return results[0] || null;
+    getObjectRecords(objectName) {
+        const records = [];
+        const prefix = `${objectName}:`;
+        for (const [key, value] of this.store.entries()) {
+            if (key.startsWith(prefix)) {
+                records.push(value);
+            }
         }
-        return null;
+        return records;
     }
     /**
-     * Create a new record.
+     * Populate worksheet with records.
      */
-    async create(objectName, data, options) {
-        // Get or create object data array
-        if (!this.data.has(objectName)) {
-            this.data.set(objectName, []);
-        }
-        const records = this.data.get(objectName);
-        // Generate ID if not provided
-        const id = data.id || this.generateId(objectName);
-        // Check if record already exists
-        if (records.some(r => r.id === id)) {
-            throw new types_1.ObjectQLError({
-                code: 'DUPLICATE_RECORD',
-                message: `Record with id '${id}' already exists in '${objectName}'`,
-                details: { objectName, id }
-            });
+    populateWorksheet(worksheet, records) {
+        if (records.length === 0) {
+            return;
         }
-        const now = new Date().toISOString();
-        const doc = {
-            id,
-            ...data,
-            created_at: data.created_at || now,
-            updated_at: data.updated_at || now
-        };
-        records.push(doc);
-        await this.syncToWorkbook(objectName);
-        return { ...doc };
-    }
-    /**
-     * Update an existing record.
-     */
-    async update(objectName, id, data, options) {
-        const records = this.data.get(objectName) || [];
-        const index = records.findIndex(r => r.id === String(id));
-        if (index === -1) {
-            if (this.config.strictMode) {
-                throw new types_1.ObjectQLError({
-                    code: 'RECORD_NOT_FOUND',
-                    message: `Record with id '${id}' not found in '${objectName}'`,
-                    details: { objectName, id }
-                });
+        // Get all unique keys from records to create headers
+        const headers = new Set();
+        records.forEach(record => {
+            Object.keys(record).forEach(key => headers.add(key));
+        });
+        const headerArray = Array.from(headers);
+        // Add header row
+        worksheet.addRow(headerArray);
+        // Add data rows
+        records.forEach(record => {
+            const row = headerArray.map(header => record[header]);
+            worksheet.addRow(row);
+        });
+        // Auto-fit columns
+        worksheet.columns.forEach((column) => {
+            if (column.header) {
+                column.width = Math.max(10, String(column.header).length + 2);
             }
-            return null;
-        }
-        const existing = records[index];
-        const doc = {
-            ...existing,
-            ...data,
-            id: existing.id, // Preserve ID
-            created_at: existing.created_at, // Preserve created_at
-            updated_at: new Date().toISOString()
-        };
-        records[index] = doc;
-        await this.syncToWorkbook(objectName);
-        return { ...doc };
+        });
     }
     /**
-     * Delete a record.
+     * Simple file locking mechanism.
      */
-    async delete(objectName, id, options) {
-        const records = this.data.get(objectName) || [];
-        const index = records.findIndex(r => r.id === String(id));
-        if (index === -1) {
-            if (this.config.strictMode) {
-                throw new types_1.ObjectQLError({
-                    code: 'RECORD_NOT_FOUND',
-                    message: `Record with id '${id}' not found in '${objectName}'`,
-                    details: { objectName, id }
-                });
+    async acquireLock(filePath) {
+        const maxRetries = 10;
+        const retryDelay = 100;
+        for (let i = 0; i < maxRetries; i++) {
+            if (!this.fileLocks.get(filePath)) {
+                this.fileLocks.set(filePath, true);
+                return;
             }
-            return false;
+            await new Promise(resolve => setTimeout(resolve, retryDelay));
         }
-        records.splice(index, 1);
-        await this.syncToWorkbook(objectName);
-        return true;
-    }
-    /**
-     * Count records matching filters.
-     */
-    async count(objectName, filters, options) {
-        const records = this.data.get(objectName) || [];
-        // Extract actual filters from query object if needed
-        let actualFilters = filters;
-        if (filters && !Array.isArray(filters) && filters.filters) {
-            actualFilters = filters.filters;
-        }
-        // If no filters or empty object/array, return total count
-        if (!actualFilters ||
-            (Array.isArray(actualFilters) && actualFilters.length === 0) ||
-            (typeof actualFilters === 'object' && !Array.isArray(actualFilters) && Object.keys(actualFilters).length === 0)) {
-            return records.length;
-        }
-        // Count only records matching filters
-        return records.filter(record => this.matchesFilters(record, actualFilters)).length;
+        throw new types_1.ObjectQLError({
+            code: 'FILE_LOCK_TIMEOUT',
+            message: `Failed to acquire lock for file: ${filePath}`,
+            details: { filePath }
+        });
     }
     /**
-     * Get distinct values for a field.
+     * Release file lock.
      */
-    async distinct(objectName, field, filters, options) {
-        const records = this.data.get(objectName) || [];
-        const values = new Set();
-        for (const record of records) {
-            if (!filters || this.matchesFilters(record, filters)) {
-                const value = record[field];
-                if (value !== undefined && value !== null) {
-                    values.add(value);
-                }
-            }
-        }
-        return Array.from(values);
+    releaseLock(filePath) {
+        this.fileLocks.delete(filePath);
     }
     /**
-     * Create multiple records at once.
+     * Override create to persist to Excel.
      */
-    async createMany(objectName, data, options) {
-        const results = [];
-        for (const item of data) {
-            const result = await this.create(objectName, item, options);
-            results.push(result);
-        }
-        return results;
+    async create(objectName, data, options) {
+        const result = await super.create(objectName, data, options);
+        await this.syncToWorkbook(objectName);
+        return result;
     }
     /**
-     * Update multiple records matching filters.
+     * Override update to persist to Excel.
      */
-    async updateMany(objectName, filters, data, options) {
-        const records = this.data.get(objectName) || [];
-        let count = 0;
-        for (let i = 0; i < records.length; i++) {
-            if (this.matchesFilters(records[i], filters)) {
-                records[i] = {
-                    ...records[i],
-                    ...data,
-                    id: records[i].id, // Preserve ID
-                    created_at: records[i].created_at, // Preserve created_at
-                    updated_at: new Date().toISOString()
-                };
-                count++;
-            }
-        }
-        if (count > 0) {
+    async update(objectName, id, data, options) {
+        const result = await super.update(objectName, id, data, options);
+        if (result) {
             await this.syncToWorkbook(objectName);
         }
-        return { modifiedCount: count };
+        return result;
     }
     /**
-     * Delete multiple records matching filters.
+     * Override delete to persist to Excel.
      */
-    async deleteMany(objectName, filters, options) {
-        const records = this.data.get(objectName) || [];
-        const initialLength = records.length;
-        const filtered = records.filter(record => !this.matchesFilters(record, filters));
-        this.data.set(objectName, filtered);
-        const deletedCount = initialLength - filtered.length;
-        if (deletedCount > 0) {
+    async delete(objectName, id, options) {
+        const result = await super.delete(objectName, id, options);
+        if (result) {
             await this.syncToWorkbook(objectName);
         }
-        return { deletedCount };
+        return result;
     }
     /**
-     * Manually save the workbook to file.
-     */
-    /**
-     * Manually save the workbook to file.
+     * Manually save all data to Excel file(s).
      */
     async save() {
         if (this.fileStorageMode === 'single-file') {
-            await this.saveWorkbook();
-        }
-        else {
-            // Save all object files in file-per-object mode
-            for (const objectName of this.data.keys()) {
-                await this.saveWorkbook(objectName);
+            // Sync all objects to single file (collect unique object names first)
+            const objectNames = new Set();
+            for (const [key] of this.store.entries()) {
+                objectNames.add(key.split(':')[0]);
             }
-        }
-    }
-    /**
-     * Disconnect (flush any pending writes).
-     */
-    async disconnect() {
-        if (this.config.autoSave) {
-            await this.save();
-        }
-    }
-    /**
-     * Execute a query using QueryAST (DriverInterface v4.0 method)
-     *
-     * This method handles all query operations using the standard QueryAST format
-     * from @objectstack/spec. It converts the AST to the legacy query format
-     * and delegates to the existing find() method.
-     *
-     * @param ast - The query AST to execute
-     * @param options - Optional execution options
-     * @returns Query results with value array and count
-     */
-    async executeQuery(ast, options) {
-        var _a;
-        const objectName = ast.object || '';
-        // Convert QueryAST to legacy query format
-        const legacyQuery = {
-            fields: ast.fields,
-            filters: this.convertFilterNodeToLegacy(ast.filters),
-            sort: (_a = ast.sort) === null || _a === void 0 ? void 0 : _a.map((s) => [s.field, s.order]),
-            limit: ast.top,
-            skip: ast.skip,
-        };
-        // Use existing find method
-        const results = await this.find(objectName, legacyQuery, options);
-        return {
-            value: results,
-            count: results.length
-        };
-    }
-    /**
-     * Execute a command (DriverInterface v4.0 method)
-     *
-     * This method handles all mutation operations (create, update, delete)
-     * using a unified command interface.
-     *
-     * @param command - The command to execute
-     * @param options - Optional execution options
-     * @returns Command execution result
-     */
-    async executeCommand(command, options) {
-        try {
-            const cmdOptions = { ...options, ...command.options };
-            switch (command.type) {
-                case 'create':
-                    if (!command.data) {
-                        throw new Error('Create command requires data');
-                    }
-                    const created = await this.create(command.object, command.data, cmdOptions);
-                    return {
-                        success: true,
-                        data: created,
-                        affected: 1
-                    };
-                case 'update':
-                    if (!command.id || !command.data) {
-                        throw new Error('Update command requires id and data');
-                    }
-                    const updated = await this.update(command.object, command.id, command.data, cmdOptions);
-                    return {
-                        success: true,
-                        data: updated,
-                        affected: 1
-                    };
-                case 'delete':
-                    if (!command.id) {
-                        throw new Error('Delete command requires id');
-                    }
-                    await this.delete(command.object, command.id, cmdOptions);
-                    return {
-                        success: true,
-                        affected: 1
-                    };
-                case 'bulkCreate':
-                    if (!command.records || !Array.isArray(command.records)) {
-                        throw new Error('BulkCreate command requires records array');
-                    }
-                    const bulkCreated = [];
-                    for (const record of command.records) {
-                        const created = await this.create(command.object, record, cmdOptions);
-                        bulkCreated.push(created);
-                    }
-                    return {
-                        success: true,
-                        data: bulkCreated,
-                        affected: command.records.length
-                    };
-                case 'bulkUpdate':
-                    if (!command.updates || !Array.isArray(command.updates)) {
-                        throw new Error('BulkUpdate command requires updates array');
-                    }
-                    const updateResults = [];
-                    for (const update of command.updates) {
-                        const result = await this.update(command.object, update.id, update.data, cmdOptions);
-                        updateResults.push(result);
-                    }
-                    return {
-                        success: true,
-                        data: updateResults,
-                        affected: command.updates.length
-                    };
-                case 'bulkDelete':
-                    if (!command.ids || !Array.isArray(command.ids)) {
-                        throw new Error('BulkDelete command requires ids array');
-                    }
-                    let deleted = 0;
-                    for (const id of command.ids) {
-                        const result = await this.delete(command.object, id, cmdOptions);
-                        if (result)
-                            deleted++;
-                    }
-                    return {
-                        success: true,
-                        affected: deleted
-                    };
-                default:
-                    throw new Error(`Unsupported command type: ${command.type}`);
+            for (const objectName of objectNames) {
+                await this.syncToSingleFileWorkbook(objectName);
             }
+            await this.saveWorkbook();
         }
-        catch (error) {
-            return {
-                success: false,
-                affected: 0,
-                error: error.message || 'Unknown error occurred'
-            };
-        }
-    }
-    /**
-     * Execute raw command (for compatibility)
-     *
-     * @param command - Command string or object
-     * @param parameters - Command parameters
-     * @param options - Execution options
-     */
-    async execute(command, parameters, options) {
-        throw new Error('Excel driver does not support raw command execution. Use executeCommand() instead.');
-    }
-    // ========== Helper Methods ==========
-    /**
-     * Convert FilterNode from QueryAST to legacy filter format.
-     *
-     * @param node - The FilterNode to convert
-     * @returns Legacy filter array format
-     */
-    convertFilterNodeToLegacy(node) {
-        if (!node)
-            return undefined;
-        switch (node.type) {
-            case 'comparison':
-                // Convert comparison node to [field, operator, value] format
-                const operator = node.operator || '=';
-                return [[node.field, operator, node.value]];
-            case 'and':
-                // Convert AND node to array with 'and' separator
-                if (!node.children || node.children.length === 0)
-                    return undefined;
-                const andResults = [];
-                for (const child of node.children) {
-                    const converted = this.convertFilterNodeToLegacy(child);
-                    if (converted) {
-                        if (andResults.length > 0) {
-                            andResults.push('and');
-                        }
-                        andResults.push(...(Array.isArray(converted) ? converted : [converted]));
-                    }
-                }
-                return andResults.length > 0 ? andResults : undefined;
-            case 'or':
-                // Convert OR node to array with 'or' separator
-                if (!node.children || node.children.length === 0)
-                    return undefined;
-                const orResults = [];
-                for (const child of node.children) {
-                    const converted = this.convertFilterNodeToLegacy(child);
-                    if (converted) {
-                        if (orResults.length > 0) {
-                            orResults.push('or');
-                        }
-                        orResults.push(...(Array.isArray(converted) ? converted : [converted]));
-                    }
-                }
-                return orResults.length > 0 ? orResults : undefined;
-            case 'not':
-                // NOT is complex - we'll just process the first child for now
-                if (node.children && node.children.length > 0) {
-                    return this.convertFilterNodeToLegacy(node.children[0]);
-                }
-                return undefined;
-            default:
-                return undefined;
-        }
-    }
-    /**
-     * Normalizes query format to support both legacy UnifiedQuery and QueryAST formats.
-     * This ensures backward compatibility while supporting the new @objectstack/spec interface.
-     *
-     * QueryAST format uses 'top' for limit, while UnifiedQuery uses 'limit'.
-     * QueryAST sort is array of {field, order}, while UnifiedQuery is array of [field, order].
-     */
-    normalizeQuery(query) {
-        if (!query)
-            return {};
-        const normalized = { ...query };
-        // Normalize limit/top
-        if (normalized.top !== undefined && normalized.limit === undefined) {
-            normalized.limit = normalized.top;
-        }
-        // Normalize sort format
-        if (normalized.sort && Array.isArray(normalized.sort)) {
-            // Check if it's already in the array format [field, order]
-            const firstSort = normalized.sort[0];
-            if (firstSort && typeof firstSort === 'object' && !Array.isArray(firstSort)) {
-                // Convert from QueryAST format {field, order} to internal format [field, order]
-                normalized.sort = normalized.sort.map((item) => [
-                    item.field,
-                    item.order || item.direction || item.dir || 'asc'
-                ]);
-            }
-        }
-        return normalized;
-    }
-    /**
-     * Apply filters to an array of records (in-memory filtering).
-     */
-    applyFilters(records, filters) {
-        if (!filters || filters.length === 0) {
-            return records;
-        }
-        return records.filter(record => this.matchesFilters(record, filters));
-    }
-    /**
-     * Check if a single record matches the filter conditions.
-     */
-    matchesFilters(record, filters) {
-        if (!filters || filters.length === 0) {
-            return true;
-        }
-        let conditions = [];
-        let operators = [];
-        for (const item of filters) {
-            if (typeof item === 'string') {
-                // Logical operator (and/or)
-                operators.push(item.toLowerCase());
-            }
-            else if (Array.isArray(item)) {
-                const [field, operator, value] = item;
-                // Handle nested filter groups
-                if (typeof field !== 'string') {
-                    // Nested group - recursively evaluate
-                    conditions.push(this.matchesFilters(record, item));
-                }
-                else {
-                    // Single condition
-                    const matches = this.evaluateCondition(record[field], operator, value);
-                    conditions.push(matches);
-                }
-            }
-        }
-        // Combine conditions with operators
-        if (conditions.length === 0) {
-            return true;
-        }
-        let result = conditions[0];
-        for (let i = 0; i < operators.length; i++) {
-            const op = operators[i];
-            const nextCondition = conditions[i + 1];
-            if (op === 'or') {
-                result = result || nextCondition;
-            }
-            else { // 'and' or default
-                result = result && nextCondition;
-            }
-        }
-        return result;
-    }
-    /**
-     * Evaluate a single filter condition.
-     */
-    evaluateCondition(fieldValue, operator, compareValue) {
-        switch (operator) {
-            case '=':
-            case '==':
-                return fieldValue === compareValue;
-            case '!=':
-            case '<>':
-                return fieldValue !== compareValue;
-            case '>':
-                return fieldValue > compareValue;
-            case '>=':
-                return fieldValue >= compareValue;
-            case '<':
-                return fieldValue < compareValue;
-            case '<=':
-                return fieldValue <= compareValue;
-            case 'in':
-                return Array.isArray(compareValue) && compareValue.includes(fieldValue);
-            case 'nin':
-            case 'not in':
-                return Array.isArray(compareValue) && !compareValue.includes(fieldValue);
-            case 'contains':
-            case 'like':
-                return String(fieldValue).toLowerCase().includes(String(compareValue).toLowerCase());
-            case 'startswith':
-            case 'starts_with':
-                return String(fieldValue).toLowerCase().startsWith(String(compareValue).toLowerCase());
-            case 'endswith':
-            case 'ends_with':
-                return String(fieldValue).toLowerCase().endsWith(String(compareValue).toLowerCase());
-            case 'between':
-                return Array.isArray(compareValue) &&
-                    fieldValue >= compareValue[0] &&
-                    fieldValue <= compareValue[1];
-            default:
-                throw new types_1.ObjectQLError({
-                    code: 'UNSUPPORTED_OPERATOR',
-                    message: `[ExcelDriver] Unsupported operator: ${operator}`,
-                });
-        }
-    }
-    /**
-     * Apply sorting to an array of records.
-     */
-    applySort(records, sort) {
-        const sorted = [...records];
-        // Apply sorts in reverse order for correct precedence
-        for (let i = sort.length - 1; i >= 0; i--) {
-            const sortItem = sort[i];
-            let field;
-            let direction;
-            if (Array.isArray(sortItem)) {
-                [field, direction] = sortItem;
-            }
-            else if (typeof sortItem === 'object') {
-                field = sortItem.field;
-                direction = sortItem.order || sortItem.direction || sortItem.dir || 'asc';
-            }
-            else {
-                continue;
+        else {
+            // Save all object files
+            const objectNames = new Set();
+            for (const [key] of this.store.entries()) {
+                objectNames.add(key.split(':')[0]);
             }
-            sorted.sort((a, b) => {
-                const aVal = a[field];
-                const bVal = b[field];
-                // Handle null/undefined
-                if (aVal == null && bVal == null)
-                    return 0;
-                if (aVal == null)
-                    return 1;
-                if (bVal == null)
-                    return -1;
-                // Compare values
-                if (aVal < bVal)
-                    return direction === 'asc' ? -1 : 1;
-                if (aVal > bVal)
-                    return direction === 'asc' ? 1 : -1;
-                return 0;
-            });
-        }
-        return sorted;
-    }
-    /**
-     * Project specific fields from a document.
-     */
-    projectFields(doc, fields) {
-        const result = {};
-        for (const field of fields) {
-            if (doc[field] !== undefined) {
-                result[field] = doc[field];
+            for (const objectName of objectNames) {
+                await this.syncToFilePerObjectWorkbook(objectName);
+                await this.saveWorkbook(objectName);
             }
         }
-        return result;
-    }
-    /**
-     * Generate a unique ID for a record.
-     */
-    generateId(objectName) {
-        const counter = (this.idCounters.get(objectName) || 0) + 1;
-        this.idCounters.set(objectName, counter);
-        // Use timestamp + counter for better uniqueness
-        const timestamp = Date.now();
-        return `${objectName}-${timestamp}-${counter}`;
     }
 }
 exports.ExcelDriver = ExcelDriver;