tina4-nodejs 3.13.25 → 3.13.26

package/CLAUDE.md

@@ -1,10 +1,10 @@
-# CLAUDE.md — AI Developer Guide for tina4-nodejs (v3.13.25)
+# CLAUDE.md — AI Developer Guide for tina4-nodejs (v3.13.26)
 
 > 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.13.25 — The Intelligent Native Application 4ramework. A convention-over-configuration structural paradigm. The developer writes TypeScript; Tina4 is invisible infrastructure.
+Tina4 for Node.js/TypeScript v3.13.26 — The Intelligent Native Application 4ramework. A convention-over-configuration structural paradigm. The developer writes TypeScript; Tina4 is invisible infrastructure.
 
 The philosophy: zero ceremony, batteries included, file system as source of truth.
 
@@ -575,7 +575,10 @@ db.delete(table, filter?, params?): DatabaseWriteResult
 db.getLastId(): string | number | null
 db.getError(): string | null
 
-// Transactions — autoCommit defaults to OFF; set TINA4_AUTOCOMMIT=true to enable
+// Transactions — autoCommit defaults to ON: a standalone write commits on its
+// own connection (durable + visible across the pool); inside startTransaction()
+// the per-statement commit is suppressed so the transaction stays atomic. Set
+// TINA4_AUTOCOMMIT=false for strict manual-commit mode.
 db.startTransaction(): void
 db.commit(): void
 db.rollback(): void
@@ -1114,7 +1117,7 @@ When adding new features, add a corresponding `test/<feature>.test.ts` file.
 ## v3 Features Summary
 
 - **45 built-in features**, zero third-party dependencies
-- **3,804 tests** passing across all modules
+- **3,748 tests** passing across all modules
 - **Race-safe `getNextId()`** with atomic sequence table (`tina4_sequences`) for SQLite/MySQL/MSSQL; PostgreSQL auto-creates sequences
 - **Frond template engine optimizations**: pre-compiled regexes, lazy loop context (copy-on-write), filter chain caching, path split caching, inline common filters (11-15% speedup)
 - **Production server auto-detect**: `npx tina4nodejs serve --production` auto-uses cluster mode

package/package.json

@@ -3,7 +3,7 @@
 
 
 
-  "version": "3.13.25",
+  "version": "3.13.26",
 
   "type": "module",
   "description": "Tina4 for Node.js/TypeScript \u2014 54 built-in features, zero dependencies",

package/packages/orm/src/database.ts

@@ -458,8 +458,17 @@ export class Database {
   /** Factory for creating new adapters (used by pool) */
   private adapterFactory: (() => Promise<DatabaseAdapter>) | null = null;
 
-  /** Whether to automatically commit after each write operation */
-  private autoCommit: boolean = process.env.TINA4_AUTOCOMMIT === "true";
+  /**
+   * Whether a standalone write auto-commits. ON by default — a write made
+   * outside an explicit transaction commits on its own connection before
+   * returning (so it's durable and visible across pooled connections). Inside
+   * startTransaction()/commit()/rollback() the per-statement commit is
+   * suppressed, so explicit transactions stay atomic. Set TINA4_AUTOCOMMIT=false
+   * for strict manual-commit mode.
+   */
+  private autoCommit: boolean = ["true", "1", "yes"].includes(
+    (process.env.TINA4_AUTOCOMMIT ?? "true").toLowerCase(),
+  );
   private lastError: string | null = null;
 
   /** Database engine type (sqlite, postgres, mysql, mssql, firebird) */
@@ -675,7 +684,7 @@ export class Database {
     try {
       const adapter = this.getNextAdapter();
       const result = await adapterExecute(adapter, sql, params);
-      if (this.autoCommit) {
+      if (this.autoCommit && !this.inExplicitTransaction()) {
         try { await adapterCommit(adapter); } catch { /* no active transaction */ }
       }
       this.lastError = null;
@@ -697,7 +706,7 @@ export class Database {
     const result = (adapter as any).insertAsync
       ? await (adapter as any).insertAsync(table, data)
       : adapter.insert(table, data);
-    if (this.autoCommit) {
+    if (this.autoCommit && !this.inExplicitTransaction()) {
       try { await adapterCommit(adapter); } catch { /* no active transaction */ }
     }
     return result;
@@ -709,7 +718,7 @@ export class Database {
     const result = (adapter as any).updateAsync
       ? await (adapter as any).updateAsync(table, data, filter ?? {}, params)
       : adapter.update(table, data, filter ?? {}, params);
-    if (this.autoCommit) {
+    if (this.autoCommit && !this.inExplicitTransaction()) {
       try { await adapterCommit(adapter); } catch { /* no active transaction */ }
     }
     return result;
@@ -721,7 +730,7 @@ export class Database {
     const result = (adapter as any).deleteAsync
       ? await (adapter as any).deleteAsync(table, filter ?? {}, params)
       : adapter.delete(table, filter ?? {}, params);
-    if (this.autoCommit) {
+    if (this.autoCommit && !this.inExplicitTransaction()) {
       try { await adapterCommit(adapter); } catch { /* no active transaction */ }
     }
     return result;
@@ -741,6 +750,16 @@ export class Database {
     }
   }
 
+  /**
+   * True while an explicit transaction is active on the current async context.
+   * startTransaction() pins an adapter into txStore; commit()/rollback() clear
+   * it. Standalone writes only auto-commit when this is false, so per-statement
+   * commits never break the atomicity of an explicit transaction.
+   */
+  private inExplicitTransaction(): boolean {
+    return !!this.txStore.getStore()?.adapter;
+  }
+
   /**
    * Start a transaction. Pins the adapter to the current async context for
    * the whole transaction so executes and the final commit/rollback all run