fix(models): call setUpdatedAt for all docs in insertMany

ggazzo · claude · ggazzo · commit 242fd594d01e · 2026-03-18T15:07:52.000-03:00
insertMany() only called setUpdatedAt() when a document already had a
valid _id. Documents without _id (the common case) got an _id generated
but skipped setUpdatedAt(), leaving them without _updatedAt in the
database. This caused API response validation failures for endpoints
using $ref schemas that require _updatedAt (e.g. subscriptions.getOne).

Align insertMany() with insertOne(), which already calls setUpdatedAt()
unconditionally.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/packages/models/src/models/BaseRaw.ts b/packages/models/src/models/BaseRaw.ts
@@ -274,7 +274,7 @@ export abstract class BaseRaw<
 		docs = docs.map((doc) => {
 			if (!doc._id || typeof doc._id !== 'string') {
 				const oid = new ObjectId();
-				return { _id: oid.toHexString(), ...doc };
+				doc = { _id: oid.toHexString(), ...doc };
 			}
 			this.setUpdatedAt(doc);
 			return doc;

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,7 @@ export abstract class BaseRaw<`
`274`	`274`	`docs = docs.map((doc) => {`
`275`	`275`	`if (!doc._id \|\| typeof doc._id !== 'string') {`
`276`	`276`	`const oid = new ObjectId();`
`277`		`- return { _id: oid.toHexString(), ...doc };`
	`277`	`+ doc = { _id: oid.toHexString(), ...doc };`
`278`	`278`	`}`
`279`	`279`	`this.setUpdatedAt(doc);`
`280`	`280`	`return doc;`