types: make lean() not clobber result type for updateOne(), etc.

[vkarpov15]

Fix #13382

Summary

Doing Model.updateOne().lean() returns an incorrect type: has _id, etc. With this change, queries now have a QueryOp generic that they can use to check the current query op. With this change, Model.deleteOne().lean(), etc. all reflect their runtime behavior: lean() is a no-op.

Examples

[AbdelrahmanHafez]

LGTM, thanks. 👍

I think it's worth taking a note to deprecate using lean on non-find operations like deleteOne/countDocuments in both TS types and JS runtime, it doesn't make sense to call them with lean.

[AbdelrahmanHafez]

LGTM, thanks. 👍

I think it's worth taking a note to deprecate using lean on non-find operations like deleteOne/countDocuments in both TS types and JS runtime, it doesn't make sense to call them with lean.

[vkarpov15]

Potentially. As a general rule of thumb, Mongoose doesn't throw errors on unrecognized or unused options, so throwing an error when using lean() on an operation that doesn't handle documents would be a bit inconsistent. I opened an issue to consider that for the future.