Replace console.log and console.error with Neutralino.debug.log in AuthStatus.tsx, AppInitializer.ts, auth.ts, and config-manager.ts.

tsjnsn · tsjnsn · commit e0b8a9ea1325 · 2025-04-15T22:29:49.000-05:00
Add error level logs to the user-facing console:
* `AuthStatus.tsx`: Add detailed error messages for authentication, refresh, check, and logout processes.
* `AppInitializer.ts`: Add error message for initialization.
* `auth.ts`: Add error message for token saving and loading.
* `config-manager.ts`: Add error message for config loading and saving.

Expand fetch error messages to show full request and response details in `AuthStatus.tsx`.
diff --git a/client/src/components/AuthStatus.tsx b/client/src/components/AuthStatus.tsx
@@ -25,7 +25,12 @@ export const AuthStatus: Component = () => {
         throw new Error(`Failed to open authentication window: ${error instanceof Error ? error.message : 'Unknown error'}`);
       }
     } catch (error) {
-      Neutralino.debug.log('Error starting authentication: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+      const errorMessage = `Error starting authentication: ${error instanceof Error ? error.message : 'Unknown error'} | Request: ${JSON.stringify({ url: `${botServerUrl}/auth?redirect_uri=${window.location.origin}/auth.html&NL_TOKEN=${window.NL_TOKEN}` })}`;
+      Neutralino.debug.log(errorMessage, 'ERROR');
+      window.consoleAddMessage?.({
+        text: errorMessage,
+        type: 'error'
+      });
     } finally {
       setIsLoading(false);
     }
@@ -60,9 +65,14 @@ export const AuthStatus: Component = () => {
         throw new Error('Failed to refresh authentication');
       }
     } catch (error) {
+      const errorMessage = `Error refreshing authentication: ${error instanceof Error ? error.message : 'Unknown error'} | Request: ${JSON.stringify({ url: `${botServerUrl}/auth/refresh`, body: { refresh_token: authData.tokens.refresh_token } })} | Response: ${JSON.stringify(error.response)}`;
       await state.auth.saveAuthData(null);
       setIsAuthenticated(false);
-      Neutralino.debug.log('Error refreshing authentication: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+      Neutralino.debug.log(errorMessage, 'ERROR');
+      window.consoleAddMessage?.({
+        text: errorMessage,
+        type: 'error'
+      });
     } finally {
       setIsLoading(false);
     }
@@ -86,7 +96,12 @@ export const AuthStatus: Component = () => {
         await authenticate();
       }
     } catch (error) {
-      Neutralino.debug.log('Error checking authentication: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+      const errorMessage = `Error checking authentication: ${error instanceof Error ? error.message : 'Unknown error'} | Request: ${JSON.stringify({ url: `${botServerUrl}/auth/check` })}`;
+      Neutralino.debug.log(errorMessage, 'ERROR');
+      window.consoleAddMessage?.({
+        text: errorMessage,
+        type: 'error'
+      });
     } finally {
       setIsLoading(false);
     }
@@ -101,7 +116,12 @@ export const AuthStatus: Component = () => {
       setIsAuthenticated(false);
       Neutralino.debug.log('Logged out successfully', 'INFO');
     } catch (error) {
-      Neutralino.debug.log('Error during logout: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+      const errorMessage = `Error during logout: ${error instanceof Error ? error.message : 'Unknown error'}`;
+      Neutralino.debug.log(errorMessage, 'ERROR');
+      window.consoleAddMessage?.({
+        text: errorMessage,
+        type: 'error'
+      });
     } finally {
       setIsLoading(false);
     }
diff --git a/client/src/services/AppInitializer.ts b/client/src/services/AppInitializer.ts
@@ -27,6 +27,10 @@ class AppInitializer {
       Neutralino.debug.log('Application initialized successfully', 'INFO');
     } catch (error) {
       Neutralino.debug.log('Error during initialization: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+      window.consoleAddMessage?.({
+        text: `Error during initialization: ${error instanceof Error ? error.message : 'Unknown error'}`,
+        type: 'error'
+      });
       throw error;
     }
   }
diff --git a/client/src/utils/auth.ts b/client/src/utils/auth.ts
@@ -38,7 +38,12 @@ export class Auth {
             if (ev.detail.tokens) {
                 await this.saveAuthData(ev.detail);
             } else {
-                Neutralino.debug.log('No access token received in authentication callback', 'ERROR');
+                const errorMessage = 'No access token received in authentication callback';
+                Neutralino.debug.log(errorMessage, 'ERROR');
+                window.consoleAddMessage?.({
+                    text: errorMessage,
+                    type: 'error'
+                });
             }
         });
     }
@@ -54,7 +59,12 @@ export class Auth {
                 Neutralino.debug.log('Auth data cleared successfully', 'INFO');
             }
         } catch (err) {
-            Neutralino.debug.log('Error saving discord details to storage: ' + (err instanceof Error ? err.message : 'Unknown error'), 'ERROR');
+            const errorMessage = `Error saving discord details to storage: ${err instanceof Error ? err.message : 'Unknown error'}`;
+            Neutralino.debug.log(errorMessage, 'ERROR');
+            window.consoleAddMessage?.({
+                text: errorMessage,
+                type: 'error'
+            });
             throw err;
         }
     }
@@ -70,7 +80,12 @@ export class Auth {
                 Neutralino.debug.log('No discord details found in storage', 'WARN');
             }
         } catch (err) {
-            Neutralino.debug.log('Error getting discord details from storage: ' + (err instanceof Error ? err.message : 'Unknown error'), 'ERROR');
+            const errorMessage = `Error getting discord details from storage: ${err instanceof Error ? err.message : 'Unknown error'}`;
+            Neutralino.debug.log(errorMessage, 'ERROR');
+            window.consoleAddMessage?.({
+                text: errorMessage,
+                type: 'error'
+            });
             this.authData = null;
         }
     }
diff --git a/client/src/utils/config-manager.ts b/client/src/utils/config-manager.ts
@@ -64,15 +64,19 @@ export default class ConfigManager {
                 const fileContent = await Neutralino.filesystem.readFile(this.configPath);
                 const loadedConfig = JSON.parse(fileContent);
                 this.config = this.deepMerge(this.defaultConfig, loadedConfig);
-                Neutralino.debug.log('Config loaded successfully', 'INFO'); // P3998
+                Neutralino.debug.log('Config loaded successfully', 'INFO');
             } else {
                 this.config = { ...this.defaultConfig };
                 await this.save();
             }
             this.isInitialized = true;
         } catch (error) {
-            Neutralino.debug.log('Error loading config: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR'); // Peb86
-            Neutralino.debug.log('Using default configuration...', 'WARN'); // Peb86
+            Neutralino.debug.log('Error loading config: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+            window.consoleAddMessage?.({
+                text: `Error loading config: ${error instanceof Error ? error.message : 'Unknown error'}`,
+                type: 'error'
+            });
+            Neutralino.debug.log('Using default configuration...', 'WARN');
             this.config = { ...this.defaultConfig };
             await this.save();
             this.isInitialized = true;
@@ -105,9 +109,13 @@ export default class ConfigManager {
     async save(): Promise<void> {
         try {
             await Neutralino.filesystem.writeFile(this.configPath, JSON.stringify(this.config, null, 2));
-            Neutralino.debug.log('Config saved successfully', 'INFO'); // P3998
+            Neutralino.debug.log('Config saved successfully', 'INFO');
         } catch (error) {
-            Neutralino.debug.log('Error saving config: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR'); // Peb86
+            Neutralino.debug.log('Error saving config: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');
+            window.consoleAddMessage?.({
+                text: `Error saving config: ${error instanceof Error ? error.message : 'Unknown error'}`,
+                type: 'error'
+            });
             throw error;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,10 @@ class AppInitializer {`
`27`	`27`	`Neutralino.debug.log('Application initialized successfully', 'INFO');`
`28`	`28`	`} catch (error) {`
`29`	`29`	`Neutralino.debug.log('Error during initialization: ' + (error instanceof Error ? error.message : 'Unknown error'), 'ERROR');`
	`30`	`+ window.consoleAddMessage?.({`
	`31`	+ text: `Error during initialization: ${error instanceof Error ? error.message : 'Unknown error'}`,
	`32`	`+ type: 'error'`
	`33`	`+ });`
`30`	`34`	`throw error;`
`31`	`35`	`}`
`32`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,12 @@ export class Auth {`
`38`	`38`	`if (ev.detail.tokens) {`
`39`	`39`	`await this.saveAuthData(ev.detail);`
`40`	`40`	`} else {`
`41`		`- Neutralino.debug.log('No access token received in authentication callback', 'ERROR');`
	`41`	`+ const errorMessage = 'No access token received in authentication callback';`
	`42`	`+ Neutralino.debug.log(errorMessage, 'ERROR');`
	`43`	`+ window.consoleAddMessage?.({`
	`44`	`+ text: errorMessage,`
	`45`	`+ type: 'error'`
	`46`	`+ });`
`42`	`47`	`}`
`43`	`48`	`});`
`44`	`49`	`}`
`@@ -54,7 +59,12 @@ export class Auth {`
`54`	`59`	`Neutralino.debug.log('Auth data cleared successfully', 'INFO');`
`55`	`60`	`}`
`56`	`61`	`} catch (err) {`
`57`		`- Neutralino.debug.log('Error saving discord details to storage: ' + (err instanceof Error ? err.message : 'Unknown error'), 'ERROR');`
	`62`	+ const errorMessage = `Error saving discord details to storage: ${err instanceof Error ? err.message : 'Unknown error'}`;
	`63`	`+ Neutralino.debug.log(errorMessage, 'ERROR');`
	`64`	`+ window.consoleAddMessage?.({`
	`65`	`+ text: errorMessage,`
	`66`	`+ type: 'error'`
	`67`	`+ });`
`58`	`68`	`throw err;`
`59`	`69`	`}`
`60`	`70`	`}`
`@@ -70,7 +80,12 @@ export class Auth {`
`70`	`80`	`Neutralino.debug.log('No discord details found in storage', 'WARN');`
`71`	`81`	`}`
`72`	`82`	`} catch (err) {`
`73`		`- Neutralino.debug.log('Error getting discord details from storage: ' + (err instanceof Error ? err.message : 'Unknown error'), 'ERROR');`
	`83`	+ const errorMessage = `Error getting discord details from storage: ${err instanceof Error ? err.message : 'Unknown error'}`;
	`84`	`+ Neutralino.debug.log(errorMessage, 'ERROR');`
	`85`	`+ window.consoleAddMessage?.({`
	`86`	`+ text: errorMessage,`
	`87`	`+ type: 'error'`
	`88`	`+ });`
`74`	`89`	`this.authData = null;`
`75`	`90`	`}`
`76`	`91`	`}`