Migrate src/events console statements to structured logger (#61073)

Author: Ebonsignori

eslint.config.ts

@@ -179,7 +179,7 @@ export default [
       'src/color-schemes/**/*.{ts,js}',
       'src/data-directory/**/*.{ts,js}',
       'src/dev-toc/**/*.{ts,js}',
-      'src/events/**/*.{ts,js}',
+      'src/events/components/**/*.{ts,js}',
       'src/fixtures/**/*.{ts,js}',
       'src/github-apps/**/*.{ts,js}',
       'src/journeys/**/*.{ts,js}',

src/events/lib/hydro.ts

@@ -18,7 +18,7 @@ const { NODE_ENV, HYDRO_SECRET, HYDRO_ENDPOINT } = process.env
 const inProd = NODE_ENV === 'production'
 
 if (inProd && (isNil(HYDRO_SECRET) || isNil(HYDRO_ENDPOINT))) {
-  console.warn(
+  logger.warn(
     'Running in production but HYDRO_SECRET and HYDRO_ENDPOINT environment variables are not set.',
   )
 }

src/events/middleware.ts

@@ -1,3 +1,4 @@
+import { createLogger } from '@/observability/logger'
 import express from 'express'
 import { omit, without, mapValues } from 'lodash-es'
 import QuickLRU from 'quick-lru'
@@ -17,6 +18,8 @@ import { DOTCOM_USER_COOKIE_NAME, STAFFONLY_COOKIE_NAME } from '@/frame/lib/cons
 import { analyzeComment, getGuessedLanguage } from './lib/analyze-comment'
 import { EventType, EventProps, EventPropsByType } from './types'
 
+const logger = createLogger(import.meta.url)
+
 const router = express.Router()
 const OMIT_FIELDS = ['type']
 const allowedTypes = new Set(without(Object.keys(schemas), 'validation'))
@@ -103,7 +106,7 @@ router.post(
           value: omit(body, OMIT_FIELDS),
         })
       } catch (eventError) {
-        console.error('Error validating event:', eventError)
+        logger.error('Error validating event', { error: eventError })
       }
     }
     if (validEvents.length > 0) {