Sm/new logger

package.json

@@ -39,20 +39,21 @@
     "messageTransformer/messageTransformer.ts"
   ],
   "dependencies": {
-    "@salesforce/bunyan": "^2.0.0",
     "@salesforce/kit": "^3.0.3",
     "@salesforce/schemas": "^1.5.1",
     "@salesforce/ts-types": "^2.0.2",
     "@types/semver": "^7.5.0",
     "ajv": "^8.12.0",
     "change-case": "^4.1.2",
-    "debug": "^3.2.7",
     "faye": "^1.4.0",
     "form-data": "^4.0.0",
     "js2xmlparser": "^4.0.1",
     "jsforce": "^2.0.0-beta.27",
     "jsonwebtoken": "9.0.0",
     "jszip": "3.10.1",
+    "pino": "^8.14.1",
+    "pino-abstract-transport": "^1.0.0",
+    "pino-pretty": "^10.0.0",
     "proper-lockfile": "^4.1.2",
     "ts-retry-promise": "^0.7.0"
   },
@@ -63,7 +64,6 @@
     "@salesforce/ts-sinon": "^1.4.8",
     "@types/benchmark": "^2.1.2",
     "@types/chai-string": "^1.4.2",
-    "@types/debug": "0.0.31",
     "@types/jsonwebtoken": "9.0.2",
     "@types/lodash": "^4.14.195",
     "@types/proper-lockfile": "^4.1.2",
@@ -174,4 +174,4 @@
       ]
     }
   }
-}
+}

src/config/config.ts

@@ -10,7 +10,7 @@ import * as fs from 'fs';
 import { keyBy, parseJsonMap, set } from '@salesforce/kit';
 import { Dictionary, ensure, isString, JsonPrimitive, Nullable } from '@salesforce/ts-types';
 import { Global } from '../global';
-import { Logger } from '../logger';
+import { Logger } from '../logger/logger';
 import { Messages } from '../messages';
 import { validateApiVersion } from '../util/sfdc';
 import { SfdcUrl } from '../util/sfdcUrl';

src/config/configFile.ts

@@ -12,7 +12,7 @@ import { dirname as pathDirname, join as pathJoin } from 'path';
 import { isPlainObject } from '@salesforce/ts-types';
 import { parseJsonMap } from '@salesforce/kit';
 import { Global } from '../global';
-import { Logger } from '../logger';
+import { Logger } from '../logger/logger';
 import { SfError } from '../sfError';
 import { resolveProjectPath, resolveProjectPathSync } from '../util/internal';
 import { BaseConfigStore, ConfigContents } from './configStore';

src/crypto/crypto.ts

@@ -11,7 +11,7 @@ import * as os from 'os';
 import { join as pathJoin } from 'path';
 import { ensure, Nullable, Optional } from '@salesforce/ts-types';
 import { AsyncOptionalCreatable, env } from '@salesforce/kit';
-import { Logger } from '../logger';
+import { Logger } from '../logger/logger';
 import { Messages } from '../messages';
 import { Cache } from '../util/cache';
 import { Global } from '../global';

src/crypto/keyChain.ts

@@ -6,7 +6,7 @@
  */
 
 import { env } from '@salesforce/kit';
-import { Logger } from '../logger';
+import { Logger } from '../logger/logger';
 import { Messages } from '../messages';
 import { KeyChain, keyChainImpl } from './keyChainImpl';
 

src/deviceOauthService.ts

@@ -12,7 +12,7 @@ import { AsyncCreatable, Duration, parseJsonMap } from '@salesforce/kit';
 import { HttpRequest, OAuth2Config } from 'jsforce';
 import { ensureString, JsonMap, Nullable } from '@salesforce/ts-types';
 import * as FormData from 'form-data';
-import { Logger } from './logger';
+import { Logger } from './logger/logger';
 import { AuthInfo, DEFAULT_CONNECTED_APP_INFO, SFDX_HTTP_HEADERS } from './org';
 import { SfError } from './sfError';
 import { Messages } from './messages';

src/exported.ts

@@ -63,7 +63,7 @@ export {
   LoggerOptions,
   LoggerStream,
   Logger,
-} from './logger';
+} from './logger/logger';
 
 export { Messages, StructuredMessage } from './messages';
 

src/lifecycleEvents.ts

@@ -6,12 +6,12 @@
  */
 
 import { AnyJson, Dictionary } from '@salesforce/ts-types';
-import * as Debug from 'debug';
 import { compare } from 'semver';
 // needed for TS to not put everything inside /lib/src
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-ignore
 import * as pjson from '../package.json';
+import { Logger } from './logger/logger';
 
 // Data of any type can be passed to the callback. Can be cast to any type that is given in emit().
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -45,7 +45,7 @@ declare const global: {
 export class Lifecycle {
   public static readonly telemetryEventName = 'telemetry';
   public static readonly warningEventName = 'warning';
-  private debug = Debug(`sfdx:${this.constructor.name}`);
+  private logger?: Logger;
 
   private constructor(private readonly listeners: Dictionary<callback[]> = {}) {}
 
@@ -154,7 +154,10 @@ export class Lifecycle {
   public on<T = AnyJson>(eventName: string, cb: (data: T) => Promise<void>): void {
     const listeners = this.getListeners(eventName);
     if (listeners.length !== 0) {
-      this.debug(
+      if (!this.logger) {
+        this.logger = Logger.childFromRoot('Lifecycle');
+      }
+      this.logger.debug(
         `${
           listeners.length + 1
         } lifecycle events with the name ${eventName} have now been registered. When this event is emitted all ${
@@ -196,7 +199,10 @@ export class Lifecycle {
   public async emit<T = AnyJson>(eventName: string, data: T): Promise<void> {
     const listeners = this.getListeners(eventName);
     if (listeners.length === 0 && eventName !== Lifecycle.warningEventName) {
-      this.debug(
+      if (!this.logger) {
+        this.logger = Logger.childFromRoot('Lifecycle');
+      }
+      this.logger.debug(
         `A lifecycle event with the name ${eventName} does not exist. An event must be registered before it can be emitted.`
       );
     } else {