tina4-nodejs 3.13.85 → 3.13.86

package/packages/core/src/session.ts

@@ -2,7 +2,7 @@
  * Tina4 Session — Pluggable session backends, zero core dependencies.
  *
  * File-based sessions by default. Redis backend available via raw TCP (no ioredis needed).
- * Database (SQLite) backend available via better-sqlite3.
+ * Database (SQLite) backend available via Node's built-in node:sqlite.
  *
  *   import { Session, RedisSessionHandler } from "@tina4/core";
  *
@@ -15,7 +15,7 @@
  *     redisPort: 6379,
  *   });
  *
- *   // Database backend (SQLite via better-sqlite3)
+ *   // Database backend (SQLite via node:sqlite)
  *   const session = new Session("database");
  *   // or: new Session("db");
  *

package/packages/core/src/sessionHandlers/databaseHandler.ts

@@ -1,14 +1,13 @@
 /**
- * Tina4 Database Session Handler — SQLite via better-sqlite3, zero extra dependencies.
+ * Tina4 Database Session Handler — SQLite via Node's built-in node:sqlite,
+ * zero extra dependencies.
  *
- * Uses the same `better-sqlite3` library the ORM already depends on.
+ * Uses the same `node:sqlite` (DatabaseSync) the ORM's SQLite adapter uses —
+ * no third-party driver, nothing to install.
  * Stores sessions in a `tina4_session` table with JSON data and expiry.
  *
  * Configure via environment variables:
  *   TINA4_DATABASE_URL  (default: "sqlite:///data/tina4_sessions.db")
- *
- * The handler dynamically imports `better-sqlite3` and throws a clear
- * error if the package is not installed.
  */
 import { DatabaseSync } from "node:sqlite";
 import type { SessionHandler } from "../session.js";
@@ -35,7 +34,7 @@ export interface DatabaseSessionConfig {
 }
 
 /**
- * Database session handler using better-sqlite3 (synchronous SQLite).
+ * Database session handler using node:sqlite (synchronous SQLite).
  *
  * Stores session data as JSON in a `tina4_session` table.
  * Expiry is checked on read; expired rows are cleaned up lazily.

package/packages/frond/dist/index.js

@@ -316,11 +316,14 @@ function resolveVar(expr, context) {
   return value;
 }
 function findOutsideQuotes(expr, needle) {
+  if (!expr.includes(needle)) return -1;
   let inQuote = null;
   let depth = 0;
   let bracketDepth = 0;
   let i = 0;
-  while (i <= expr.length - needle.length) {
+  const needleLen = needle.length;
+  const lastStart = expr.length - needleLen;
+  while (i <= lastStart) {
     const ch = expr[i];
     if ((ch === '"' || ch === "'") && depth === 0 && bracketDepth === 0) {
       if (inQuote === null) {
@@ -339,7 +342,7 @@ function findOutsideQuotes(expr, needle) {
     else if (ch === ")") depth--;
     else if (ch === "[") bracketDepth++;
     else if (ch === "]") bracketDepth--;
-    if (depth === 0 && bracketDepth === 0 && expr.slice(i, i + needle.length) === needle) {
+    if (depth === 0 && bracketDepth === 0 && expr.startsWith(needle, i)) {
       return i;
     }
     i++;
@@ -347,13 +350,16 @@ function findOutsideQuotes(expr, needle) {
   return -1;
 }
 function splitOutsideQuotes(expr, sep) {
+  if (!expr.includes(sep)) return [expr];
   const parts = [];
   let currentStart = 0;
   let inQuote = null;
   let depth = 0;
   let bracketDepth = 0;
   let i = 0;
-  while (i <= expr.length - sep.length) {
+  const sepLen = sep.length;
+  const lastStart = expr.length - sepLen;
+  while (i <= lastStart) {
     const ch = expr[i];
     if ((ch === '"' || ch === "'") && depth === 0 && bracketDepth === 0) {
       if (inQuote === null) {
@@ -372,9 +378,9 @@ function splitOutsideQuotes(expr, sep) {
     else if (ch === ")") depth--;
     else if (ch === "[") bracketDepth++;
     else if (ch === "]") bracketDepth--;
-    if (depth === 0 && bracketDepth === 0 && expr.slice(i, i + sep.length) === sep) {
+    if (depth === 0 && bracketDepth === 0 && expr.startsWith(sep, i)) {
       parts.push(expr.slice(currentStart, i));
-      i += sep.length;
+      i += sepLen;
       currentStart = i;
       continue;
     }

package/packages/frond/src/engine.ts

@@ -444,11 +444,23 @@ function resolveVar(expr: string, context: Record<string, unknown>): unknown {
 }
 
 function findOutsideQuotes(expr: string, needle: string): number {
+  // Fast path. This is the hottest function in a render: profiling the Python
+  // twin showed 415,200 calls and 53% of render time for one 20-row loop
+  // template, and the overwhelming majority return -1 because the needle simply
+  // is not in the expression. includes() is a native scan, so bailing here skips
+  // the whole JS character loop. Exact, not a heuristic: a needle absent from
+  // the string cannot be present outside quotes either.
+  if (!expr.includes(needle)) return -1;
+
   let inQuote: string | null = null;
   let depth = 0;
   let bracketDepth = 0;
   let i = 0;
-  while (i <= expr.length - needle.length) {
+  // Hoisted out of the loop condition -- both lengths were recomputed on every
+  // single iteration.
+  const needleLen = needle.length;
+  const lastStart = expr.length - needleLen;
+  while (i <= lastStart) {
     const ch = expr[i];
     if ((ch === '"' || ch === "'") && depth === 0 && bracketDepth === 0) {
       if (inQuote === null) {
@@ -464,7 +476,9 @@ function findOutsideQuotes(expr: string, needle: string): number {
     else if (ch === ")") depth--;
     else if (ch === "[") bracketDepth++;
     else if (ch === "]") bracketDepth--;
-    if (depth === 0 && bracketDepth === 0 && expr.slice(i, i + needle.length) === needle) {
+    // startsWith(needle, i) rather than slice(i, i + needleLen) === needle: the
+    // slice allocated a throwaway string at every character position.
+    if (depth === 0 && bracketDepth === 0 && expr.startsWith(needle, i)) {
       return i;
     }
     i++;
@@ -473,13 +487,20 @@ function findOutsideQuotes(expr: string, needle: string): number {
 }
 
 function splitOutsideQuotes(expr: string, sep: string): string[] {
+  // Fast path, same reasoning as findOutsideQuotes: no separator anywhere means
+  // no split, and includes() is a native scan versus a JS character loop.
+  if (!expr.includes(sep)) return [expr];
+
   const parts: string[] = [];
   let currentStart = 0;
   let inQuote: string | null = null;
   let depth = 0;
   let bracketDepth = 0;
   let i = 0;
-  while (i <= expr.length - sep.length) {
+  // Hoisted out of the loop condition -- recomputed every iteration before.
+  const sepLen = sep.length;
+  const lastStart = expr.length - sepLen;
+  while (i <= lastStart) {
     const ch = expr[i];
     if ((ch === '"' || ch === "'") && depth === 0 && bracketDepth === 0) {
       if (inQuote === null) {
@@ -495,9 +516,10 @@ function splitOutsideQuotes(expr: string, sep: string): string[] {
     else if (ch === ")") depth--;
     else if (ch === "[") bracketDepth++;
     else if (ch === "]") bracketDepth--;
-    if (depth === 0 && bracketDepth === 0 && expr.slice(i, i + sep.length) === sep) {
+    // startsWith avoids allocating a throwaway slice at every position.
+    if (depth === 0 && bracketDepth === 0 && expr.startsWith(sep, i)) {
       parts.push(expr.slice(currentStart, i));
-      i += sep.length;
+      i += sepLen;
       currentStart = i;
       continue;
     }