chore: make elicitation message more specific

Author: himanshusinghs

src/tools/mongodb/delete/dropIndex.ts

@@ -84,9 +84,14 @@ export class DropIndexTool extends MongoDBToolBase {
         };
     }
 
-    protected getConfirmationMessage({ database, collection, indexName }: ToolArgs<typeof this.argsShape>): string {
+    protected getConfirmationMessage({
+        database,
+        collection,
+        indexName,
+        type,
+    }: ToolArgs<typeof this.argsShape>): string {
         return (
-            `You are about to drop the \`${indexName}\` index from the \`${database}.${collection}\` namespace:\n\n` +
+            `You are about to drop the ${type === "search" ? "search index" : "index"} named \`${indexName}\` from the \`${database}.${collection}\` namespace:\n\n` +
             "This operation will permanently remove the index and might affect the performance of queries relying on this index.\n\n" +
             "**Do you confirm the execution of the action?**"
         );

tests/integration/tools/mongodb/delete/dropIndex.test.ts

@@ -275,7 +275,7 @@ describe.each([{ vectorSearchEnabled: false }, { vectorSearchEnabled: true }])(
                         expect(mockElicitInput.mock).toHaveBeenCalledWith({
                             // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
                             message: expect.stringContaining(
-                                "You are about to drop the `year_1` index from the `mflix.movies` namespace"
+                                "You are about to drop the index named `year_1` from the `mflix.movies` namespace"
                             ),
                             requestedSchema: Elicitation.CONFIRMATION_SCHEMA,
                         });
@@ -300,7 +300,7 @@ describe.each([{ vectorSearchEnabled: false }, { vectorSearchEnabled: true }])(
                         expect(mockElicitInput.mock).toHaveBeenCalledWith({
                             // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
                             message: expect.stringContaining(
-                                "You are about to drop the `year_1` index from the `mflix.movies` namespace"
+                                "You are about to drop the index named `year_1` from the `mflix.movies` namespace"
                             ),
                             requestedSchema: Elicitation.CONFIRMATION_SCHEMA,
                         });
@@ -449,7 +449,7 @@ describe.each([{ vectorSearchEnabled: false }, { vectorSearchEnabled: true }])(
                             expect(mockElicitInput.mock).toHaveBeenCalledWith({
                                 // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
                                 message: expect.stringContaining(
-                                    "You are about to drop the `searchIdx` index from the `mflix.movies` namespace"
+                                    "You are about to drop the search index named `searchIdx` from the `mflix.movies` namespace"
                                 ),
                                 requestedSchema: Elicitation.CONFIRMATION_SCHEMA,
                             });
@@ -476,7 +476,7 @@ describe.each([{ vectorSearchEnabled: false }, { vectorSearchEnabled: true }])(
                             expect(mockElicitInput.mock).toHaveBeenCalledWith({
                                 // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
                                 message: expect.stringContaining(
-                                    "You are about to drop the `searchIdx` index from the `mflix.movies` namespace"
+                                    "You are about to drop the search index named `searchIdx` from the `mflix.movies` namespace"
                                 ),
                                 requestedSchema: Elicitation.CONFIRMATION_SCHEMA,
                             });