fix: Only capture logs if enableLogs is true (#1235)

Author: timfish

src/main/integrations/child-process.ts

@@ -80,6 +80,7 @@ export const childProcessIntegration = defineIntegration((userOptions: Partial<O
       // only hook these events if we're after more than just the unresponsive event
       if (allReasons.length > 0) {
         const clientOptions = client.getOptions() as ElectronMainOptions;
+        const enableLogs = !!clientOptions?.enableLogs;
 
         app.on('child-process-gone', (_, details) => {
           const { reason } = details;
@@ -100,11 +101,13 @@ export const childProcessIntegration = defineIntegration((userOptions: Partial<O
               data: details,
             });
 
-            log(messageFmt, {
-              exitCode: details.exitCode,
-              name: details.name,
-              serviceName: details.serviceName,
-            });
+            if (enableLogs) {
+              log(messageFmt, {
+                exitCode: details.exitCode,
+                name: details.name,
+                serviceName: details.serviceName,
+              });
+            }
           }
         });
 
@@ -128,9 +131,11 @@ export const childProcessIntegration = defineIntegration((userOptions: Partial<O
               data: details,
             });
 
-            log(messageFmt, {
-              exitCode: details.exitCode,
-            });
+            if (enableLogs) {
+              log(messageFmt, {
+                exitCode: details.exitCode,
+              });
+            }
           }
         });
       }

src/main/integrations/electron-breadcrumbs.ts

@@ -110,55 +110,58 @@ export const electronBreadcrumbsIntegration = defineIntegration(
       ...normalizeOptions(userOptions),
     };
 
-    function patchEventEmitter(
-      emitter: NodeJS.EventEmitter | WebContents | BrowserWindow,
-      category: string,
-      shouldCapture: EventFunction | undefined | false,
-      id?: number | undefined,
-    ): void {
-      const emit = emitter.emit.bind(emitter) as (event: string, ...args: unknown[]) => boolean;
-
-      emitter.emit = (event: string, ...args: unknown[]) => {
-        // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
-        if (shouldCapture && shouldCapture(event)) {
-          const breadcrumb: Breadcrumb = {
-            category: 'electron',
-            message: `${category}.${event}`,
-            timestamp: new Date().getTime() / 1_000,
-            type: 'ui',
-          };
-
-          if (id) {
-            breadcrumb.data = { ...getRendererProperties(id) };
-
-            if (!options.captureWindowTitles && breadcrumb.data?.title) {
-              delete breadcrumb.data?.title;
-            }
-          }
-
-          addBreadcrumb(breadcrumb);
-
-          const attributes: Record<string, unknown> = {};
-
-          if (breadcrumb.data?.id) {
-            attributes.id = breadcrumb.data.id;
-          }
-
-          if (breadcrumb.data?.url) {
-            attributes.url = breadcrumb.data.url;
-          }
-
-          logger.info(logger.fmt`electron.${category}.${event}`, attributes);
-        }
-
-        return emit(event, ...args);
-      };
-    }
-
     return {
       name: 'ElectronBreadcrumbs',
       setup(client: NodeClient) {
         const clientOptions = client.getOptions() as ElectronMainOptions | undefined;
+        const enableLogs = !!clientOptions?.enableLogs;
+
+        function patchEventEmitter(
+          emitter: NodeJS.EventEmitter | WebContents | BrowserWindow,
+          category: string,
+          shouldCapture: EventFunction | undefined | false,
+          id?: number | undefined,
+        ): void {
+          const emit = emitter.emit.bind(emitter) as (event: string, ...args: unknown[]) => boolean;
+
+          emitter.emit = (event: string, ...args: unknown[]) => {
+            // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
+            if (shouldCapture && shouldCapture(event)) {
+              const breadcrumb: Breadcrumb = {
+                category: 'electron',
+                message: `${category}.${event}`,
+                timestamp: new Date().getTime() / 1_000,
+                type: 'ui',
+              };
+
+              if (id) {
+                breadcrumb.data = { ...getRendererProperties(id) };
+
+                if (!options.captureWindowTitles && breadcrumb.data?.title) {
+                  delete breadcrumb.data?.title;
+                }
+              }
+
+              addBreadcrumb(breadcrumb);
+
+              const attributes: Record<string, unknown> = {};
+
+              if (breadcrumb.data?.id) {
+                attributes.id = breadcrumb.data.id;
+              }
+
+              if (breadcrumb.data?.url) {
+                attributes.url = breadcrumb.data.url;
+              }
+
+              if (enableLogs) {
+                logger.info(logger.fmt`electron.${category}.${event}`, attributes);
+              }
+            }
+
+            return emit(event, ...args);
+          };
+        }
 
         trackRendererProperties();