@quazardous/qdadm 2.2.0 → 2.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quazardous/qdadm",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Vue 3 framework for admin dashboards with PrimeVue",
   "author": "quazardous",
   "license": "MIT",

package/src/entity/EntityManager.cache.ts

@@ -90,7 +90,9 @@ export function applyCacheMethods(EntityManagerClass: { prototype: any }): void
   Object.defineProperty(proto, 'overflow', {
     get(this: Self): boolean {
       const cache = this._cache
-      return cache.valid && cache.total > cache.items.length
+      // overflowed: total exceeded effectiveThreshold, cache was never
+      // filled (#1204) — the cache is unusable either way.
+      return cache.overflowed === true || (cache.valid && cache.total > cache.items.length)
     },
     configurable: true,
   })
@@ -398,6 +400,7 @@ export function applyCacheMethods(EntityManagerClass: { prototype: any }): void
     cache.valid = false
     cache.items = []
     cache.total = 0
+    cache.overflowed = false
     cache.loadedAt = null
     this._cacheLoading = null
     // Also clear detail cache

package/src/entity/EntityManager.crud.ts

@@ -182,6 +182,7 @@ export function applyCrudMethods(EntityManagerClass: { prototype: any }): void {
         cache.items = items
         cache.total = total
         cache.valid = true
+        cache.overflowed = false
         cache.loadedAt = Date.now()
         // Resolve parent fields for search (book.title, user.username, etc.)
         await this._resolveSearchFields(items)
@@ -191,6 +192,11 @@ export function applyCrudMethods(EntityManagerClass: { prototype: any }): void {
         // Fire-and-forget: load full cache in background
         this._loadCacheInBackground()
       }
+    } else if (canUseCache && this.isCacheEnabled && total > this.effectiveThreshold) {
+      // Entity outgrew the threshold: the cache can never hold it. Mark it
+      // so query() stops filtering an empty cache and goes to the API
+      // instead of returning an empty list (#1204).
+      cache.overflowed = true
     }
 
     return { items, total, fromCache: false }

package/src/entity/EntityManager.query.ts

@@ -31,7 +31,7 @@ export function applyQueryMethods(EntityManagerClass: { prototype: any }): void
     // Ensure cache is filled (via list) — skip for dynamic endpoints
     // since list() disables caching when resolveStorage provides an endpoint
     const cache = this._cache
-    if (!isDynamicEndpoint && !cache.valid && this.isCacheEnabled) {
+    if (!isDynamicEndpoint && !cache.valid && !cache.overflowed && this.isCacheEnabled) {
       await this.list(
         { page_size: this.effectiveThreshold },
         routingContext ?? undefined
@@ -40,8 +40,11 @@ export function applyQueryMethods(EntityManagerClass: { prototype: any }): void
 
     let result: ListResult<any>
 
-    // Dynamic endpoint, overflow, or cache disabled → use API directly
-    if (isDynamicEndpoint || this.overflow || !this.isCacheEnabled) {
+    // Dynamic endpoint, overflow, cache disabled, or cache still not
+    // valid (fill failed / threshold exceeded) → use API directly. The
+    // cache is an optional layer: it must never turn into an empty
+    // result when it can't hold the data (#1204).
+    if (isDynamicEndpoint || this.overflow || !this.isCacheEnabled || !cache.valid) {
       result = await this.list(params, routingContext ?? undefined)
     } else {
       // Full cache available - filter locally

package/src/entity/EntityManager.ts

@@ -114,6 +114,7 @@ export class EntityManager<T extends EntityRecord = EntityRecord> {
     total: 0,
     loadedAt: null,
     valid: false,
+    overflowed: false,
   }
   protected _cacheLoading: Promise<boolean> | null = null
 

package/src/entity/EntityManager.types.ts

@@ -26,6 +26,8 @@ export interface CacheState<T> {
   total: number
   loadedAt: number | null
   valid: boolean
+  /** Entity total exceeded effectiveThreshold — cache can't hold it (#1204). */
+  overflowed?: boolean
 }
 
 /**

package/src/generated/managers/testManager.ts

@@ -1,46 +0,0 @@
-/**
- * TestManager - Auto-generated EntityManager
- *
- * Generated by qdadm/gen/generateManagers
- * DO NOT EDIT MANUALLY - Changes will be overwritten
- *
- * Entity: test
- * Endpoint: /test
- */
-
-import { EntityManager } from '@quazardous/qdadm'
-import type { EntityRecord } from '@quazardous/qdadm'
-import { ApiStorage } from '@quazardous/qdadm'
-
-export interface TestEntity extends EntityRecord {
-}
-
-/**
- * Schema definition for test
- */
-export const testSchema = {
-  name: "test",
-  endpoint: "/test",
-  fields: {}
-} as const
-
-/**
- * Storage options for test
- */
-const storageOptions = {
-  endpoint: "/test"
-}
-
-/**
- * TestManager instance
- *
- * Provides CRUD operations for test entity.
- */
-export const testManager = new EntityManager<TestEntity>({
-  ...{
-  name: "test",
-  idField: "id",
-  fields: {}
-},
-  storage: new ApiStorage<TestEntity>(storageOptions)
-})

package/src/kernel/Kernel.routing.ts

@@ -212,7 +212,11 @@ export function applyRoutingMethods(KernelClass: { prototype: any }): void {
       }
 
       const entity = to.meta?.entity as string | undefined
-      if (entity && this.orchestrator) {
+      // Entity not registered → nothing to check yet (e.g. child route
+      // declared before its manager loads) — allow. This is the ONLY
+      // allowed pass-through; a failing access check must fail CLOSED,
+      // not silently allow (#1190).
+      if (entity && this.orchestrator && this.orchestrator.isRegistered(entity)) {
         try {
           const manager = this.orchestrator.get(entity)
           if (manager && !manager.canRead()) {
@@ -232,8 +236,13 @@ export function applyRoutingMethods(KernelClass: { prototype: any }): void {
             next({ path: '/' })
             return
           }
-        } catch {
-          // Entity not registered - allow navigation
+        } catch (err) {
+          console.error(
+            `[qdadm] Access check failed for ${to.path} (entity: ${entity}) — denying navigation:`,
+            err
+          )
+          next({ path: '/' })
+          return
         }
       }