@ethlete/query 5.31.1 → 5.31.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @ethlete/query
 
+## 5.31.2
+
+### Patch Changes
+
+- [`a26a8c6`](https://github.com/ethlete-io/ethdk/commit/a26a8c6c374fd60152e7bd430500d2b1777f4be8) Thanks [@TomTomB](https://github.com/TomTomB)! - Fix gql query variables not getting parsed as json when transported via query params
+
 ## 5.31.1
 
 ### Patch Changes

package/fesm2022/ethlete-query.mjs

@@ -122,7 +122,19 @@ const buildQueryString = (params, config) => {
     const ignoredValuesFns = config?.ignoredValuesFns ?? [isNaN$1, isEmptyString];
     const queryParams = [];
     function processValue(key, value) {
-        if (Array.isArray(value)) {
+        if (config?.useJsonStringify) {
+            if (ignoredValues.includes(value)) {
+                return false;
+            }
+            if (ignoredValuesFns.some((fn) => fn(value))) {
+                return false;
+            }
+            const encodedKey = encodeURIComponent(key);
+            const encodedValue = encodeURIComponent(JSON.stringify(value));
+            queryParams.push(`${encodedKey}=${encodedValue}`);
+            return true;
+        }
+        else if (Array.isArray(value)) {
             let currentFilteredIndex = 0;
             for (const arrayValue of value) {
                 const nestedKey = writeArrayIndexes ? `${key}[${currentFilteredIndex}]` : `${key}[]`;
@@ -704,12 +716,19 @@ class QueryCreator {
         // eslint-disable-next-line @typescript-eslint/ban-ts-comment
         // @ts-ignore
         this.prepare = (args) => {
+            const shouldUseJsonStringifyForQs = isGqlQueryConfig(this._queryConfig) &&
+                (this._queryConfig.transferVia === 'GET' ||
+                    (this._client.config.request?.gql?.transferVia === 'GET' && !this._queryConfig.transferVia));
+            const originalQueryParams = this._client.config.request?.queryParams;
+            const queryParamConfig = originalQueryParams
+                ? { ...originalQueryParams, useJsonStringify: shouldUseJsonStringifyForQs }
+                : undefined;
             const route = buildRoute({
                 base: this._client.config.baseRoute,
                 route: this._queryConfig.route,
                 pathParams: args?.pathParams,
                 queryParams: computeQueryQueryParams({ config: this._queryConfig, client: this._client, args }),
-                queryParamConfig: this._client.config.request?.queryParams,
+                queryParamConfig,
             });
             const cacheKey = (args?.config?.queryStoreCacheKey ?? '') + buildQueryCacheKey$1(route, args);
             if (shouldCacheQuery$1(this._queryConfig.method) && !args?.config?.skipQueryStore) {
@@ -4249,7 +4268,9 @@ const createQueryRepository = (config) => {
             route: options.route,
             pathParams: args?.pathParams,
             queryParams: args?.queryParams,
-            queryParamConfig: config.queryString,
+            queryParamConfig: options.internalRunQueryOptions?.forceJsonStringifyInQueryParams
+                ? { useJsonStringify: true }
+                : config.queryString,
         });
         const key = shouldCache &&
             buildQueryCacheKey(`${options.key ? options.key + '_' : ''}${route}`, {
@@ -4576,6 +4597,7 @@ const createGqlExecuteFn = (executeOptions) => {
             options,
             internalOptions: {
                 useQueryRepositoryCache: executeOptions.creatorInternals.method === 'QUERY',
+                forceJsonStringifyInQueryParams: executeOptions.creatorInternals.method === 'QUERY',
             },
             transformResponse: (originalResponse) => {
                 if (originalResponse && 'data' in originalResponse) {