remove 'IndexInformationOperation'

Author: baileympearson

src/collection.ts

@@ -55,7 +55,6 @@ import {
   type DropIndexesOptions,
   DropIndexOperation,
   type IndexDescription,
-  IndexInformationOperation,
   type IndexSpecification,
   type ListIndexesOptions
 } from './operations/indexes';
@@ -684,7 +683,7 @@ export class Collection<TSchema extends Document = Document> {
     indexes: string | string[],
     options?: IndexInformationOptions
   ): Promise<boolean> {
-    const indexNames: Set<string> = new Set([indexes].flat());
+    const indexNames: string[] = [indexes].flat();
     const allIndexes: string[] = await this.listIndexes(options)
       .map(({ name }) => name)
       .toArray();
@@ -698,10 +697,12 @@ export class Collection<TSchema extends Document = Document> {
    * @param options - Optional settings for the command
    */
   async indexInformation(options?: IndexInformationOptions): Promise<Document> {
-    return executeOperation(
-      this.client,
-      new IndexInformationOperation(this.s.db, this.collectionName, resolveOptions(this, options))
-    );
+    const resolvedOptions: IndexInformationOptions = {
+      readPreference: options?.readPreference,
+      session: options?.session,
+      full: false
+    };
+    return this.indexes(resolvedOptions);
   }
 
   /**
@@ -806,7 +807,19 @@ export class Collection<TSchema extends Document = Document> {
    * @param options - Optional settings for the command
    */
   async indexes(options?: IndexInformationOptions): Promise<Document[]> {
-    return this.listIndexes(options).toArray();
+    const indexes = await this.listIndexes(options).toArray();
+    const full = options?.full === true;
+    if (full) {
+      return indexes;
+    }
+
+    const info: Record<string, Array<[string, unknown]>> = {};
+    for (const { name, key } of indexes) {
+      info[name] = Object.entries(key);
+    }
+
+    // @ts-expect-error The return type is broken
+    return info;
   }
 
   /**

src/db.ts

@@ -23,11 +23,7 @@ import {
   type DropDatabaseOptions
 } from './operations/drop';
 import { executeOperation } from './operations/execute_operation';
-import {
-  type CreateIndexesOptions,
-  IndexInformationOperation,
-  type IndexSpecification
-} from './operations/indexes';
+import { type CreateIndexesOptions, type IndexSpecification } from './operations/indexes';
 import type { CollectionInfo, ListCollectionsOptions } from './operations/list_collections';
 import { ProfilingLevelOperation, type ProfilingLevelOptions } from './operations/profiling_level';
 import { RemoveUserOperation, type RemoveUserOptions } from './operations/remove_user';
@@ -476,10 +472,7 @@ export class Db {
    * @param options - Optional settings for the command
    */
   async indexInformation(name: string, options?: IndexInformationOptions): Promise<Document> {
-    return executeOperation(
-      this.client,
-      new IndexInformationOperation(this, name, resolveOptions(this, options))
-    );
+    return this.collection(name).indexInformation(resolveOptions(this, options));
   }
 
   /**

src/operations/indexes.ts

@@ -1,9 +1,7 @@
 import type { Document } from '../bson';
 import type { Collection } from '../collection';
-import type { Db } from '../db';
-import { MongoCompatibilityError, MONGODB_ERROR_CODES, MongoError } from '../error';
+import { MongoCompatibilityError } from '../error';
 import { type OneOrMore } from '../mongo_types';
-import { ReadPreference } from '../read_preference';
 import type { Server } from '../sdam/server';
 import type { ClientSession } from '../sessions';
 import { isObject, maxWireVersion, type MongoDBNamespace } from '../utils';
@@ -13,8 +11,7 @@ import {
   type CommandOperationOptions,
   type OperationParent
 } from './command';
-import { indexInformation, type IndexInformationOptions } from './common_functions';
-import { AbstractOperation, Aspect, defineAspects } from './operation';
+import { Aspect, defineAspects } from './operation';
 
 const VALID_INDEX_OPTIONS = new Set([
   'background',
@@ -337,35 +334,6 @@ export class ListIndexesOperation extends CommandOperation<Document> {
   }
 }
 
-/** @internal */
-export class IndexInformationOperation extends AbstractOperation<Document> {
-  override options: IndexInformationOptions;
-  db: Db;
-  name: string;
-
-  constructor(db: Db, name: string, options?: IndexInformationOptions) {
-    super(options);
-    this.options = options ?? {};
-    this.db = db;
-    this.name = name;
-  }
-
-  override get commandName() {
-    return 'listIndexes' as const;
-  }
-
-  override async execute(server: Server, session: ClientSession | undefined): Promise<Document> {
-    const db = this.db;
-    const name = this.name;
-
-    return indexInformation(db, name, {
-      ...this.options,
-      readPreference: this.readPreference,
-      session
-    });
-  }
-}
-
 defineAspects(ListIndexesOperation, [
   Aspect.READ_OPERATION,
   Aspect.RETRYABLE,

test/integration/crud/abstract_operation.test.ts

@@ -20,8 +20,7 @@ describe('abstract operation', async function () {
       'IsCappedOperation',
       'BulkWriteOperation',
       'IndexOperation',
-      'CollectionsOperation',
-      'IndexInformationOperation'
+      'CollectionsOperation'
     ];
 
     const sameServerOnlyOperationSubclasses = ['GetMoreOperation', 'KillCursorsOperation'];
@@ -156,11 +155,6 @@ describe('abstract operation', async function () {
         subclassType: mongodb.ListIndexesOperation,
         correctCommandName: 'listIndexes'
       },
-      {
-        subclassCreator: () => new mongodb.IndexInformationOperation(db, 'a', {}),
-        subclassType: mongodb.IndexInformationOperation,
-        correctCommandName: 'listIndexes'
-      },
       {
         subclassCreator: () =>
           new mongodb.InsertOperation(collection.fullNamespace, [{ a: 1 }], {}),