remove common logger

Author: charleslien

backend/src/__tests__/cost-aggregation.integration.test.ts

@@ -116,17 +116,6 @@ describe('Cost Aggregation Integration Tests', () => {
       },
       withLoggerContext: async (context: any, fn: () => Promise<any>) => fn(),
     }))
-
-    // Mock logger for common (used by billing package)
-    mockModule('@codebuff/common/util/logger', () => ({
-      logger: {
-        debug: () => {},
-        error: () => {},
-        info: () => {},
-        warn: () => {},
-      },
-      withLoggerContext: async (context: any, fn: () => Promise<any>) => fn(),
-    }))
   })
 
   beforeEach(async () => {

backend/src/process-str-replace.ts

@@ -1,7 +1,7 @@
-import { logger } from '@codebuff/common/util/logger'
 import { createPatch } from 'diff'
 
 import { tryToDoStringReplacementWithExtraIndentation } from './generate-diffs-prompt'
+import { logger } from './util/logger'
 
 function normalizeLineEndings(params: { str: string }): string {
   return params.str.replace(/\r\n/g, '\n')
@@ -47,7 +47,9 @@ export async function processStrReplace(params: {
       continue
     }
 
-    const normalizedCurrentContent = normalizeLineEndings({ str: currentContent })
+    const normalizedCurrentContent = normalizeLineEndings({
+      str: currentContent,
+    })
     const normalizedOldStr = normalizeLineEndings({ str: oldStr })
     const normalizedNewStr = normalizeLineEndings({ str: newStr })
 

common/src/util/logger.ts

@@ -25,7 +25,6 @@ Currently depends on code from the `common` package for:
 
 - Database access (`common/db`)
 - Stripe integration (`common/util/stripe`)
-- Logging (`common/util/logger`)
 - Database schema (`common/db/schema`)
 - Types (`common/types/usage`)
 - Date utilities (`common/util/date`)

packages/billing/knowledge.md

@@ -1,6 +1,6 @@
 import { db } from '@codebuff/common/db'
 import * as schema from '@codebuff/common/db/schema'
-import { logger } from '@codebuff/common/util/logger'
+import { getErrorObject } from '@codebuff/common/util/error'
 import { and, gte, lt, sql } from 'drizzle-orm'
 
 async function calculateMonthlyUsage(month: string) {
@@ -87,7 +87,10 @@ async function calculateMonthlyUsage(month: string) {
       }
     })
   } catch (error) {
-    logger.error({ error }, 'Error calculating monthly usage')
+    console.error(
+      { error: getErrorObject(error) },
+      'Error calculating monthly usage',
+    )
     throw error
   }
 }