chore(devtools-connect): align MongoLogWriter interfaces between packages (#570)

Because the OIDC plugin's `MongoLogWriter` class has an optional `debug()`
method and the devtools-connect one didn't, we accidentally missed
adding this debug method in mongodb-js/mongosh@b340d790ed9c159.
Aligning these interfaces should prevent this from happening again
in the future.

Author: addaleax

packages/devtools-connect/src/index.ts

@@ -7,5 +7,5 @@ export type {
   AgentWithInitialize,
   ConnectMongoClientResult,
 } from './connect';
-export { hookLogger } from './log-hook';
+export { hookLogger, MongoLogWriter } from './log-hook';
 export { oidcServerRequestHandler } from './oidc/handler';

packages/devtools-connect/src/log-hook.ts

@@ -11,15 +11,18 @@ import type {
   ConnectRetryAfterTLSErrorEvent,
 } from './types';
 
-import { hookLoggerToMongoLogWriter as oidcHookLogger } from '@mongodb-js/oidc-plugin';
-import { hookLogger as proxyHookLogger } from '@mongodb-js/devtools-proxy-support';
+import {
+  hookLoggerToMongoLogWriter as oidcHookLogger,
+  MongoLogWriter as OIDCMongoLogWriter,
+} from '@mongodb-js/oidc-plugin';
+import {
+  hookLogger as proxyHookLogger,
+  MongoLogWriter as ProxyMongoLogWriter,
+} from '@mongodb-js/devtools-proxy-support';
 
-interface MongoLogWriter {
-  info(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
-  warn(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
-  error(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
-  mongoLogId(this: void, id: number): unknown;
-}
+export interface MongoLogWriter
+  extends OIDCMongoLogWriter,
+    ProxyMongoLogWriter {}
 
 export function hookLogger(
   emitter: ConnectLogEmitter,

packages/devtools-proxy-support/src/index.ts

@@ -13,5 +13,10 @@ export {
   RequestInfo,
   RequestInit,
 } from './fetch';
-export { ProxyEventMap, ProxyLogEmitter, hookLogger } from './logging';
+export {
+  ProxyEventMap,
+  ProxyLogEmitter,
+  hookLogger,
+  MongoLogWriter,
+} from './logging';
 export { systemCA, resetSystemCACache } from './system-ca';

packages/devtools-proxy-support/src/logging.ts

@@ -77,10 +77,10 @@ export interface ProxyLogEmitter {
   ): unknown;
 }
 
-interface MongoLogWriter {
-  info(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
-  warn(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
-  error(c: string, id: unknown, ctx: string, msg: string, attr?: any): void;
+export interface MongoLogWriter {
+  info(c: string, id: unknown, ctx: string, msg: string, attr?: unknown): void;
+  warn(c: string, id: unknown, ctx: string, msg: string, attr?: unknown): void;
+  error(c: string, id: unknown, ctx: string, msg: string, attr?: unknown): void;
   mongoLogId(this: void, id: number): unknown;
 }