refactor: remove throws from error handling to preserve original behaviour

Author: DanielMicrosoft

src/web/src/services/commandApi.ts

@@ -1,5 +1,4 @@
 import axios from "axios";
-import { apiErrorHandler } from "./errorHandler";
 
 export const commandApi = {
   getCommand: async (leafUrl: string): Promise<any> => {
@@ -99,11 +98,7 @@ export const commandApi = {
       arg: string;
     },
   ): Promise<any> => {
-    try {
-      const response = await axios.post(subresourceUrl, data);
-      return response.data;
-    } catch (err: any) {
-      apiErrorHandler.handleApiError(err, "Failed to create subresource");
-    }
+    const response = await axios.post(subresourceUrl, data);
+    return response.data;
   },
 } as const;

src/web/src/services/errorHandler.ts

@@ -11,10 +11,4 @@ export const apiErrorHandler = {
   isHttpError: (err: any, statusCode: number): boolean => {
     return err.response?.status === statusCode;
   },
-
-  handleApiError: (err: any, context: string = ""): never => {
-    console.error(context, err);
-    const message = apiErrorHandler.getErrorMessage(err);
-    throw new Error(message);
-  },
 } as const;