From a29cdbb454592467a3124e1fd014d1972dcea8bb Mon Sep 17 00:00:00 2001 From: Justin Edmund Date: Wed, 3 Dec 2025 17:13:54 -0800 Subject: [PATCH] fix cache key to include query params was generating same cache key for /artifact_skills and /artifact_skills?group=group_i causing filtered requests to return cached unfiltered results --- src/lib/api/adapters/base.adapter.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/lib/api/adapters/base.adapter.ts b/src/lib/api/adapters/base.adapter.ts index a0777ebe..83bd6bbe 100644 --- a/src/lib/api/adapters/base.adapter.ts +++ b/src/lib/api/adapters/base.adapter.ts @@ -100,7 +100,8 @@ export abstract class BaseAdapter { ? options.body : JSON.stringify(options.body) : undefined - const requestId = this.generateRequestId(path, options.method, bodyString) + // Use full URL (including query params) for cache key to differentiate filtered requests + const requestId = this.generateRequestId(url, options.method, bodyString) // Check cache first if caching is enabled (support both cacheTime and cacheTTL) const cacheTime = options.cacheTTL ?? options.cacheTime ?? this.options.cacheTime