Collection: fix updateMany success return type UpdateResult<I> to allow custom defined _id types

Author: devmatteini

.changeset/forty-lemons-fix.md

@@ -0,0 +1,5 @@
+---
+"effect-mongodb": patch
+---
+
+Fix `Collection.updateMany` success return type `UpdateResult<I>` to allow custom defined `_id` types

packages/effect-mongodb/dtslint/Collection.ts

@@ -81,10 +81,10 @@ F.pipe(collection, Collection.deleteMany({ birthday: "2024-11-28" }))
 // updateMany
 // -------------------------------------------------------------------------------------
 
-// $ExpectType Effect<UpdateResult<Document>, MongoError, never>
+// $ExpectType Effect<UpdateResult<{ readonly birthday: string; }>, MongoError, never>
 Collection.updateMany(collection, { birthday: "2024-11-28" }, { $set: { birthday: "2024-11-29" } })
 
-// $ExpectType Effect<UpdateResult<Document>, MongoError, never>
+// $ExpectType Effect<UpdateResult<{ readonly birthday: string; }>, MongoError, never>
 F.pipe(collection, Collection.updateMany({ birthday: "2024-11-28" }, { $set: { birthday: "2024-11-29" } }))
 
 // -------------------------------------------------------------------------------------

packages/effect-mongodb/src/Collection.ts

@@ -187,23 +187,23 @@ export const updateMany: {
     filter: Filter<I>,
     update: UpdateFilter<I> | ReadonlyArray<Document>,
     options?: UpdateOptions
-  ): <A extends Document, R>(
-    collection: Collection<A, I, R>
-  ) => Effect.Effect<UpdateResult, MongoError.MongoError, R>
+  ): <A extends Document, I2 extends I, R>(
+    collection: Collection<A, I2, R>
+  ) => Effect.Effect<UpdateResult<I2>, MongoError.MongoError, R>
   <A extends Document, I extends Document, R>(
     collection: Collection<A, I, R>,
     filter: Filter<I>,
     update: UpdateFilter<I> | ReadonlyArray<Document>,
     options?: UpdateOptions
-  ): Effect.Effect<UpdateResult, MongoError.MongoError, R>
+  ): Effect.Effect<UpdateResult<I>, MongoError.MongoError, R>
 } = F.dual(
   (args) => isCollection(args[0]),
   <A extends Document, I extends Document, R>(
     collection: Collection<A, I, R>,
     filter: Filter<I>,
     update: UpdateFilter<I> | ReadonlyArray<Document>,
     options?: UpdateOptions
-  ): Effect.Effect<UpdateResult, MongoError.MongoError, R> =>
+  ): Effect.Effect<UpdateResult<I>, MongoError.MongoError, R> =>
     Effect.promise(() =>
       collection.collection.updateMany(
         filter,