refactor(backend): change logger level from info to trace for detailed logging

Author: Lasim

services/backend/src/cron/cronManager.ts

@@ -62,7 +62,7 @@ export class CronManager {
 
     try {
       const task = cron.schedule(schedule, async () => {
-        this.logger.info({
+        this.logger.trace({
           job: jobName,
           jobType,
           schedule,
@@ -81,7 +81,7 @@ export class CronManager {
             maxAttempts ? { maxAttempts } : undefined
           );
 
-          this.logger.info({
+          this.logger.trace({
             job: jobName,
             jobId: job.id,
             jobType,

services/backend/src/hooks/authHook.ts

@@ -37,13 +37,13 @@ export async function authHook(
     const sessionId = lucia.readSessionCookie(request.headers.cookie ?? '');
 
     if (!sessionId) {
-      request.log.debug('Auth hook: No session cookie found');
+      request.log.trace('Auth hook: No session cookie found');
       request.user = null;
       request.session = null;
       return; // Proceed as unauthenticated
     }
 
-    request.log.debug(`Auth hook: Found session ID: ${sessionId}`);
+    request.log.trace(`Auth hook: Found session ID: ${sessionId}`);
 
     // Manual session validation to avoid Lucia SQL syntax issues
     const db = getDb();
@@ -66,7 +66,7 @@ export async function authHook(
     .limit(1);
 
     if (sessionResult.length === 0) {
-      request.log.debug(`Auth hook: Session ${sessionId} not found`);
+      request.log.trace(`Auth hook: Session ${sessionId} not found`);
       const sessionCookie = lucia.createBlankSessionCookie();
       reply.setCookie(sessionCookie.name, sessionCookie.value, sessionCookie.attributes);
       request.user = null;
@@ -78,7 +78,7 @@ export async function authHook(
 
     // Check if session is expired
     if (sessionData.expiresAt < Date.now()) {
-      request.log.debug(`Auth hook: Session ${sessionId} is expired`);
+      request.log.trace(`Auth hook: Session ${sessionId} is expired`);
       // Delete expired session
       await db.delete(authSession).where(eq(authSession.id, sessionId));
       const sessionCookie = lucia.createBlankSessionCookie();
@@ -106,8 +106,8 @@ export async function authHook(
       fresh: false
     };
 
-    request.log.debug(`Auth hook: Session ${sessionId} is valid for user ${user.id}`);
-    
+    request.log.trace(`Auth hook: Session ${sessionId} is valid for user ${user.id}`);
+
     request.user = user;
     request.session = session;
     // No explicit done() call, Fastify awaits the promise

services/backend/src/middleware/oauthMiddleware.ts

@@ -100,7 +100,7 @@ export function requireOAuthScope(requiredScope: string) {
     // Skip scope check if user authenticated via cookie (not OAuth2)
     if (!request.tokenPayload) {
       // User authenticated via cookie session - skip OAuth scope validation
-      request.log.debug({
+      request.log.trace({
         operation: 'oauth_scope_check',
         userId: request.user?.id,
         requiredScope,
@@ -147,7 +147,7 @@ export function requireAnyOAuthScope(requiredScopes: string[]) {
     // Skip scope check if user authenticated via cookie (not OAuth2)
     if (!request.tokenPayload) {
       // User authenticated via cookie session - skip OAuth scope validation
-      request.log.debug({
+      request.log.trace({
         operation: 'oauth_scope_check',
         userId: request.user?.id,
         requiredScopes,
@@ -244,7 +244,7 @@ export function requireAuthenticationAny() {
   return async (request: FastifyRequest, reply: FastifyReply) => {
     // Check if user is already authenticated via cookie (from authHook)
     if (request.user && request.session) {
-      request.log.debug({
+      request.log.trace({
         operation: 'dual_auth_middleware',
         userId: request.user.id,
         authType: 'cookie',

services/backend/src/middleware/roleMiddleware.ts

@@ -336,7 +336,7 @@ export function requireTeamPermission(
 
       const userId = request.user.id;
 
-      request.log.debug({
+      request.log.trace({
         operation: 'team_permission_check',
         userId,
         teamId,

services/backend/src/middleware/satelliteAuthMiddleware.ts

@@ -130,7 +130,7 @@ export function requireUserOrSatelliteAuth() {
   return async (request: FastifyRequest, reply: FastifyReply) => {
     // Check if user is already authenticated via cookie (from authHook)
     if (request.user && request.session) {
-      request.log.debug({
+      request.log.trace({
         operation: 'hybrid_auth_middleware',
         userId: request.user.id,
         authType: 'user_cookie',

services/backend/src/routes/mcp/categories/list.ts

@@ -79,7 +79,7 @@ export default async function listCategories(server: FastifyInstance) {
       const authType = request.tokenPayload ? 'oauth2' : 'cookie';
       const userId = request.user.id;
 
-      request.log.debug({
+      request.log.trace({
         operation: 'list_mcp_categories',
         userId,
         authType,

services/backend/src/services/jobProcessorService.ts

@@ -146,13 +146,13 @@ export class JobProcessorService {
       }
 
       // Execute the worker
-      this.logger.info({ jobId: job.id, jobType: job.type, attempt: job.attempts + 1 }, 'Executing job');
-      
+      this.logger.trace({ jobId: job.id, jobType: job.type, attempt: job.attempts + 1 }, 'Executing job');
+
       const result: WorkerResult = await worker.execute(payload, job.id);
 
       if (result.success) {
         await this.jobQueueService.updateJobStatus(job.id, 'completed', undefined, result.data);
-        this.logger.info({ jobId: job.id, jobType: job.type }, 'Job completed successfully');
+        this.logger.trace({ jobId: job.id, jobType: job.type }, 'Job completed successfully');
 
         // Update batch progress if job is part of a batch
         if (job.batch_id) {

services/backend/src/services/jobQueueService.ts

@@ -56,9 +56,9 @@ export class JobQueueService {
       };
 
       await this.db.insert(this.queueJobs).values(jobData);
-      
-      this.logger.info({ jobId, type, scheduledFor, batchId }, 'Job created');
-      
+
+      this.logger.trace({ jobId, type, scheduledFor, batchId }, 'Job created');
+
       return jobData as Job;
     } catch (error) {
       this.logger.error({ error, type }, 'Failed to create job');
@@ -110,7 +110,7 @@ export class JobQueueService {
         })
         .where(eq(this.queueJobs.id, jobId));
 
-      this.logger.debug({ jobId }, 'Job marked as processing');
+      this.logger.trace({ jobId }, 'Job marked as processing');
     } catch (error) {
       this.logger.error({ error, jobId }, 'Failed to mark job as processing');
       throw error;
@@ -145,7 +145,7 @@ export class JobQueueService {
         .set(updateData)
         .where(eq(this.queueJobs.id, jobId));
 
-      this.logger.info({ jobId, status, error }, 'Job status updated');
+      this.logger.trace({ jobId, status, error }, 'Job status updated');
     } catch (error) {
       this.logger.error({ error, jobId, status }, 'Failed to update job status');
       throw error;

services/backend/src/services/mcpHealthCheckService.ts

@@ -676,14 +676,14 @@ export class McpHealthCheckService {
     oauthFailed: number;
     apiKeyRequested: number;
   }> {
-    this.logger.info({
+    this.logger.trace({
       operation: 'credential_validation_started'
     }, 'Starting credential validation run');
 
     const installations = await this.getInstallationsNeedingCredentialCheck();
 
     if (installations.length === 0) {
-      this.logger.debug({
+      this.logger.trace({
         operation: 'credential_validation_complete',
         installationsChecked: 0
       }, 'No installations need credential validation');
@@ -755,7 +755,7 @@ export class McpHealthCheckService {
       }
     }
 
-    this.logger.info({
+    this.logger.trace({
       operation: 'credential_validation_complete',
       installationsChecked: installations.length,
       oauthValidated,

services/backend/src/workers/cleanupMcpServerLogsWorker.ts

@@ -34,7 +34,7 @@ export class CleanupMcpServerLogsWorker implements Worker {
 
     const { maxLogsPerInstallation } = payload as CleanupPayload;
 
-    this.logger.info({
+    this.logger.trace({
       jobId,
       maxLogsPerInstallation,
       operation: 'cleanup_mcp_server_logs'
@@ -59,7 +59,7 @@ export class CleanupMcpServerLogsWorker implements Worker {
 
       const totalDeleted = serverLogsDeleted + requestLogsDeleted;
 
-      this.logger.info({
+      this.logger.trace({
         jobId,
         serverLogsDeleted,
         requestLogsDeleted,