ts-patch-mongoose 3.1.0 → 3.1.2

package/README.md

@@ -15,16 +15,16 @@ Patch history (audit log) & events plugin for mongoose
 
 ## Motivation
 
-ts-patch-mongoose is a plugin for mongoose
+ts-patch-mongoose is a plugin for mongoose.
 \
-I need to track changes of mongoose models and save them as patch history (audit log) in separate collection. Changes must also emit events that I can subscribe to and react in other parts of my application. I also want to omit some fields from patch history.
+I need to track changes of mongoose models and save them as patch history (audit log) in a separate collection. Changes must also emit events that I can subscribe to and react in other parts of my application. I also want to omit some fields from patch history.
 
 ## Supports and tested with
 
 ```json
 {
   "node": "20.x || 22.x || 24.x",
-  "mongoose": ">=6.6.x || 7.x || 8.x || 9.x",
+  "mongoose": ">=6.6.0 || 7.x || 8.x || 9.x",
 }
 ```
 
@@ -53,9 +53,9 @@ bun add ts-patch-mongoose mongoose
 
 Works with any Node.js framework — Express, Fastify, Koa, Hono, Nest, etc.
 \
-How to use it with express [ts-express-tsx](https://github.com/ilovepixelart/ts-express-tsx)
+How to use it with Express: [ts-express-tsx](https://github.com/ilovepixelart/ts-express-tsx)
 
-Create your event constants `events.ts`
+Create your event constants in `events.ts`
 
 ```typescript
 export const BOOK_CREATED = 'book-created'
@@ -77,33 +77,34 @@ export type Book = {
 }
 ```
 
-Setup your mongoose model `Book.ts`
+Set up your mongoose model in `Book.ts`
 
 ```typescript
 import { Schema, model } from 'mongoose'
 
-import type { HydratedDocument, Types } from 'mongoose'
+import type { HydratedDocument } from 'mongoose'
 import type { Book } from '../types'
 
 import { patchHistoryPlugin, setPatchHistoryTTL } from 'ts-patch-mongoose'
 import { BOOK_CREATED, BOOK_UPDATED, BOOK_DELETED } from '../constants/events'
 
-// You can set patch history TTL in plain english or in milliseconds as you wish.
+// You can set patch history TTL in plain English or in milliseconds as you wish.
 // This will determine how long you want to keep patch history.
 // You don't need to use this global config in case you want to keep patch history forever.
-// Execute this method after you connected to you database somewhere in your application.
-setPatchHistoryTTL('1 month')
+// Execute this method after you connected to your database somewhere in your application.
+// Optional second argument for custom error handling
+setPatchHistoryTTL('1 month', (error) => console.error('TTL setup failed:', error))
 
 const BookSchema = new Schema<Book>({
-  name: {
-    title: String,
+  title: {
+    type: String,
     required: true
   },
   description: {
     type: String,
   },
   authorId: {
-    type: Types.ObjectId,
+    type: Schema.Types.ObjectId,
     required: true
   }
 }, { timestamps: true })
@@ -117,25 +118,25 @@ BookSchema.plugin(patchHistoryPlugin, {
   // You can omit some properties in case you don't want to save them to patch history
   omit: ['__v', 'createdAt', 'updatedAt'],
 
-  // Addition options for patchHistoryPlugin plugin
-  // Everything bellow is optional and just shows you what you can do:
+  // Additional options for patchHistoryPlugin
+  // Everything below is optional and just shows you what you can do:
 
-  // Code bellow is abstract example, you can use any other way to get user, reason, metadata
-  // These three properties will be added to patch history document automatically and give you flexibility to track who, why and when made changes to your documents
+  // Code below is an abstract example, you can use any other way to get user, reason, metadata
+  // These three properties will be added to patch history document automatically and gives you flexibility to track who, why and when made changes to your documents
   getUser: async (doc: HydratedDocument<Book>) => {
     // For example: get user from http context
     // You should return an object, in case you want to save user to patch history
     return httpContext.get('user') as Record<string, unknown>
   },
 
-  // Reason of document (create/update/delete) like: 'Excel upload', 'Manual update', 'API call', etc.
+  // Reason for the document change (create/update/delete) like: 'Excel upload', 'Manual update', 'API call', etc.
   getReason: async (doc: HydratedDocument<Book>) => {
     // For example: get reason from http context, or any other place of your application
-    // You shout return a string, in case you want to save reason to patch history
+    // You should return a string, in case you want to save reason to patch history
     return httpContext.get('reason') as string
   },
 
-  // You can provide any information you want to save in along with patch history
+  // You can provide any information you want to save along with patch history
   getMetadata: async (doc: HydratedDocument<Book>) => {
     // For example: get metadata from http context, or any other place of your application
     // You should return an object, in case you want to save metadata to patch history
@@ -143,15 +144,20 @@ BookSchema.plugin(patchHistoryPlugin, {
   },
 
   // Do something before deleting documents
-  // This method will be executed before deleting document or documents and always returns a nonempty array of documents
+  // This method will be executed before deleting document or documents and always returns a non-empty array of documents
   preDelete: async (docs) => {
     const bookIds = docs.map((doc) => doc._id)
     await SomeOtherModel.deleteMany({ bookId: { $in: bookIds } })
   },
 
-  // In case you just want to track changes in your models using events below.
-  // And don't want to save changes to patch history collection
-  patchHistoryDisabled: true, 
+  // Custom error handler for history write failures (defaults to console.error)
+  onError: (error) => {
+    console.error('Patch history error:', error)
+  },
+
+  // In case you just want to track changes in your models using events
+  // and don't want to save changes to patch history collection
+  // patchHistoryDisabled: true,
 })
 
 const Book = model('Book', BookSchema)
@@ -188,7 +194,7 @@ patchEventEmitter.on(BOOK_UPDATED, ({ doc, oldDoc, patch }) => {
 patchEventEmitter.on(BOOK_DELETED, ({ oldDoc }) => {
   try {
     console.log('Event - book deleted', oldDoc)
-    // Do something with doc here
+    // Do something with oldDoc here
   } catch (error) {
     console.error(error)
   }

package/dist/index.cjs

@@ -100,9 +100,10 @@ const ms = (val) => {
   if (str.length > 100) return Number.NaN;
   const match = RE.exec(str);
   if (!match) return Number.NaN;
-  const n = Number.parseFloat(match[1] ?? "");
-  const type = (match[2] ?? "ms").toLowerCase();
-  return n * (UNITS[type] ?? 0);
+  const [, numStr, unitStr] = match;
+  const n = Number.parseFloat(String(numStr));
+  const type = (unitStr ?? "ms").toLowerCase();
+  return n * UNITS[type];
 };
 
 const isArray = Array.isArray;
@@ -385,7 +386,7 @@ const updatePatch = async (opts, context, current, original) => {
   if (history) {
     let version = 0;
     const lastHistory = await HistoryModel.findOne({ collectionId: original._id }).sort("-version").exec();
-    if (lastHistory && lastHistory.version >= 0) {
+    if (lastHistory) {
       version = lastHistory.version + 1;
     }
     const [user, reason, metadata] = await getData(opts, current);
@@ -472,7 +473,7 @@ const updateMethods = ["update", "updateOne", "replaceOne", "updateMany", "findO
 const trackChangedFields = (fields, updated, changed) => {
   if (!fields) return;
   for (const key of Object.keys(fields)) {
-    const root = key.split(".")[0];
+    const [root = key] = key.split(".");
     changed.set(root, updated[root]);
   }
 };
@@ -490,7 +491,7 @@ const assignUpdate = (document, update, commands) => {
   let updated = powerAssign.assign(document.toObject(toObjectOptions), update);
   const changedByCommand = /* @__PURE__ */ new Map();
   for (const command of commands) {
-    const op = Object.keys(command)[0];
+    const [op = ""] = Object.keys(command);
     const fields = command[op];
     try {
       updated = powerAssign.assign(updated, command);
@@ -558,7 +559,8 @@ const updateHooksInitialize = (schema, opts) => {
     let current = null;
     for (const query of candidates) {
       if (current || isEmpty(query)) continue;
-      current = await model.findOne(query).sort({ _id: -1 }).lean().exec();
+      const found = await model.findOne(query).sort({ _id: -1 }).lean().exec();
+      current = found;
     }
     if (current) {
       this._context.createdDocs = [current];
@@ -590,13 +592,14 @@ const patchHistoryPlugin = (schema, opts) => {
     await createPatch(opts, context);
   });
   if (isMongooseLessThan8) {
-    schema.pre(remove, { document: true, query: false }, async function() {
+    const legacySchema = schema;
+    legacySchema.pre(remove, { document: true, query: false }, async function() {
       const original = this.toObject(toObjectOptions);
       if (opts.preDelete && !isEmpty(original)) {
         await opts.preDelete([original]);
       }
     });
-    schema.post(remove, { document: true, query: false }, async function() {
+    legacySchema.post(remove, { document: true, query: false }, async function() {
       const original = this.toObject(toObjectOptions);
       const model = this.constructor;
       const context = {

package/dist/index.d.cts

@@ -13,6 +13,8 @@ interface History {
     reason?: string;
     metadata?: object;
     patch?: Operation[];
+    createdAt?: Date;
+    updatedAt?: Date;
 }
 interface PatchEvent<T> {
     oldDoc?: HydratedDocument<T>;

package/dist/index.d.cts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.cts","sources":["../src/types.ts","../src/em.ts","../src/ms.ts","../src/helpers.ts","../src/index.ts"],"mappings":";;;;AAGM,UAAW,OAAO;;;;kBAIR,KAAK,CAAC,QAAQ;;;;;;YAMpB,SAAS;;AAGb,UAAW,UAAU;aAChB,gBAAgB;UACnB,gBAAgB;YACd,SAAS;;AAGb,UAAW,YAAY;;;;;kBAKb,gBAAgB;kBAChB,gBAAgB;;;;AAK1B,KAAM,WAAW,MAAM,KAAK;;cAAiC,YAAY;;AAEzE,KAAM,IAAI,GAAG,MAAM;AAEnB,KAAM,QAAQ,GAAG,MAAM;AAEvB,UAAW,aAAa;;;;;;oBAMZ,gBAAgB,QAAQ,OAAO,CAAC,IAAI,IAAI,IAAI;sBAC1C,gBAAgB,QAAQ,OAAO;wBAC7B,gBAAgB,QAAQ,OAAO,CAAC,QAAQ,IAAI,QAAQ;;;uBAGrD,gBAAgB,UAAU,OAAO;sBAClC,KAAK;;;ACjDzB,cAAM,iBAAkB,SAAQ,YAAY;;AAC5C,cAAM,EAAE,mBAA0B;;ACKlC,cAAa,KAAK;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAqCZ,KAAM,IAAI,gBAAgB,KAAK;AAE/B,KAAM,QAAQ,sCAAsC,IAAI,kBAAkB,IAAI;;AC6FpF,cAAa,kBAAkB,QAAe,QAAQ,oBAAoB,KAAK,cAAY,OAAO;;AC1HlG,cAAa,kBAAkB,cAAe,MAAM,WAAW,aAAa","names":[]}
+{"version":3,"file":"index.d.cts","sources":["../src/types.ts","../src/em.ts","../src/ms.ts","../src/helpers.ts","../src/index.ts"],"mappings":";;;;AAGM,UAAW,OAAO;;;;kBAIR,KAAK,CAAC,QAAQ;;;;;;YAMpB,SAAS;gBACL,IAAI;gBACJ,IAAI;;AAGZ,UAAW,UAAU;aAChB,gBAAgB;UACnB,gBAAgB;YACd,SAAS;;AAGb,UAAW,YAAY;;;;;kBAKb,gBAAgB;kBAChB,gBAAgB;;;;AAK1B,KAAM,WAAW,MAAM,KAAK;;cAAiC,YAAY;;AAEzE,KAAM,IAAI,GAAG,MAAM;AAEnB,KAAM,QAAQ,GAAG,MAAM;AAEvB,UAAW,aAAa;;;;;;oBAMZ,gBAAgB,QAAQ,OAAO,CAAC,IAAI,IAAI,IAAI;sBAC1C,gBAAgB,QAAQ,OAAO;wBAC7B,gBAAgB,QAAQ,OAAO,CAAC,QAAQ,IAAI,QAAQ;;;uBAGrD,gBAAgB,UAAU,OAAO;sBAClC,KAAK;;;ACnDzB,cAAM,iBAAkB,SAAQ,YAAY;;AAC5C,cAAM,EAAE,mBAA0B;;ACKlC,cAAa,KAAK;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAqCZ,KAAM,IAAI,gBAAgB,KAAK;AAE/B,KAAM,QAAQ,sCAAsC,IAAI,kBAAkB,IAAI;;AC6FpF,cAAa,kBAAkB,QAAe,QAAQ,oBAAoB,KAAK,cAAY,OAAO;;AC1HlG,cAAa,kBAAkB,cAAe,MAAM,WAAW,aAAa","names":[]}

package/dist/index.d.mts

@@ -13,6 +13,8 @@ interface History {
     reason?: string;
     metadata?: object;
     patch?: Operation[];
+    createdAt?: Date;
+    updatedAt?: Date;
 }
 interface PatchEvent<T> {
     oldDoc?: HydratedDocument<T>;

package/dist/index.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.mts","sources":["../src/types.ts","../src/em.ts","../src/ms.ts","../src/helpers.ts","../src/index.ts"],"mappings":";;;;AAGM,UAAW,OAAO;;;;kBAIR,KAAK,CAAC,QAAQ;;;;;;YAMpB,SAAS;;AAGb,UAAW,UAAU;aAChB,gBAAgB;UACnB,gBAAgB;YACd,SAAS;;AAGb,UAAW,YAAY;;;;;kBAKb,gBAAgB;kBAChB,gBAAgB;;;;AAK1B,KAAM,WAAW,MAAM,KAAK;;cAAiC,YAAY;;AAEzE,KAAM,IAAI,GAAG,MAAM;AAEnB,KAAM,QAAQ,GAAG,MAAM;AAEvB,UAAW,aAAa;;;;;;oBAMZ,gBAAgB,QAAQ,OAAO,CAAC,IAAI,IAAI,IAAI;sBAC1C,gBAAgB,QAAQ,OAAO;wBAC7B,gBAAgB,QAAQ,OAAO,CAAC,QAAQ,IAAI,QAAQ;;;uBAGrD,gBAAgB,UAAU,OAAO;sBAClC,KAAK;;;ACjDzB,cAAM,iBAAkB,SAAQ,YAAY;;AAC5C,cAAM,EAAE,mBAA0B;;ACKlC,cAAa,KAAK;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAqCZ,KAAM,IAAI,gBAAgB,KAAK;AAE/B,KAAM,QAAQ,sCAAsC,IAAI,kBAAkB,IAAI;;AC6FpF,cAAa,kBAAkB,QAAe,QAAQ,oBAAoB,KAAK,cAAY,OAAO;;AC1HlG,cAAa,kBAAkB,cAAe,MAAM,WAAW,aAAa","names":[]}
+{"version":3,"file":"index.d.mts","sources":["../src/types.ts","../src/em.ts","../src/ms.ts","../src/helpers.ts","../src/index.ts"],"mappings":";;;;AAGM,UAAW,OAAO;;;;kBAIR,KAAK,CAAC,QAAQ;;;;;;YAMpB,SAAS;gBACL,IAAI;gBACJ,IAAI;;AAGZ,UAAW,UAAU;aAChB,gBAAgB;UACnB,gBAAgB;YACd,SAAS;;AAGb,UAAW,YAAY;;;;;kBAKb,gBAAgB;kBAChB,gBAAgB;;;;AAK1B,KAAM,WAAW,MAAM,KAAK;;cAAiC,YAAY;;AAEzE,KAAM,IAAI,GAAG,MAAM;AAEnB,KAAM,QAAQ,GAAG,MAAM;AAEvB,UAAW,aAAa;;;;;;oBAMZ,gBAAgB,QAAQ,OAAO,CAAC,IAAI,IAAI,IAAI;sBAC1C,gBAAgB,QAAQ,OAAO;wBAC7B,gBAAgB,QAAQ,OAAO,CAAC,QAAQ,IAAI,QAAQ;;;uBAGrD,gBAAgB,UAAU,OAAO;sBAClC,KAAK;;;ACnDzB,cAAM,iBAAkB,SAAQ,YAAY;;AAC5C,cAAM,EAAE,mBAA0B;;ACKlC,cAAa,KAAK;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAqCZ,KAAM,IAAI,gBAAgB,KAAK;AAE/B,KAAM,QAAQ,sCAAsC,IAAI,kBAAkB,IAAI;;AC6FpF,cAAa,kBAAkB,QAAe,QAAQ,oBAAoB,KAAK,cAAY,OAAO;;AC1HlG,cAAa,kBAAkB,cAAe,MAAM,WAAW,aAAa","names":[]}

package/dist/index.mjs

@@ -98,9 +98,10 @@ const ms = (val) => {
   if (str.length > 100) return Number.NaN;
   const match = RE.exec(str);
   if (!match) return Number.NaN;
-  const n = Number.parseFloat(match[1] ?? "");
-  const type = (match[2] ?? "ms").toLowerCase();
-  return n * (UNITS[type] ?? 0);
+  const [, numStr, unitStr] = match;
+  const n = Number.parseFloat(String(numStr));
+  const type = (unitStr ?? "ms").toLowerCase();
+  return n * UNITS[type];
 };
 
 const isArray = Array.isArray;
@@ -383,7 +384,7 @@ const updatePatch = async (opts, context, current, original) => {
   if (history) {
     let version = 0;
     const lastHistory = await HistoryModel.findOne({ collectionId: original._id }).sort("-version").exec();
-    if (lastHistory && lastHistory.version >= 0) {
+    if (lastHistory) {
       version = lastHistory.version + 1;
     }
     const [user, reason, metadata] = await getData(opts, current);
@@ -470,7 +471,7 @@ const updateMethods = ["update", "updateOne", "replaceOne", "updateMany", "findO
 const trackChangedFields = (fields, updated, changed) => {
   if (!fields) return;
   for (const key of Object.keys(fields)) {
-    const root = key.split(".")[0];
+    const [root = key] = key.split(".");
     changed.set(root, updated[root]);
   }
 };
@@ -488,7 +489,7 @@ const assignUpdate = (document, update, commands) => {
   let updated = assign(document.toObject(toObjectOptions), update);
   const changedByCommand = /* @__PURE__ */ new Map();
   for (const command of commands) {
-    const op = Object.keys(command)[0];
+    const [op = ""] = Object.keys(command);
     const fields = command[op];
     try {
       updated = assign(updated, command);
@@ -556,7 +557,8 @@ const updateHooksInitialize = (schema, opts) => {
     let current = null;
     for (const query of candidates) {
       if (current || isEmpty(query)) continue;
-      current = await model.findOne(query).sort({ _id: -1 }).lean().exec();
+      const found = await model.findOne(query).sort({ _id: -1 }).lean().exec();
+      current = found;
     }
     if (current) {
       this._context.createdDocs = [current];
@@ -588,13 +590,14 @@ const patchHistoryPlugin = (schema, opts) => {
     await createPatch(opts, context);
   });
   if (isMongooseLessThan8) {
-    schema.pre(remove, { document: true, query: false }, async function() {
+    const legacySchema = schema;
+    legacySchema.pre(remove, { document: true, query: false }, async function() {
       const original = this.toObject(toObjectOptions);
       if (opts.preDelete && !isEmpty(original)) {
         await opts.preDelete([original]);
       }
     });
-    schema.post(remove, { document: true, query: false }, async function() {
+    legacySchema.post(remove, { document: true, query: false }, async function() {
       const original = this.toObject(toObjectOptions);
       const model = this.constructor;
       const context = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-patch-mongoose",
-  "version": "3.1.0",
+  "version": "3.1.2",
   "description": "Patch history & events for mongoose models",
   "author": "ilovepixelart",
   "license": "MIT",
@@ -40,11 +40,7 @@
   },
   "files": [
     "dist",
-    "src",
-    "tests",
-    "tsconfig.json",
-    "vite.config.mts",
-    "biome.json"
+    "src"
   ],
   "type": "module",
   "exports": {
@@ -67,25 +63,26 @@
     "test": "vitest run --coverage",
     "test:open": "vitest run --coverage && open-cli coverage/lcov-report/index.html",
     "type:check": "tsc --noEmit",
+    "type:check:tests": "tsc --noEmit -p tests/tsconfig.json",
     "build": "pkgroll --clean-dist",
-    "release": "npm install && npm run biome && npm run type:check && npm run build && np --no-publish"
+    "release": "npm install && npm run biome && npm run type:check && npm run type:check:tests && npm run build && np --no-publish"
   },
   "dependencies": {
     "fast-json-patch": "3.1.1",
     "power-assign": "0.2.10"
   },
   "devDependencies": {
-    "@biomejs/biome": "2.4.9",
-    "@types/node": "25.5.0",
-    "@vitest/coverage-v8": "4.1.2",
+    "@biomejs/biome": "2.4.11",
+    "@types/node": "25.6.0",
+    "@vitest/coverage-v8": "4.1.4",
     "mongodb-memory-server": "11.0.1",
-    "mongoose": "9.3.3",
-    "np": "11.0.2",
+    "mongoose": "9.4.1",
+    "np": "11.0.3",
     "open-cli": "9.0.0",
     "pkgroll": "2.27.0",
     "simple-git-hooks": "2.13.1",
     "typescript": "5.9.3",
-    "vitest": "4.1.2"
+    "vitest": "4.1.4"
   },
   "peerDependencies": {
     "mongoose": ">=6.6.0 < 10"
@@ -99,6 +96,8 @@
   },
   "overrides": {
     "tmp": "0.2.5",
-    "file-type": "21.3.2"
+    "file-type": "21.3.2",
+    "lodash": "4.18.1",
+    "vite": "8.0.8"
   }
 }

package/src/hooks/update-hooks.ts

@@ -10,7 +10,7 @@ const updateMethods = ['update', 'updateOne', 'replaceOne', 'updateMany', 'findO
 const trackChangedFields = (fields: Record<string, unknown> | undefined, updated: Record<string, unknown>, changed: Map<string, unknown>): void => {
   if (!fields) return
   for (const key of Object.keys(fields)) {
-    const root = key.split('.')[0] as string
+    const [root = key] = key.split('.')
     changed.set(root, updated[root])
   }
 }
@@ -31,7 +31,7 @@ export const assignUpdate = <T>(document: HydratedDocument<T>, update: UpdateQue
   const changedByCommand = new Map<string, unknown>()
 
   for (const command of commands) {
-    const op = Object.keys(command)[0] as string
+    const [op = ''] = Object.keys(command)
     const fields = command[op] as Record<string, unknown> | undefined
     try {
       updated = assign(updated, command)
@@ -113,7 +113,12 @@ export const updateHooksInitialize = <T>(schema: Schema<T>, opts: PluginOptions<
     let current: HydratedDocument<T> | null = null
     for (const query of candidates) {
       if (current || isEmpty(query)) continue
-      current = (await model.findOne(query).sort({ _id: -1 }).lean().exec()) as HydratedDocument<T>
+      const found = await model
+        .findOne(query as never)
+        .sort({ _id: -1 })
+        .lean()
+        .exec()
+      current = found as HydratedDocument<T>
     }
 
     if (current) {

package/src/index.ts

@@ -36,9 +36,13 @@ export const patchHistoryPlugin = <T>(schema: Schema<T>, opts: PluginOptions<T>)
   // In Mongoose 7, doc.deleteOne() returned a promise that resolved to doc.
   // In Mongoose 8, doc.deleteOne() returns a query for easier chaining, as well as consistency with doc.updateOne().
   if (isMongooseLessThan8) {
-    // @ts-expect-error - Mongoose 7 and below
-    schema.pre(remove, { document: true, query: false }, async function () {
-      // @ts-expect-error - Mongoose 7 and below
+    type LegacySchema = {
+      pre(name: string, options: { document: boolean; query: boolean }, fn: (this: HydratedDocument<T>) => Promise<void>): void
+      post(name: string, options: { document: boolean; query: boolean }, fn: (this: HydratedDocument<T>) => Promise<void>): void
+    }
+    const legacySchema = schema as unknown as LegacySchema
+
+    legacySchema.pre(remove, { document: true, query: false }, async function (this: HydratedDocument<T>) {
       const original = this.toObject(toObjectOptions) as HydratedDocument<T>
 
       if (opts.preDelete && !isEmpty(original)) {
@@ -46,8 +50,7 @@ export const patchHistoryPlugin = <T>(schema: Schema<T>, opts: PluginOptions<T>)
       }
     })
 
-    // @ts-expect-error - Mongoose 7 and below
-    schema.post(remove, { document: true, query: false }, async function (this: HydratedDocument<T>) {
+    legacySchema.post(remove, { document: true, query: false }, async function (this: HydratedDocument<T>) {
       const original = this.toObject(toObjectOptions) as HydratedDocument<T>
       const model = this.constructor as Model<T>
 

package/src/ms.ts

@@ -60,7 +60,8 @@ export const ms = (val: Duration): number => {
   const match = RE.exec(str)
   if (!match) return Number.NaN
 
-  const n = Number.parseFloat(match[1] ?? '')
-  const type = (match[2] ?? 'ms').toLowerCase()
-  return n * (UNITS[type as Unit] ?? 0)
+  const [, numStr, unitStr] = match
+  const n = Number.parseFloat(String(numStr))
+  const type = (unitStr ?? 'ms').toLowerCase() as Unit
+  return n * UNITS[type]
 }

package/src/patch.ts

@@ -23,7 +23,7 @@ export const getJsonOmit = <T>(opts: PluginOptions<T>, doc: HydratedDocument<T>)
 }
 
 export const getObjectOmit = <T>(opts: PluginOptions<T>, doc: HydratedDocument<T>): Partial<T> => {
-  return applyOmit(isFunction(doc?.toObject) ? doc.toObject() : doc, opts)
+  return applyOmit(isFunction(doc?.toObject) ? (doc.toObject() as Partial<T>) : doc, opts)
 }
 
 const getOptionalField = async <T, R>(fn: ((doc: HydratedDocument<T>) => Promise<R> | R) | undefined, doc?: HydratedDocument<T>): Promise<R | undefined> => {
@@ -43,7 +43,7 @@ export const getValue = <T>(item: PromiseSettledResult<T>): T | undefined => {
   return item.status === 'fulfilled' ? item.value : undefined
 }
 
-export const getData = async <T>(opts: PluginOptions<T>, doc: HydratedDocument<T>): Promise<[User | undefined, string | undefined, Metadata | undefined]> => {
+export const getData = async <T>(opts: PluginOptions<T>, doc?: HydratedDocument<T>): Promise<[User | undefined, string | undefined, Metadata | undefined]> => {
   return Promise.allSettled([getUser(opts, doc), getReason(opts, doc), getMetadata(opts, doc)]).then(([user, reason, metadata]) => {
     return [getValue(user), getValue(reason), getValue(metadata)]
   })
@@ -127,7 +127,7 @@ export const updatePatch = async <T>(opts: PluginOptions<T>, context: PatchConte
       .sort('-version')
       .exec()
 
-    if (lastHistory && lastHistory.version >= 0) {
+    if (lastHistory) {
       version = lastHistory.version + 1
     }
 

package/src/types.ts

@@ -12,6 +12,8 @@ export interface History {
   reason?: string
   metadata?: object
   patch?: Operation[]
+  createdAt?: Date
+  updatedAt?: Date
 }
 
 export interface PatchEvent<T> {

package/biome.json

@@ -1,47 +0,0 @@
-{
-  "$schema": "https://biomejs.dev/schemas/2.4.9/schema.json",
-  "vcs": { "enabled": false, "clientKind": "git", "useIgnoreFile": false },
-  "files": {
-    "ignoreUnknown": false,
-    "includes": ["src/**/*.ts", "tests/**/*.ts"]
-  },
-  "formatter": { "enabled": true, "indentStyle": "space", "indentWidth": 2 },
-  "assist": {
-    "actions": {
-      "source": {
-        "organizeImports": {
-          "level": "on",
-          "options": {
-            "groups": [
-              "vitest",
-              ":BLANK_LINE:",
-              ":NODE:",
-              { "type": false },
-              ":BLANK_LINE:"
-            ]
-          }
-        }
-      }
-    }
-  },
-  "linter": {
-    "enabled": true,
-    "rules": {
-      "recommended": true
-    }
-  },
-  "javascript": {
-    "formatter": {
-      "trailingCommas": "all",
-      "quoteStyle": "single",
-      "semicolons": "asNeeded",
-      "lineWidth": 320
-    },
-    "globals": ["Atomics", "SharedArrayBuffer"]
-  },
-  "json": {
-    "formatter": {
-      "trailingCommas": "none"
-    }
-  }
-}

package/tests/constants/events.ts

@@ -1,7 +0,0 @@
-export const USER_CREATED = 'user-created'
-export const USER_UPDATED = 'user-updated'
-export const USER_DELETED = 'user-deleted'
-
-export const GLOBAL_CREATED = 'global-created'
-export const GLOBAL_UPDATED = 'global-updated'
-export const GLOBAL_DELETED = 'global-deleted'

package/tests/em.test.ts

@@ -1,70 +0,0 @@
-import { afterEach, describe, expect, it, vi } from 'vitest'
-
-import { patchEventEmitter } from '../src/index'
-import { emitEvent } from '../src/patch'
-
-describe('em', () => {
-  afterEach(() => {
-    patchEventEmitter.removeAllListeners()
-  })
-
-  it('should subscribe and count', () => {
-    let count = 0
-    const fn = () => {
-      count++
-    }
-    patchEventEmitter.on('test', fn)
-    patchEventEmitter.emit('test')
-    expect(count).toBe(1)
-    patchEventEmitter.off('test', fn)
-    patchEventEmitter.emit('test')
-    expect(count).toBe(1)
-  })
-
-  it('emitEvent', () => {
-    const fn = vi.fn()
-    patchEventEmitter.on('test', fn)
-
-    const context = {
-      op: 'test',
-      modelName: 'Test',
-      collectionName: 'tests',
-    }
-
-    // @ts-expect-error expected
-    emitEvent(context, 'test', { doc: { name: 'test' } })
-    expect(fn).toHaveBeenCalledOnce()
-  })
-
-  it('emitEvent ignore', () => {
-    const fn = vi.fn()
-    patchEventEmitter.on('test', fn)
-
-    const context = {
-      ignoreEvent: true,
-      op: 'test',
-      modelName: 'Test',
-      collectionName: 'tests',
-    }
-
-    // @ts-expect-error expected
-    emitEvent(context, 'test', { doc: { name: 'test' } })
-    expect(fn).toHaveBeenCalledTimes(0)
-  })
-
-  it('emitEvent should not throw when listener throws', () => {
-    const fn = () => {
-      throw new Error('listener error')
-    }
-    patchEventEmitter.on('throw-test', fn)
-
-    const context = {
-      op: 'test',
-      modelName: 'Test',
-      collectionName: 'tests',
-    }
-
-    // @ts-expect-error expected
-    expect(() => emitEvent(context, 'throw-test', { doc: { name: 'test' } })).not.toThrow()
-  })
-})