Remove `skipId` param for documents

[vkarpov15]

Similar to #4625 . This option is only used to avoid setting a default id on replaceOne() and findOneAndReplace(), and it would be cleaner to just delete _id before sending the operation to the server as opposed to passing this option everywhere.

[AbdelrahmanHafez]

What if users pass a new Document() to Model.replaceOne(...), we'll need to remove the newly generated _id automatically, correct?

[vkarpov15]

Yes we would need to remove the newly generated _id in a different way. Another alternative would be to replace skipId with options.skipId. My primary issue with skipId is that it is taking up some prime real estate as the 3rd parameter to the document constructor, and doesn't provide much value to the end user.