Improve errors

jest.config.js

@@ -16,7 +16,6 @@ const config = {
     'jest-watch-typeahead/filename',
     'jest-watch-typeahead/testname',
   ],
-  globalSetup: './jest-disable-built-in-fetch.js',
   projects: [
     {
       preset: 'ts-jest',
@@ -28,6 +27,8 @@ const config = {
         'env/',
         'token.test.ts',
       ],
+      // make sure built-in Node.js fetch doesn't get replaced for consistency
+      globals: { fetch: global.fetch, AbortController: global.AbortController },
     },
     {
       preset: 'ts-jest',

package.json

@@ -84,7 +84,6 @@
     "@types/jest": "^29.5.11",
     "@typescript-eslint/eslint-plugin": "^6.19.0",
     "@typescript-eslint/parser": "^6.19.0",
-    "abort-controller": "^3.0.0",
     "brotli-size": "^4.0.0",
     "eslint": "^8.56.0",
     "eslint-config-prettier": "^9.1.0",

src/errors/index.ts

@@ -1,6 +1,5 @@
-export * from './http-error-handler';
 export * from './meilisearch-api-error';
-export * from './meilisearch-communication-error';
+export * from './meilisearch-request-error';
 export * from './meilisearch-error';
 export * from './meilisearch-timeout-error';
 export * from './version-hint-message';

src/errors/meilisearch-api-error.ts

@@ -1,30 +1,20 @@
-import { MeiliSearchErrorInfo } from '../types';
+import { MeiliSearchErrorResponse } from '../types';
 import { MeiliSearchError } from './meilisearch-error';
 
-const MeiliSearchApiError = class extends MeiliSearchError {
-  httpStatus: number;
-  code: string;
-  link: string;
-  type: string;
-  stack?: string;
+export class MeiliSearchApiError extends MeiliSearchError {
+  override name = 'MeiliSearchApiError';
+  override cause?: MeiliSearchErrorResponse;
+  readonly response: Response;
 
-  constructor(error: MeiliSearchErrorInfo, status: number) {
-    super(error.message);
+  constructor(response: Response, responseBody?: MeiliSearchErrorResponse) {
+    super(
+      responseBody?.message ?? `${response.status}: ${response.statusText}`,
+    );
 
-    // Make errors comparison possible. ex: error instanceof MeiliSearchApiError.
-    Object.setPrototypeOf(this, MeiliSearchApiError.prototype);
+    this.response = response;
 
-    this.name = 'MeiliSearchApiError';
-
-    this.code = error.code;
-    this.type = error.type;
-    this.link = error.link;
-    this.message = error.message;
-    this.httpStatus = status;
-
-    if (Error.captureStackTrace) {
-      Error.captureStackTrace(this, MeiliSearchApiError);
+    if (responseBody !== undefined) {
+      this.cause = responseBody;
     }
   }
-};
-export { MeiliSearchApiError };
+}

src/errors/meilisearch-error.ts

@@ -1,16 +1,7 @@
-class MeiliSearchError extends Error {
-  constructor(message: string) {
-    super(message);
+export class MeiliSearchError extends Error {
+  override name = 'MeiliSearchError';
 
-    // Make errors comparison possible. ex: error instanceof MeiliSearchError.
-    Object.setPrototypeOf(this, MeiliSearchError.prototype);
-
-    this.name = 'MeiliSearchError';
-
-    if (Error.captureStackTrace) {
-      Error.captureStackTrace(this, MeiliSearchError);
-    }
+  constructor(...params: ConstructorParameters<typeof Error>) {
+    super(...params);
   }
 }
-
-export { MeiliSearchError };

src/errors/meilisearch-request-error.ts

@@ -0,0 +1,9 @@
+import { MeiliSearchError } from './meilisearch-error';
+
+export class MeiliSearchRequestError extends MeiliSearchError {
+  override name = 'MeiliSearchRequestError';
+
+  constructor(url: string, cause: unknown) {
+    super(`Request to ${url} has failed`, { cause });
+  }
+}

src/errors/meilisearch-timeout-error.ts

@@ -1,18 +1,9 @@
 import { MeiliSearchError } from './meilisearch-error';
 
-class MeiliSearchTimeOutError extends MeiliSearchError {
+export class MeiliSearchTimeOutError extends MeiliSearchError {
+  override name = 'MeiliSearchTimeOutError';
+
   constructor(message: string) {
     super(message);
-
-    // Make errors comparison possible. ex: error instanceof MeiliSearchTimeOutError.
-    Object.setPrototypeOf(this, MeiliSearchTimeOutError.prototype);
-
-    this.name = 'MeiliSearchTimeOutError';
-
-    if (Error.captureStackTrace) {
-      Error.captureStackTrace(this, MeiliSearchTimeOutError);
-    }
   }
 }
-
-export { MeiliSearchTimeOutError };

src/http-requests.ts

@@ -3,8 +3,8 @@ import { PACKAGE_VERSION } from './package-version';
 
 import {
   MeiliSearchError,
-  httpResponseErrorHandler,
-  httpErrorHandler,
+  MeiliSearchApiError,
+  MeiliSearchRequestError,
 } from './errors';
 
 import { addTrailingSlash, addProtocolIfNotPresent } from './utils';
@@ -143,35 +143,36 @@ class HttpRequests {
     }
 
     const headers = { ...this.headers, ...config.headers };
+    const responsePromise = this.fetchWithTimeout(
+      constructURL.toString(),
+      {
+        ...config,
+        ...this.requestConfig,
+        method,
+        body,
+        headers,
+      },
+      this.requestTimeout,
+    );
 
-    try {
-      const result = this.fetchWithTimeout(
-        constructURL.toString(),
-        {
-          ...config,
-          ...this.requestConfig,
-          method,
-          body,
-          headers,
-        },
-        this.requestTimeout,
-      );
-
-      // When using a custom HTTP client, the response is returned to allow the user to parse/handle it as they see fit
-      if (this.httpClient) {
-        return await result;
-      }
+    const response = await responsePromise.catch((error: unknown) => {
+      throw new MeiliSearchRequestError(constructURL.toString(), error);
+    });
+
+    // When using a custom HTTP client, the response is returned to allow the user to parse/handle it as they see fit
+    if (this.httpClient !== undefined) {
+      return response;
+    }
 
-      const response = await result.then((res: any) =>
-        httpResponseErrorHandler(res),
-      );
-      const parsedBody = await response.json().catch(() => undefined);
+    const responseBody = await response.text();
+    const parsedResponse =
+      responseBody === '' ? undefined : JSON.parse(responseBody);
 
-      return parsedBody;
-    } catch (e: any) {
-      const stack = e.stack;
-      httpErrorHandler(e, stack, constructURL.toString());
+    if (!response.ok) {
+      throw new MeiliSearchApiError(response, parsedResponse);
     }
+
+    return parsedResponse;
   }
 
   async fetchWithTimeout(

src/indexes.ts

@@ -9,7 +9,7 @@
 
 import {
   MeiliSearchError,
-  MeiliSearchCommunicationError,
+  MeiliSearchRequestError,
   versionErrorHintMessage,
   MeiliSearchApiError,
 } from './errors';
@@ -360,7 +360,7 @@ class Index<T extends Record<string, any> = Record<string, any>> {
           Promise<ResourceResults<D[]>>
         >(url, parameters);
       } catch (e) {
-        if (e instanceof MeiliSearchCommunicationError) {
+        if (e instanceof MeiliSearchRequestError) {
           e.message = versionErrorHintMessage(e.message, 'getDocuments');
         } else if (e instanceof MeiliSearchApiError) {
           e.message = versionErrorHintMessage(e.message, 'getDocuments');
@@ -585,10 +585,7 @@ class Index<T extends Record<string, any> = Record<string, any>> {
 
       return new EnqueuedTask(task);
     } catch (e) {
-      if (
-        e instanceof MeiliSearchCommunicationError &&
-        isDocumentsDeletionQuery
-      ) {
+      if (e instanceof MeiliSearchRequestError && isDocumentsDeletionQuery) {
         e.message = versionErrorHintMessage(e.message, 'deleteDocuments');
       } else if (e instanceof MeiliSearchApiError) {
         e.message = versionErrorHintMessage(e.message, 'deleteDocuments');

src/types/types.ts

@@ -546,7 +546,7 @@ export type TaskObject = Omit<EnqueuedTaskObject, 'taskUid'> & {
     // Query parameters used to filter the tasks
     originalFilter?: string;
   };
-  error: MeiliSearchErrorInfo | null;
+  error: MeiliSearchErrorResponse | null;
   duration: string;
   startedAt: string;
   finishedAt: string;
@@ -655,13 +655,16 @@ export interface FetchError extends Error {
   code: string;
 }
 
-export type MeiliSearchErrorInfo = {
-  code: string;
-  link: string;
+export type MeiliSearchErrorResponse = {
   message: string;
+  // @TODO: Could be typed, but will it be kept updated? https://www.meilisearch.com/docs/reference/errors/error_codes
+  code: string;
+  // @TODO: Could be typed https://www.meilisearch.com/docs/reference/errors/overview#errors
   type: string;
+  link: string;
 };
 
+// @TODO: This doesn't seem to be up to date, and its usefullness comes into question.
 export const ErrorStatusCode = {
   /** @see https://www.meilisearch.com/docs/reference/errors/error_codes#index_creation_failed */
   INDEX_CREATION_FAILED: 'index_creation_failed',