@product7/feedback-sdk 1.0.1 → 1.0.3

package/src/core/APIService.js

@@ -7,64 +7,36 @@ export class APIService {
 		this.sessionExpiry = null;
 		this.userContext = config.userContext || null;
 
-		// Construct workspace-specific API URL
 		if (config.apiUrl) {
-			// If explicitly provided, use it
 			this.baseURL = config.apiUrl;
 		} else if (this.workspace) {
-			// Construct from workspace: workspace.api.staging.product7.io/api/v1
 			this.baseURL = `https://${this.workspace}.api.staging.product7.io/api/v1`;
 		} else {
-			// Fallback to default
 			this.baseURL = 'https://api.staging.product7.io/api/v1';
 		}
 
-		console.log('[APIService] Using API URL:', this.baseURL);
-
-		// Try to load existing session from localStorage
 		this._loadStoredSession();
 	}
 
 	async init(userContext = null) {
-		console.log('[APIService] Starting initialization...');
-
 		if (userContext) {
 			this.userContext = userContext;
 		}
 
-		// Check if we have a valid session token
 		if (this.isSessionValid()) {
-			console.log('[APIService] Found valid existing session');
 			return { sessionToken: this.sessionToken };
 		}
 
-		// Try to get user context from various sources if not provided
-		if (!this.userContext) {
-			console.log(
-				'[APIService] No user context provided, attempting auto-detection...'
-			);
-			this.userContext = this._getUserContextFromStorage();
-		}
-
 		if (!this.userContext || !this.workspace) {
 			const error = `Missing ${!this.workspace ? 'workspace' : 'user context'} for initialization`;
-			console.error('[APIService]', error);
 			throw new APIError(400, error);
 		}
 
-		console.log('[APIService] User context detected:', this.userContext);
-
 		const payload = {
 			workspace: this.workspace,
 			user: this.userContext,
 		};
 
-		console.log(
-			'[APIService] Making init request to:',
-			`${this.baseURL}/widget/init`
-		);
-		console.log('[APIService] Payload:', payload);
-
 		try {
 			const response = await this._makeRequest('/widget/init', {
 				method: 'POST',
@@ -74,13 +46,8 @@ export class APIService {
 				},
 			});
 
-			console.log('[APIService] Init response:', response);
-
-			// Store session token and expiry
 			this.sessionToken = response.session_token;
 			this.sessionExpiry = new Date(Date.now() + response.expires_in * 1000);
-
-			// Store session in localStorage for persistence
 			this._storeSession();
 
 			return {
@@ -89,7 +56,6 @@ export class APIService {
 				expiresIn: response.expires_in,
 			};
 		} catch (error) {
-			console.error('[APIService] Init failed:', error);
 			throw new APIError(
 				error.status || 500,
 				`Failed to initialize widget: ${error.message}`,
@@ -99,7 +65,6 @@ export class APIService {
 	}
 
 	async submitFeedback(feedbackData) {
-		// Ensure we have a valid session
 		if (!this.isSessionValid()) {
 			await this.init();
 		}
@@ -109,8 +74,7 @@ export class APIService {
 		}
 
 		const payload = {
-			board:
-				feedbackData.board_id || feedbackData.board || feedbackData.boardId,
+			board: feedbackData.board_id || feedbackData.board || feedbackData.boardId,
 			title: feedbackData.title,
 			content: feedbackData.content,
 			attachments: feedbackData.attachments || [],
@@ -128,13 +92,10 @@ export class APIService {
 
 			return response;
 		} catch (error) {
-			// If session expired, try to reinitialize once
 			if (error.status === 401) {
 				this.sessionToken = null;
 				this.sessionExpiry = null;
 				await this.init();
-
-				// Retry the request with new session
 				return this.submitFeedback(feedbackData);
 			}
 
@@ -154,12 +115,8 @@ export class APIService {
 
 	setUserContext(userContext) {
 		this.userContext = userContext;
-		// Store in localStorage for persistence
 		if (typeof localStorage !== 'undefined') {
-			localStorage.setItem(
-				'feedbackSDK_userContext',
-				JSON.stringify(userContext)
-			);
+			localStorage.setItem('feedbackSDK_userContext', JSON.stringify(userContext));
 		}
 	}
 
@@ -172,99 +129,10 @@ export class APIService {
 		this.sessionExpiry = null;
 		if (typeof localStorage !== 'undefined') {
 			localStorage.removeItem('feedbackSDK_session');
+			localStorage.removeItem('feedbackSDK_userContext');
 		}
 	}
 
-	_getUserContextFromStorage() {
-		if (typeof localStorage === 'undefined') return null;
-
-		console.log('[APIService] Attempting to detect user from storage...');
-
-		try {
-			// Try to get from feedbackSDK specific storage first
-			const stored = localStorage.getItem('feedbackSDK_userContext');
-			if (stored) {
-				console.log('[APIService] Found user context in feedbackSDK storage');
-				return JSON.parse(stored);
-			}
-
-			// Try to get from window object (if set by parent application)
-			if (typeof window !== 'undefined' && window.FeedbackSDKUserContext) {
-				console.log(
-					'[APIService] Found user context in window.FeedbackSDKUserContext'
-				);
-				return window.FeedbackSDKUserContext;
-			}
-
-			// Check window.currentUser
-			if (typeof window !== 'undefined' && window.currentUser) {
-				console.log('[APIService] Found user context in window.currentUser');
-				return this._mapUserData(window.currentUser);
-			}
-
-			// Try to extract from existing session storage
-			const authSources = ['auth', 'currentUser', 'userSession', 'user'];
-
-			for (const key of authSources) {
-				const sessionData = localStorage.getItem(key);
-				if (sessionData) {
-					console.log(`[APIService] Found data in localStorage.${key}`);
-					const parsed = JSON.parse(sessionData);
-
-					// Handle nested user data (like {user: {...}, token: ...})
-					const userData = parsed.user || parsed;
-
-					if (userData && (userData.id || userData.user_id || userData.email)) {
-						console.log('[APIService] Successfully mapped user data');
-						return this._mapUserData(userData);
-					}
-				}
-			}
-
-			console.log('[APIService] No user context found in any storage location');
-			return null;
-		} catch (error) {
-			console.warn(
-				'[FeedbackSDK] Failed to load user context from storage:',
-				error
-			);
-			return null;
-		}
-	}
-
-	_mapUserData(userData) {
-		console.log('[APIService] Mapping user data:', userData);
-
-		const mapped = {
-			user_id: userData.id || userData.user_id || userData.userId,
-			email: userData.email,
-			name: userData.name || userData.displayName || userData.full_name,
-			custom_fields: {},
-			company: {},
-		};
-
-		// Map company data if available
-		if (userData.company || userData.organization) {
-			const company = userData.company || userData.organization;
-			mapped.company = {
-				id: company.id || company.company_id,
-				name: company.name || company.company_name,
-				monthly_spend: company.monthly_spend || company.spend,
-			};
-		}
-
-		// Map any additional custom fields
-		const customFieldKeys = ['plan', 'role', 'tier', 'subscription'];
-		customFieldKeys.forEach((key) => {
-			if (userData[key]) {
-				mapped.custom_fields[key] = userData[key];
-			}
-		});
-
-		console.log('[APIService] Mapped result:', mapped);
-		return mapped;
-	}
-
 	_storeSession() {
 		if (typeof localStorage === 'undefined') return;
 
@@ -276,7 +144,7 @@ export class APIService {
 			};
 			localStorage.setItem('feedbackSDK_session', JSON.stringify(sessionData));
 		} catch (error) {
-			console.warn('[FeedbackSDK] Failed to store session:', error);
+			// Silently fail if localStorage is not available
 		}
 	}
 
@@ -293,14 +161,12 @@ export class APIService {
 
 			return this.isSessionValid();
 		} catch (error) {
-			console.warn('[FeedbackSDK] Failed to load stored session:', error);
 			return false;
 		}
 	}
 
 	async _makeRequest(endpoint, options = {}) {
 		const url = `${this.baseURL}${endpoint}`;
-		console.log('[APIService] Making request to:', url);
 
 		try {
 			const response = await fetch(url, options);
@@ -311,8 +177,7 @@ export class APIService {
 
 				try {
 					responseData = await response.json();
-					errorMessage =
-						responseData.message || responseData.error || errorMessage;
+					errorMessage = responseData.message || responseData.error || errorMessage;
 				} catch (e) {
 					errorMessage = (await response.text()) || errorMessage;
 				}
@@ -330,9 +195,7 @@ export class APIService {
 			if (error instanceof APIError) {
 				throw error;
 			}
-
-			// Network or other errors
 			throw new APIError(0, error.message, null);
 		}
 	}
-}
+}

package/src/core/FeedbackSDK.js

@@ -11,7 +11,6 @@ export class FeedbackSDK {
 		this.widgets = new Map();
 		this.eventBus = new EventBus();
 
-		// Initialize API service
 		this.apiService = new APIService({
 			apiUrl: this.config.apiUrl,
 			workspace: this.config.workspace,
@@ -27,10 +26,8 @@ export class FeedbackSDK {
 		}
 
 		try {
-			// Initialize the API service (this will handle the /widget/init call)
 			const initData = await this.apiService.init(this.config.userContext);
 
-			// Merge any server-provided config with local config
 			if (initData.config) {
 				this.config = deepMerge(this.config, initData.config);
 			}
@@ -55,9 +52,7 @@ export class FeedbackSDK {
 
 	createWidget(type = 'button', options = {}) {
 		if (!this.initialized) {
-			throw new SDKError(
-				'SDK must be initialized before creating widgets. Call init() first.'
-			);
+			throw new SDKError('SDK must be initialized before creating widgets. Call init() first.');
 		}
 
 		const widgetId = generateId('widget');
@@ -72,7 +67,6 @@ export class FeedbackSDK {
 		try {
 			const widget = WidgetFactory.create(type, widgetOptions);
 			this.widgets.set(widgetId, widget);
-
 			this.eventBus.emit('widget:created', { widget, type });
 			return widget;
 		} catch (error) {
@@ -111,7 +105,6 @@ export class FeedbackSDK {
 		const oldConfig = { ...this.config };
 		this.config = this._validateAndMergeConfig(newConfig, this.config);
 
-		// Update all existing widgets with new config
 		for (const widget of this.widgets.values()) {
 			widget.handleConfigUpdate(this.config);
 		}
@@ -131,23 +124,17 @@ export class FeedbackSDK {
 	}
 
 	getUserContext() {
-		return (
-			this.config.userContext ||
-			(this.apiService ? this.apiService.getUserContext() : null)
-		);
+		return this.config.userContext || (this.apiService ? this.apiService.getUserContext() : null);
 	}
 
 	async reinitialize(newUserContext = null) {
-		// Clear current session
 		this.apiService.clearSession();
 		this.initialized = false;
 
-		// Update user context if provided
 		if (newUserContext) {
 			this.setUserContext(newUserContext);
 		}
 
-		// Reinitialize
 		return this.init();
 	}
 
@@ -191,24 +178,12 @@ export class FeedbackSDK {
 			debug: false,
 		};
 
-		const mergedConfig = deepMerge(
-			deepMerge(defaultConfig, existingConfig),
-			newConfig
-		);
-
-		// Validate required config
-		const requiredFields = ['workspace'];
-		const missingFields = requiredFields.filter(
-			(field) => !mergedConfig[field]
-		);
-
-		if (missingFields.length > 0) {
-			throw new ConfigError(
-				`Missing required configuration: ${missingFields.join(', ')}`
-			);
+		const mergedConfig = deepMerge(deepMerge(defaultConfig, existingConfig), newConfig);
+
+		if (!mergedConfig.workspace) {
+			throw new ConfigError('Missing required configuration: workspace');
 		}
 
-		// Validate userContext structure if provided
 		if (mergedConfig.userContext) {
 			this._validateUserContext(mergedConfig.userContext);
 		}
@@ -218,12 +193,9 @@ export class FeedbackSDK {
 
 	_validateUserContext(userContext) {
 		if (!userContext.user_id && !userContext.email) {
-			throw new ConfigError(
-				'User context must include at least user_id or email'
-			);
+			throw new ConfigError('User context must include at least user_id or email');
 		}
 
-		// Validate structure matches expected API format
 		const validStructure = {
 			user_id: 'string',
 			email: 'string',
@@ -234,9 +206,7 @@ export class FeedbackSDK {
 
 		for (const [key, expectedType] of Object.entries(validStructure)) {
 			if (userContext[key] && typeof userContext[key] !== expectedType) {
-				throw new ConfigError(
-					`User context field '${key}' must be of type '${expectedType}'`
-				);
+				throw new ConfigError(`User context field '${key}' must be of type '${expectedType}'`);
 			}
 		}
 	}
@@ -247,7 +217,6 @@ export class FeedbackSDK {
 		this.updateConfig = this.updateConfig.bind(this);
 	}
 
-	// Static helper methods
 	static create(config) {
 		return new FeedbackSDK(config);
 	}
@@ -258,9 +227,7 @@ export class FeedbackSDK {
 		return sdk;
 	}
 
-	// Utility methods for external integrations
 	static extractUserContextFromAuth(authData) {
-		// Helper method to extract user context from common auth structures
 		if (!authData) return null;
 
 		return {
@@ -272,14 +239,11 @@ export class FeedbackSDK {
 				plan: authData.plan || authData.subscription?.plan,
 				...(authData.custom_fields || {}),
 			},
-			company:
-				authData.company || authData.organization
-					? {
-							id: authData.company?.id || authData.organization?.id,
-							name: authData.company?.name || authData.organization?.name,
-							monthly_spend: authData.company?.monthly_spend,
-						}
-					: undefined,
+			company: authData.company || authData.organization ? {
+				id: authData.company?.id || authData.organization?.id,
+				name: authData.company?.name || authData.organization?.name,
+				monthly_spend: authData.company?.monthly_spend,
+			} : undefined,
 		};
 	}
-}
+}

package/src/index.js

@@ -17,25 +17,11 @@ import { TabWidget } from './widgets/TabWidget.js';
 import { WidgetFactory } from './widgets/WidgetFactory.js';
 
 function injectStyles() {
-	console.log('injectStyles called');
-	console.log('document exists:', typeof document !== 'undefined');
-	console.log('CSS_STYLES exists:', !!CSS_STYLES);
-
-	if (
-		typeof document !== 'undefined' &&
-		!document.querySelector('#feedback-sdk-styles')
-	) {
-		console.log('Injecting CSS...');
+	if (typeof document !== 'undefined' && !document.querySelector('#feedback-sdk-styles')) {
 		const style = document.createElement('style');
 		style.id = 'feedback-sdk-styles';
 		style.textContent = CSS_STYLES;
 		document.head.appendChild(style);
-		console.log(
-			'CSS injected, style element created:',
-			!!document.querySelector('#feedback-sdk-styles')
-		);
-	} else {
-		console.log('CSS already exists or document not ready');
 	}
 }
 
@@ -44,11 +30,6 @@ function autoInit() {
 		injectStyles();
 
 		const config = { ...window.FeedbackSDKConfig };
-
-		if (!config.userContext) {
-			config.userContext = getUserContextFromEnvironment();
-		}
-
 		const sdk = new FeedbackSDK(config);
 
 		sdk
@@ -76,30 +57,17 @@ function autoInit() {
 
 				if (typeof CustomEvent !== 'undefined') {
 					const event = new CustomEvent('FeedbackSDKReady', {
-						detail: {
-							sdk,
-							config: config,
-							initData: initData,
-						},
+						detail: { sdk, config, initData },
 					});
 					window.dispatchEvent(event);
 				}
-
-				console.log(
-					'[FeedbackSDK] Successfully initialized with session:',
-					initData.sessionToken ? 'Yes' : 'No'
-				);
 			})
 			.catch((error) => {
 				console.error('[FeedbackSDK] Auto-initialization failed:', error);
 
 				if (typeof CustomEvent !== 'undefined') {
 					const event = new CustomEvent('FeedbackSDKError', {
-						detail: {
-							error,
-							config: config,
-							phase: 'initialization',
-						},
+						detail: { error, config, phase: 'initialization' },
 					});
 					window.dispatchEvent(event);
 				}
@@ -107,70 +75,6 @@ function autoInit() {
 	}
 }
 
-function getUserContextFromEnvironment() {
-	if (typeof window === 'undefined') return null;
-
-	if (window.FeedbackSDKUserContext) {
-		return window.FeedbackSDKUserContext;
-	}
-
-	const authSources = [
-		() => window.auth0?.user,
-		() => window.firebase?.auth()?.currentUser,
-		() => window.amplify?.Auth?.currentAuthenticatedUser(),
-
-		() => window.currentUser,
-		() => window.user,
-		() => window.userData,
-
-		() => window.app?.user,
-		() => window.store?.getState?.()?.user,
-		() => window.App?.currentUser,
-	];
-
-	for (const getAuth of authSources) {
-		try {
-			const authData = getAuth();
-			if (authData) {
-				const userContext = FeedbackSDK.extractUserContextFromAuth(authData);
-				if (userContext && (userContext.user_id || userContext.email)) {
-					console.log(
-						'[FeedbackSDK] Auto-detected user context from',
-						getAuth.name || 'unknown source'
-					);
-					return userContext;
-				}
-			}
-		} catch (error) {
-			continue;
-		}
-	}
-
-	try {
-		const storedAuth =
-			localStorage.getItem('auth') ||
-			localStorage.getItem('user') ||
-			localStorage.getItem('session');
-		if (storedAuth) {
-			const authData = JSON.parse(storedAuth);
-			const userContext = FeedbackSDK.extractUserContextFromAuth(authData);
-			if (userContext && (userContext.user_id || userContext.email)) {
-				console.log(
-					'[FeedbackSDK] Auto-detected user context from localStorage'
-				);
-				return userContext;
-			}
-		}
-	} catch (error) {
-		// Continue
-	}
-
-	console.warn(
-		'[FeedbackSDK] No user context found. Widget initialization may require manual user context setting.'
-	);
-	return null;
-}
-
 function handleDOMReady() {
 	if (typeof document !== 'undefined') {
 		if (document.readyState === 'loading') {
@@ -278,4 +182,4 @@ export {
 	ValidationError,
 	WidgetError,
 	WidgetFactory,
-};
+};

package/types/index.d.ts

@@ -0,0 +1,42 @@
+declare module '@product7/feedback-sdk' {
+	export interface FeedbackConfig {
+		workspace: string;
+		debug?: boolean;
+		boardId?: string;
+		position?: 'bottom-right' | 'bottom-left' | 'top-right' | 'top-left';
+	}
+
+	export interface ButtonWidget {
+		mount(container?: string | HTMLElement): this;
+		destroy(): void;
+		show(): this;
+		hide(): this;
+		openModal(): void;
+		closeModal(): void;
+	}
+
+	export class FeedbackSDK {
+		constructor(config: FeedbackConfig);
+		init(): Promise<any>;
+		createWidget(
+			type: 'button',
+			options?: Partial<FeedbackConfig>
+		): ButtonWidget;
+		readonly initialized: boolean;
+	}
+
+	interface FeedbackSDKExport {
+		FeedbackSDK: typeof FeedbackSDK;
+		ButtonWidget: any;
+		create: (config: FeedbackConfig) => FeedbackSDK;
+		getInstance: () => FeedbackSDK | null;
+		isReady: () => boolean;
+		version: string;
+		instance: FeedbackSDK | null;
+		onReady: (callback: (sdk: FeedbackSDK) => void) => void;
+		onError: (callback: (error: Error) => void) => void;
+	}
+
+	const FeedbackSDKDefault: FeedbackSDKExport;
+	export default FeedbackSDKDefault;
+}

package/src/types/index.d.ts

@@ -1,12 +0,0 @@
-declare module '@product7/feedback-sdk' {
-	export class FeedbackSDK {
-		constructor(config: any);
-		init(): Promise<any>;
-		createWidget(type: string, options?: any): any;
-	}
-
-	export class ButtonWidget {
-		mount(container?: string | HTMLElement): this;
-		destroy(): void;
-	}
-}