Change logger to singleton (#564)

Closes PRO-722.

This PR moves the logger to singleton, we can keep all tags centrally. Will be needed for the sessionId change I'm doing.

packages/puppeteer/src/install.ts

@@ -1,7 +1,8 @@
+import { initLogger, logger } from "@replay-cli/shared/logger";
 import { installLatestRuntimeRelease } from "@replay-cli/shared/runtime/installLatestRuntimeRelease";
-import { logger } from "@replay-cli/shared/runtime/logger";
 
 async function install() {
+  initLogger("puppeteer");
   try {
     await installLatestRuntimeRelease();
   } finally {

packages/shared/src/graphql/fetchAuthIdsFromGraphQL.ts

@@ -1,6 +1,6 @@
 import { base64Decode } from "../strings/decode";
 import { GraphQLError } from "./GraphQLError";
-import { logger } from "./debug";
+import { logger } from "../logger";
 import { queryGraphQL } from "./queryGraphQL";
 
 export type AuthIds = { userId: string | null; workspaceId: string | null };

packages/shared/src/graphql/queryGraphQL.ts

@@ -1,7 +1,7 @@
 import { fetch } from "undici";
 import { replayApiServer } from "../config";
 import { getUserAgent } from "../userAgent";
-import { logger } from "./debug";
+import { logger } from "../logger";
 
 export async function queryGraphQL(name: string, query: string, variables = {}, apiKey?: string) {
   const options = {

packages/shared/src/logger/logger.ts

@@ -37,6 +37,7 @@ class Logger {
       basicAuth: GRAFANA_BASIC_AUTH,
       format: winston.format.json(),
       replaceTimestamp: true,
+      timeout: 5000,
       onConnectionError: err => this.localDebugger("Grafana connection error", err),
       gracefulShutdown: true,
     });
@@ -94,4 +95,17 @@ class Logger {
   }
 }
 
-export { Logger };
+let logger: Logger;
+
+// This should be called with the name once at the entry point.
+// For example, with the Playwright plugin, it is called in the Reporter interface constructor.
+function initLogger(name: string) {
+  if (logger) {
+    console.warn(`Logger already initialized.`);
+    return logger;
+  }
+  logger = new Logger(name);
+  return logger;
+}
+
+export { initLogger, logger };

packages/shared/src/runtime/getLatestRuntimeRelease.ts

@@ -2,7 +2,7 @@ import assert from "node:assert/strict";
 import { fetch } from "undici";
 import { replayAppHost } from "../config";
 import { runtimeMetadata } from "./config";
-import { logger } from "./logger";
+import { logger } from "../logger/logger";
 import { Release } from "./types";
 
 const { architecture, platform, runtime } = runtimeMetadata;

packages/shared/src/runtime/getRuntimePath.ts

@@ -1,6 +1,6 @@
 import { getReplayPath } from "../getReplayPath";
 import { runtimeMetadata } from "./config";
-import { logger } from "./logger";
+import { logger } from "../logger/logger";
 
 export function getRuntimePath() {
   const overridePathKey = `REPLAY_CHROMIUM_EXECUTABLE_PATH`;

packages/shared/src/runtime/installLatestRuntimeRelease.ts

@@ -7,7 +7,7 @@ import { writeToCache } from "../cache";
 import { dim, link } from "../theme";
 import { metadataPath, runtimeBasePath, runtimeMetadata } from "./config";
 import { getLatestRuntimeRelease } from "./getLatestRuntimeRelease";
-import { logger } from "./logger";
+import { logger } from "../logger/logger";
 import { MetadataJSON } from "./types";
 
 const MAX_DOWNLOAD_ATTEMPTS = 5;
@@ -84,7 +84,6 @@ export async function installLatestRuntimeRelease(): Promise<Result | undefined>
     };
   } catch (error) {
     logger.debug(`Browser installation failed: ${error}`);
-
     // TODO [PRO-711]
     // TODO [PRO-712] Add Mixpanel tracking "failed"
   }