feat: switch to optional dependencies

Author: noomorph

build.mjs

@@ -1,10 +1,12 @@
 #!/usr/bin/env node
 
 import { cosmiconfig } from 'cosmiconfig';
+import { useLoggingForCLI } from './utils/cli-logger.mjs';
 import { logger } from './utils/logger.mjs';
 import { build } from './index.mjs';
 
 try {
+  await useLoggingForCLI();
   const explorer = cosmiconfig('esbuild-jest');
   const esbuildJestBaseConfig = await explorer.search();
   await logger.trace.complete(esbuildJestBaseConfig, 'esbuild-jest-cli', () => build(esbuildJestBaseConfig.config));

package.json

@@ -32,21 +32,23 @@
   },
   "homepage": "https://github.com/wix-incubator/esbuild-jest-cli#readme",
   "dependencies": {
-    "bunyamin": "^1.5.1",
-    "bunyan": "^2.0.0",
-    "bunyan-debug-stream": "^3.1.0",
-    "cosmiconfig": "^8.1.3",
+    "bunyamin": "^1.6.3",
     "find-up": "^7.0.0",
     "import-from": "^4.0.0",
     "lodash.merge": "^4.6.2",
     "resolve-from": "^5.0.0"
   },
+  "optionalDependencies": {
+    "bunyan": "^2.0.0",
+    "bunyan-debug-stream": "^3.1.0",
+    "cosmiconfig": "^9.0.0"
+  },
   "peerDependencies": {
     "esbuild": ">=0.17.4"
   },
   "devDependencies": {
     "@jest/transform": "^29.6.3",
     "@jest/types": "^29.6.3",
-    "esbuild": "^0.21.4"
+    "esbuild": "^0.24.2"
   }
 }

utils/cli-logger.mjs

@@ -0,0 +1,60 @@
+import * as fs from 'node:fs';
+import * as path from 'node:path';
+import {
+  bunyamin,
+  threadGroups,
+  traceEventStream,
+} from 'bunyamin';
+
+const PACKAGE_NAME = 'esbuild-jest-cli';
+
+export async function useLoggingForCLI() {
+  const bunyan = await import('bunyan');
+  const bds = await import('bunyan-debug-stream');
+  const logPath = process.env.BUNYAMIN_LOG;
+
+  const logger = bunyan.createLogger({
+    name: PACKAGE_NAME,
+    streams: [
+      {
+        type: 'raw',
+        level: 'warn',
+        stream: bds.create({
+          out: process.stderr,
+          showMetadata: false,
+          showDate: false,
+          showPid: false,
+          showProcess: false,
+          showLoggerName: false,
+          showLevel: false,
+          prefixers: {
+            cat: (value) => String(value).split(',', 1)[0],
+          },
+        }),
+      },
+      ...(logPath
+        ? [
+          {
+            type: 'raw',
+            level: 'trace',
+            stream: traceEventStream({
+              filePath: ensureCleanLog(logPath),
+              threadGroups,
+            }),
+          },
+        ]
+        : []),
+    ],
+  });
+
+  bunyamin.useLogger(logger, 1);
+}
+
+function ensureCleanLog(filePath) {
+  fs.mkdirSync(path.dirname(filePath), { recursive: true });
+  if (fs.existsSync(filePath)) {
+    fs.unlinkSync(filePath);
+  }
+
+  return filePath;
+}

utils/logger.mjs

@@ -1,14 +1,9 @@
-import * as fs from 'node:fs';
-import * as path from 'node:path';
 import {
   bunyamin,
   nobunyamin,
   isDebug,
   threadGroups,
-  traceEventStream,
 } from 'bunyamin';
-import { createLogger } from 'bunyan';
-import { create as createDebugStream } from 'bunyan-debug-stream';
 
 const PACKAGE_NAME = 'esbuild-jest-cli';
 
@@ -23,8 +18,6 @@ threadGroups.add({
   maxConcurrency: 100500,
 });
 
-bunyamin.useLogger(createBunyanImpl(), 1);
-
 export const logger = bunyamin.child({ tid: PACKAGE_NAME });
 export const optimizedLogger = isDebug(PACKAGE_NAME)
   ? logger
@@ -35,50 +28,3 @@ const noop = () => {};
 export const optimizeTracing = isDebug(PACKAGE_NAME)
   ? ((f) => f)
   : (() => noop);
-
-function createBunyanImpl() {
-  const logPath = process.env.BUNYAMIN_LOG;
-
-  return createLogger({
-    name: PACKAGE_NAME,
-    streams: [
-      {
-        type: 'raw',
-        level: 'warn',
-        stream: createDebugStream({
-          out: process.stderr,
-          showMetadata: false,
-          showDate: false,
-          showPid: false,
-          showProcess: false,
-          showLoggerName: false,
-          showLevel: false,
-          prefixers: {
-            cat: (value) => String(value).split(',', 1)[0],
-          },
-        }),
-      },
-      ...(logPath
-        ? [
-          {
-            type: 'raw',
-            level: 'trace',
-            stream: traceEventStream({
-              filePath: ensureCleanLog(logPath),
-              threadGroups,
-            }),
-          },
-        ]
-        : []),
-    ],
-  });
-}
-
-function ensureCleanLog(filePath) {
-  fs.mkdirSync(path.dirname(filePath), { recursive: true });
-  if (fs.existsSync(filePath)) {
-    fs.unlinkSync(filePath);
-  }
-
-  return filePath;
-}