chore: further feedback

Author: himanshusinghs

src/common/connectionManager.ts

@@ -6,7 +6,7 @@ import { type ConnectionInfo, generateConnectionInfoFromCliArgs } from "@mongosh
 import type { DeviceId } from "../helpers/deviceId.js";
 import type { DriverOptions, UserConfig } from "./config.js";
 import { MongoDBError, ErrorCodes } from "./errors.js";
-import { type CompositeLogger, LogId } from "./logger.js";
+import { type LoggerBase, LogId } from "./logger.js";
 import { packageInfo } from "./packageInfo.js";
 import { type AppNameComponents, setAppNameParamIfMissing } from "../helpers/connectionOptions.js";
 
@@ -123,7 +123,7 @@ export class MCPConnectionManager extends ConnectionManager {
     constructor(
         private userConfig: UserConfig,
         private driverOptions: DriverOptions,
-        private logger: CompositeLogger,
+        private logger: LoggerBase,
         deviceId: DeviceId,
         bus?: EventEmitter
     ) {
@@ -132,7 +132,6 @@ export class MCPConnectionManager extends ConnectionManager {
         this.bus.on("mongodb-oidc-plugin:auth-failed", this.onOidcAuthFailed.bind(this));
         this.bus.on("mongodb-oidc-plugin:auth-succeeded", this.onOidcAuthSucceeded.bind(this));
         this.deviceId = deviceId;
-        this.clientName = "unknown";
     }
 
     async connect(settings: ConnectionSettings): Promise<AnyConnectionState> {

src/lib.ts

@@ -1,16 +1,13 @@
 export { Server, type ServerOptions } from "./server.js";
 export { Session, type SessionOptions } from "./common/session.js";
 export { defaultUserConfig, type UserConfig } from "./common/config.js";
-export { LoggerBase, CompositeLogger, type LogPayload, type LoggerType, type LogLevel } from "./common/logger.js";
+export { LoggerBase, type LogPayload, type LoggerType, type LogLevel } from "./common/logger.js";
 export { StreamableHttpRunner } from "./transports/streamableHttp.js";
 export { type ConnectionManagerFactoryFn } from "./transports/base.js";
 export {
     ConnectionManager,
     type AnyConnectionState,
     type ConnectionState,
-    type ConnectionStateConnected,
-    type ConnectionStateConnecting,
     type ConnectionStateDisconnected,
-    type ConnectionStateErrored,
 } from "./common/connectionManager.js";
 export { Telemetry } from "./telemetry/telemetry.js";

src/transports/base.ts

@@ -11,7 +11,7 @@ import type { ConnectionManager } from "../common/connectionManager.js";
 import { DeviceId } from "../helpers/deviceId.js";
 
 export type ConnectionManagerFactoryFn = (createParams: {
-    logger: CompositeLogger;
+    logger: LoggerBase;
     deviceId: DeviceId;
 }) => Promise<ConnectionManager>;
 

tests/integration/build.test.ts

@@ -43,7 +43,6 @@ describe("Build Test", () => {
         expect(cjsKeys).toEqual(esmKeys);
         expect(cjsKeys).toEqual(
             expect.arrayContaining([
-                "CompositeLogger",
                 "ConnectionManager",
                 "LoggerBase",
                 "Server",