From 7534b44e6ef4093109a084c4ed8f674607b6edd1 Mon Sep 17 00:00:00 2001 From: Philipinho <16838612+Philipinho@users.noreply.github.com> Date: Sun, 19 Apr 2026 21:50:34 +0100 Subject: [PATCH] refactor(server): preserve cache-failure stack trace and reuse hasSearch --- apps/server/src/core/base/query-cache/base-query-router.ts | 5 ++--- apps/server/src/core/base/services/base-row.service.ts | 5 +++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/apps/server/src/core/base/query-cache/base-query-router.ts b/apps/server/src/core/base/query-cache/base-query-router.ts index edf0706d..9047a899 100644 --- a/apps/server/src/core/base/query-cache/base-query-router.ts +++ b/apps/server/src/core/base/query-cache/base-query-router.ts @@ -29,9 +29,8 @@ export class BaseQueryRouter { const hasSearch = !!args.search; if (!hasFilter && !hasSorts && !hasSearch) return 'postgres'; - // v1: any search stays on Postgres. Trgm search also stays on Postgres - // until the loader populates `search_text`; re-evaluate after that lands. - if (args.search) return 'postgres'; + // v1: any search stays on Postgres — loader doesn't populate search_text yet. + if (hasSearch) return 'postgres'; const count = await this.baseRowRepo.countActiveRows(args.baseId, { workspaceId: args.workspaceId, diff --git a/apps/server/src/core/base/services/base-row.service.ts b/apps/server/src/core/base/services/base-row.service.ts index 74a78f75..4fbaf131 100644 --- a/apps/server/src/core/base/services/base-row.service.ts +++ b/apps/server/src/core/base/services/base-row.service.ts @@ -227,10 +227,11 @@ export class BaseRowService { pagination, }); } catch (err) { + const error = err as Error; this.logger.warn( - `Cache list failed for base ${dto.baseId}, falling back to Postgres`, - err as Error, + `Cache list failed for base ${dto.baseId}, falling back to Postgres: ${error.message}`, ); + if (error.stack) this.logger.warn(error.stack); } }