chore: comments

Author: durran

src/cursor/client_bulk_write_cursor.ts

@@ -15,7 +15,7 @@ import {
 
 /** @public */
 export interface ClientBulkWriteCursorOptions
-  extends AbstractCursorOptions,
+  extends Omit<AbstractCursorOptions, 'maxAwaitTimeMS' | 'tailable' | 'awaitData'>,
     ClientBulkWriteOptions {}
 
 /**

src/error.ts

@@ -617,7 +617,7 @@ export class MongoGCPError extends MongoOIDCError {
 }
 
 /**
- * An error generated when a ChangeStream operation fails to execute.
+ * An error indicating that an error occurred when processing bulk write results.
  *
  * @public
  * @category Error

src/operations/client_bulk_write/results_merger.ts

@@ -74,9 +74,8 @@ export class ClientBulkWriteResultsMerger {
             const result: ClientUpdateResult = {
               matchedCount: document.n,
               modifiedCount: document.nModified ?? 0,
-              // We do specifically want to check undefined here since a null _id is valid.
-              // eslint-disable-next-line no-restricted-syntax
-              didUpsert: document.upserted?._id !== undefined
+              // Check if the bulk did actually upsert.
+              didUpsert: document.upserted != null
             };
             if (document.upserted) {
               result.upsertedId = document.upserted._id;