Merge pull request Automattic#15085 from Automattic/vkarpov15/Automatticgh-15041

types: add splice() to DocumentArray to allow adding partial objects with splice()

Author: vkarpov15

test/types/docArray.test.ts

@@ -162,3 +162,20 @@ function gh14469() {
   const jsonNames = doc?.names[0]?.toJSON();
   expectType<string>(jsonNames?.firstName);
 }
+
+function gh15041() {
+  const subDoc = {
+    name: { type: String, required: true },
+    age: { type: Number, required: true }
+  };
+
+  const testSchema = new Schema({
+    subdocArray: { type: [subDoc], required: true }
+  });
+
+  const TestModel = model('Test', testSchema);
+
+  const doc = new TestModel({ subdocArray: [{ name: 'John', age: 30 }] });
+  type TestModelDoc = ReturnType<(typeof TestModel)['hydrate']>
+  expectType<TestModelDoc['subdocArray'][0][]>(doc.subdocArray.splice(0, 1, { name: 'Bill' }));
+}

types/types.d.ts

@@ -60,19 +60,21 @@ declare module 'mongoose' {
 
     class Decimal128 extends mongodb.Decimal128 { }
 
-    class DocumentArray<T> extends Types.Array<T extends Types.Subdocument ? T : Types.Subdocument<InferId<T>, any, T> & T> {
+    class DocumentArray<T, THydratedDocumentType extends Types.Subdocument<any> = Types.Subdocument<InferId<T>, any, T> & T> extends Types.Array<THydratedDocumentType> {
       /** DocumentArray constructor */
       constructor(values: AnyObject[]);
 
       isMongooseDocumentArray: true;
 
       /** Creates a subdocument casted to this schema. */
-      create(obj: any): T extends Types.Subdocument ? T : Types.Subdocument<InferId<T>> & T;
+      create(obj: any): THydratedDocumentType;
 
       /** Searches array items for the first document with a matching _id. */
-      id(id: any): (T extends Types.Subdocument ? T : Types.Subdocument<InferId<T>> & T) | null;
+      id(id: any): THydratedDocumentType | null;
 
       push(...args: (AnyKeys<T> & AnyObject)[]): number;
+
+      splice(start: number, deleteCount?: number, ...args: (AnyKeys<T> & AnyObject)[]): THydratedDocumentType[];
     }
 
     class Map<V> extends global.Map<string, V> {