fix: use better error handling

Author: deepfates

src/replicate_client.ts

@@ -719,12 +719,16 @@ export class ReplicateClient {
       const response = await this.makeRequest<ReplicateModel>(
         "GET",
         `/models/${owner}/${name}`
-      );
+      ).catch((error) => {
+        throw ErrorHandler.parseAPIError(error);
+      });
 
       // Get model versions
       const versionsResponse = await this.makeRequest<
         ReplicatePage<ModelVersion>
-      >("GET", `/models/${owner}/${name}/versions`);
+      >("GET", `/models/${owner}/${name}/versions`).catch((error) => {
+        throw ErrorHandler.parseAPIError(error);
+      });
 
       const model: Model = {
         id: `${response.owner}/${response.name}`,
@@ -763,6 +767,9 @@ export class ReplicateClient {
       });
       return model;
     } catch (error) {
+      if (error instanceof Promise) {
+        throw new ReplicateError("Failed to fetch model details");
+      }
       throw ErrorHandler.parseAPIError(error as Response);
     }
   }

src/tools/handlers.ts

@@ -25,6 +25,9 @@ function getErrorMessage(error: unknown): string {
   if (error instanceof Error) {
     return error.message;
   }
+  if (error instanceof Promise) {
+    return "An asynchronous error occurred. Please try again.";
+  }
   return String(error);
 }