@symbo.ls/sdk 2.32.11 → 2.32.13

package/dist/node/utils/services.js

@@ -222,8 +222,6 @@ const SERVICE_METHODS = {
   suspendUser: "admin",
   promoteToAdmin: "admin",
   demoteFromAdmin: "admin",
-  // Utility methods
-  getHealthStatus: "core",
   // Screenshot methods
   createScreenshotProject: "screenshot",
   getProjectScreenshots: "screenshot",
@@ -236,7 +234,33 @@ const SERVICE_METHODS = {
   getComponentScreenshot: "screenshot",
   getScreenshotByKey: "screenshot",
   getQueueStatistics: "screenshot",
-  refreshThumbnail: "screenshot"
+  refreshThumbnail: "screenshot",
+  // Tracking methods
+  configureTracking: "tracking",
+  trackEvent: "tracking",
+  trackError: "tracking",
+  captureException: "tracking",
+  logMessage: "tracking",
+  logDebug: "tracking",
+  logInfo: "tracking",
+  logWarning: "tracking",
+  logWarn: "tracking",
+  logErrorMessage: "tracking",
+  logError: "tracking",
+  addBreadcrumb: "tracking",
+  trackMeasurement: "tracking",
+  trackView: "tracking",
+  setUser: "tracking",
+  clearUser: "tracking",
+  setSession: "tracking",
+  clearSession: "tracking",
+  setGlobalAttributes: "tracking",
+  setGlobalAttribute: "tracking",
+  removeGlobalAttribute: "tracking",
+  flushQueue: "tracking",
+  getClient: "tracking",
+  isEnabled: "tracking",
+  isInitialized: "tracking"
 };
 export {
   SERVICE_METHODS

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@symbo.ls/sdk",
-  "version": "2.32.11",
+  "version": "2.32.13",
   "type": "module",
   "main": "dist/esm/index.js",
   "module": "dist/esm/index.js",
@@ -46,10 +46,12 @@
     "test:user": "cross-env NODE_ENV=$NODE_ENV npx tape integration-tests/index.js integration-tests/user/*.test.js | tap-spec"
   },
   "dependencies": {
-    "@domql/element": "^2.32.11",
-    "@domql/utils": "^2.32.11",
-    "@symbo.ls/router": "^2.32.11",
-    "@symbo.ls/socket": "^2.32.11",
+    "@domql/element": "^2.32.13",
+    "@domql/utils": "^2.32.13",
+    "@grafana/faro-web-sdk": "^1.19.0",
+    "@grafana/faro-web-tracing": "^1.19.0",
+    "@symbo.ls/router": "^2.32.13",
+    "@symbo.ls/socket": "^2.32.13",
     "acorn": "^8.14.0",
     "acorn-walk": "^8.3.4",
     "dexie": "^4.0.11",
@@ -71,5 +73,5 @@
     "tap-spec": "^5.0.0",
     "tape": "^5.9.0"
   },
-  "gitHead": "45bd289bc2a652a700941e7743f1066ac5312430"
+  "gitHead": "c9fa0cbf09841fd8c586d525a1f4e96f3ba11569"
 }

package/src/config/environment.js

@@ -10,7 +10,9 @@ const CONFIG = {
     // Feature toggles that apply across all environments by default
     features: {
       newUserOnboarding: true,
-      betaFeatures: false
+      betaFeatures: false,
+      // Tracking is enabled by default unless overridden per environment
+      trackingEnabled: true
     }
   },
 
@@ -24,12 +26,16 @@ const CONFIG = {
     basedProject: 'platform-v2-sm', // For based api
     basedOrg: 'symbols', // For based api
     githubClientId: 'Ov23liAFrsR0StbAO6PO', // For github api
-    grafanaUrl:
-      'https://faro-collector-prod-us-east-0.grafana.net/collect/aef64330db80bdfeaac084317bf72f99', // For grafana tracing
-    grafanaAppName: 'Localhost Symbols',
+    grafanaUrl: '', // For grafana tracing
+    grafanaAppName: 'Symbols Localhost',
     // Environment-specific feature toggles (override common)
     features: {
-      betaFeatures: true // Enable beta features in local dev
+      // Disable tracking by default on localhost/dev machines
+      trackingEnabled: false,
+      // Enable beta features in local dev
+      betaFeatures: true,
+      // Preserve common defaults explicitly for local
+      newUserOnboarding: true
     },
     typesenseCollectionName: 'docs',
     typesenseApiKey: 'vZya3L2zpq8L6iI5WWMUZJZABvT63VDb',
@@ -57,8 +63,7 @@ const CONFIG = {
     basedProject: 'platform-v2-sm',
     basedOrg: 'symbols',
     githubClientId: 'Ov23liHxyWFBxS8f1gnF',
-    grafanaUrl:
-      'https://faro-collector-prod-us-east-0.grafana.net/collect/7a3ba473cee2025c68513667024316b8', // For grafana tracing
+    grafanaUrl: '', // For grafana tracing
     grafanaAppName: 'Symbols Test',
     typesenseCollectionName: 'docs',
     typesenseApiKey: 'awmcVpbWqZi9IUgmvslp1C5LKDU8tMjA',
@@ -70,8 +75,7 @@ const CONFIG = {
     socketUrl: 'https://upcoming.api.symbols.app',
     apiUrl: 'https://upcoming.api.symbols.app',
     githubClientId: 'Ov23liWF7NvdZ056RV5J',
-    grafanaUrl:
-      'https://faro-collector-prod-us-east-0.grafana.net/collect/7a3ba473cee2025c68513667024316b8', // For grafana tracing
+    grafanaUrl: '', // For grafana tracing
     grafanaAppName: 'Symbols Upcoming',
     typesenseCollectionName: 'docs',
     typesenseApiKey: 'awmcVpbWqZi9IUgmvslp1C5LKDU8tMjA',
@@ -86,8 +90,7 @@ const CONFIG = {
     basedProject: 'platform-v2-sm',
     basedOrg: 'symbols',
     githubClientId: 'Ov23ligwZDQVD0VfuWNa',
-    grafanaUrl:
-      'https://faro-collector-prod-us-east-0.grafana.net/collect/7a3ba473cee2025c68513667024316b8', // For grafana tracing
+    grafanaUrl: '', // For grafana tracing
     grafanaAppName: 'Symbols Staging',
     typesenseCollectionName: 'docs',
     typesenseApiKey: 'awmcVpbWqZi9IUgmvslp1C5LKDU8tMjA',
@@ -135,6 +138,11 @@ export const getConfig = () => {
     // Create the final config with environment variable overrides
     const finalConfig = {
       ...envConfig,
+      // Deep-merge feature flags so env-specific overrides don't drop common defaults
+      features: {
+        ...(CONFIG.common.features || {}),
+        ...((CONFIG[env] && CONFIG[env].features) || {})
+      },
       socketUrl: process.env.SYMBOLS_APP_SOCKET_URL || envConfig.socketUrl,
       apiUrl: process.env.SYMBOLS_APP_API_URL || envConfig.apiUrl,
       basedEnv: process.env.SYMBOLS_APP_BASED_ENV || envConfig.basedEnv,

package/src/index.js

@@ -1,6 +1,5 @@
 import {
   createAuthService,
-  createCoreService,
   createCollabService,
   createProjectService,
   createPlanService,
@@ -11,13 +10,24 @@ import {
   createPullRequestService,
   createAdminService,
   createSubscriptionService,
-  createScreenshotService
+  createScreenshotService,
+  createTrackingService
 } from './services/index.js'
 
 import { SERVICE_METHODS } from './utils/services.js'
 import environment from './config/environment.js'
 import { rootBus } from './state/rootEventBus.js'
 
+const isBrowserEnvironment = () => typeof window !== 'undefined'
+
+export const isLocalhost = () => {
+  if (!isBrowserEnvironment()) {
+    return false
+  }
+  const host = window.location && window.location.hostname
+  return host === 'localhost' || host === '127.0.0.1' || host === '::1' || host === '' || !host
+}
+
 export class SDK {
   constructor(options = {}) {
     this._services = new Map()
@@ -48,13 +58,6 @@ export class SDK {
           options: this._options
         })
       ),
-      this._initService(
-        'core',
-        createCoreService({
-          context: this._context,
-          options: this._options
-        })
-      ),
       this._initService(
         'collab',
         createCollabService({
@@ -132,6 +135,13 @@ export class SDK {
           context: this._context,
           options: this._options
         })
+      ),
+      this._initService(
+        'tracking',
+        createTrackingService({
+          context: this._context,
+          options: this._options
+        })
       )
     ])
 
@@ -156,16 +166,36 @@ export class SDK {
   }
 
   _validateOptions(options) {
+    const onLocalhost = isLocalhost()
+    const hasGrafanaUrl = Boolean(environment.grafanaUrl)
     const defaults = {
       useNewServices: true, // Use new service implementations by default
       apiUrl: environment.apiUrl,
       socketUrl: environment.socketUrl,
       timeout: 30000,
       retryAttempts: 3,
-      debug: false
+      debug: false,
+      tracking: {
+        // Force-disabled on localhost or when no Grafana URL is configured
+        enabled: onLocalhost ? false : (hasGrafanaUrl ? environment.features.trackingEnabled : false),
+      }
+    }
+
+    const merged = {
+      ...defaults,
+      ...options,
+      tracking: {
+        ...defaults.tracking,
+        ...(options.tracking || {})
+      }
     }
 
-    return { ...defaults, ...options }
+    // Enforce disabled tracking on localhost or when no Grafana URL configured, even if overridden
+    if (onLocalhost || !hasGrafanaUrl) {
+      merged.tracking.enabled = false
+    }
+
+    return merged
   }
 
   // Get service instance
@@ -179,7 +209,7 @@ export class SDK {
   // Update context
   updateContext(newContext) {
     // Do not persist authToken in SDK context; TokenManager is the source of truth
-    const { authToken, ...sanitized } = newContext || {}
+    const { ...sanitized } = newContext || {}
 
     this._context = {
       ...this._context,
@@ -265,7 +295,6 @@ export default SDK
 // Export services for direct usage
 export {
   createAuthService,
-  createCoreService,
   createCollabService,
   createProjectService,
   createPlanService,
@@ -275,7 +304,8 @@ export {
   createBranchService,
   createPullRequestService,
   createAdminService,
-  createSubscriptionService
+  createSubscriptionService,
+  createTrackingService
 } from './services/index.js'
 
 // Export environment configuration

package/src/services/BaseService.js

@@ -71,6 +71,34 @@ export class BaseService {
     this._error = error
   }
 
+  _getTrackingService () {
+    const services = this._context?.services
+    const tracking = services?.tracking
+    if (!tracking || typeof tracking.trackError !== 'function') {return null}
+    return tracking
+  }
+
+  _shouldTrackErrors () {
+    const name = this?.constructor?.name
+    return name !== 'TrackingService'
+  }
+
+  _trackServiceError (error, details = {}) {
+    if (!this._shouldTrackErrors()) {return}
+    try {
+      const tracking = this._getTrackingService()
+      if (!tracking) {return}
+      const context = {
+        service: this?.constructor?.name || 'UnknownService',
+        apiUrl: this._apiUrl || null,
+        ...details
+      }
+      tracking.trackError(error instanceof Error ? error : new Error(String(error)), context)
+    } catch {
+      // Do not let tracking failures affect service flow
+    }
+  }
+
   _requireAuth () {
     if (!this._context.authToken) {
       throw new Error('Authentication required')
@@ -132,11 +160,26 @@ export class BaseService {
         } catch {
           // Use default error message
         }
+        // Track HTTP error before throwing
+        this._trackServiceError(
+          new Error(error.message || error.error || `HTTP ${response.status}: ${response.statusText}`),
+          {
+            endpoint,
+            methodName: options.methodName,
+            status: response.status,
+            statusText: response.statusText
+          }
+        )
         throw new Error(error.message || error.error || 'Request failed', { cause: error })
       }
 
       return response.status === 204 ? null : response.json()
     } catch (error) {
+      // Track network/other request errors before rethrowing
+      this._trackServiceError(error, {
+        endpoint,
+        methodName: options.methodName
+      })
       throw new Error(`Request failed: ${error.message}`, { cause: error })
     }
   }

package/src/services/DnsService.js

@@ -57,7 +57,7 @@ export class DnsService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get custom host: ${error.message}`)
+      throw new Error(`Failed to get custom host: ${error.message}`, { cause: error })
     }
   }
 
@@ -136,7 +136,7 @@ export class DnsService extends BaseService {
       throw new Error(response.message)
     } catch (error) {
       throw new Error(
-        `Failed to update project custom domains: ${error.message}`
+        `Failed to update project custom domains: ${error.message}`, { cause: error }
       )
     }
   }
@@ -325,7 +325,7 @@ export class DnsService extends BaseService {
         needsVerification: false
       }
     } catch (error) {
-      throw new Error(`Failed to verify domain ownership: ${error.message}`)
+      throw new Error(`Failed to verify domain ownership: ${error.message}`, { cause: error })
     }
   }
 
@@ -348,7 +348,7 @@ export class DnsService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to get project domains: ${error.message}`)
+      throw new Error(`Failed to get project domains: ${error.message}`, { cause: error })
     }
   }
 
@@ -374,7 +374,7 @@ export class DnsService extends BaseService {
       }
       throw new Error(response.message)
     } catch (error) {
-      throw new Error(`Failed to remove project custom domain: ${error.message}`)
+      throw new Error(`Failed to remove project custom domain: ${error.message}`, { cause: error })
     }
   }