fix: logger

Author: izadoesdev

apps/basket/src/lib/logger.ts

@@ -1,12 +1,3 @@
-// import { logger as sharedLogger } from "@databuddy/shared/logger";
+import { createLogger } from "@databuddy/shared/logger";
 
-// export const logger = sharedLogger.child({
-// 	service: "basket",
-// });
-
-import pino from "pino";
-
-export const logger = pino({
-	level: "debug",
-	name: "basket",
-});
+export const logger = createLogger("basket");

packages/shared/src/utils/logger.ts

@@ -1,33 +1,59 @@
 import pino from "pino";
 
-const token = process.env.AXIOM_TOKEN as string;
-const dataset = process.env.AXIOM_DATASET as string;
+type AxiomConfig = {
+    token: string;
+    dataset: string;
+};
+
+function canLoadAxiomTransport(): boolean {
+    try {
+        require.resolve("@axiomhq/pino");
+        return true;
+    } catch {
+        return false;
+    }
+}
 
-const createLoggerConfig = (name: string): pino.LoggerOptions => {
+function createLoggerConfig(
+    name: string,
+    axiomConfig?: AxiomConfig
+): pino.LoggerOptions {
     const config: pino.LoggerOptions = {
         level: "debug",
         name,
     };
 
-    if (token && dataset) {
-        try {
-            config.transport = {
-                target: "@axiomhq/pino",
-                options: {
-                    token,
-                    dataset,
-                },
-            };
-        } catch {
-            // Transport not available, fallback to default logger
-        }
+    // Only set up Axiom transport if:
+    // 1. Axiom config is provided
+    // 2. The @axiomhq/pino module can be resolved
+    if (axiomConfig?.token && axiomConfig?.dataset && canLoadAxiomTransport()) {
+        config.transport = {
+            target: "@axiomhq/pino",
+            options: {
+                token: axiomConfig.token,
+                dataset: axiomConfig.dataset,
+            },
+        };
     }
 
     return config;
-};
+}
+
+const defaultAxiomConfig: AxiomConfig | undefined =
+    process.env.AXIOM_TOKEN && process.env.AXIOM_DATASET
+        ? {
+            token: process.env.AXIOM_TOKEN,
+            dataset: process.env.AXIOM_DATASET,
+        }
+        : undefined;
 
-export const logger = pino(createLoggerConfig("databuddy"));
+export const logger = pino(
+    createLoggerConfig("databuddy", defaultAxiomConfig)
+);
 
-export function createLogger(name: string) {
-    return pino(createLoggerConfig(name));
+export function createLogger(
+    name: string,
+    axiomConfig?: AxiomConfig
+): pino.Logger {
+    return pino(createLoggerConfig(name, axiomConfig));
 }