@symbo.ls/sdk 2.32.2 → 2.32.5

package/src/services/AuthService.js

@@ -1,6 +1,4 @@
 import { BaseService } from './BaseService.js'
-import environment from '../config/environment.js'
-import { getTokenManager } from '../utils/TokenManager.js'
 import {
   ROLE_PERMISSIONS,
   TIER_FEATURES,
@@ -18,147 +16,11 @@ 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
   }
 
-  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')
-    }
-  }
+  // Use BaseService.init/_request/_requireReady implementations
 
   // ==================== AUTH METHODS ====================
 
@@ -174,7 +36,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Registration failed: ${error.message}`)
+      throw new Error(`Registration failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -200,9 +62,6 @@ 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) {
@@ -210,7 +69,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Login failed: ${error.message}`)
+      throw new Error(`Login failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -227,15 +86,13 @@ 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}`)
+      throw new Error(`Logout failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -251,7 +108,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Token refresh failed: ${error.message}`)
+      throw new Error(`Token refresh failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -282,9 +139,6 @@ 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) {
@@ -292,7 +146,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Google auth failed: ${error.message}`)
+      throw new Error(`Google auth failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -323,9 +177,6 @@ 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) {
@@ -333,7 +184,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`GitHub auth failed: ${error.message}`)
+      throw new Error(`GitHub auth failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -364,9 +215,6 @@ 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) {
@@ -374,7 +222,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Google auth callback failed: ${error.message}`)
+      throw new Error(`Google auth callback failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -390,7 +238,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password reset request failed: ${error.message}`)
+      throw new Error(`Password reset request failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -406,7 +254,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password reset confirmation failed: ${error.message}`)
+      throw new Error(`Password reset confirmation failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -422,7 +270,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Registration confirmation failed: ${error.message}`)
+      throw new Error(`Registration confirmation failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -438,7 +286,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password change request failed: ${error.message}`)
+      throw new Error(`Password change request failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -455,7 +303,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Password change confirmation failed: ${error.message}`)
+      throw new Error(`Password change confirmation failed: ${error.message}`, { cause: error })
     }
   }
 
@@ -471,7 +319,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user profile: ${error.message}`)
+      throw new Error(`Failed to get user profile: ${error.message}`, { cause: error })
     }
   }
 
@@ -595,7 +443,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user profile: ${error.message}`)
+      throw new Error(`Failed to get user profile: ${error.message}`, { cause: error })
     }
   }
 
@@ -612,7 +460,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to update user profile: ${error.message}`)
+      throw new Error(`Failed to update user profile: ${error.message}`, { cause: error })
     }
   }
 
@@ -636,7 +484,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user projects: ${error.message}`)
+      throw new Error(`Failed to get user projects: ${error.message}`, { cause: error })
     }
   }
 
@@ -655,7 +503,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user: ${error.message}`)
+      throw new Error(`Failed to get user: ${error.message}`, { cause: error })
     }
   }
 
@@ -674,7 +522,7 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get user by email: ${error.message}`)
+      throw new Error(`Failed to get user by email: ${error.message}`, { cause: error })
     }
   }
 
@@ -690,6 +538,11 @@ 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)
@@ -715,7 +568,12 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get project role: ${error.message}`)
+      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 })
     }
   }
 
@@ -729,6 +587,11 @@ 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)
@@ -754,7 +617,12 @@ export class AuthService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get project role by key: ${error.message}`)
+      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 })
     }
   }
 
@@ -882,7 +750,7 @@ export class AuthService extends BaseService {
     try {
       return await this.getMe()
     } catch (error) {
-      throw new Error(`Failed to get current user: ${error.message}`)
+      throw new Error(`Failed to get current user: ${error.message}`, { cause: error })
     }
   }
 
@@ -1082,7 +950,6 @@ export class AuthService extends BaseService {
     }
     // Clear project role cache
     this._projectRoleCache.clear()
-    this._initialized = false
     this._setReady(false)
   }
 }

package/src/services/BaseService.js

@@ -23,28 +23,14 @@ export class BaseService {
         throw new Error('Service base URL not configured')
       }
 
-      // Initialize token manager
+      // Initialize token manager (singleton). TokenManager handles persistence
       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)
@@ -54,7 +40,10 @@ export class BaseService {
 
   // Update context
   updateContext (context) {
-    this._context = { ...this._context, ...context }
+    // Mutate in place to preserve the shared reference with the SDK context
+    if (context && typeof context === 'object') {
+      Object.assign(this._context, context)
+    }
   }
 
   // Get service status
@@ -123,12 +112,6 @@ 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 {
@@ -149,12 +132,12 @@ export class BaseService {
         } catch {
           // Use default error message
         }
-        throw new Error(error.message || error.error || 'Request failed')
+        throw new Error(error.message || error.error || 'Request failed', { cause: error })
       }
 
       return response.status === 204 ? null : response.json()
     } catch (error) {
-      throw new Error(`Request failed: ${error.message}`)
+      throw new Error(`Request failed: ${error.message}`, { cause: error })
     }
   }
 

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}`)
+      throw new Error(`Failed to list branches: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to create branch: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to delete branch: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to rename branch: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to get branch changes: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to merge branch: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to reset branch: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to publish version: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to check if branch exists: ${error.message}`, { cause: error })
     }
   }
 
@@ -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}`)
+      throw new Error(`Failed to get branch status: ${error.message}`, { cause: error })
     }
   }