fix: error codes

Author: fmenezes

src/errors.ts

@@ -1,8 +1,6 @@
 export enum ErrorCodes {
     NotConnectedToMongoDB = 1_000_000,
     InvalidParams = 1_000_001,
-    CloseServiceProvider = 1_000_007,
-    DeleteDatabaseUser = 1_000_008,
     MisconfiguredConnectionString = 1_000_001
 }
 

src/logger.ts

@@ -11,6 +11,7 @@ export const LogId = {
     serverInitialized: mongoLogId(1_000_002),
 
     atlasCheckCredentials: mongoLogId(1_001_001),
+    atlasDeleteDatabaseUserFailure: mongoLogId(1_001_002),
 
     telemetryDisabled: mongoLogId(1_002_001),
     telemetryEmitFailure: mongoLogId(1_002_002),
@@ -22,6 +23,7 @@ export const LogId = {
     toolDisabled: mongoLogId(1_003_003),
 
     mongodbConnectFailure: mongoLogId(1_004_001),
+    mongodbDisconnectFailure: mongoLogId(1_004_002),
 } as const;
 
 abstract class LoggerBase {

src/session.ts

@@ -1,9 +1,7 @@
 import { NodeDriverServiceProvider } from "@mongosh/service-provider-node-driver";
 import { ApiClient, ApiClientCredentials } from "./common/atlas/apiClient.js";
 import { Implementation } from "@modelcontextprotocol/sdk/types.js";
-import logger from "./logger.js";
-import { mongoLogId } from "mongodb-log-writer";
-import { ErrorCodes } from "./errors.js";
+import logger, { LogId } from "./logger.js";
 import EventEmitter from "events";
 
 export interface SessionOptions {
@@ -62,11 +60,7 @@ export class Session extends EventEmitter<{
                 await this.serviceProvider.close(true);
             } catch (err: unknown) {
                 const error = err instanceof Error ? err : new Error(String(err));
-                logger.error(
-                    mongoLogId(ErrorCodes.CloseServiceProvider),
-                    "Error closing service provider:",
-                    error.message
-                );
+                logger.error(LogId.mongodbDisconnectFailure, "Error closing service provider:", error.message);
             }
             this.serviceProvider = undefined;
         }
@@ -87,7 +81,7 @@ export class Session extends EventEmitter<{
             const error = err instanceof Error ? err : new Error(String(err));
 
             logger.error(
-                mongoLogId(ErrorCodes.DeleteDatabaseUser),
+                LogId.atlasDeleteDatabaseUserFailure,
                 "atlas-connect-cluster",
                 `Error deleting previous database user: ${error.message}`
             );