tina4-nodejs 3.10.12 → 3.10.14

package/CLAUDE.md

@@ -1,10 +1,10 @@
-# CLAUDE.md — AI Developer Guide for tina4-nodejs (v3.10.12)
+# CLAUDE.md — AI Developer Guide for tina4-nodejs (v3.10.14)
 
 > This file helps AI assistants (Claude, Copilot, Cursor, etc.) understand and work on this codebase effectively.
 
 ## What This Project Is
 
-Tina4 for Node.js/TypeScript v3.10.12 — a convention-over-configuration structural paradigm. **Not a framework.** The developer writes TypeScript; Tina4 is invisible infrastructure.
+Tina4 for Node.js/TypeScript v3.10.14 — a convention-over-configuration structural paradigm. **Not a framework.** The developer writes TypeScript; Tina4 is invisible infrastructure.
 
 The philosophy: zero ceremony, batteries included, file system as source of truth.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tina4-nodejs",
-  "version": "3.10.12",
+  "version": "3.10.14",
   "type": "module",
   "description": "This is not a framework. Tina4 for Node.js/TypeScript — zero deps, 38 built-in features.",
   "keywords": ["tina4", "framework", "web", "api", "orm", "graphql", "websocket", "typescript"],

package/packages/orm/src/baseModel.ts

@@ -266,6 +266,7 @@ export class BaseModel {
         this[pk] = result.lastInsertRowid;
       }
     }
+    db.commit();
   }
 
   /**
@@ -294,6 +295,7 @@ export class BaseModel {
         [pkValue],
       );
     }
+    db.commit();
   }
 
   /**
@@ -459,6 +461,7 @@ export class BaseModel {
       const sql = `CREATE TABLE IF NOT EXISTS "${this.tableName}" (${colDefs.join(", ")})`;
       db.execute(sql);
     }
+    db.commit();
   }
 
   /**
@@ -505,6 +508,7 @@ export class BaseModel {
       `DELETE FROM "${ModelClass.tableName}" WHERE "${pkCol}" = ?`,
       [pkValue],
     );
+    db.commit();
   }
 
   /**
@@ -529,6 +533,7 @@ export class BaseModel {
       `UPDATE "${ModelClass.tableName}" SET is_deleted = 0 WHERE "${pkCol}" = ?`,
       [pkValue],
     );
+    db.commit();
     this.is_deleted = 0;
   }
 

package/packages/orm/src/database.ts

@@ -208,6 +208,9 @@ export class Database {
   /** Whether to automatically commit after each write operation */
   private autoCommit: boolean = process.env.TINA4_AUTOCOMMIT === "true";
 
+  /** Database engine type (sqlite, postgres, mysql, mssql, firebird) */
+  private dbType: string = "sqlite";
+
   /**
    * Create a Database wrapping an existing adapter.
    * For creating a Database from a URL, use the async static factories:
@@ -227,6 +230,8 @@ export class Database {
    * @param pool - Number of pooled connections (0 = single, N>0 = round-robin)
    */
   static async create(url: string, username?: string, password?: string, pool: number = 0): Promise<Database> {
+    const parsed = parseDatabaseUrl(url, username, password);
+
     if (pool > 0) {
       // Pooled mode — create all adapters eagerly
       const adapters: DatabaseAdapter[] = [];
@@ -243,13 +248,16 @@ export class Database {
       db.poolIndex = 0;
       db.adapter = null;  // Don't use single-adapter path
       db.adapterFactory = () => createAdapterFromUrl(url, username, password);
+      db.dbType = parsed.type;
       return db;
     }
 
     // Single-connection mode — current behavior
     const adapter = await createAdapterFromUrl(url, username, password);
     setAdapter(adapter);
-    return new Database(adapter);
+    const db = new Database(adapter);
+    db.dbType = parsed.type;
+    return db;
   }
 
   /**
@@ -391,6 +399,55 @@ export class Database {
     if (id === null) return 0;
     return typeof id === "bigint" ? id.toString() : id;
   }
+
+  /**
+   * Pre-generate the next available primary key ID using engine-aware strategies.
+   *
+   * - Firebird: auto-creates a generator if missing, then increments via GEN_ID.
+   * - PostgreSQL: tries nextval() on the standard sequence, falls through to MAX+1.
+   * - SQLite/MySQL/MSSQL: uses MAX(pk) + 1.
+   * - Returns 1 if the table is empty or does not exist.
+   */
+  getNextId(table: string, pkColumn = "id", generatorName?: string): number {
+    const adapter = this.getNextAdapter();
+
+    // Firebird — use generators
+    if (this.dbType === "firebird") {
+      const genName = generatorName ?? `GEN_${table.toUpperCase()}_ID`;
+
+      // Auto-create the generator if it does not exist
+      try {
+        adapter.execute(`CREATE GENERATOR ${genName}`);
+      } catch {
+        // Generator already exists — ignore
+      }
+
+      const row = adapter.fetchOne<Record<string, unknown>>(`SELECT GEN_ID(${genName}, 1) AS NEXT_ID FROM RDB$DATABASE`);
+      return Number(row?.NEXT_ID ?? row?.next_id ?? 1);
+    }
+
+    // PostgreSQL — try sequence first, fall through to MAX
+    if (this.dbType === "postgres") {
+      const seqName = `${table.toLowerCase()}_${pkColumn.toLowerCase()}_seq`;
+      try {
+        const row = adapter.fetchOne<Record<string, unknown>>(`SELECT nextval('${seqName}') AS next_id`);
+        if (row?.next_id != null) {
+          return Number(row.next_id);
+        }
+      } catch {
+        // No sequence — fall through to MAX
+      }
+    }
+
+    // SQLite / MySQL / MSSQL / PostgreSQL fallback — MAX + 1
+    try {
+      const row = adapter.fetchOne<Record<string, unknown>>(`SELECT MAX(${pkColumn}) + 1 AS next_id FROM ${table}`);
+      const nextId = row?.next_id;
+      return nextId != null ? Number(nextId) : 1;
+    } catch {
+      return 1;
+    }
+  }
 }
 
 /**