refactor: remove verbose info and debug logging statements (#1731)

Author: jonathanlab

apps/code/src/main/db/service.ts

@@ -41,7 +41,6 @@ export class DatabaseService {
       this._sqlite.pragma("foreign_keys = ON");
       this._db = drizzle(this._sqlite, { schema, casing: "snake_case" });
       migrate(this._db, { migrationsFolder: MIGRATIONS_FOLDER });
-      log.info("Database ready");
     } catch (error) {
       log.error("Database initialization failed", error);
       throw error;

apps/code/src/main/protocols/mcp-sandbox.ts

@@ -28,10 +28,6 @@ const log = logger.scope("mcp-sandbox protocol");
 export function registerMcpSandboxProtocol(): void {
   const mainSession = session.fromPartition("persist:main");
 
-  log.info("Registering protocol handler on persist:main", {
-    htmlLength: sandboxProxyHtml.length,
-  });
-
   mainSession.protocol.handle("mcp-sandbox", (request) => {
     log.debug("Serving proxy HTML", { url: request.url });
 

apps/code/src/main/services/agent/service.ts

@@ -427,11 +427,9 @@ export class AgentService extends TypedEventEmitter<AgentServiceEvents> {
   public hasActiveSessions(): boolean {
     for (const session of this.sessions.values()) {
       if (session.promptPending || session.inFlightMcpToolCalls.size > 0) {
-        log.info("Active session found", { sessionId: session.taskRunId });
         return true;
       }
     }
-    log.debug("No active sessions found");
     return false;
   }
 
@@ -890,14 +888,6 @@ When creating pull requests, add the following footer at the end of the PR descr
     this.recordActivity(sessionId);
     this.sleepService.acquire(sessionId);
 
-    const promptJson = JSON.stringify(finalPrompt);
-    log.info("Sending prompt to agent", {
-      sessionId,
-      blockCount: finalPrompt.length,
-      blocks: promptJson.slice(0, 10000),
-      totalSize: promptJson.length,
-    });
-
     try {
       const result = await session.clientSideConnection.prompt({
         sessionId: getAgentSessionId(session),
@@ -994,8 +984,6 @@ When creating pull requests, add the following footer at the end of the PR descr
       ) {
         session.config.permissionMode = updatedModeOption.currentValue;
       }
-
-      log.info("Session config option updated", { sessionId, configId, value });
     } catch (err) {
       log.error("Failed to set session config option", {
         sessionId,

apps/code/src/main/services/auth-proxy/service.ts

@@ -34,7 +34,6 @@ export class AuthProxyService {
         const addr = this.server?.address();
         if (typeof addr === "object" && addr) {
           this.port = addr.port;
-          log.info("Auth proxy started", { port: this.port });
           resolve(this.getProxyUrl());
         } else {
           reject(new Error("Failed to get proxy address"));
@@ -118,12 +117,6 @@ export class AuthProxyService {
       return;
     }
 
-    log.debug("Proxying request", {
-      method: req.method,
-      incoming: req.url,
-      target: targetUrl.toString(),
-    });
-
     const strippedAuthHeaders = new Set([
       "authorization",
       "x-api-key",
@@ -173,11 +166,6 @@ export class AuthProxyService {
         options,
       );
 
-      log.debug("Proxy response", {
-        url,
-        status: response.status,
-      });
-
       const responseHeaders: Record<string, string> = {};
       const stripHeaders = new Set([
         "transfer-encoding",

apps/code/src/main/services/connectivity/service.ts

@@ -25,7 +25,6 @@ export class ConnectivityService extends TypedEventEmitter<ConnectivityEvents> {
   @postConstruct()
   init(): void {
     this.isOnline = net.isOnline();
-    log.info("Initial connectivity status", { isOnline: this.isOnline });
 
     this.startPolling();
   }

apps/code/src/main/services/deep-link/service.ts

@@ -25,9 +25,6 @@ export class DeepLinkService {
     // Skip protocol registration in development to avoid hijacking deep links
     // from the production app. OAuth uses HTTP callback in dev mode anyway.
     if (process.defaultApp) {
-      log.info(
-        "Skipping protocol registration in development (using HTTP callback for OAuth)",
-      );
       return;
     }
 
@@ -38,17 +35,13 @@ export class DeepLinkService {
     }
 
     this.protocolRegistered = true;
-    log.info(
-      `Registered '${PROTOCOL}' and legacy [${LEGACY_PROTOCOLS.join(", ")}] protocol handlers`,
-    );
   }
 
   public registerHandler(key: string, handler: DeepLinkHandler): void {
     if (this.handlers.has(key)) {
       log.warn(`Overwriting existing handler for key: ${key}`);
     }
     this.handlers.set(key, handler);
-    log.info(`Registered deep link handler for key: ${key}`);
   }
 
   public unregisterHandler(key: string): void {

apps/code/src/main/services/mcp-apps/service.ts

@@ -154,7 +154,6 @@ export class McpAppsService extends TypedEventEmitter<McpAppsServiceEvents> {
       throw new Error(`No server config for: ${serverName}`);
     }
 
-    log.info("Creating lazy MCP connection", { serverName, url: config.url });
     const connectionPromise = this.createConnection(config);
     this.pendingConnections.set(serverName, connectionPromise);
 

apps/code/src/main/services/mcp-proxy/service.ts

@@ -201,24 +201,12 @@ export class McpProxyService {
             status: response.status,
             body: bodyText.slice(0, 2000),
           });
-        } else {
-          log.debug("MCP proxy response", {
-            id,
-            url,
-            status: response.status,
-          });
         }
 
         this.writeBufferedResponse(response, buf, res);
         return;
       }
 
-      log.debug("MCP proxy response", {
-        id,
-        url,
-        status: response.status,
-        streaming: true,
-      });
       this.writeStreamingResponse(response, res);
     } catch (err) {
       log.error("MCP proxy forward error", { id, url, err });

apps/code/src/main/services/notification/service.ts

@@ -19,7 +19,6 @@ export class NotificationService {
   @postConstruct()
   init(): void {
     app.on("browser-window-focus", () => this.clearDockBadge());
-    log.info("Notification service initialized");
   }
 
   send(title: string, body: string, silent: boolean, taskId?: string): void {
@@ -47,7 +46,6 @@ export class NotificationService {
     });
 
     notification.show();
-    log.info("Notification sent", { title, body, silent, taskId });
   }
 
   showDockBadge(): void {
@@ -59,7 +57,6 @@ export class NotificationService {
     } else if (process.platform === "win32") {
       getMainWindow()?.flashFrame(true);
     }
-    log.info("Dock badge shown");
   }
 
   bounceDock(): void {
@@ -80,6 +77,5 @@ export class NotificationService {
     } else if (process.platform === "win32") {
       getMainWindow()?.flashFrame(false);
     }
-    log.info("Dock badge cleared");
   }
 }

apps/code/src/main/services/oauth/service.ts

@@ -56,7 +56,6 @@ export class OAuthService {
     this.deepLinkService.registerHandler("callback", (_path, searchParams) =>
       this.handleOAuthCallback(searchParams),
     );
-    log.info("Registered OAuth callback handler for deep links");
   }
 
   private handleOAuthCallback(searchParams: URLSearchParams): boolean {