Merge pull request #159 from tech-sushant/screenshot-array

fix: Enhance ApiClient to support responseType in requests

Author: gaurav-singh-9227

src/lib/apiClient.ts

@@ -12,6 +12,7 @@ type RequestOptions = {
   params?: Record<string, string | number>;
   body?: any;
   timeout?: number;
+  responseType?: AxiosRequestConfig["responseType"];
   raise_error?: boolean; // default: true
 };
 
@@ -163,12 +164,14 @@ class ApiClient {
     headers,
     params,
     timeout,
+    responseType,
     raise_error = true,
   }: RequestOptions): Promise<ApiResponse<T>> {
     const config: AxiosRequestConfig = {
       headers,
       params,
       timeout,
+      responseType,
       httpsAgent: this.axiosAgent,
     };
     return this.requestWrapper<T>(

src/tools/automate-utils/fetch-screenshots.ts

@@ -59,7 +59,10 @@ async function convertUrlsToBase64(
 ): Promise<Array<{ url: string; base64: string }>> {
   const screenshots = await Promise.all(
     urls.map(async (url) => {
-      const response = await apiClient.get({ url });
+      const response = await apiClient.get({ 
+        url, 
+        responseType: "arraybuffer"
+      });
       // Axios returns response.data as a Buffer for binary data
       const base64 = Buffer.from(response.data).toString("base64");