@zerotal/orm 1.15.1 → 1.15.2

package/CHANGELOG.md

@@ -8,6 +8,25 @@ follows the Zerotal monorepo's unified versioning.
 
 ## [Unreleased]
 
+## [1.15.2] — 2026-10-23
+
+### Fixed
+
+- **The ORM runs on Bun 1.4.4.** Every statement it assembled from a runtime string — the
+  string form of `DB.raw()`, the query builder, schema and migration DDL, the `SAVEPOINT`
+  a nested transaction issues, the validator's `unique` and `exists` rules — reached Bun's
+  `sql` function as a template object built with `Object.assign(parts, { raw: parts })`,
+  whose `raw` is enumerable. Bun 1.4.4 refuses that ("has an enumerable 'raw' property, so
+  it is not a template object"), so on that version and later none of those statements ran:
+  the package's own suite failed 288 tests and then hung. Earlier Bun accepted it, which is
+  why nothing noticed until the CI job that runs on the latest Bun did.
+
+  All of them now take the object from one helper, `_templateStrings()`, which defines
+  `raw` the way a real template object has it and freezes the result. `@zerotal/notifications`
+  and `@zerotal/testing` had built the same object by hand and use it too. One test runs the
+  ORM's paths through a connection that rejects an enumerable `raw`; another fails if any
+  package's source builds one again.
+
 ## [1.15.1] — 2026-09-13
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zerotal/orm",
-  "version": "1.15.1",
+  "version": "1.15.2",
   "license": "MIT",
   "maturity": "stable",
   "private": false,
@@ -31,8 +31,8 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@zerotal/core": "1.15.1",
-    "@zerotal/validator": "1.15.1"
+    "@zerotal/core": "1.15.2",
+    "@zerotal/validator": "1.15.2"
   },
   "devDependencies": {
     "typescript": "^5.8.0"

package/src/db/DB.ts

@@ -17,6 +17,7 @@ import {
   allowNPlusOne as _allowNPlusOne,
   type NPlusOneOptions,
 } from "./NPlusOneDetector.ts";
+import { _templateStrings } from "./templateStrings.ts";
 
 // Test-override escape hatch — set by _setDbConnection() in test files.
 // Production code resolves the connection from the container instead.
@@ -140,7 +141,7 @@ let _savepointCounter = 0;
 
 /** Run a raw, parameter-less statement on a specific connection (SAVEPOINT, etc.). */
 function _runRaw(conn: SQLInstance, sql: string): Promise<unknown> {
-  const arr = Object.assign([sql], { raw: [sql] }) as unknown as TemplateStringsArray;
+  const arr = _templateStrings([sql]);
   return conn(arr);
 }
 
@@ -160,7 +161,7 @@ export function _rawStatement(sql: string): Promise<unknown> {
 /** Run a parameterised statement (`?` placeholders) on a specific connection. */
 function _runParams(conn: SQLInstance, sql: string, params: unknown[]): Promise<unknown> {
   const parts = sql.split("?");
-  const tpl = Object.assign(parts, { raw: parts }) as unknown as TemplateStringsArray;
+  const tpl = _templateStrings(parts);
   return conn(tpl, ...params);
 }
 
@@ -246,7 +247,7 @@ export const DB = {
     if (typeof sql === "string") {
       const bindings: unknown[] = Array.isArray(rest[0]) ? (rest[0] as unknown[]) : rest;
       const parts = sql.split("?");
-      const tpl = Object.assign(parts, { raw: parts }) as unknown as TemplateStringsArray;
+      const tpl = _templateStrings(parts);
       const rows = await conn<T>(tpl, ...bindings);
       FrameworkEvents.emit(
         new QueryExecuted(

package/src/db/QueryBuilder.ts

@@ -20,6 +20,7 @@ import type {
   SimplePaginateResult,
 } from "./types.ts";
 import { withPaginationHelpers, withSimplePaginationHelpers } from "./types.ts";
+import { _templateStrings } from "./templateStrings.ts";
 
 /**
  * The comparison operators accepted in a value-comparison slot. Exported so
@@ -204,8 +205,7 @@ function _getCachedTemplate(strings: string[]): TemplateStringsArray {
   const key = strings.join("\x00");
   let tpl = _tplCache.get(key);
   if (!tpl) {
-    const arr = [...strings];
-    tpl = Object.assign(arr, { raw: arr }) as TemplateStringsArray;
+    tpl = _templateStrings(strings);
     if (_tplCache.size >= _TPL_CACHE_MAX) {
       // FIFO eviction: drop the oldest interned shape.
       const oldest = _tplCache.keys().next().value;

package/src/db/templateStrings.ts

@@ -0,0 +1,19 @@
+/**
+ * Build the `strings` argument for a Bun SQL tagged-template call from plain strings.
+ *
+ * Bun's `sql` function wants what the language hands a tag: a frozen array whose `raw`
+ * is a property you cannot see when you enumerate it. `Object.assign(parts, { raw: parts })`
+ * makes the property enumerable, and Bun 1.4.4 refuses that with "has an enumerable 'raw'
+ * property, so it is not a template object" — every statement built from a runtime-assembled
+ * string failed, which is nearly everything the ORM sends. `defineProperty` leaves `raw`
+ * non-enumerable, as a real template object has it.
+ *
+ * Copies its input, so the caller's array is neither frozen nor given a `raw`.
+ *
+ * @internal Shared by the ORM and by the packages that issue SQL of their own.
+ */
+export function _templateStrings(parts: readonly string[]): TemplateStringsArray {
+  const strings = [...parts];
+  Object.defineProperty(strings, "raw", { value: strings });
+  return Object.freeze(strings) as TemplateStringsArray;
+}

package/src/index.ts

@@ -97,6 +97,7 @@ export type { InsertPayload, UpdatePayload } from "./model/payload.ts";
 export { ModelQueryBuilder, _globalScopeRegistry } from "./model/ModelQueryBuilder.ts";
 export type { GlobalScopeCallback, RelationConstraint } from "./model/ModelQueryBuilder.ts";
 export { DB } from "./db/DB.ts";
+export { _templateStrings } from "./db/templateStrings.ts";
 export {
   _getConnection,
   _setDbConnection,

package/src/provider/DatabaseProvider.ts

@@ -25,6 +25,7 @@ import {
   RUN_MIGRATIONS_PATH,
   _mintDiagnosisToken,
 } from "../diagnostics/runMigrationsEndpoint.ts";
+import { _templateStrings } from "../db/templateStrings.ts";
 
 // Extend the core container registry so 'db' is a typed binding.
 declare module "@zerotal/core" {
@@ -171,27 +172,17 @@ export class DatabaseProvider extends ServiceProvider {
           let rows: unknown[];
           if (ignoreId !== undefined) {
             const strs = [`SELECT 1 FROM ${table} WHERE ${column} = `, ` AND id != `, ` LIMIT 1`];
-            rows = await conn(
-              Object.assign(strs, { raw: strs }) as unknown as TemplateStringsArray,
-              value,
-              ignoreId,
-            );
+            rows = await conn(_templateStrings(strs), value, ignoreId);
           } else {
             const strs = [`SELECT 1 FROM ${table} WHERE ${column} = `, ` LIMIT 1`];
-            rows = await conn(
-              Object.assign(strs, { raw: strs }) as unknown as TemplateStringsArray,
-              value,
-            );
+            rows = await conn(_templateStrings(strs), value);
           }
           return rows.length === 0; // true = unique (no duplicate found)
         }
 
         // exists
         const strs = [`SELECT 1 FROM ${table} WHERE ${column} = `, ` LIMIT 1`];
-        const rows = await conn(
-          Object.assign(strs, { raw: strs }) as unknown as TemplateStringsArray,
-          value,
-        );
+        const rows = await conn(_templateStrings(strs), value);
         return rows.length > 0; // true = exists (row found)
       });
     } catch {

package/src/schema/MigrationRunner.ts

@@ -8,6 +8,7 @@ import { MigrationRan } from "../events.ts";
 import { dialectFor } from "../db/QueryBuilder.ts";
 import { getDialect } from "../db/dialects/index.ts";
 import { TransactionContext } from "../db/TransactionContext.ts";
+import { _templateStrings } from "../db/templateStrings.ts";
 
 // ── Public types ──────────────────────────────────────────────────────────────
 
@@ -375,9 +376,7 @@ export class MigrationRunner {
   /** Execute a DDL string with no parameters. */
   private async _ddl(sql: string): Promise<void> {
     const strings = [sql];
-    const tpl = Object.assign(strings, {
-      raw: strings,
-    }) as TemplateStringsArray;
+    const tpl = _templateStrings(strings);
     await this._active()(tpl);
   }
 
@@ -389,22 +388,20 @@ export class MigrationRunner {
   /** SELECT with exactly one bound parameter. */
   private async _selectParam<T>(sql: string, value: unknown): Promise<T[]> {
     const parts = sql.split("?");
-    const tpl = Object.assign(parts, { raw: parts }) as TemplateStringsArray;
+    const tpl = _templateStrings(parts);
     return this._active()<T>(tpl, value);
   }
 
   /** DML with N bound parameters. */
   private async _exec(sql: string, ...values: unknown[]): Promise<void> {
     const parts = sql.split("?");
-    const tpl = Object.assign(parts, { raw: parts }) as TemplateStringsArray;
+    const tpl = _templateStrings(parts);
     await this._active()(tpl, ...values);
   }
 
   private async _exec0<T>(sql: string): Promise<T[]> {
     const strings = [sql];
-    const tpl = Object.assign(strings, {
-      raw: strings,
-    }) as TemplateStringsArray;
+    const tpl = _templateStrings(strings);
     return this._active()<T>(tpl);
   }
 

package/src/schema/Schema.ts

@@ -2,6 +2,7 @@ import { _getScopedDbConnection } from "../db/DB.ts";
 import { _getDialect } from "../model/BaseModel.ts";
 import { getDialect } from "../db/dialects/index.ts";
 import { Blueprint } from "./Blueprint.ts";
+import { _templateStrings } from "../db/templateStrings.ts";
 
 // ── Helpers ───────────────────────────────────────────────────────────────────
 
@@ -16,7 +17,7 @@ async function ddl(sql: string): Promise<void> {
   // migration runner's `begin()` wrapper decorative.
   const conn = _getScopedDbConnection();
   const strings = [sql];
-  const tpl = Object.assign(strings, { raw: strings }) as TemplateStringsArray;
+  const tpl = _templateStrings(strings);
   await conn(tpl);
 }
 
@@ -29,7 +30,7 @@ async function query<T = Record<string, unknown>>(sql: string, params: unknown[]
   // the tables that transaction has created, which a pooled connection cannot.
   const conn = _getScopedDbConnection();
   const parts = sql.split("?");
-  const tpl = Object.assign(parts, { raw: parts }) as TemplateStringsArray;
+  const tpl = _templateStrings(parts);
   return conn<T>(tpl, ...params);
 }
 

package/src/schema/SchemaInspector.ts

@@ -1,5 +1,6 @@
 import { _getDbConnection } from "../db/DB.ts";
 import { _getDialect } from "../model/BaseModel.ts";
+import { _templateStrings } from "../db/templateStrings.ts";
 
 // ── DB column descriptor (normalised across dialects) ─────────────────────────
 
@@ -23,7 +24,7 @@ export interface LiveTable {
 async function queryRaw<T>(sql: string): Promise<T[]> {
   const conn = _getDbConnection();
   const strings = [sql];
-  const tpl = Object.assign(strings, { raw: strings }) as TemplateStringsArray;
+  const tpl = _templateStrings(strings);
   return conn<T>(tpl);
 }
 
@@ -31,7 +32,7 @@ async function queryRaw<T>(sql: string): Promise<T[]> {
 async function queryParam<T>(sql: string, value: unknown): Promise<T[]> {
   const conn = _getDbConnection();
   const parts = sql.split("?");
-  const tpl = Object.assign(parts, { raw: parts }) as TemplateStringsArray;
+  const tpl = _templateStrings(parts);
   return conn<T>(tpl, value);
 }