Fix reporting of server-side errors in "bw sync". (#6855)

Co-authored-by: SmithThe4th <gsmith@bitwarden.com>

apps/cli/src/models/response.ts

@@ -1,21 +1,36 @@
 // FIXME: Update this file to be type safe and remove this and next line
 // @ts-strict-ignore
+import { ErrorResponse } from "@bitwarden/common/models/response/error.response";
+
 import { BaseResponse } from "./response/base.response";
 
+function getErrorMessage(error: unknown): string {
+  if (typeof error === "string") {
+    return error;
+  }
+  if (error instanceof ErrorResponse) {
+    const message = error.getSingleMessage();
+    if (message) {
+      return message;
+    }
+  }
+  if (error instanceof Error) {
+    return String(error);
+  }
+  if (error) {
+    const errorWithMessage: { message?: unknown } = error; // To placate TypeScript.
+    if (errorWithMessage.message && typeof errorWithMessage.message === "string") {
+      return errorWithMessage.message;
+    }
+  }
+  return JSON.stringify(error);
+}
+
 export class Response {
   static error(error: any, data?: any): Response {
     const res = new Response();
     res.success = false;
-    if (typeof error === "string") {
+    res.message = getErrorMessage(error);
-      res.message = error;
-    } else {
-      res.message =
-        error.message != null
-          ? error.message
-          : error.toString() === "[object Object]"
-            ? JSON.stringify(error)
-            : error.toString();
-    }
     res.data = data;
     return res;
   }

apps/cli/src/vault/sync.command.ts

@@ -21,7 +21,9 @@ export class SyncCommand {
       const res = new MessageResponse("Syncing complete.", null);
       return Response.success(res);
     } catch (e) {
-      return Response.error("Syncing failed: " + e.toString());
+      const response = Response.error(e);
+      response.message = "Syncing failed: " + response.message;
+      return response;
     }
   }
 

libs/common/src/services/api.service.ts

@@ -1948,7 +1948,6 @@ export class ApiService implements ApiServiceAbstraction {
           responseJson.error === "invalid_grant")
       ) {
         await this.logoutCallback("invalidGrantError");
-        return null;
       }
     }