dobo 2.19.1 → 2.20.0

package/lib/factory/model/_util.js

@@ -430,3 +430,12 @@ export function preparePagination (filter = {}, options = {}) {
   const sort = buildSort(sortInput, options.allowSortUnindexed)
   return { limit, page, skip, sort }
 }
+
+export async function clearCache (id) {
+  const { clear } = this.app.bajoCache
+  if (!clear) return
+  await clear({ key: `dobo|${this.name}|getRecord|${id}` })
+  await clear({ key: `dobo|${this.name}|findRecord` })
+  await clear({ key: `dobo|${this.name}|findAllRecord` })
+  await clear({ key: `dobo|${this.name}|findOneRecord` })
+}

package/lib/factory/model/find-all-record.js

@@ -4,45 +4,50 @@ const action = 'findAllRecord'
 async function native (...args) {
   const { isSet } = this.app.lib.aneka
   const { getDefaultValues, t } = this.app.dobo
-  const { pick, omit } = this.app.lib._
+  const { pick, omit, cloneDeep } = this.app.lib._
   const [params = {}, opts = {}] = args
+  const { get, set } = this.app.bajoCache ?? {}
   opts.dataOnly = opts.dataOnly ?? true
   const { dataOnly } = opts
   const { filter, options } = await getFilterAndOptions.call(this, params, opts, action)
   const { hardCap, warnings } = getDefaultValues(options)
   if (dataOnly) options.count = false
   const { noResultSanitizer } = options
-  try {
-    await execHook.call(this, 'beforeFindRecord', filter, options)
-    await execModelHook.call(this, 'beforeFindRecord', filter, options)
-    await execDynHook.call(this, 'beforeFindRecord', filter, options)
-    let result = options.record ?? (await this.driver._findAllRecord(this, filter, options)) ?? {}
-    result.limit = filter.limit
-    result.filter = pick(filter, ['query', 'match', 'sort'])
-    result.warnings = result.warnings ?? []
-    if (!options.count) result = omit(result, ['count', 'pages'])
-    else if (options.count && result.count > hardCap) {
-      result.warnings.push(t('hardCapWarning%s%s', result.count, hardCap))
-      result.count = hardCap
-      result.hardCapped = true
+  await execHook.call(this, 'beforeFindRecord', filter, options)
+  await execModelHook.call(this, 'beforeFindRecord', filter, options)
+  await execDynHook.call(this, 'beforeFindRecord', filter, options)
+  const cFilter = cloneDeep(filter)
+  if (get) {
+    const resp = await get({ model: this, action, filter: cFilter, options })
+    if (resp) {
+      resp.cached = true
+      return dataOnly ? resp.data : resp
     }
-    result.pages = options.count ? Math.ceil(result.count / filter.limit) : undefined
-    if (!warnings) delete result.warnings
+  }
+  let result = options.record ?? (await this.driver._findAllRecord(this, filter, options)) ?? {}
+  result.limit = filter.limit
+  result.filter = pick(filter, ['query', 'match', 'sort'])
+  result.warnings = result.warnings ?? []
+  if (!options.count) result = omit(result, ['count', 'pages'])
+  else if (options.count && result.count > hardCap) {
+    result.warnings.push(t('hardCapWarning%s%s', result.count, hardCap))
+    result.count = hardCap
+    result.hardCapped = true
+  }
+  result.pages = options.count ? Math.ceil(result.count / filter.limit) : undefined
+  if (!warnings) delete result.warnings
 
-    if (!noResultSanitizer) {
-      for (const idx in result.data) {
-        result.data[idx] = await this.sanitizeRecord(result.data[idx], options)
-      }
+  if (!noResultSanitizer) {
+    for (const idx in result.data) {
+      result.data[idx] = await this.sanitizeRecord(result.data[idx], options)
     }
-    if (isSet(options.refs)) await getMultiRefs.call(this, result.data, options)
-    await execDynHook.call(this, 'afterFindRecord', filter, result, options)
-    await execModelHook.call(this, 'afterFindRecord', filter, result, options)
-    await execHook.call(this, 'afterFindRecord', filter, result, options)
-    return dataOnly ? result.data : result
-  } catch (err) {
-    if (err.code === 'cachedResult') return err.data
-    throw err
   }
+  if (isSet(options.refs)) await getMultiRefs.call(this, result.data, options)
+  await execDynHook.call(this, 'afterFindRecord', filter, result, options)
+  await execModelHook.call(this, 'afterFindRecord', filter, result, options)
+  await execHook.call(this, 'afterFindRecord', filter, result, options)
+  if (set) await set({ model: this, action, filter: cFilter, options, result })
+  return dataOnly ? result.data : result
 }
 
 async function loop (...args) {

package/lib/factory/model/find-record.js

@@ -69,45 +69,50 @@ async function findRecord (...args) {
   const { getDefaultValues, t } = this.app.dobo
   const [params = {}, opts = {}] = args
   const { isSet } = this.app.lib.aneka
-  const { pick, omit } = this.app.lib._
+  const { pick, omit, cloneDeep } = this.app.lib._
+  const { get, set } = this.app.bajoCache ?? {}
   opts.dataOnly = opts.dataOnly ?? true
   const { dataOnly } = opts
   const { filter, options } = await getFilterAndOptions.call(this, params, opts, action)
   const { hardCap, warnings } = getDefaultValues(options)
   if (dataOnly) options.count = false
   const { noResultSanitizer } = options
-  try {
-    await execHook.call(this, 'beforeFindRecord', filter, options)
-    await execModelHook.call(this, 'beforeFindRecord', filter, options)
-    await execDynHook.call(this, 'beforeFindRecord', filter, options)
-    let result = options.record ?? (await this.driver._findRecord(this, filter, options)) ?? {}
-    result.page = filter.page
-    result.limit = filter.limit
-    result.filter = pick(filter, ['query', 'match', 'sort'])
-    result.warnings = result.warnings ?? []
-    if (!options.count) result = omit(result, ['count', 'pages'])
-    else if (options.count && result.count > hardCap) {
-      result.warnings.push(t('hardCapWarning%s%s', result.count, hardCap))
-      result.count = hardCap
-      result.hardCapped = true
+  await execHook.call(this, 'beforeFindRecord', filter, options)
+  await execModelHook.call(this, 'beforeFindRecord', filter, options)
+  await execDynHook.call(this, 'beforeFindRecord', filter, options)
+  const cFilter = cloneDeep(filter)
+  if (get) {
+    const resp = await get({ model: this, action, filter: cFilter, options })
+    if (resp) {
+      resp.cached = true
+      return dataOnly ? resp.data : resp
     }
-    result.pages = options.count ? Math.ceil(result.count / filter.limit) : undefined
-    if (!warnings) delete result.warnings
+  }
+  let result = options.record ?? (await this.driver._findRecord(this, filter, options)) ?? {}
+  result.page = filter.page
+  result.limit = filter.limit
+  result.filter = pick(filter, ['query', 'search', 'sort'])
+  result.warnings = result.warnings ?? []
+  if (!options.count) result = omit(result, ['count', 'pages'])
+  else if (options.count && result.count > hardCap) {
+    result.warnings.push(t('hardCapWarning%s%s', result.count, hardCap))
+    result.count = hardCap
+    result.hardCapped = true
+  }
+  result.pages = options.count ? Math.ceil(result.count / filter.limit) : undefined
+  if (!warnings) delete result.warnings
 
-    if (!noResultSanitizer) {
-      for (const idx in result.data) {
-        result.data[idx] = await this.sanitizeRecord(result.data[idx], options)
-      }
+  if (!noResultSanitizer) {
+    for (const idx in result.data) {
+      result.data[idx] = await this.sanitizeRecord(result.data[idx], options)
     }
-    if (isSet(options.refs)) await getMultiRefs.call(this, result.data, options)
-    await execDynHook.call(this, 'afterFindRecord', filter, result, options)
-    await execModelHook.call(this, 'afterFindRecord', filter, result, options)
-    await execHook.call(this, 'afterFindRecord', filter, result, options)
-    return dataOnly ? result.data : result
-  } catch (err) {
-    if (err.code === 'cachedResult') return err.data
-    throw err
   }
+  if (isSet(options.refs)) await getMultiRefs.call(this, result.data, options)
+  await execDynHook.call(this, 'afterFindRecord', filter, result, options)
+  await execModelHook.call(this, 'afterFindRecord', filter, result, options)
+  await execHook.call(this, 'afterFindRecord', filter, result, options)
+  if (set) await set({ model: this, action, filter: cFilter, options, result })
+  return dataOnly ? result.data : result
 }
 
 export default findRecord

package/lib/factory/model/get-record.js

@@ -48,29 +48,33 @@ async function getRecord (...args) {
   const { getDefaultValues } = this.app.dobo
   const { isEmpty } = this.app.lib._
   const { isSet } = this.app.lib.aneka
+  const { get, set } = this.app.bajoCache ?? {}
   opts.dataOnly = opts.dataOnly ?? true
   const { dataOnly } = opts
   const { options } = await getFilterAndOptions.call(this, null, opts, action)
   const { noResultSanitizer } = options
   id = this.sanitizeId(id)
-  try {
-    await execHook.call(this, 'beforeGetRecord', id, options)
-    await execModelHook.call(this, 'beforeGetRecord', id, options)
-    await execDynHook.call(this, 'beforeGetRecord', id, options)
-    const result = options.record ?? (await this.driver._getRecord(this, id, options)) ?? {}
-    const { warnings } = getDefaultValues(options)
-    if (!warnings) delete result.warnings
-    if (isEmpty(result.data) && !options.throwNotFound) return dataOnly ? undefined : { data: undefined }
-    if (!noResultSanitizer) result.data = await this.sanitizeRecord(result.data, options)
-    if (isSet(options.refs)) await getSingleRef.call(this, result.data, options)
-    await execDynHook.call(this, 'afterGetRecord', id, result, options)
-    await execModelHook.call(this, 'afterGetRecord', id, result, options)
-    await execHook.call(this, 'afterGetRecord', id, result, options)
-    return dataOnly ? result.data : result
-  } catch (err) {
-    if (err.code === 'cachedResult') return err.data
-    throw err
+  await execHook.call(this, 'beforeGetRecord', id, options)
+  await execModelHook.call(this, 'beforeGetRecord', id, options)
+  await execDynHook.call(this, 'beforeGetRecord', id, options)
+  if (get) {
+    const resp = await get({ model: this, action, id, options })
+    if (resp) {
+      resp.cached = true
+      return dataOnly ? resp.data : resp
+    }
   }
+  const result = options.record ?? (await this.driver._getRecord(this, id, options)) ?? {}
+  const { warnings } = getDefaultValues(options)
+  if (!warnings) delete result.warnings
+  if (isEmpty(result.data) && !options.throwNotFound) return dataOnly ? undefined : { data: undefined }
+  if (!noResultSanitizer) result.data = await this.sanitizeRecord(result.data, options)
+  if (isSet(options.refs)) await getSingleRef.call(this, result.data, options)
+  await execDynHook.call(this, 'afterGetRecord', id, result, options)
+  await execModelHook.call(this, 'afterGetRecord', id, result, options)
+  await execHook.call(this, 'afterGetRecord', id, result, options)
+  if (set) await set({ model: this, action, id, options, result })
+  return dataOnly ? result.data : result
 }
 
 export default getRecord

package/lib/factory/model/remove-record.js

@@ -1,4 +1,4 @@
-import { getFilterAndOptions, execHook, execModelHook, execDynHook, getSingleRef, handleReq } from './_util.js'
+import { getFilterAndOptions, execHook, execModelHook, execDynHook, getSingleRef, handleReq, clearCache } from './_util.js'
 const action = 'removeRecord'
 
 /**
@@ -45,6 +45,7 @@ async function removeRecord (...args) {
   await execModelHook.call(this, 'beforeRemoveRecord', id, options)
   await execDynHook.call(this, 'beforeRemoveRecord', id, options)
   const result = options.record ?? (await this.driver._removeRecord(this, id, options)) ?? {}
+  await clearCache.call(this, id)
   if (noResult) return
   const { warnings } = getDefaultValues(options)
   if (!warnings) delete result.warnings

package/lib/factory/model/update-record.js

@@ -1,4 +1,4 @@
-import { getFilterAndOptions, execHook, execValidation, execModelHook, execDynHook, getSingleRef, handleReq } from './_util.js'
+import { getFilterAndOptions, execHook, execValidation, execModelHook, execDynHook, getSingleRef, handleReq, clearCache } from './_util.js'
 import { onlyTypes } from './create-record.js'
 const action = 'updateRecord'
 
@@ -67,6 +67,7 @@ async function updateRecord (...args) {
   await execDynHook.call(this, 'beforeUpdateRecord', id, input, options)
   if (!noValidation) await execValidation.call(this, input, options)
   const result = options.record ?? (await this.driver._updateRecord(this, id, input, options)) ?? {}
+  await clearCache.call(this, id)
   if (noResult) return
   const { warnings } = getDefaultValues(options)
   if (!warnings) delete result.warnings

package/lib/factory/model/upsert-record.js

@@ -1,4 +1,4 @@
-import { getFilterAndOptions, execHook, execModelHook, execDynHook, execValidation, getSingleRef, handleReq } from './_util.js'
+import { getFilterAndOptions, execHook, execModelHook, execDynHook, execValidation, getSingleRef, handleReq, clearCache } from './_util.js'
 const action = 'upsertRecord'
 
 async function native (body = {}, opts = {}) {
@@ -16,6 +16,7 @@ async function native (body = {}, opts = {}) {
   await execModelHook.call(this, 'beforeUpsertRecord', input, options)
   await execDynHook.call(this, 'beforeUpsertRecord', input, options)
   const result = options.record ?? (await this.driver._upsertRecord(this, input, options)) ?? {}
+  await clearCache.call(this, body.id)
   if (noResult) return
   const { warnings } = getDefaultValues(options)
   if (!warnings) delete result.warnings
@@ -39,7 +40,7 @@ async function manual (body = {}, options = {}) {
     } catch (err) {
     }
   }
-  const id = get(old, 'dara.id')
+  const id = get(old, 'data.id')
   if (id) return await this.updateRecord(old.data.id, omit(body, ['id']), options)
   return await this.createRecord(body, options)
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dobo",
-  "version": "2.19.1",
+  "version": "2.20.0",
   "description": "DBMS for Bajo Framework",
   "main": "index.js",
   "scripts": {

package/wiki/CHANGES.md

@@ -2,9 +2,10 @@
 
 ## 2026-04-21
 
-- [2.5.1] Bug fix in ```collect-models.js```, now values are sanitized with ```parseObject()```
-- [2.5.1] Bug fix in ```options.dataOnly``` on all model methods
-- [2.5.1] Add ```options.noMagic```
+- [2.19.1] Bug fix in ```collect-models.js```, now values are sanitized with ```parseObject()```
+- [2.19.1] Bug fix in ```options.dataOnly``` on all model methods
+- [2.19.1] Add ```options.noMagic```
+- [2.20.0] Revert back to NOT using hooks for caching
 
 ## 2026-04-19