address comments

Author: blva

src/tools/mongodb/mongodbTool.ts

@@ -75,11 +75,11 @@ export abstract class MongoDBToolBase extends ToolBase {
         return this.session.connectToMongoDB(connectionString, this.config.connectOptions);
     }
 
-    protected resolveToolMetadata(
+    protected resolveTelemetryMetadata(
         args: ToolArgs<typeof this.argsShape>,
         extra: RequestHandlerExtra<ServerRequest, ServerNotification>
     ): ToolMetadata {
-        const metadata = super.resolveToolMetadata(args, extra);
+        const metadata = super.resolveTelemetryMetadata(args, extra);
 
         // Add projectId to the metadata if running a MongoDB operation to an Atlas cluster
         if (this.session.connectedAtlasCluster?.projectId) {

src/tools/tool.ts

@@ -139,10 +139,10 @@ export abstract class ToolBase {
      * @param args - The arguments passed to the tool
      * @returns The tool metadata
      */
-    protected resolveToolMetadata(...args: Parameters<ToolCallback<typeof this.argsShape>>): ToolMetadata {
+    protected resolveTelemetryMetadata(...args: Parameters<ToolCallback<typeof this.argsShape>>): ToolMetadata {
         const toolMetadata: ToolMetadata = {};
         try {
-            if (!args[0] || typeof args[0] !== "object") {
+            if (!args.length) {
                 return toolMetadata;
             }
 
@@ -192,7 +192,7 @@ export abstract class ToolBase {
             return;
         }
         const duration = Date.now() - startTime;
-        const metadata = this.resolveToolMetadata(...args);
+        const metadata = this.resolveTelemetryMetadata(...args);
         const event: ToolEvent = {
             timestamp: new Date().toISOString(),
             source: "mdbmcp",