@toa.io/storages.mongodb 1.0.0-alpha.332 → 1.0.0-alpha.335

package/CHANGELOG.md

@@ -3,6 +3,29 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [1.0.0-alpha.335](https://github.com/toa-io/toa/compare/v1.0.0-alpha.334...v1.0.0-alpha.335) (2026-10-26)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.334](https://github.com/toa-io/toa/compare/v1.0.0-alpha.333...v1.0.0-alpha.334) (2026-10-26)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.333](https://github.com/toa-io/toa/compare/v1.0.0-alpha.332...v1.0.0-alpha.333) (2026-10-26)
+
+### Bug Fixes
+
+* **storages.mongodb:** run a set again where it met a commit in flight ([2e293fe](https://github.com/toa-io/toa/commit/2e293fe1b72c3eac65f659f0061e312945829b6e))
+
+
 # [1.0.0-alpha.332](https://github.com/toa-io/toa/compare/v1.0.0-alpha.331...v1.0.0-alpha.332) (2026-10-23)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toa.io/storages.mongodb",
-  "version": "1.0.0-alpha.332",
+  "version": "1.0.0-alpha.335",
   "type": "module",
   "description": "Toa MongoDB Storage Connector",
   "author": "temich <tema.gurtovoy@gmail.com>",
@@ -21,12 +21,12 @@
   },
   "dependencies": {
     "@toa.io/conveyor": "1.0.0-alpha.332",
-    "@toa.io/core": "1.0.0-alpha.332",
-    "@toa.io/definitions": "1.0.0-alpha.332",
+    "@toa.io/core": "1.0.0-alpha.333",
+    "@toa.io/definitions": "1.0.0-alpha.335",
     "@toa.io/generic": "1.0.0-alpha.332",
     "@toa.io/pointer": "1.0.0-alpha.332",
     "mongodb": "7.7.0",
     "openspan": "1.0.0-alpha.330"
   },
-  "gitHead": "37dbdefd96c01df050ea0d46a647da7f2f9f93f2"
+  "gitHead": "07f9c76f161c9422d9126deaecf468ec50bb54a3"
 }

package/src/bulk.js

@@ -0,0 +1,26 @@
+import { MongoClientBulkWriteError, MongoServerError } from 'mongodb'
+
+/**
+ * The client's bulk write, where a command the server refused is thrown as the server sent it.
+ *
+ * The driver wraps that refusal in an error that carries none of its labels, and a transaction
+ * runs again only what is labelled as worth running again: a write conflict with a commit in
+ * flight would end the transaction as a failure of its own. A write the server refused one by
+ * one is left as it is, read off `writeErrors`.
+ *
+ * @param {import('mongodb').MongoClient} client
+ * @param {import('mongodb').AnyClientBulkWriteModel[]} operations
+ * @param {import('mongodb').ClientBulkWriteOptions} options
+ */
+export async function bulkWrite(client, operations, options) {
+  try {
+    return await client.bulkWrite(operations, options)
+  } catch (error) {
+    if (
+      error instanceof MongoClientBulkWriteError &&
+      error.cause instanceof MongoServerError
+    )
+      throw error.cause
+    else throw error
+  }
+}

package/src/outbox.js

@@ -1,5 +1,6 @@
 import { environment } from '@toa.io/generic'
 import { index, prune } from './indexes.js'
+import { bulkWrite } from './bulk.js'
 
 /**
  * The outbox rows of one component. Its lifecycle is the Client's, so it is not a Connector.
@@ -30,7 +31,8 @@ export class Outbox {
     const namespace = this.#collection.namespace
 
     // the client's bulk write, not `insertMany`: see `massStore` in ./storage.js
-    await this.#collection.client.bulkWrite(
+    await bulkWrite(
+      this.#collection.client,
       rows.map((row) => ({ name: 'insertOne', namespace, document: to(row) })),
       { session }
     )

package/src/storage.js

@@ -4,6 +4,7 @@ import { translate } from './translate.js'
 import { codec } from './record.js'
 import { Inbox } from './inbox.js'
 import { Outbox } from './outbox.js'
+import { bulkWrite } from './bulk.js'
 import { Migrations } from './migrations.js'
 import { Streams } from './streams.js'
 import { conflicted, query } from './measurements.js'
@@ -332,7 +333,7 @@ export class Storage extends Connector {
             // the client's bulk write, not the collection's: the latter takes a `timeoutMS` the
             // URL sets for one the call passed, and refuses it inside a transaction that has
             // its own
-            async () => await this.#collection.client.bulkWrite(operations, { session })
+            async () => await bulkWrite(this.#collection.client, operations, { session })
           )
         } catch (error) {
           if (!collidedOnId(firstWriteError(error))) throw error

package/test/bulk.test.js

@@ -0,0 +1,49 @@
+import { describe, it, mock } from 'node:test'
+import assert from 'node:assert/strict'
+import { MongoClientBulkWriteError, MongoServerError } from 'mongodb'
+
+import { bulkWrite } from '../src/bulk.js'
+
+describe('bulkWrite', () => {
+  it('should throw what the server refused the command with, labels and all', async () => {
+    const conflict = new MongoServerError({
+      message: 'Write conflict during plan execution and yielding is disabled.',
+      code: 112,
+      errorLabels: ['TransientTransactionError']
+    })
+
+    const wrapped = new MongoClientBulkWriteError({
+      message: 'Mongo client bulk write encountered an error during execution'
+    })
+
+    wrapped.cause = conflict
+
+    const client = { bulkWrite: mock.fn(async () => Promise.reject(wrapped)) }
+
+    await assert.rejects(bulkWrite(client, [], {}), (error) => {
+      assert.equal(error, conflict)
+      assert.equal(error.hasErrorLabel('TransientTransactionError'), true)
+
+      return true
+    })
+  })
+
+  it('should throw what refused a write one by one as it is', async () => {
+    const refused = new MongoClientBulkWriteError({
+      message: 'Mongo client bulk write encountered errors during execution'
+    })
+
+    refused.writeErrors.set(0, { code: 11000, message: 'E11000 duplicate key error' })
+
+    const client = { bulkWrite: mock.fn(async () => Promise.reject(refused)) }
+
+    await assert.rejects(bulkWrite(client, [], {}), (error) => error === refused)
+  })
+
+  it('should answer what the client answers', async () => {
+    const result = { matchedCount: 1, upsertedCount: 0 }
+    const client = { bulkWrite: mock.fn(async () => result) }
+
+    assert.equal(await bulkWrite(client, [], {}), result)
+  })
+})