From 71ea9efda7132c3b800e93531d3172f65f6b5451 Mon Sep 17 00:00:00 2001 From: Matthew Raymer Date: Fri, 18 Jul 2025 05:28:00 +0000 Subject: [PATCH] Remove debugging console.log statements Clean up all temporary debugging console.log statements added during registration status troubleshooting. Remove debug output from multiple components while preserving essential error logging and functionality. Changes: - PlatformServiceMixin.ts: Remove debug logging from $saveUserSettings and $saveMySettings - AccountViewView.vue: Remove debug logging from mounted, initializeState, checkLimits, and onRecheckLimits - UsageLimitsSection.vue: Remove debug logging from lifecycle hooks and recheckLimits - IdentitySwitcherView.vue: Remove debug logging from switchAccount method All core functionality preserved including error handling with logger.error() and user notifications. Codebase now production-ready without debugging noise. --- src/components/UsageLimitsSection.vue | 15 +++--------- src/utils/PlatformServiceMixin.ts | 16 ------------ src/views/AccountViewView.vue | 35 --------------------------- src/views/IdentitySwitcherView.vue | 11 ++------- 4 files changed, 5 insertions(+), 72 deletions(-) diff --git a/src/components/UsageLimitsSection.vue b/src/components/UsageLimitsSection.vue index 0e25bb24..8eecf9a2 100644 --- a/src/components/UsageLimitsSection.vue +++ b/src/components/UsageLimitsSection.vue @@ -98,20 +98,11 @@ export default class UsageLimitsSection extends Vue { @Prop({ required: false }) imageLimits?: any; mounted() { - console.log('[DEBUG] UsageLimitsSection mounted'); - console.log('[DEBUG] loadingLimits prop:', this.loadingLimits); - console.log('[DEBUG] limitsMessage prop:', this.limitsMessage); - console.log('[DEBUG] activeDid prop:', this.activeDid); - console.log('[DEBUG] endorserLimits prop:', this.endorserLimits); - console.log('[DEBUG] imageLimits prop:', this.imageLimits); + // Component mounted } updated() { - console.log('[DEBUG] UsageLimitsSection updated'); - console.log('[DEBUG] loadingLimits prop:', this.loadingLimits); - console.log('[DEBUG] limitsMessage prop:', this.limitsMessage); - console.log('[DEBUG] endorserLimits prop:', this.endorserLimits); - console.log('[DEBUG] imageLimits prop:', this.imageLimits); + // Component updated } readableDate(dateString: string | undefined): string { @@ -125,7 +116,7 @@ export default class UsageLimitsSection extends Vue { @Emit("recheck-limits") recheckLimits() { - console.log('[DEBUG] recheckLimits called'); + // Emit recheck-limits event } } diff --git a/src/utils/PlatformServiceMixin.ts b/src/utils/PlatformServiceMixin.ts index 660ae611..bfbc86d0 100644 --- a/src/utils/PlatformServiceMixin.ts +++ b/src/utils/PlatformServiceMixin.ts @@ -778,17 +778,12 @@ export const PlatformServiceMixin = { changes: Partial, ): Promise { try { - console.log('[DEBUG] $saveUserSettings - did:', did); - console.log('[DEBUG] $saveUserSettings - changes:', changes); - // Remove fields that shouldn't be updated const { id, ...safeChanges } = changes; // eslint-disable-next-line @typescript-eslint/no-unused-vars void id; safeChanges.accountDid = did; - console.log('[DEBUG] $saveUserSettings - safeChanges:', safeChanges); - if (Object.keys(safeChanges).length === 0) return true; const setParts: string[] = []; @@ -801,21 +796,14 @@ export const PlatformServiceMixin = { } }); - console.log('[DEBUG] $saveUserSettings - setParts:', setParts); - console.log('[DEBUG] $saveUserSettings - params:', params); - if (setParts.length === 0) return true; params.push(did); const sql = `UPDATE settings SET ${setParts.join(", ")} WHERE accountDid = ?`; - console.log('[DEBUG] $saveUserSettings - SQL:', sql); - console.log('[DEBUG] $saveUserSettings - Final params:', params); await this.$dbExec(sql, params); - console.log('[DEBUG] $saveUserSettings - Database update successful'); return true; } catch (error) { - console.log('[DEBUG] $saveUserSettings - Error:', error); logger.error( "[PlatformServiceMixin] Error saving user settings:", error, @@ -833,14 +821,10 @@ export const PlatformServiceMixin = { async $saveMySettings(changes: Partial): Promise { // eslint-disable-next-line @typescript-eslint/no-explicit-any const currentDid = (this as any).activeDid; - console.log('[DEBUG] $saveMySettings - changes:', changes); - console.log('[DEBUG] $saveMySettings - currentDid:', currentDid); if (!currentDid) { - console.log('[DEBUG] $saveMySettings - No DID, using $saveSettings'); return await this.$saveSettings(changes); } - console.log('[DEBUG] $saveMySettings - Using $saveUserSettings for DID:', currentDid); return await this.$saveUserSettings(currentDid, changes); }, diff --git a/src/views/AccountViewView.vue b/src/views/AccountViewView.vue index 755db527..33ef065f 100644 --- a/src/views/AccountViewView.vue +++ b/src/views/AccountViewView.vue @@ -970,10 +970,7 @@ export default class AccountViewView extends Vue { // Check limits for any user with an activeDid (this will also check registration status) if (this.activeDid) { - console.log('[DEBUG] Calling checkLimits from mounted for user with activeDid'); await this.checkLimits(); - } else { - console.log('[DEBUG] Not calling checkLimits - no activeDid available'); } // Only check service worker on web platform - Capacitor/Electron don't support it @@ -1014,20 +1011,13 @@ export default class AccountViewView extends Vue { * Initializes component state with values from the database or defaults. */ async initializeState(): Promise { - console.log('[DEBUG] AccountViewView - initializeState called'); - // First get the master settings to see the active DID const masterSettings = await this.$settings(); - console.log('[DEBUG] AccountViewView - Master settings activeDid:', masterSettings.activeDid); // Then get the account-specific settings const settings: AccountSettings = await this.$accountSettings(); - console.log('[DEBUG] AccountViewView - Account settings loaded for DID:', settings.activeDid); - console.log('[DEBUG] AccountViewView - Account settings isRegistered:', settings?.isRegistered); this.activeDid = settings.activeDid || ""; - console.log('[DEBUG] initializeState - activeDid:', this.activeDid); - console.log('[DEBUG] initializeState - settings.isRegistered:', settings?.isRegistered); this.apiServer = settings.apiServer || ""; this.apiServerInput = settings.apiServer || ""; this.givenName = @@ -1056,9 +1046,6 @@ export default class AccountViewView extends Vue { this.warnIfTestServer = !!settings.warnIfTestServer; this.webPushServer = settings.webPushServer || this.webPushServer; this.webPushServerInput = settings.webPushServer || this.webPushServerInput; - - console.log('[DEBUG] initializeState complete - isRegistered:', this.isRegistered); - console.log('[DEBUG] initializeState complete - activeDid:', this.activeDid); } // call fn, copy text to the clipboard, then redo fn after 2 seconds @@ -1401,17 +1388,11 @@ export default class AccountViewView extends Vue { } async checkLimits(): Promise { - console.log('[DEBUG] checkLimits called'); - console.log('[DEBUG] activeDid:', this.activeDid); - console.log('[DEBUG] isRegistered:', this.isRegistered); - this.loadingLimits = true; try { const did = this.activeDid; - console.log('[DEBUG] did value:', did); if (!did) { - console.log('[DEBUG] No DID found, setting NO_IDENTIFIER message'); this.limitsMessage = ACCOUNT_VIEW_CONSTANTS.LIMITS.NO_IDENTIFIER; return; } @@ -1422,16 +1403,11 @@ export default class AccountViewView extends Vue { webPushServer: this.webPushServer, }); - console.log('[DEBUG] Calling fetchImageRateLimits for DID:', did); const imageResp = await fetchImageRateLimits(this.axios, did); - console.log('[DEBUG] Image rate limits response status:', imageResp.status); - console.log('[DEBUG] Image rate limits response data:', imageResp.data); if (imageResp.status === 200) { this.imageLimits = imageResp.data; - console.log('[DEBUG] Image limits set successfully'); } else { - console.log('[DEBUG] Image rate limits failed, status:', imageResp.status); await this.$saveSettings({ profileImageUrl: "", }); @@ -1441,21 +1417,15 @@ export default class AccountViewView extends Vue { return; } - console.log('[DEBUG] Calling fetchEndorserRateLimits for DID:', did); - console.log('[DEBUG] API server:', this.apiServer); const endorserResp = await fetchEndorserRateLimits( this.apiServer, this.axios, did, ); - console.log('[DEBUG] Endorser rate limits response status:', endorserResp.status); - console.log('[DEBUG] Endorser rate limits response data:', endorserResp.data); if (endorserResp.status === 200) { this.endorserLimits = endorserResp.data; - console.log('[DEBUG] Endorser limits set successfully'); } else { - console.log('[DEBUG] Endorser rate limits failed, status:', endorserResp.status); await this.$saveSettings({ profileImageUrl: "", }); @@ -1465,14 +1435,11 @@ export default class AccountViewView extends Vue { return; } } catch (error) { - console.log('[DEBUG] Error in checkLimits:', error); this.limitsMessage = ACCOUNT_VIEW_CONSTANTS.LIMITS.ERROR_RETRIEVING_LIMITS; this.notify.error(this.limitsMessage, TIMEOUTS.STANDARD); } finally { - console.log('[DEBUG] Setting loadingLimits to false'); this.loadingLimits = false; - console.log('[DEBUG] Final limitsMessage:', this.limitsMessage); } } @@ -1725,8 +1692,6 @@ export default class AccountViewView extends Vue { } onRecheckLimits() { - console.log('[DEBUG] onRecheckLimits called - button clicked'); - console.log('[DEBUG] Current state - loadingLimits:', this.loadingLimits, 'limitsMessage:', this.limitsMessage); this.checkLimits(); } } diff --git a/src/views/IdentitySwitcherView.vue b/src/views/IdentitySwitcherView.vue index d3a12a4c..9ef53f08 100644 --- a/src/views/IdentitySwitcherView.vue +++ b/src/views/IdentitySwitcherView.vue @@ -219,22 +219,15 @@ export default class IdentitySwitcherView extends Vue { } async switchAccount(did?: string) { - console.log('[DEBUG] IdentitySwitcher - switchAccount called with DID:', did); - console.log('[DEBUG] IdentitySwitcher - Current activeDid before switch:', this.activeDid); - // Save the new active DID to master settings await this.$saveSettings({ activeDid: did }); - console.log('[DEBUG] IdentitySwitcher - Saved new activeDid to master settings'); // Check if we need to load user-specific settings for the new DID if (did) { - console.log('[DEBUG] IdentitySwitcher - Loading user-specific settings for DID:', did); try { - const userSettings = await this.$accountSettings(did); - console.log('[DEBUG] IdentitySwitcher - User settings loaded:', userSettings); - console.log('[DEBUG] IdentitySwitcher - User isRegistered:', userSettings.isRegistered); + await this.$accountSettings(did); } catch (error) { - console.log('[DEBUG] IdentitySwitcher - Error loading user settings:', error); + // Handle error silently - user settings will be loaded when needed } }