@naturalcycles/db-lib 9.7.2 → 9.9.0

package/dist/commondao/common.dao.d.ts

@@ -4,7 +4,7 @@ import { AsyncMapper, BaseDBEntity, CommonLogger, JsonSchemaObject, JsonSchemaRo
 import { AjvSchema, ObjectSchema, ReadableTyped } from '@naturalcycles/nodejs-lib';
 import { CommonDBTransactionOptions, DBPatch, DBTransaction, RunQueryResult } from '../db.model';
 import { DBQuery, RunnableDBQuery } from '../query/dbQuery';
-import { CommonDaoCfg, CommonDaoCreateOptions, CommonDaoOptions, CommonDaoSaveBatchOptions, CommonDaoSaveOptions, CommonDaoStreamDeleteOptions, CommonDaoStreamForEachOptions, CommonDaoStreamOptions, CommonDaoStreamSaveOptions } from './common.dao.model';
+import { CommonDaoCfg, CommonDaoCreateOptions, CommonDaoOptions, CommonDaoPatchOptions, CommonDaoSaveBatchOptions, CommonDaoSaveOptions, CommonDaoStreamDeleteOptions, CommonDaoStreamForEachOptions, CommonDaoStreamOptions, CommonDaoStreamSaveOptions } from './common.dao.model';
 /**
  * Lowest common denominator API between supported Databases.
  *
@@ -78,15 +78,6 @@ export declare class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity
      * "Returns", just to have a type of "Saved"
      */
     assignIdCreatedUpdated<T extends BaseDBEntity>(obj: Partial<T>, opt?: CommonDaoOptions): T;
-    /**
-     * 1. Applies the patch
-     * 2. If object is the same after patching - skips saving it
-     * 3. Otherwise - saves the patched object and returns it
-     *
-     * Similar to `save` with skipIfEquals.
-     * Similar to `patch`, but doesn't load the object from the Database.
-     */
-    savePatch(bm: BM, patch: Partial<BM>, opt?: CommonDaoSaveBatchOptions<DBM>): Promise<BM>;
     /**
      * Convenience method to replace 3 operations (loading+patching+saving) with one:
      *
@@ -107,7 +98,7 @@ export declare class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity
      * Otherwise, similar behavior as patchById.
      * It still loads the row from the DB.
      */
-    patch(bm: BM, patch: Partial<BM>, opt?: CommonDaoSaveBatchOptions<DBM>): Promise<BM>;
+    patch(bm: BM, patch: Partial<BM>, opt?: CommonDaoPatchOptions<DBM>): Promise<BM>;
     /**
      * Like patch, but runs all operations within a Transaction.
      */
@@ -158,7 +149,8 @@ export declare class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity
      * Does NOT mutate the object.
      * Validates (unless `skipValidation=true` passed).
      */
-    validateAndConvert<T>(obj: Partial<T>, schema: ObjectSchema<T> | AjvSchema<T> | ZodSchema<T> | undefined, opt?: CommonDaoOptions): any;
+    validateAndConvert<T>(obj: Partial<T>, schema: ObjectSchema<T> | AjvSchema<T> | ZodSchema<T> | undefined, op?: 'load' | 'save', // this is to skip validation if validateOnLoad/Save is false
+    opt?: CommonDaoOptions): any;
     getTableSchema(): Promise<JsonSchemaRootObject<DBM>>;
     createTable(schema: JsonSchemaObject<DBM>, opt?: CommonDaoCreateOptions): Promise<void>;
     /**

package/dist/commondao/common.dao.js

@@ -27,6 +27,8 @@ class CommonDao {
             assignGeneratedIds: false,
             useCreatedProperty: true,
             useUpdatedProperty: true,
+            validateOnLoad: true,
+            validateOnSave: true,
             logger: console,
             ...cfg,
             hooks: {
@@ -49,7 +51,7 @@ class CommonDao {
         const bm = this.cfg.hooks.beforeCreate(part);
         // First assignIdCreatedUpdated, then validate!
         this.assignIdCreatedUpdated(bm, opt);
-        return this.validateAndConvert(bm, this.cfg.bmSchema, opt);
+        return this.validateAndConvert(bm, this.cfg.bmSchema, undefined, opt);
     }
     async getById(id, opt = {}) {
         if (!id)
@@ -245,7 +247,6 @@ class CommonDao {
     async streamQueryForEach(q, mapper, opt = {}) {
         q.table = opt.table || q.table;
         opt.skipValidation = opt.skipValidation !== false; // default true
-        opt.skipConversion = opt.skipConversion !== false; // default true
         opt.errorMode ||= js_lib_1.ErrorMode.SUPPRESS;
         const partialQuery = !!q._selectedFieldNames;
         const op = `streamQueryForEach(${q.pretty()})`;
@@ -284,7 +285,6 @@ class CommonDao {
     async streamQueryAsDBMForEach(q, mapper, opt = {}) {
         q.table = opt.table || q.table;
         opt.skipValidation = opt.skipValidation !== false; // default true
-        opt.skipConversion = opt.skipConversion !== false; // default true
         opt.errorMode ||= js_lib_1.ErrorMode.SUPPRESS;
         const partialQuery = !!q._selectedFieldNames;
         const op = `streamQueryAsDBMForEach(${q.pretty()})`;
@@ -326,7 +326,6 @@ class CommonDao {
     streamQueryAsDBM(q, opt = {}) {
         q.table = opt.table || q.table;
         opt.skipValidation = opt.skipValidation !== false; // default true
-        opt.skipConversion = opt.skipConversion !== false; // default true
         opt.errorMode ||= js_lib_1.ErrorMode.SUPPRESS;
         const partialQuery = !!q._selectedFieldNames;
         const stream = this.cfg.db.streamQuery(q, opt);
@@ -357,7 +356,6 @@ class CommonDao {
     streamQuery(q, opt = {}) {
         q.table = opt.table || q.table;
         opt.skipValidation = opt.skipValidation !== false; // default true
-        opt.skipConversion = opt.skipConversion !== false; // default true
         opt.errorMode ||= js_lib_1.ErrorMode.SUPPRESS;
         const stream = this.cfg.db.streamQuery(q, opt);
         const partialQuery = !!q._selectedFieldNames;
@@ -444,27 +442,6 @@ class CommonDao {
         return obj;
     }
     // SAVE
-    /**
-     * 1. Applies the patch
-     * 2. If object is the same after patching - skips saving it
-     * 3. Otherwise - saves the patched object and returns it
-     *
-     * Similar to `save` with skipIfEquals.
-     * Similar to `patch`, but doesn't load the object from the Database.
-     */
-    async savePatch(bm, patch, opt) {
-        const patched = {
-            ...bm,
-            ...patch,
-        };
-        if ((0, js_lib_1._deepJsonEquals)(bm, patched)) {
-            // Skipping the save operation, as data is the same
-            return bm;
-        }
-        // Actually apply the patch by mutating the original object (by design)
-        Object.assign(bm, patch);
-        return await this.save(bm, opt);
-    }
     /**
      * Convenience method to replace 3 operations (loading+patching+saving) with one:
      *
@@ -516,18 +493,28 @@ class CommonDao {
             // and should just continue as-is
             return await this.patchInTransaction(bm, patch, opt);
         }
-        const loaded = await this.getById(bm.id, opt);
-        if (loaded) {
-            Object.assign(loaded, patch);
-            if ((0, js_lib_1._deepJsonEquals)(loaded, bm)) {
+        if (opt.skipDBRead) {
+            const bmBefore = (0, js_lib_1._deepCopy)(bm);
+            Object.assign(bm, patch);
+            if ((0, js_lib_1._deepJsonEquals)(bm, bmBefore)) {
                 // Skipping the save operation, as data is the same
                 return bm;
             }
-            // Make `bm` exactly the same as `loaded`
-            (0, js_lib_1._objectAssignExact)(bm, loaded);
         }
         else {
-            Object.assign(bm, patch);
+            const loaded = await this.getById(bm.id, opt);
+            if (loaded) {
+                Object.assign(loaded, patch);
+                if ((0, js_lib_1._deepJsonEquals)(loaded, bm)) {
+                    // Skipping the save operation, as data is the same
+                    return bm;
+                }
+                // Make `bm` exactly the same as `loaded`
+                (0, js_lib_1._objectAssignExact)(bm, loaded);
+            }
+            else {
+                Object.assign(bm, patch);
+            }
         }
         return await this.save(bm, opt);
     }
@@ -548,7 +535,7 @@ class CommonDao {
             // We compare with convertedBM, to account for cases when some extra property is assigned to bm,
             // which should be removed post-validation, but it breaks the "equality check"
             // Post-validation the equality check should work as intended
-            const convertedBM = this.validateAndConvert(bm, this.cfg.bmSchema, opt);
+            const convertedBM = this.validateAndConvert(bm, this.cfg.bmSchema, 'save', opt);
             if ((0, js_lib_1._deepJsonEquals)(convertedBM, opt.skipIfEquals)) {
                 // Skipping the save operation
                 return bm;
@@ -693,7 +680,6 @@ class CommonDao {
         this.requireWriteAccess();
         const table = opt.table || this.cfg.table;
         opt.skipValidation ??= true;
-        opt.skipConversion ??= true;
         opt.errorMode ||= js_lib_1.ErrorMode.SUPPRESS;
         if (this.cfg.immutable && !opt.allowMutability && !opt.saveMethod) {
             opt = { ...opt, saveMethod: 'insert' };
@@ -828,7 +814,7 @@ class CommonDao {
         // DBM > BM
         const bm = ((await this.cfg.hooks.beforeDBMToBM?.(dbm)) || dbm);
         // Validate/convert BM
-        return this.validateAndConvert(bm, this.cfg.bmSchema, opt);
+        return this.validateAndConvert(bm, this.cfg.bmSchema, 'load', opt);
     }
     async dbmsToBM(dbms, opt = {}) {
         return await (0, js_lib_1.pMap)(dbms, async (dbm) => await this.dbmToBM(dbm, opt));
@@ -837,7 +823,7 @@ class CommonDao {
         if (bm === undefined)
             return;
         // bm gets assigned to the new reference
-        bm = this.validateAndConvert(bm, this.cfg.bmSchema, opt);
+        bm = this.validateAndConvert(bm, this.cfg.bmSchema, 'save', opt);
         // BM > DBM
         return ((await this.cfg.hooks.beforeBMToDBM?.(bm)) || bm);
     }
@@ -868,7 +854,8 @@ class CommonDao {
      * Does NOT mutate the object.
      * Validates (unless `skipValidation=true` passed).
      */
-    validateAndConvert(obj, schema, opt = {}) {
+    validateAndConvert(obj, schema, op, // this is to skip validation if validateOnLoad/Save is false
+    opt = {}) {
         // Kirill 2021-10-18: I realized that there's little reason to keep removing null values
         // So, from now on we'll preserve them
         // "undefined" values, I believe, are/were not saved to/from DB anyway (due to e.g JSON.stringify removing them)
@@ -880,7 +867,10 @@ class CommonDao {
         // and they can be annoying with snapshot tests
         obj = (0, js_lib_1._filterUndefinedValues)(obj);
         // Return as is if no schema is passed or if `skipConversion` is set
-        if (!schema || opt.skipConversion) {
+        if (!schema ||
+            opt.skipValidation ||
+            (op === 'load' && !this.cfg.validateOnLoad) ||
+            (op === 'save' && !this.cfg.validateOnSave)) {
             return obj;
         }
         // This will Convert and Validate
@@ -908,7 +898,7 @@ class CommonDao {
             convertedValue = vr.value;
         }
         // If we care about validation and there's an error
-        if (error && !opt.skipValidation) {
+        if (error) {
             const processedError = this.cfg.hooks.onValidationError(error);
             if (processedError)
                 throw processedError;

package/dist/commondao/common.dao.model.d.ts

@@ -101,6 +101,16 @@ export interface CommonDaoCfg<BM extends BaseDBEntity, DBM extends BaseDBEntity
      */
     bmSchema?: ObjectSchema<BM> | AjvSchema<BM> | ZodSchema<BM>;
     excludeFromIndexes?: (keyof DBM)[];
+    /**
+     * Defaults to true.
+     * If set to false - load (read) operations will skip validation (and conversion).
+     */
+    validateOnLoad?: boolean;
+    /**
+     * Defaults to true.
+     * If set to false - save (write) operations will skip validation (and conversion).
+     */
+    validateOnSave?: boolean;
     /**
      * Defaults to false.
      * Setting it to true will set saveMethod to `insert` for save/saveBatch, which will
@@ -163,20 +173,12 @@ export interface CommonDaoCfg<BM extends BaseDBEntity, DBM extends BaseDBEntity
  */
 export interface CommonDaoOptions extends CommonDBOptions {
     /**
-     * If true - will ignore the validation result, but will STILL DO the validation step, which will DO conversion
-     * (according to Joi schema).
-     *
-     * Set skipConversion=true (or raw=true) to bypass conversion step as well (e.g for performance reasons).
+     * Defaults to false.
      *
-     * @default false
+     * If set to true - will disable validation (and conversion).
+     * One possible use case of doing this is - performance (as validation/conversion takes time, especially with Joi).
      */
     skipValidation?: boolean;
-    /**
-     * If true - will SKIP the joi validation AND conversion steps alltogether. To improve performance of DAO.
-     *
-     * @default false
-     */
-    skipConversion?: boolean;
     /**
      * @default false
      */
@@ -209,6 +211,12 @@ export interface CommonDaoSaveOptions<BM extends BaseDBEntity, DBM extends BaseD
      */
     skipIfEquals?: BM;
 }
+export interface CommonDaoPatchOptions<DBM extends BaseDBEntity> extends CommonDaoSaveBatchOptions<DBM> {
+    /**
+     * If true - patch will skip loading from DB, and will just optimistically patch passed object.
+     */
+    skipDBRead?: boolean;
+}
 /**
  * All properties default to undefined.
  */
@@ -235,10 +243,6 @@ export interface CommonDaoStreamOptions<IN> extends CommonDaoOptions, TransformL
      * @default true (for streams)
      */
     skipValidation?: boolean;
-    /**
-     * @default true (for streams)
-     */
-    skipConversion?: boolean;
     /**
      * @default ErrorMode.SUPPRESS for returning ReadableStream, because .pipe() has no concept of "error propagation"
      * @default ErrorMode.SUPPRESS for .forEach() streams as well, but overridable

package/dist/testing/timeSeriesTest.util.js

@@ -4,6 +4,6 @@ exports.createTestTimeSeries = void 0;
 const js_lib_1 = require("@naturalcycles/js-lib");
 function createTestTimeSeries(count = 10) {
     const ts = Date.now();
-    return (0, js_lib_1._range)(1, count + 1).map(i => [ts - i * 60000, (0, js_lib_1._randomInt)(10, 20)]);
+    return (0, js_lib_1._range)(1, count + 1).map(i => [ts - i * 60_000, (0, js_lib_1._randomInt)(10, 20)]);
 }
 exports.createTestTimeSeries = createTestTimeSeries;

package/package.json

@@ -8,7 +8,7 @@
     "@naturalcycles/nodejs-lib": "^13.1.1"
   },
   "devDependencies": {
-    "@naturalcycles/bench-lib": "^1.0.0",
+    "@naturalcycles/bench-lib": "^2.0.0",
     "@naturalcycles/dev-lib": "^13.0.0",
     "@types/node": "^20.2.1",
     "jest": "^29.0.0"
@@ -40,7 +40,7 @@
   "engines": {
     "node": ">=18.12"
   },
-  "version": "9.7.2",
+  "version": "9.9.0",
   "description": "Lowest Common Denominator API to supported Databases",
   "keywords": [
     "db",

package/src/commondao/common.dao.model.ts

@@ -129,6 +129,18 @@ export interface CommonDaoCfg<BM extends BaseDBEntity, DBM extends BaseDBEntity
 
   excludeFromIndexes?: (keyof DBM)[]
 
+  /**
+   * Defaults to true.
+   * If set to false - load (read) operations will skip validation (and conversion).
+   */
+  validateOnLoad?: boolean
+
+  /**
+   * Defaults to true.
+   * If set to false - save (write) operations will skip validation (and conversion).
+   */
+  validateOnSave?: boolean
+
   /**
    * Defaults to false.
    * Setting it to true will set saveMethod to `insert` for save/saveBatch, which will
@@ -203,22 +215,13 @@ export interface CommonDaoCfg<BM extends BaseDBEntity, DBM extends BaseDBEntity
  */
 export interface CommonDaoOptions extends CommonDBOptions {
   /**
-   * If true - will ignore the validation result, but will STILL DO the validation step, which will DO conversion
-   * (according to Joi schema).
-   *
-   * Set skipConversion=true (or raw=true) to bypass conversion step as well (e.g for performance reasons).
+   * Defaults to false.
    *
-   * @default false
+   * If set to true - will disable validation (and conversion).
+   * One possible use case of doing this is - performance (as validation/conversion takes time, especially with Joi).
    */
   skipValidation?: boolean
 
-  /**
-   * If true - will SKIP the joi validation AND conversion steps alltogether. To improve performance of DAO.
-   *
-   * @default false
-   */
-  skipConversion?: boolean
-
   /**
    * @default false
    */
@@ -257,6 +260,14 @@ export interface CommonDaoSaveOptions<BM extends BaseDBEntity, DBM extends BaseD
   skipIfEquals?: BM
 }
 
+export interface CommonDaoPatchOptions<DBM extends BaseDBEntity>
+  extends CommonDaoSaveBatchOptions<DBM> {
+  /**
+   * If true - patch will skip loading from DB, and will just optimistically patch passed object.
+   */
+  skipDBRead?: boolean
+}
+
 /**
  * All properties default to undefined.
  */
@@ -294,11 +305,6 @@ export interface CommonDaoStreamOptions<IN>
    */
   skipValidation?: boolean
 
-  /**
-   * @default true (for streams)
-   */
-  skipConversion?: boolean
-
   /**
    * @default ErrorMode.SUPPRESS for returning ReadableStream, because .pipe() has no concept of "error propagation"
    * @default ErrorMode.SUPPRESS for .forEach() streams as well, but overridable

package/src/commondao/common.dao.ts

@@ -51,6 +51,7 @@ import {
   CommonDaoHooks,
   CommonDaoLogLevel,
   CommonDaoOptions,
+  CommonDaoPatchOptions,
   CommonDaoSaveBatchOptions,
   CommonDaoSaveOptions,
   CommonDaoStreamDeleteOptions,
@@ -80,6 +81,8 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
       assignGeneratedIds: false,
       useCreatedProperty: true,
       useUpdatedProperty: true,
+      validateOnLoad: true,
+      validateOnSave: true,
       logger: console,
       ...cfg,
       hooks: {
@@ -103,7 +106,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
     const bm = this.cfg.hooks!.beforeCreate!(part)
     // First assignIdCreatedUpdated, then validate!
     this.assignIdCreatedUpdated(bm, opt)
-    return this.validateAndConvert(bm, this.cfg.bmSchema, opt)
+    return this.validateAndConvert(bm, this.cfg.bmSchema, undefined, opt)
   }
 
   // GET
@@ -352,7 +355,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   ): Promise<void> {
     q.table = opt.table || q.table
     opt.skipValidation = opt.skipValidation !== false // default true
-    opt.skipConversion = opt.skipConversion !== false // default true
     opt.errorMode ||= ErrorMode.SUPPRESS
 
     const partialQuery = !!q._selectedFieldNames
@@ -402,7 +404,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   ): Promise<void> {
     q.table = opt.table || q.table
     opt.skipValidation = opt.skipValidation !== false // default true
-    opt.skipConversion = opt.skipConversion !== false // default true
     opt.errorMode ||= ErrorMode.SUPPRESS
 
     const partialQuery = !!q._selectedFieldNames
@@ -451,7 +452,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   streamQueryAsDBM(q: DBQuery<DBM>, opt: CommonDaoStreamOptions<DBM> = {}): ReadableTyped<DBM> {
     q.table = opt.table || q.table
     opt.skipValidation = opt.skipValidation !== false // default true
-    opt.skipConversion = opt.skipConversion !== false // default true
     opt.errorMode ||= ErrorMode.SUPPRESS
 
     const partialQuery = !!q._selectedFieldNames
@@ -490,7 +490,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   streamQuery(q: DBQuery<DBM>, opt: CommonDaoStreamOptions<BM> = {}): ReadableTyped<BM> {
     q.table = opt.table || q.table
     opt.skipValidation = opt.skipValidation !== false // default true
-    opt.skipConversion = opt.skipConversion !== false // default true
     opt.errorMode ||= ErrorMode.SUPPRESS
 
     const stream = this.cfg.db.streamQuery<DBM>(q, opt)
@@ -604,31 +603,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   }
 
   // SAVE
-  /**
-   * 1. Applies the patch
-   * 2. If object is the same after patching - skips saving it
-   * 3. Otherwise - saves the patched object and returns it
-   *
-   * Similar to `save` with skipIfEquals.
-   * Similar to `patch`, but doesn't load the object from the Database.
-   */
-  async savePatch(bm: BM, patch: Partial<BM>, opt?: CommonDaoSaveBatchOptions<DBM>): Promise<BM> {
-    const patched: BM = {
-      ...bm,
-      ...patch,
-    }
-
-    if (_deepJsonEquals(bm, patched)) {
-      // Skipping the save operation, as data is the same
-      return bm
-    }
-
-    // Actually apply the patch by mutating the original object (by design)
-    Object.assign(bm, patch)
-
-    return await this.save(bm, opt)
-  }
-
   /**
    * Convenience method to replace 3 operations (loading+patching+saving) with one:
    *
@@ -686,7 +660,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
    * Otherwise, similar behavior as patchById.
    * It still loads the row from the DB.
    */
-  async patch(bm: BM, patch: Partial<BM>, opt: CommonDaoSaveBatchOptions<DBM> = {}): Promise<BM> {
+  async patch(bm: BM, patch: Partial<BM>, opt: CommonDaoPatchOptions<DBM> = {}): Promise<BM> {
     if (this.cfg.patchInTransaction && !opt.tx) {
       // patchInTransaction means that we should run this op in Transaction
       // But if opt.tx is passed - means that we are already in a Transaction,
@@ -694,20 +668,29 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
       return await this.patchInTransaction(bm, patch, opt)
     }
 
-    const loaded = await this.getById(bm.id, opt)
-
-    if (loaded) {
-      Object.assign(loaded, patch)
-
-      if (_deepJsonEquals(loaded, bm)) {
+    if (opt.skipDBRead) {
+      const bmBefore = _deepCopy(bm)
+      Object.assign(bm, patch)
+      if (_deepJsonEquals(bm, bmBefore)) {
         // Skipping the save operation, as data is the same
         return bm
       }
-
-      // Make `bm` exactly the same as `loaded`
-      _objectAssignExact(bm, loaded)
     } else {
-      Object.assign(bm, patch)
+      const loaded = await this.getById(bm.id, opt)
+
+      if (loaded) {
+        Object.assign(loaded, patch)
+
+        if (_deepJsonEquals(loaded, bm)) {
+          // Skipping the save operation, as data is the same
+          return bm
+        }
+
+        // Make `bm` exactly the same as `loaded`
+        _objectAssignExact(bm, loaded)
+      } else {
+        Object.assign(bm, patch)
+      }
     }
 
     return await this.save(bm, opt)
@@ -736,7 +719,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
       // We compare with convertedBM, to account for cases when some extra property is assigned to bm,
       // which should be removed post-validation, but it breaks the "equality check"
       // Post-validation the equality check should work as intended
-      const convertedBM = this.validateAndConvert(bm as Partial<BM>, this.cfg.bmSchema, opt)
+      const convertedBM = this.validateAndConvert(bm as Partial<BM>, this.cfg.bmSchema, 'save', opt)
       if (_deepJsonEquals(convertedBM, opt.skipIfEquals)) {
         // Skipping the save operation
         return bm as BM
@@ -915,7 +898,6 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
 
     const table = opt.table || this.cfg.table
     opt.skipValidation ??= true
-    opt.skipConversion ??= true
     opt.errorMode ||= ErrorMode.SUPPRESS
 
     if (this.cfg.immutable && !opt.allowMutability && !opt.saveMethod) {
@@ -1092,7 +1074,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
     const bm = ((await this.cfg.hooks!.beforeDBMToBM?.(dbm)) || dbm) as Partial<BM>
 
     // Validate/convert BM
-    return this.validateAndConvert(bm, this.cfg.bmSchema, opt)
+    return this.validateAndConvert(bm, this.cfg.bmSchema, 'load', opt)
   }
 
   async dbmsToBM(dbms: DBM[], opt: CommonDaoOptions = {}): Promise<BM[]> {
@@ -1109,7 +1091,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
     if (bm === undefined) return
 
     // bm gets assigned to the new reference
-    bm = this.validateAndConvert(bm, this.cfg.bmSchema, opt)
+    bm = this.validateAndConvert(bm, this.cfg.bmSchema, 'save', opt)
 
     // BM > DBM
     return ((await this.cfg.hooks!.beforeBMToDBM?.(bm!)) || bm) as DBM
@@ -1153,6 +1135,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
   validateAndConvert<T>(
     obj: Partial<T>,
     schema: ObjectSchema<T> | AjvSchema<T> | ZodSchema<T> | undefined,
+    op?: 'load' | 'save', // this is to skip validation if validateOnLoad/Save is false
     opt: CommonDaoOptions = {},
   ): any {
     // Kirill 2021-10-18: I realized that there's little reason to keep removing null values
@@ -1167,7 +1150,12 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
     obj = _filterUndefinedValues(obj)
 
     // Return as is if no schema is passed or if `skipConversion` is set
-    if (!schema || opt.skipConversion) {
+    if (
+      !schema ||
+      opt.skipValidation ||
+      (op === 'load' && !this.cfg.validateOnLoad) ||
+      (op === 'save' && !this.cfg.validateOnSave)
+    ) {
       return obj
     }
 
@@ -1198,7 +1186,7 @@ export class CommonDao<BM extends BaseDBEntity, DBM extends BaseDBEntity = BM> {
     }
 
     // If we care about validation and there's an error
-    if (error && !opt.skipValidation) {
+    if (error) {
       const processedError = this.cfg.hooks!.onValidationError!(error)
 
       if (processedError) throw processedError