@stonyx/orm 0.3.2-beta.6 → 0.3.2-beta.8

package/config/environment.js

@@ -1,91 +1,12 @@
-const {
-  ORM_ACCESS_PATH,
-  ORM_MODEL_PATH,
-  ORM_REST_ROUTE,
-  ORM_SERIALIZER_PATH,
-  ORM_TRANSFORM_PATH,
-  ORM_VIEW_PATH,
-  ORM_USE_REST_SERVER,
-  DB_AUTO_SAVE,
-  DB_FILE,
-  DB_MODE,
-  DB_DIRECTORY,
-  DB_SCHEMA_PATH,
-  DB_SAVE_INTERVAL,
-  MYSQL_HOST,
-  MYSQL_PORT,
-  MYSQL_USER,
-  MYSQL_PASSWORD,
-  MYSQL_DATABASE,
-  MYSQL_CONNECTION_LIMIT,
-  MYSQL_MIGRATIONS_DIR,
-  PG_HOST,
-  PG_PORT,
-  PG_USER,
-  PG_PASSWORD,
-  PG_DATABASE,
-  PG_CONNECTION_LIMIT,
-  PG_MIGRATIONS_DIR,
-  TIMESCALE_HOST,
-  TIMESCALE_PORT,
-  TIMESCALE_USER,
-  TIMESCALE_PASSWORD,
-  TIMESCALE_DATABASE,
-  TIMESCALE_CONNECTION_LIMIT,
-  TIMESCALE_MIGRATIONS_DIR,
-} = process.env;
-
-export default {
-  logColor: 'white',
-  logMethod: 'db',
-  
-  db: {
-    autosave: DB_AUTO_SAVE ?? 'false', // 'true' (cron interval), 'false' (disabled), 'onUpdate' (save after each write op)
-    file: DB_FILE ?? 'db.json',
-    mode: DB_MODE ?? 'file', // 'file' (single db.json) or 'directory' (one file per collection)
-    directory: DB_DIRECTORY ?? 'db', // directory name for collection files when mode is 'directory'
-    saveInterval: DB_SAVE_INTERVAL ?? 60 * 60, // 1 hour
-    schema: DB_SCHEMA_PATH ?? './config/db-schema.js'
-  },
-  paths: {
-    access: ORM_ACCESS_PATH ?? './access', // Optional for restServer access hooks
-    model: ORM_MODEL_PATH ?? './models',
-    serializer: ORM_SERIALIZER_PATH ?? './serializers',
-    transform: ORM_TRANSFORM_PATH ?? './transforms',
-    view: ORM_VIEW_PATH ?? './views'
-  },
-  mysql: MYSQL_HOST ? {
-    host: MYSQL_HOST ?? 'localhost',
-    port: parseInt(MYSQL_PORT ?? '3306'),
-    user: MYSQL_USER ?? 'root',
-    password: MYSQL_PASSWORD ?? '',
-    database: MYSQL_DATABASE ?? 'stonyx',
-    connectionLimit: parseInt(MYSQL_CONNECTION_LIMIT ?? '10'),
-    migrationsDir: MYSQL_MIGRATIONS_DIR ?? 'migrations',
-    migrationsTable: '__migrations',
-  } : undefined,
-  postgres: PG_HOST ? {
-    host: PG_HOST ?? 'localhost',
-    port: parseInt(PG_PORT ?? '5432'),
-    user: PG_USER ?? 'postgres',
-    password: PG_PASSWORD ?? '',
-    database: PG_DATABASE ?? 'stonyx',
-    connectionLimit: parseInt(PG_CONNECTION_LIMIT ?? '10'),
-    migrationsDir: PG_MIGRATIONS_DIR ?? 'migrations',
-    migrationsTable: '__migrations',
-  } : undefined,
-  timescale: TIMESCALE_HOST ? {
-    host: TIMESCALE_HOST ?? 'localhost',
-    port: parseInt(TIMESCALE_PORT ?? '5432'),
-    user: TIMESCALE_USER ?? 'postgres',
-    password: TIMESCALE_PASSWORD ?? '',
-    database: TIMESCALE_DATABASE ?? 'stonyx',
-    connectionLimit: parseInt(TIMESCALE_CONNECTION_LIMIT ?? '10'),
-    migrationsDir: TIMESCALE_MIGRATIONS_DIR ?? 'migrations',
-    migrationsTable: '__migrations',
-  } : undefined,
-  restServer: {
-    enabled: ORM_USE_REST_SERVER ?? 'true', // Whether to load restServer for automatic route setup or
-    route: ORM_REST_ROUTE ?? '/',
-  }
-}
+// project configuration, override-able by listed environment variables
+const {
+  DEBUG,
+  NODE_ENV,
+} = process.env;
+
+const environment = NODE_ENV ?? 'development';
+
+export default {
+  environment,
+  debug: DEBUG ?? environment === 'development',
+}

package/config/environment.ts

@@ -0,0 +1,91 @@
+const {
+  ORM_ACCESS_PATH,
+  ORM_MODEL_PATH,
+  ORM_REST_ROUTE,
+  ORM_SERIALIZER_PATH,
+  ORM_TRANSFORM_PATH,
+  ORM_VIEW_PATH,
+  ORM_USE_REST_SERVER,
+  DB_AUTO_SAVE,
+  DB_FILE,
+  DB_MODE,
+  DB_DIRECTORY,
+  DB_SCHEMA_PATH,
+  DB_SAVE_INTERVAL,
+  MYSQL_HOST,
+  MYSQL_PORT,
+  MYSQL_USER,
+  MYSQL_PASSWORD,
+  MYSQL_DATABASE,
+  MYSQL_CONNECTION_LIMIT,
+  MYSQL_MIGRATIONS_DIR,
+  PG_HOST,
+  PG_PORT,
+  PG_USER,
+  PG_PASSWORD,
+  PG_DATABASE,
+  PG_CONNECTION_LIMIT,
+  PG_MIGRATIONS_DIR,
+  TIMESCALE_HOST,
+  TIMESCALE_PORT,
+  TIMESCALE_USER,
+  TIMESCALE_PASSWORD,
+  TIMESCALE_DATABASE,
+  TIMESCALE_CONNECTION_LIMIT,
+  TIMESCALE_MIGRATIONS_DIR,
+} = process.env;
+
+export default {
+  logColor: 'white',
+  logMethod: 'db',
+  
+  db: {
+    autosave: DB_AUTO_SAVE ?? 'false', // 'true' (cron interval), 'false' (disabled), 'onUpdate' (save after each write op)
+    file: DB_FILE ?? 'db.json',
+    mode: DB_MODE ?? 'file', // 'file' (single db.json) or 'directory' (one file per collection)
+    directory: DB_DIRECTORY ?? 'db', // directory name for collection files when mode is 'directory'
+    saveInterval: DB_SAVE_INTERVAL ?? 60 * 60, // 1 hour
+    schema: DB_SCHEMA_PATH ?? './config/db-schema.js'
+  },
+  paths: {
+    access: ORM_ACCESS_PATH ?? './access', // Optional for restServer access hooks
+    model: ORM_MODEL_PATH ?? './models',
+    serializer: ORM_SERIALIZER_PATH ?? './serializers',
+    transform: ORM_TRANSFORM_PATH ?? './transforms',
+    view: ORM_VIEW_PATH ?? './views'
+  },
+  mysql: MYSQL_HOST ? {
+    host: MYSQL_HOST ?? 'localhost',
+    port: parseInt(MYSQL_PORT ?? '3306'),
+    user: MYSQL_USER ?? 'root',
+    password: MYSQL_PASSWORD ?? '',
+    database: MYSQL_DATABASE ?? 'stonyx',
+    connectionLimit: parseInt(MYSQL_CONNECTION_LIMIT ?? '10'),
+    migrationsDir: MYSQL_MIGRATIONS_DIR ?? 'migrations',
+    migrationsTable: '__migrations',
+  } : undefined,
+  postgres: PG_HOST ? {
+    host: PG_HOST ?? 'localhost',
+    port: parseInt(PG_PORT ?? '5432'),
+    user: PG_USER ?? 'postgres',
+    password: PG_PASSWORD ?? '',
+    database: PG_DATABASE ?? 'stonyx',
+    connectionLimit: parseInt(PG_CONNECTION_LIMIT ?? '10'),
+    migrationsDir: PG_MIGRATIONS_DIR ?? 'migrations',
+    migrationsTable: '__migrations',
+  } : undefined,
+  timescale: TIMESCALE_HOST ? {
+    host: TIMESCALE_HOST ?? 'localhost',
+    port: parseInt(TIMESCALE_PORT ?? '5432'),
+    user: TIMESCALE_USER ?? 'postgres',
+    password: TIMESCALE_PASSWORD ?? '',
+    database: TIMESCALE_DATABASE ?? 'stonyx',
+    connectionLimit: parseInt(TIMESCALE_CONNECTION_LIMIT ?? '10'),
+    migrationsDir: TIMESCALE_MIGRATIONS_DIR ?? 'migrations',
+    migrationsTable: '__migrations',
+  } : undefined,
+  restServer: {
+    enabled: ORM_USE_REST_SERVER ?? 'true', // Whether to load restServer for automatic route setup or
+    route: ORM_REST_ROUTE ?? '/',
+  }
+}

package/dist/index.d.ts

@@ -8,6 +8,7 @@ import { createRecord, updateRecord } from './manage-record.js';
 import { count, avg, sum, min, max } from './aggregates.js';
 export { default } from './main.js';
 export { store, relationships } from './main.js';
+export type { PersistErrorDetail } from './main.js';
 export { Model, View, Serializer };
 export { attr, belongsTo, hasMany, createRecord, updateRecord };
 export { count, avg, sum, min, max };

package/dist/main.d.ts

@@ -15,6 +15,12 @@ export interface OrmDB {
     save(): Promise<void>;
     init(): Promise<void>;
 }
+export interface PersistErrorDetail {
+    operation: 'create' | 'update' | 'delete';
+    modelName: string;
+    recordId: unknown;
+    error: Error;
+}
 export default class Orm {
     static initialized: boolean;
     static relationships: Map<string, Map<string, unknown>>;
@@ -29,6 +35,7 @@ export default class Orm {
     options: OrmOptions;
     sqlDb?: SqlDb;
     db?: OrmDB | SqlDb;
+    private _persistErrorHandler;
     constructor(options?: OrmOptions);
     init(): Promise<void>;
     startup(): Promise<void>;
@@ -39,6 +46,15 @@ export default class Orm {
         serializerClass: unknown;
     };
     isView(modelName: string): boolean;
+    /**
+     * Register a callback to be invoked when a fire-and-forget SQL persist fails.
+     * Without a handler, persist errors are logged via log.error (backwards-compatible).
+     */
+    onPersistError(handler: ((detail: PersistErrorDetail) => void) | null): void;
+    /**
+     * Emit a persist error to the registered handler, or fall back to log.error.
+     */
+    emitPersistError(detail: PersistErrorDetail): void;
     warn(message: string): void;
 }
 export declare const store: Store;

package/dist/main.js

@@ -41,6 +41,7 @@ export default class Orm {
     options;
     sqlDb;
     db;
+    _persistErrorHandler = null;
     constructor(options = {}) {
         if (Orm.instance)
             return Orm.instance;
@@ -168,6 +169,31 @@ export default class Orm {
         const modelClassPrefix = kebabCaseToPascalCase(modelName);
         return !!this.views[`${modelClassPrefix}View`];
     }
+    /**
+     * Register a callback to be invoked when a fire-and-forget SQL persist fails.
+     * Without a handler, persist errors are logged via log.error (backwards-compatible).
+     */
+    onPersistError(handler) {
+        this._persistErrorHandler = handler;
+    }
+    /**
+     * Emit a persist error to the registered handler, or fall back to log.error.
+     */
+    emitPersistError(detail) {
+        const fallbackLog = () => log.error?.(`[ORM] Failed to persist ${detail.operation} for ${detail.modelName}:${String(detail.recordId)}: ${detail.error.message}`);
+        if (this._persistErrorHandler) {
+            try {
+                this._persistErrorHandler(detail);
+            }
+            catch (handlerError) {
+                fallbackLog();
+                log.error?.(`[ORM] onPersistError handler threw: ${handlerError instanceof Error ? handlerError.message : String(handlerError)}`);
+            }
+        }
+        else {
+            fallbackLog();
+        }
+    }
     // Queue warnings to avoid the same error from being logged in the same iteration
     warn(message) {
         this.warnings.add(message);

package/dist/manage-record.js

@@ -2,12 +2,12 @@ import Orm, { store } from '@stonyx/orm';
 import OrmRecord from './record.js';
 import { getGlobalRegistry, getPendingRegistry, getPendingBelongsToRegistry, getBelongsToRegistry, getHasManyRegistry } from './relationships.js';
 import { isOrmRecord } from './utils.js';
-import log from 'stonyx/log';
 const defaultOptions = {
     isDbRecord: false,
     serialize: true,
     transform: true
 };
+let pendingIdCounter = 0;
 export function createRecord(modelName, rawData = {}, userOptions = {}) {
     const orm = Orm.instance;
     const { initialized } = Orm;
@@ -85,7 +85,12 @@ export function createRecord(modelName, rawData = {}, userOptions = {}) {
     if (shouldPersist) {
         const response = { data: { id: record.id } };
         orm.sqlDb.persist('create', modelName, { rawData }, response).catch((err) => {
-            log.error?.(`[ORM] Failed to persist create for ${modelName}:${String(record.id)}`, err);
+            orm.emitPersistError({
+                operation: 'create',
+                modelName,
+                recordId: record.id,
+                error: err instanceof Error ? err : new Error(String(err)),
+            });
         });
     }
     return record;
@@ -107,7 +112,12 @@ export function updateRecord(record, rawData, userOptions = {}) {
     const shouldPersist = orm?.sqlDb && !options.isDbRecord && !userOptions._relationshipKey && !options._skipAutoPersist;
     if (shouldPersist && modelName) {
         orm.sqlDb.persist('update', modelName, { record, oldState }, {}).catch((err) => {
-            log.error?.(`[ORM] Failed to persist update for ${modelName}:${String(record.id)}`, err);
+            orm.emitPersistError({
+                operation: 'update',
+                modelName,
+                recordId: record.id,
+                error: err instanceof Error ? err : new Error(String(err)),
+            });
         });
     }
 }
@@ -120,9 +130,11 @@ export function updateRecord(record, rawData, userOptions = {}) {
 function assignRecordId(modelName, rawData) {
     if (rawData.id)
         return;
-    // In SQL mode with numeric IDs, defer to database auto-increment
+    // In SQL mode with numeric IDs, defer to database auto-increment.
+    // Use unique negative integers — they survive the number transform (parseInt preserves negatives)
+    // and avoid NaN store-key collisions that string pending IDs caused.
     if (Orm.instance?.sqlDb && !isStringIdModel(modelName)) {
-        rawData.id = `__pending_${Date.now()}_${Math.random()}`;
+        rawData.id = -(++pendingIdCounter);
         rawData.__pendingSqlId = true;
         return;
     }

package/dist/mysql/mysql-db.d.ts

@@ -16,6 +16,7 @@ interface PersistContext {
     record?: OrmRecord;
     recordId?: unknown;
     oldState?: Record<string, unknown>;
+    rawData?: Record<string, unknown>;
 }
 interface PersistResponse {
     data?: {

package/dist/mysql/mysql-db.js

@@ -321,8 +321,10 @@ export default class MysqlDB {
         if (!record)
             return;
         const insertData = this._recordToRow(record, schema);
-        // For auto-increment models, remove the pending ID
-        const isPendingId = record.__data.__pendingSqlId;
+        // For auto-increment models, remove the pending ID.
+        // Check context.rawData (not record.__data) because __pendingSqlId is not a model
+        // attribute and gets lost during serialization.
+        const isPendingId = context.rawData?.__pendingSqlId === true;
         if (isPendingId) {
             delete insertData.id;
         }

package/dist/postgres/postgres-db.js

@@ -376,8 +376,10 @@ export default class PostgresDB {
         if (!record)
             return;
         const insertData = this._recordToRow(record, schema, context.rawData);
-        // For auto-increment models, remove the pending ID
-        const isPendingId = record.__data.__pendingSqlId;
+        // For auto-increment models, remove the pending ID.
+        // Check context.rawData (not record.__data) because __pendingSqlId is not a model
+        // attribute and gets lost during serialization.
+        const isPendingId = context.rawData?.__pendingSqlId === true;
         if (isPendingId) {
             delete insertData.id;
         }

package/dist/store.js

@@ -115,7 +115,12 @@ export default class Store {
         // Auto-persist delete to SQL
         if (id && Orm.instance?.sqlDb) {
             Orm.instance.sqlDb.persist('delete', key, { recordId: id }, {}).catch((err) => {
-                console.error(`[ORM] Failed to persist delete for ${key}:${id}`, err);
+                Orm.instance.emitPersistError({
+                    operation: 'delete',
+                    modelName: key,
+                    recordId: id,
+                    error: err instanceof Error ? err : new Error(String(err)),
+                });
             });
         }
         if (id)

package/package.json

@@ -4,7 +4,7 @@
     "stonyx-async",
     "stonyx-module"
   ],
-  "version": "0.3.2-beta.6",
+  "version": "0.3.2-beta.8",
   "description": "",
   "main": "dist/index.js",
   "type": "module",
@@ -61,9 +61,9 @@
   },
   "homepage": "https://github.com/abofs/stonyx-orm#readme",
   "dependencies": {
-    "@stonyx/cron": "0.2.1-beta.29",
+    "@stonyx/cron": "0.2.1-beta.38",
     "@stonyx/events": "0.1.1-beta.9",
-    "stonyx": "0.2.3-beta.11"
+    "stonyx": "0.2.3-beta.53"
   },
   "peerDependencies": {
     "@stonyx/rest-server": ">=0.2.1-beta.14",
@@ -83,17 +83,18 @@
   },
   "devDependencies": {
     "@stonyx/rest-server": "0.2.1-beta.30",
-    "@stonyx/utils": "0.2.3-beta.7",
+    "@stonyx/utils": "0.2.3-beta.23",
     "@types/node": "^25.6.0",
     "mysql2": "^3.20.0",
     "pg": "^8.20.0",
     "qunit": "^2.24.1",
     "sinon": "^21.0.0",
+    "tsx": "^4.21.0",
     "typescript": "^5.8.3"
   },
   "scripts": {
     "build": "tsc",
     "build:test": "tsc -p tsconfig.test.json",
-    "test": "npm run build && npm run build:test && stonyx test 'dist-test/test/**/*-test.js'"
+    "test": "pnpm build && NODE_ENV=test node --import tsx/esm --import ./test/setup.ts node_modules/qunit/bin/qunit.js 'test/**/*-test.ts'"
   }
 }

package/src/index.ts

@@ -26,6 +26,7 @@ import { count, avg, sum, min, max } from './aggregates.js';
 
 export { default } from './main.js';
 export { store, relationships } from './main.js';
+export type { PersistErrorDetail } from './main.js';
 export { Model, View, Serializer }; // base classes
 export { attr, belongsTo, hasMany, createRecord, updateRecord }; // helpers
 export { count, avg, sum, min, max }; // aggregate helpers

package/src/main.ts

@@ -49,6 +49,13 @@ const defaultOptions: OrmOptions = {
   dbType: 'json'
 }
 
+export interface PersistErrorDetail {
+  operation: 'create' | 'update' | 'delete';
+  modelName: string;
+  recordId: unknown;
+  error: Error;
+}
+
 export default class Orm {
   static initialized: boolean = false;
   static relationships: Map<string, Map<string, unknown>> = new Map();
@@ -65,6 +72,8 @@ export default class Orm {
   sqlDb?: SqlDb;
   db?: OrmDB | SqlDb;
 
+  private _persistErrorHandler: ((detail: PersistErrorDetail) => void) | null = null;
+
   constructor(options: OrmOptions = {}) {
     if (Orm.instance) return Orm.instance;
 
@@ -214,6 +223,32 @@ export default class Orm {
     return !!this.views[`${modelClassPrefix}View`];
   }
 
+  /**
+   * Register a callback to be invoked when a fire-and-forget SQL persist fails.
+   * Without a handler, persist errors are logged via log.error (backwards-compatible).
+   */
+  onPersistError(handler: ((detail: PersistErrorDetail) => void) | null): void {
+    this._persistErrorHandler = handler;
+  }
+
+  /**
+   * Emit a persist error to the registered handler, or fall back to log.error.
+   */
+  emitPersistError(detail: PersistErrorDetail): void {
+    const fallbackLog = () => log.error?.(`[ORM] Failed to persist ${detail.operation} for ${detail.modelName}:${String(detail.recordId)}: ${detail.error.message}`);
+
+    if (this._persistErrorHandler) {
+      try {
+        this._persistErrorHandler(detail);
+      } catch (handlerError) {
+        fallbackLog();
+        log.error?.(`[ORM] onPersistError handler threw: ${handlerError instanceof Error ? handlerError.message : String(handlerError)}`);
+      }
+    } else {
+      fallbackLog();
+    }
+  }
+
   // Queue warnings to avoid the same error from being logged in the same iteration
   warn(message: string): void {
     this.warnings.add(message);

package/src/manage-record.ts

@@ -3,7 +3,6 @@ import OrmRecord from './record.js';
 import { getGlobalRegistry, getPendingRegistry, getPendingBelongsToRegistry, getBelongsToRegistry, getHasManyRegistry } from './relationships.js';
 import type Serializer from './serializer.js';
 import { isOrmRecord } from './utils.js';
-import log from 'stonyx/log';
 
 interface CreateRecordOptions {
   isDbRecord?: boolean;
@@ -27,6 +26,8 @@ const defaultOptions: CreateRecordOptions = {
   transform: true
 };
 
+let pendingIdCounter = 0;
+
 export function createRecord(modelName: string, rawData: { [key: string]: unknown } = {}, userOptions: CreateRecordOptions = {}): OrmRecord {
   const orm = Orm.instance;
   const { initialized } = Orm;
@@ -118,7 +119,12 @@ export function createRecord(modelName: string, rawData: { [key: string]: unknow
   if (shouldPersist) {
     const response = { data: { id: record.id } };
     orm!.sqlDb!.persist('create', modelName, { rawData }, response).catch((err: unknown) => {
-      log.error?.(`[ORM] Failed to persist create for ${modelName}:${String(record.id)}`, err);
+      orm!.emitPersistError({
+        operation: 'create',
+        modelName,
+        recordId: record.id,
+        error: err instanceof Error ? err : new Error(String(err)),
+      });
     });
   }
 
@@ -146,7 +152,12 @@ export function updateRecord(record: OrmRecord, rawData: unknown, userOptions: C
   const shouldPersist = orm?.sqlDb && !options.isDbRecord && !userOptions._relationshipKey && !options._skipAutoPersist;
   if (shouldPersist && modelName) {
     orm!.sqlDb!.persist('update', modelName, { record, oldState }, {}).catch((err: unknown) => {
-      log.error?.(`[ORM] Failed to persist update for ${modelName}:${String(record.id)}`, err);
+      orm!.emitPersistError({
+        operation: 'update',
+        modelName,
+        recordId: record.id,
+        error: err instanceof Error ? err : new Error(String(err)),
+      });
     });
   }
 }
@@ -160,9 +171,11 @@ export function updateRecord(record: OrmRecord, rawData: unknown, userOptions: C
 function assignRecordId(modelName: string, rawData: { [key: string]: unknown }): void {
   if (rawData.id) return;
 
-  // In SQL mode with numeric IDs, defer to database auto-increment
+  // In SQL mode with numeric IDs, defer to database auto-increment.
+  // Use unique negative integers — they survive the number transform (parseInt preserves negatives)
+  // and avoid NaN store-key collisions that string pending IDs caused.
   if (Orm.instance?.sqlDb && !isStringIdModel(modelName)) {
-    rawData.id = `__pending_${Date.now()}_${Math.random()}`;
+    rawData.id = -(++pendingIdCounter);
     rawData.__pendingSqlId = true;
     return;
   }

package/src/mysql/mysql-db.ts

@@ -21,6 +21,7 @@ interface PersistContext {
   record?: OrmRecord;
   recordId?: unknown;
   oldState?: Record<string, unknown>;
+  rawData?: Record<string, unknown>;
 }
 
 interface PersistResponse {
@@ -420,8 +421,10 @@ export default class MysqlDB {
 
     const insertData = this._recordToRow(record, schema);
 
-    // For auto-increment models, remove the pending ID
-    const isPendingId = record.__data.__pendingSqlId;
+    // For auto-increment models, remove the pending ID.
+    // Check context.rawData (not record.__data) because __pendingSqlId is not a model
+    // attribute and gets lost during serialization.
+    const isPendingId = context.rawData?.__pendingSqlId === true;
 
     if (isPendingId) {
       delete insertData.id;

package/src/postgres/postgres-db.ts

@@ -494,8 +494,10 @@ export default class PostgresDB {
 
     const insertData = this._recordToRow(record, schema, context.rawData);
 
-    // For auto-increment models, remove the pending ID
-    const isPendingId = record.__data.__pendingSqlId;
+    // For auto-increment models, remove the pending ID.
+    // Check context.rawData (not record.__data) because __pendingSqlId is not a model
+    // attribute and gets lost during serialization.
+    const isPendingId = context.rawData?.__pendingSqlId === true;
 
     if (isPendingId) {
       delete insertData.id;

package/src/store.ts

@@ -179,7 +179,12 @@ export default class Store {
     // Auto-persist delete to SQL
     if (id && Orm.instance?.sqlDb) {
       Orm.instance.sqlDb.persist('delete', key, { recordId: id }, {}).catch((err: unknown) => {
-        console.error(`[ORM] Failed to persist delete for ${key}:${id}`, err);
+        Orm.instance.emitPersistError({
+          operation: 'delete',
+          modelName: key,
+          recordId: id,
+          error: err instanceof Error ? err : new Error(String(err)),
+        });
       });
     }