The commit here introduces a potentially-breaking change, straying from the docs on how the _id of an upserted document is returned.

result.upsertedId was previously an object containing _id, whereas it is now the value of _id.

Is this considered a breaking change? It definitely broke a lot of our parsing and only on a patch incrementation.