refactor(common): remove redundant logs

Author: jamesgeorge007

packages/hoppscotch-common/src/components/collections/ImportExport.vue

@@ -141,29 +141,17 @@ const importToPersonalWorkspace = async (collections: HoppCollection[]) => {
           // Replace local collections with backend collections
           setRESTCollections(fetchResult.right)
         } else {
-          console.warn(
-            "Failed to fetch collections from backend after import:",
-            fetchResult.left
-          )
-          // Still append to local store as fallback
+          // Failed to fetch, append to local store as fallback
           appendRESTCollections(collections)
         }
 
         return E.right({ success: true })
       }
       // Backend import failed, fall back to local storage
-      console.warn(
-        "Backend import failed, falling back to local storage:",
-        res.left
-      )
       appendRESTCollections(collections)
       return E.right({ success: true })
-    } catch (error) {
+    } catch {
       // Backend import failed, fall back to local storage
-      console.warn(
-        "Backend import failed, falling back to local storage:",
-        error
-      )
       appendRESTCollections(collections)
       return E.right({ success: true })
     }

packages/hoppscotch-common/src/components/collections/graphql/ImportExport.vue

@@ -258,31 +258,19 @@ const handleImportToStore = async (gqlCollections: HoppCollection[]) => {
           // Replace local collections with backend collections
           setGraphqlCollections(fetchResult.right)
         } else {
-          console.warn(
-            "Failed to fetch collections from backend after import:",
-            fetchResult.left
-          )
-          // Still append to local store as fallback
+          // Failed to fetch, append to local store as fallback
           appendGraphqlCollections(gqlCollections)
         }
 
         toast.success(t("state.file_imported"))
         return
       }
       // Backend import failed, fall back to local storage
-      console.warn(
-        "Backend import failed, falling back to local storage:",
-        res.left
-      )
       appendGraphqlCollections(gqlCollections)
       toast.success(t("state.file_imported"))
       return
-    } catch (error) {
+    } catch {
       // Backend import failed, fall back to local storage
-      console.warn(
-        "Backend import failed, falling back to local storage:",
-        error
-      )
       appendGraphqlCollections(gqlCollections)
       toast.success(t("state.file_imported"))
       return

packages/hoppscotch-common/src/components/mockServer/CreateMockServer.vue

@@ -476,7 +476,6 @@ const createMockServer = async () => {
     TE.match(
       (error) => {
         // `error` here is the message string produced by the mutation helper.
-        console.error("Failed to create mock server:", error)
         // Show the backend-provided error message if available, otherwise fallback to generic
         toast.error(String(error) || t("error.something_went_wrong"))
         loading.value = false
@@ -508,8 +507,7 @@ const toggleMockServer = async () => {
   await pipe(
     updateMockServer(existingMockServer.value.id, { isActive: newActiveState }),
     TE.match(
-      (error) => {
-        console.error("Failed to update mock server:", error)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
       },

packages/hoppscotch-common/src/components/mockServer/EditMockServer.vue

@@ -221,8 +221,7 @@ const updateMockServer = async () => {
   await pipe(
     updateMockServerMutation(props.mockServer.id, payload),
     TE.match(
-      (error) => {
-        console.error("Failed to update mock server:", error)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
       },
@@ -253,8 +252,7 @@ const toggleMockServer = async () => {
   await pipe(
     updateMockServerMutation(props.mockServer.id, { isActive: newActiveState }),
     TE.match(
-      (error) => {
-        console.error("Failed to update mock server:", error)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
       },

packages/hoppscotch-common/src/components/mockServer/LogSection.vue

@@ -104,11 +104,7 @@ const formattedContent = computed(() => {
 const copyContent = () => {
   if (!props.content) return
 
-  try {
-    copyToClipboard(formattedContent.value)
-    copySuccess.value = true
-  } catch (error) {
-    console.error("Failed to copy content:", error)
-  }
+  copyToClipboard(formattedContent.value)
+  copySuccess.value = true
 }
 </script>

packages/hoppscotch-common/src/components/mockServer/MockServerDashboard.vue

@@ -282,8 +282,7 @@ const toggleMockServer = async (mockServer: MockServer) => {
   await pipe(
     updateMockServerMutation(mockServer.id, { isActive: newActiveState }),
     TE.match(
-      (error) => {
-        console.error("Failed to update mock server:", error)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
       },
@@ -325,8 +324,7 @@ const confirmDelete = async () => {
   await pipe(
     deleteMockServerMutation(mockServer.id),
     TE.match(
-      (error) => {
-        console.error("Failed to delete mock server:", error)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
         pendingMockServerToDelete.value = null

packages/hoppscotch-common/src/components/mockServer/MockServerLogs.vue

@@ -143,8 +143,7 @@ const fetchLogs = async () => {
   await pipe(
     getMockServerLogs(props.mockServerID),
     TE.match(
-      (err) => {
-        console.error("Failed to load logs", err)
+      () => {
         toast.error(t("error.something_went_wrong"))
         loading.value = false
       },
@@ -172,8 +171,7 @@ const confirmDelete = async () => {
     await pipe(
       deleteMockServerLog(logToDelete.value),
       TE.match(
-        (err) => {
-          console.error("Failed to delete log", err)
+        () => {
           toast.error(t("error.something_went_wrong"))
         },
         (res) => {

packages/hoppscotch-common/src/helpers/backend/mutations/MockServer.ts

@@ -224,7 +224,6 @@ export const getErrorMessage = (err: GQLError<string> | string | Error) => {
   if (!gErr) return t("error.something_went_wrong")
 
   if (gErr.type === "network_error") {
-    console.error(gErr.error)
     return t("error.network_error")
   }
 

packages/hoppscotch-common/src/helpers/backend/mutations/UserCollection.ts

@@ -77,9 +77,7 @@ function convertUserRequestToHoppRequest(
     }
 
     return request
-  } catch (error) {
-    console.warn("Failed to parse user request data:", error)
-
+  } catch {
     // Return a default request if parsing fails
     if (userRequest.type === ReqType.Rest) {
       const defaultRequest = getDefaultRESTRequest()
@@ -115,8 +113,7 @@ function parseUserCollectionData(data: string | null | undefined) {
       headers: parsedData?.headers || defaultDataProps.headers,
       variables: parsedData?.variables || defaultDataProps.variables,
     }
-  } catch (error) {
-    console.warn("Failed to parse user collection data:", error)
+  } catch {
     return defaultDataProps
   }
 }