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
This commit is contained in:
Justin Edmund 2025-12-03 17:13:54 -08:00
parent 644c596173
commit a29cdbb454

View file

@ -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