Add more helpful error messaging to handleAuth

Author: ajlai-orkes

src/integration-tests/AuthorizationClient.test.ts

@@ -15,7 +15,6 @@ import {
 describe("AuthorizationClient", () => {
   jest.setTimeout(60000);
 
-  const clientPromise = orkesConductorClient();
   const suffix = Date.now();
 
   let authClient: AuthorizationClient;
@@ -31,8 +30,19 @@ describe("AuthorizationClient", () => {
   const groupsToCleanup: string[] = [];
 
   beforeAll(async () => {
-    const client = await clientPromise;
-    const clients = new OrkesClients(client);
+    // Retry client creation to handle transient auth failures in CI
+    const maxAttempts = 3;
+    let client;
+    for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+      try {
+        client = await orkesConductorClient();
+        break;
+      } catch (e) {
+        if (attempt === maxAttempts) throw e;
+        await new Promise((r) => setTimeout(r, 2000 * attempt));
+      }
+    }
+    const clients = new OrkesClients(client!);
     authClient = clients.getAuthorizationClient();
     metadataClient = clients.getMetadataClient();
 
@@ -57,6 +67,8 @@ describe("AuthorizationClient", () => {
   });
 
   afterAll(async () => {
+    // Skip cleanup if beforeAll failed (e.g. auth error in CI) and clients were never set
+    if (!authClient || !metadataClient) return;
     // Cleanup groups first (they reference users)
     for (const gid of groupsToCleanup) {
       try {

src/sdk/createConductorClient/helpers/handleAuth.ts

@@ -54,10 +54,14 @@ export const handleAuth = async (
         logger?.debug(`Auth error code from server: ${errorCode}`);
       }
 
+      const statusHint =
+        response?.status != null ? ` (HTTP ${response.status})` : "";
       const message =
         error && typeof error === "object" && "message" in error
-          ? String((error as { message: unknown }).message)
-          : "Unknown error";
+          ? String((error as { message: unknown }).message) + statusHint
+          : error && typeof error === "object" && "error" in error
+            ? String((error as { error: unknown }).error) + statusHint
+            : `Unknown error${statusHint}`;
       throw new ConductorSdkError(
         `Failed to generate authorization token: ${message}`,
         error instanceof Error ? error : undefined