Collection/DocumentCollection.dropIndex return type is the same as mongodb driver

Author: devmatteini

.changeset/red-donkeys-turn.md

@@ -0,0 +1,5 @@
+---
+"effect-mongodb": patch
+---
+
+`Collection/DocumentCollection.dropIndex` return type is the same as mongodb driver

packages/effect-mongodb/dtslint/Collection.ts

@@ -169,10 +169,10 @@ F.pipe(collection, Collection.createIndex({ birthday: 1 }))
 // dropIndex
 // -------------------------------------------------------------------------------------
 
-// $ExpectType Effect<void, MongoError, never>
+// $ExpectType Effect<Document, MongoError, never>
 Collection.dropIndex(collection, "birthday_1")
 
-// $ExpectType Effect<void, MongoError, never>
+// $ExpectType Effect<Document, MongoError, never>
 F.pipe(collection, Collection.dropIndex("birthday_1"))
 
 // -------------------------------------------------------------------------------------

packages/effect-mongodb/dtslint/DocumentCollection.ts

@@ -178,10 +178,10 @@ F.pipe(collection, DocumentCollection.createIndex({ birthday: 1 }))
 // dropIndex
 // -------------------------------------------------------------------------------------
 
-// $ExpectType Effect<void, MongoError, never>
+// $ExpectType Effect<Document, MongoError, never>
 DocumentCollection.dropIndex(collection, "birthday_1")
 
-// $ExpectType Effect<void, MongoError, never>
+// $ExpectType Effect<Document, MongoError, never>
 F.pipe(collection, DocumentCollection.dropIndex("birthday_1"))
 
 // -------------------------------------------------------------------------------------

packages/effect-mongodb/src/Collection.ts

@@ -412,26 +412,24 @@ export const createIndex: {
     )
 )
 
-// TODO: review return type. Should we return Document like mongodb driver?
 export const dropIndex: {
   (indexName: string, options?: DropIndexesOptions): <A extends Document, I extends Document, R>(
     collection: Collection<A, I, R>
-  ) => Effect.Effect<void, MongoError.MongoError, R>
+  ) => Effect.Effect<Document, MongoError.MongoError, R>
   <A extends Document, I extends Document, R>(
     collection: Collection<A, I, R>,
     indexName: string,
     options?: DropIndexesOptions
-  ): Effect.Effect<void, MongoError.MongoError, R>
+  ): Effect.Effect<Document, MongoError.MongoError, R>
 } = F.dual(
   (args) => isCollection(args[0]),
   <A extends Document, I extends Document, R>(
     collection: Collection<A, I, R>,
     indexName: string,
     options?: DropIndexesOptions
-  ): Effect.Effect<void, MongoError.MongoError, R> =>
+  ): Effect.Effect<Document, MongoError.MongoError, R> =>
     F.pipe(
       Effect.promise(() => collection.collection.dropIndex(indexName, options)),
-      Effect.asVoid,
       Effect.catchAllDefect(mongoErrorOrDie(makeSource(collection, "dropIndex")))
     )
 )

packages/effect-mongodb/src/DocumentCollection.ts

@@ -391,28 +391,26 @@ export const createIndex: {
     )
 )
 
-// TODO: review return type. Should we return Document like mongodb driver?
 export const dropIndex: {
   (
     indexName: string,
     options?: DropIndexesOptions
-  ): (collection: DocumentCollection) => Effect.Effect<void, MongoError.MongoError>
+  ): (collection: DocumentCollection) => Effect.Effect<Document, MongoError.MongoError>
   (
     collection: DocumentCollection,
     indexName: string,
     options?: DropIndexesOptions
-  ): Effect.Effect<void, MongoError.MongoError>
+  ): Effect.Effect<Document, MongoError.MongoError>
 } = F.dual(
   (args) => isDocumentCollection(args[0]),
   (
     collection: DocumentCollection,
     indexName: string,
     options?: DropIndexesOptions
-  ): Effect.Effect<void, MongoError.MongoError> =>
+  ): Effect.Effect<Document, MongoError.MongoError> =>
     F.pipe(
       Effect.promise(() => collection.collection.dropIndex(indexName, options)),
-      Effect.asVoid,
-      Effect.catchAllDefect(MongoError.mongoErrorDie<void>("dropIndex error"))
+      Effect.catchAllDefect(MongoError.mongoErrorDie<Document>("dropIndex error"))
     )
 )