@symbo.ls/sdk 2.32.0 → 2.32.2

package/src/services/AuthService.js

@@ -1,4 +1,6 @@
 import { BaseService } from './BaseService.js'
+import environment from '../config/environment.js'
+import { getTokenManager } from '../utils/TokenManager.js'
 import {
   ROLE_PERMISSIONS,
   TIER_FEATURES,
@@ -16,11 +18,147 @@ export class AuthService extends BaseService {
       'pro2',
       'enterprise'
     ])
+    this._initialized = false
+    this._apiUrl = null
+    this._tokenManager = null
     this._projectRoleCache = new Map() // Cache for project roles
     this._roleCacheExpiry = 5 * 60 * 1000 // 5 minutes cache expiry
   }
 
-  // Use BaseService.init/_request/_requireReady implementations
+  init({ context }) {
+    try {
+      const { appKey, authToken } = context || this._context
+
+      // Get base URL from environment config
+      this._apiUrl = environment.apiUrl
+
+      if (!this._apiUrl) {
+        throw new Error('Auth service base URL not configured')
+      }
+
+      // Initialize token manager
+      this._tokenManager = getTokenManager({
+        apiUrl: this._apiUrl,
+        onTokenRefresh: (tokens) => {
+          // Update context with new token
+          this.updateContext({ authToken: tokens.accessToken })
+        },
+        onTokenExpired: () => {
+          // Clear context token
+          this.updateContext({ authToken: null })
+        },
+        onTokenError: (error) => {
+          console.error('Token management error:', error)
+        }
+      })
+
+      if (authToken && !this._tokenManager.hasTokens()) {
+        this._tokenManager.setTokens({ access_token: authToken })
+      }
+
+      // Store masked configuration info
+      this._info = {
+        config: {
+          apiUrl: this._apiUrl,
+          appKey: appKey
+            ? `${appKey.substr(0, 4)}...${appKey.substr(-4)}`
+            : null,
+          hasToken: Boolean(authToken)
+        }
+      }
+
+      this._initialized = true
+      this._setReady()
+    } catch (error) {
+      this._setError(error)
+      throw error
+    }
+  }
+
+  // Helper method to make HTTP requests
+  async _request(endpoint, options = {}) {
+    const url = `${this._apiUrl}/core${endpoint}`
+
+    const defaultHeaders = {}
+
+    // Only set Content-Type for JSON requests, not for FormData
+    if (!(options.body instanceof FormData)) {
+      defaultHeaders['Content-Type'] = 'application/json'
+    }
+
+    // Use TokenManager for automatic token management
+    if (this._requiresInit(options.methodName) && this._tokenManager) {
+      try {
+        // Ensure we have a valid token (will refresh if needed)
+        const validToken = await this._tokenManager.ensureValidToken()
+
+        if (validToken) {
+          const authHeader = this._tokenManager.getAuthHeader()
+          if (authHeader) {
+            defaultHeaders.Authorization = authHeader
+          }
+        }
+      } catch (error) {
+        console.warn(
+          'Token management failed, proceeding without authentication:',
+          error
+        )
+      }
+    } else if (this._requiresInit(options.methodName)) {
+      // Fallback to context token if TokenManager not available
+      const { authToken } = this._context
+      if (authToken) {
+        defaultHeaders.Authorization = `Bearer ${authToken}`
+      }
+    }
+
+    try {
+      const response = await fetch(url, {
+        ...options,
+        headers: {
+          ...defaultHeaders,
+          ...options.headers
+        }
+      })
+
+      if (!response.ok) {
+        let error = {
+          message: `HTTP ${response.status}: ${response.statusText}`
+        }
+        try {
+          error = await response.json()
+        } catch {
+          // Use default error message
+        }
+        throw new Error(error.message || error.error || 'Request failed')
+      }
+
+      return response.status === 204 ? null : response.json()
+    } catch (error) {
+      throw new Error(`Request failed: ${error.message}`)
+    }
+  }
+
+  _requiresInit(methodName) {
+    const noInitMethods = new Set([
+      'register',
+      'login',
+      'googleAuth',
+      'googleAuthCallback',
+      'githubAuth',
+      'requestPasswordReset',
+      'confirmPasswordReset',
+      'confirmRegistration',
+      'verifyEmail'
+    ])
+    return !noInitMethods.has(methodName)
+  }
+
+  _requireReady(methodName) {
+    if (this._requiresInit(methodName) && !this._initialized) {
+      throw new Error('Auth service not initialized')
+    }
+  }
 
   // ==================== AUTH METHODS ====================
 
@@ -36,7 +174,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Registration failed: ${error.message}`, { cause: error })
+      throw new Error(`Registration failed: ${error.message}`)
     }
   }
 
@@ -62,6 +200,9 @@ export class AuthService extends BaseService {
         if (this._tokenManager) {
           this._tokenManager.setTokens(tokenData)
         }
+
+        // Update context for backward compatibility
+        this.updateContext({ authToken: tokens.accessToken })
       }
 
       if (response.success) {
@@ -69,7 +210,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Login failed: ${error.message}`, { cause: error })
+      throw new Error(`Login failed: ${error.message}`)
     }
   }
 
@@ -86,13 +227,15 @@ export class AuthService extends BaseService {
       if (this._tokenManager) {
         this._tokenManager.clearTokens()
       }
+      this.updateContext({ authToken: null })
     } catch (error) {
       // Even if the API call fails, clear local tokens
       if (this._tokenManager) {
         this._tokenManager.clearTokens()
       }
+      this.updateContext({ authToken: null })
 
-      throw new Error(`Logout failed: ${error.message}`, { cause: error })
+      throw new Error(`Logout failed: ${error.message}`)
     }
   }
 
@@ -108,7 +251,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Token refresh failed: ${error.message}`, { cause: error })
+      throw new Error(`Token refresh failed: ${error.message}`)
     }
   }
 
@@ -139,6 +282,9 @@ export class AuthService extends BaseService {
         if (this._tokenManager) {
           this._tokenManager.setTokens(tokenData)
         }
+
+        // Update context for backward compatibility
+        this.updateContext({ authToken: tokens.accessToken })
       }
 
       if (response.success) {
@@ -146,7 +292,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Google auth failed: ${error.message}`, { cause: error })
+      throw new Error(`Google auth failed: ${error.message}`)
     }
   }
 
@@ -177,6 +323,9 @@ export class AuthService extends BaseService {
         if (this._tokenManager) {
           this._tokenManager.setTokens(tokenData)
         }
+
+        // Update context for backward compatibility
+        this.updateContext({ authToken: tokens.accessToken })
       }
 
       if (response.success) {
@@ -184,7 +333,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`GitHub auth failed: ${error.message}`, { cause: error })
+      throw new Error(`GitHub auth failed: ${error.message}`)
     }
   }
 
@@ -215,6 +364,9 @@ export class AuthService extends BaseService {
         if (this._tokenManager) {
           this._tokenManager.setTokens(tokenData)
         }
+
+        // Update context for backward compatibility
+        this.updateContext({ authToken: tokens.accessToken })
       }
 
       if (response.success) {
@@ -222,7 +374,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Google auth callback failed: ${error.message}`, { cause: error })
+      throw new Error(`Google auth callback failed: ${error.message}`)
     }
   }
 
@@ -238,7 +390,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password reset request failed: ${error.message}`, { cause: error })
+      throw new Error(`Password reset request failed: ${error.message}`)
     }
   }
 
@@ -254,7 +406,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password reset confirmation failed: ${error.message}`, { cause: error })
+      throw new Error(`Password reset confirmation failed: ${error.message}`)
     }
   }
 
@@ -270,7 +422,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Registration confirmation failed: ${error.message}`, { cause: error })
+      throw new Error(`Registration confirmation failed: ${error.message}`)
     }
   }
 
@@ -286,7 +438,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password change request failed: ${error.message}`, { cause: error })
+      throw new Error(`Password change request failed: ${error.message}`)
     }
   }
 
@@ -303,7 +455,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password change confirmation failed: ${error.message}`, { cause: error })
+      throw new Error(`Password change confirmation failed: ${error.message}`)
     }
   }
 
@@ -319,7 +471,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user profile: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get user profile: ${error.message}`)
     }
   }
 
@@ -443,7 +595,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user profile: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get user profile: ${error.message}`)
     }
   }
 
@@ -460,7 +612,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to update user profile: ${error.message}`, { cause: error })
+      throw new Error(`Failed to update user profile: ${error.message}`)
     }
   }
 
@@ -484,7 +636,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user projects: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get user projects: ${error.message}`)
     }
   }
 
@@ -503,7 +655,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get user: ${error.message}`)
     }
   }
 
@@ -522,7 +674,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user by email: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get user by email: ${error.message}`)
     }
   }
 
@@ -538,11 +690,6 @@ export class AuthService extends BaseService {
       throw new Error('Project ID is required')
     }
 
-    // If there are no valid tokens, treat user as guest for public access
-    if (!this.hasValidTokens()) {
-      return 'guest'
-    }
-
     // Check cache first
     const cacheKey = `role_${projectId}`
     const cached = this._projectRoleCache.get(cacheKey)
@@ -568,12 +715,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      const message = error?.message || ''
-      // If request failed due to missing/invalid auth, default to guest
-      if (/401|403|unauthorized|no token|invalid token/iu.test(message)) {
-        return 'guest'
-      }
-      throw new Error(`Failed to get project role: ${message}`, { cause: error })
+      throw new Error(`Failed to get project role: ${error.message}`)
     }
   }
 
@@ -587,11 +729,6 @@ export class AuthService extends BaseService {
       throw new Error('Project key is required')
     }
 
-    // If there are no valid tokens, treat user as guest for public access
-    if (!this.hasValidTokens()) {
-      return 'guest'
-    }
-
     // Check cache first
     const cacheKey = `role_key_${projectKey}`
     const cached = this._projectRoleCache.get(cacheKey)
@@ -617,12 +754,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      const message = error?.message || ''
-      // If request failed due to missing/invalid auth, default to guest
-      if (/401|403|unauthorized|no token|invalid token/iu.test(message)) {
-        return 'guest'
-      }
-      throw new Error(`Failed to get project role by key: ${message}`, { cause: error })
+      throw new Error(`Failed to get project role by key: ${error.message}`)
     }
   }
 
@@ -750,7 +882,7 @@ export class AuthService extends BaseService {
     try {
       return await this.getMe()
     } catch (error) {
-      throw new Error(`Failed to get current user: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get current user: ${error.message}`)
     }
   }
 
@@ -950,6 +1082,7 @@ export class AuthService extends BaseService {
     }
     // Clear project role cache
     this._projectRoleCache.clear()
+    this._initialized = false
     this._setReady(false)
   }
 }

package/src/services/BaseService.js

@@ -23,14 +23,28 @@ export class BaseService {
         throw new Error('Service base URL not configured')
       }
 
-      // Initialize token manager (singleton). TokenManager handles persistence
+      // Initialize token manager
       this._tokenManager = getTokenManager({
         apiUrl: this._apiUrl,
+        onTokenRefresh: tokens => {
+          // Update context with new token
+          this.updateContext({ authToken: tokens.accessToken })
+        },
+        onTokenExpired: () => {
+          // Clear context token
+          this.updateContext({ authToken: null })
+        },
         onTokenError: error => {
           console.error('Token management error:', error)
         }
       })
 
+      // Set initial token if provided
+      const { authToken } = this._context
+      if (authToken && !this._tokenManager.hasTokens()) {
+        this._tokenManager.setTokens({ access_token: authToken })
+      }
+
       this._setReady()
     } catch (error) {
       this._setError(error)
@@ -40,10 +54,7 @@ export class BaseService {
 
   // Update context
   updateContext (context) {
-    // Mutate in place to preserve the shared reference with the SDK context
-    if (context && typeof context === 'object') {
-      Object.assign(this._context, context)
-    }
+    this._context = { ...this._context, ...context }
   }
 
   // Get service status
@@ -112,6 +123,12 @@ export class BaseService {
           error
         )
       }
+    } else if (this._requiresInit(options.methodName)) {
+      // Fallback to context token if TokenManager not available
+      const { authToken } = this._context
+      if (authToken) {
+        defaultHeaders.Authorization = `Bearer ${authToken}`
+      }
     }
 
     try {
@@ -132,12 +149,12 @@ export class BaseService {
         } catch {
           // Use default error message
         }
-        throw new Error(error.message || error.error || 'Request failed', { cause: error })
+        throw new Error(error.message || error.error || 'Request failed')
       }
 
       return response.status === 204 ? null : response.json()
     } catch (error) {
-      throw new Error(`Request failed: ${error.message}`, { cause: error })
+      throw new Error(`Request failed: ${error.message}`)
     }
   }
 

package/src/services/BranchService.js

@@ -22,7 +22,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to list branches: ${error.message}`, { cause: error })
+      throw new Error(`Failed to list branches: ${error.message}`)
     }
   }
 
@@ -51,7 +51,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to create branch: ${error.message}`, { cause: error })
+      throw new Error(`Failed to create branch: ${error.message}`)
     }
   }
 
@@ -83,7 +83,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to delete branch: ${error.message}`, { cause: error })
+      throw new Error(`Failed to delete branch: ${error.message}`)
     }
   }
 
@@ -121,7 +121,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to rename branch: ${error.message}`, { cause: error })
+      throw new Error(`Failed to rename branch: ${error.message}`)
     }
   }
 
@@ -165,7 +165,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get branch changes: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get branch changes: ${error.message}`)
     }
   }
 
@@ -220,7 +220,7 @@ export class BranchService extends BaseService {
       ) {
         throw new Error(`Merge conflicts detected: ${error.message}`)
       }
-      throw new Error(`Failed to merge branch: ${error.message}`, { cause: error })
+      throw new Error(`Failed to merge branch: ${error.message}`)
     }
   }
 
@@ -251,7 +251,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to reset branch: ${error.message}`, { cause: error })
+      throw new Error(`Failed to reset branch: ${error.message}`)
     }
   }
 
@@ -280,7 +280,7 @@ export class BranchService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to publish version: ${error.message}`, { cause: error })
+      throw new Error(`Failed to publish version: ${error.message}`)
     }
   }
 
@@ -322,7 +322,7 @@ export class BranchService extends BaseService {
       const branches = await this.listBranches(projectId)
       return branches?.data?.includes(branchName) || false
     } catch (error) {
-      throw new Error(`Failed to check if branch exists: ${error.message}`, { cause: error })
+      throw new Error(`Failed to check if branch exists: ${error.message}`)
     }
   }
 
@@ -405,7 +405,7 @@ export class BranchService extends BaseService {
         canRename: exists && branchName !== 'main'
       }
     } catch (error) {
-      throw new Error(`Failed to get branch status: ${error.message}`, { cause: error })
+      throw new Error(`Failed to get branch status: ${error.message}`)
     }
   }