@toa.io/storages.mongodb 1.0.0-alpha.32 → 1.0.0-alpha.320

package/CHANGELOG.md

@@ -0,0 +1,282 @@
+# Change Log
+
+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.320](https://github.com/toa-io/toa/compare/v1.0.0-alpha.319...v1.0.0-alpha.320) (2026-09-28)
+
+### Features
+
+* page a stream by CREATED, and stop it after its first page ([038dc52](https://github.com/toa-io/toa/commit/038dc52f2db1a4dd58c6ec58a00de8914b3d38a3))
+
+
+# [1.0.0-alpha.319](https://github.com/toa-io/toa/compare/v1.0.0-alpha.318...v1.0.0-alpha.319) (2026-09-27)
+
+### Features
+
+* **storages.mongodb:** read a set and its changes from a token ([d0a8955](https://github.com/toa-io/toa/commit/d0a8955683b7566e704f7e9cde69f67272c7be1b))
+
+
+# [1.0.0-alpha.318](https://github.com/toa-io/toa/compare/v1.0.0-alpha.317...v1.0.0-alpha.318) (2026-09-23)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.317](https://github.com/toa-io/toa/compare/v1.0.0-alpha.316...v1.0.0-alpha.317) (2026-09-23)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.316](https://github.com/toa-io/toa/compare/v1.0.0-alpha.315...v1.0.0-alpha.316) (2026-09-23)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.315](https://github.com/toa-io/toa/compare/v1.0.0-alpha.314...v1.0.0-alpha.315) (2026-09-23)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.314](https://github.com/toa-io/toa/compare/v1.0.0-alpha.313...v1.0.0-alpha.314) (2026-09-19)
+
+* feat(core)!: let an operation state how long its calls are remembered ([af177c2](https://github.com/toa-io/toa/commit/af177c2ab3d00d475840b5ef9e612f04e82b72bd))
+
+### BREAKING CHANGES
+
+* `inbox.retention` under 600 is refused. See
+  migrations/313.md.
+
+
+# [1.0.0-alpha.313](https://github.com/toa-io/toa/compare/v1.0.0-alpha.312...v1.0.0-alpha.313) (2026-09-18)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.312](https://github.com/toa-io/toa/compare/v1.0.0-alpha.311...v1.0.0-alpha.312) (2026-09-18)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.311](https://github.com/toa-io/toa/compare/v1.0.0-alpha.310...v1.0.0-alpha.311) (2026-09-17)
+
+### Features
+
+* name the database after the scope ([4d7203f](https://github.com/toa-io/toa/commit/4d7203fd18828895272a9dae76e351d92b49277f))
+
+
+# [1.0.0-alpha.310](https://github.com/toa-io/toa/compare/v1.0.0-alpha.309...v1.0.0-alpha.310) (2026-09-16)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.309](https://github.com/toa-io/toa/compare/v1.0.0-alpha.308...v1.0.0-alpha.309) (2026-09-16)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.308](https://github.com/toa-io/toa/compare/v1.0.0-alpha.307...v1.0.0-alpha.308) (2026-09-15)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.307](https://github.com/toa-io/toa/compare/v1.0.0-alpha.306...v1.0.0-alpha.307) (2026-09-15)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.306](https://github.com/toa-io/toa/compare/v1.0.0-alpha.305...v1.0.0-alpha.306) (2026-09-14)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.305](https://github.com/toa-io/toa/compare/v1.0.0-alpha.304...v1.0.0-alpha.305) (2026-09-13)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.304](https://github.com/toa-io/toa/compare/v1.0.0-alpha.303...v1.0.0-alpha.304) (2026-09-13)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.303](https://github.com/toa-io/toa/compare/v1.0.0-alpha.302...v1.0.0-alpha.303) (2026-09-12)
+
+### Features
+
+* **metrics:** measure storage, stash, blob storages and fetch ([252298b](https://github.com/toa-io/toa/commit/252298bb9f34421694da4ee89742d62b47c39dfb))
+
+### Performance Improvements
+
+* **storages.mongodb:** rename a record's _id to id in place ([4aeb2a1](https://github.com/toa-io/toa/commit/4aeb2a1d61f7f2b0ddab203b5887152e8917ef7f))
+
+
+# [1.0.0-alpha.302](https://github.com/toa-io/toa/compare/v1.0.0-alpha.301...v1.0.0-alpha.302) (2026-09-11)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.301](https://github.com/toa-io/toa/compare/v1.0.0-alpha.300...v1.0.0-alpha.301) (2026-09-11)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.300](https://github.com/toa-io/toa/compare/v1.0.0-alpha.299...v1.0.0-alpha.300) (2026-09-11)
+
+* feat(core)!: an operation may ask to run once ([dca616c](https://github.com/toa-io/toa/commit/dca616cb380a7a389e277875d3bd5ca55a6e4d45))
+
+### Bug Fixes
+
+* **extensions:** what a factory remembers does not outlive the tree it was made for ([dd36631](https://github.com/toa-io/toa/commit/dd3663160f500eadfb701bf840ef3741a2d075f0))
+
+### Features
+
+* **core:** an assignment may ask to run once as well ([10fbcbd](https://github.com/toa-io/toa/commit/10fbcbd86a5e82960e65f62c09d0a465ae8d0353))
+
+### BREAKING CHANGES
+
+* `Storage.store`, `.upsert` and `.ensure` take one more
+  argument, and `Storage` has two more members. A connector that ignores them
+  works exactly as it did; a component that asks it for `once` is refused at
+  boot. See migrations/299.md.
+
+
+# [1.0.0-alpha.299](https://github.com/toa-io/toa/compare/v1.0.0-alpha.298...v1.0.0-alpha.299) (2026-09-10)
+
+### Features
+
+* **core:** a record carries the region that wrote it ([13ed187](https://github.com/toa-io/toa/commit/13ed187604f06dec66feec2e26ccf9dae30bb048))
+* **core:** an outbox row is published to destinations ([cfb77b4](https://github.com/toa-io/toa/commit/cfb77b4826ff0f9560769a0996981e143d6525af))
+
+
+# [1.0.0-alpha.298](https://github.com/toa-io/toa/compare/v1.0.0-alpha.297...v1.0.0-alpha.298) (2026-09-08)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.297](https://github.com/toa-io/toa/compare/v1.0.0-alpha.296...v1.0.0-alpha.297) (2026-09-08)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.296](https://github.com/toa-io/toa/compare/v1.0.0-alpha.295...v1.0.0-alpha.296) (2026-09-08)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.295](https://github.com/toa-io/toa/compare/v1.0.0-alpha.294...v1.0.0-alpha.295) (2026-09-07)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.294](https://github.com/toa-io/toa/compare/v1.0.0-alpha.293...v1.0.0-alpha.294) (2026-09-07)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.293](https://github.com/toa-io/toa/compare/v1.0.0-alpha.292...v1.0.0-alpha.293) (2026-09-07)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.292](https://github.com/toa-io/toa/compare/v1.0.0-alpha.291...v1.0.0-alpha.292) (2026-09-07)
+
+### Features
+
+* **definitions:** what a package declares is read from one package ([7052341](https://github.com/toa-io/toa/commit/70523411b5d9e2b204c999aa02365173f3bb518d))
+
+
+# [1.0.0-alpha.291](https://github.com/toa-io/toa/compare/v1.0.0-alpha.290...v1.0.0-alpha.291) (2026-09-07)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.289](https://github.com/toa-io/toa/compare/v1.0.0-alpha.288...v1.0.0-alpha.289) (2026-09-07)
+
+* A deploy moves only what changed, and a component sees none of the runtime's environment (#1073) ([f38e3db](https://github.com/toa-io/toa/commit/f38e3db533f24db866c57bfa1ef294eff1eaf499)), closes [#1073](https://github.com/toa-io/toa/issues/1073) [#1064](https://github.com/toa-io/toa/issues/1064) [#1066](https://github.com/toa-io/toa/issues/1066) [#1067](https://github.com/toa-io/toa/issues/1067) [#1068](https://github.com/toa-io/toa/issues/1068) [#1069](https://github.com/toa-io/toa/issues/1069) [#1071](https://github.com/toa-io/toa/issues/1071) [#1070](https://github.com/toa-io/toa/issues/1070) [#1072](https://github.com/toa-io/toa/issues/1072)
+
+### BREAKING CHANGES
+
+* a component that read `process.env.TOA_*` reads `context` instead;
+  `echo(input)` no longer substitutes from the environment; a bash operation sees no
+  `TOA_*`; images no longer set `USER node` — see migrations/289.md.
+
+
+# [1.0.0-alpha.288](https://github.com/toa-io/toa/compare/v1.0.0-alpha.287...v1.0.0-alpha.288) (2026-09-06)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.287](https://github.com/toa-io/toa/compare/v1.0.0-alpha.286...v1.0.0-alpha.287) (2026-09-06)
+
+### Features
+
+* **mongodb:** a migration says what it is doing ([a917a81](https://github.com/toa-io/toa/commit/a917a81fdc94eb23fd73182c0edf891e6b2df843))

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@toa.io/storages.mongodb",
-  "version": "1.0.0-alpha.32",
+  "version": "1.0.0-alpha.320",
+  "type": "module",
   "description": "Toa MongoDB Storage Connector",
   "author": "temich <tema.gurtovoy@gmail.com>",
   "homepage": "https://github.com/toa-io/toa#readme",
@@ -19,13 +20,13 @@
     "test": "echo \"Error: run tests from root\" && exit 1"
   },
   "dependencies": {
-    "@toa.io/console": "1.0.0-alpha.32",
-    "@toa.io/conveyor": "1.0.0-alpha.32",
-    "@toa.io/core": "1.0.0-alpha.32",
-    "@toa.io/generic": "1.0.0-alpha.32",
-    "@toa.io/pointer": "1.0.0-alpha.32",
-    "mongodb": "6.3.0",
-    "saslprep": "1.0.3"
+    "@toa.io/conveyor": "1.0.0-alpha.311",
+    "@toa.io/core": "1.0.0-alpha.320",
+    "@toa.io/definitions": "1.0.0-alpha.320",
+    "@toa.io/generic": "1.0.0-alpha.311",
+    "@toa.io/pointer": "1.0.0-alpha.311",
+    "mongodb": "7.6.0",
+    "openspan": "1.0.0-alpha.305"
   },
-  "gitHead": "93ceb53797363abb77eb9c550db77e906691d42a"
+  "gitHead": "5ef92ab939a596e4ac820965c59c015820eba7b2"
 }

package/readme.md

@@ -0,0 +1,38 @@
+# MongoDB Storage
+
+## Tracing
+
+Commands are recorded as `client` spans within the trace of the current invocation.
+
+Spans are named `{command} {collection}` and carry `db.*` attributes following the
+[OpenTelemetry semantic conventions](https://opentelemetry.io/docs/specs/semconv/database/mongodb/):
+`db.system`, `db.namespace`, `db.collection.name`, `db.operation.name`.
+
+Commands executed outside of a sampled trace context (e.g. index management on startup)
+and internal driver commands (`hello`, `ping`, authentication) are not recorded.
+
+Monitoring is client-side only and does not affect the MongoDB server. Span recording
+adds no waiting to the query path: exporting is buffered and happens in the background.
+
+## Stream tokens
+
+A [stream](/documentation/collections.md) ends with a token of changes where two things hold:
+
+- **MongoDB runs as a replica set**, which it does wherever there is an outbox.
+- **The collection keeps images** of what a record was before each change, which is how a change
+  that takes an entry out of the collection is told from a change outside it. They cost a copy of every
+  changed record for as long as the oplog holds it, so a collection keeps them only by a
+  [migration](/documentation/component/declaration.md#migrations):
+
+  ```yaml
+  # migrations/0003-images.yaml
+  - images: true
+  ```
+
+  `- images: false` stops keeping them, and a token issued before is then refused with `410`.
+
+A token lasts as long as the oplog of the replica set holds the point it names — its window, which
+the size of the oplog and the rate of writes decide.
+
+A read from a token scans the oplog from the point it names: it costs what the whole replica set
+wrote since, whatever of it concerns the collection.

package/src/client.js

@@ -1,28 +1,67 @@
-'use strict'
-
 /**
  * @typedef {import('mongodb').MongoClient} MongoClient
  * @typedef {{ count: number, client: MongoClient }} Instance
  * @typedef {import('@toa.io/core').Locator} Locator
  */
 
-const { Connector } = require('@toa.io/core')
-const { resolve } = require('@toa.io/pointer')
-const { ID } = require('./deployment')
-const { MongoClient } = require('mongodb')
+import { console } from 'openspan'
+import { environment } from '@toa.io/generic'
+import { Connector } from '@toa.io/core'
+import { resolve } from '@toa.io/pointer'
+import { ID } from '@toa.io/definitions/storages.mongodb'
+import { MongoClient } from 'mongodb'
 
 /**
  * @type {Record<string, Promise<Instance>>}
  */
 const INSTANCES = {}
 
-class Client extends Connector {
+export class Client extends Connector {
+  name
+
   /**
    * @public
    * @type {import('mongodb').Collection}
    */
   collection
 
+  /**
+   * The outbox rows of this component, absent unless something consumes its events. Created
+   * eagerly beside the entity collection, because a transaction cannot create a collection and
+   * an index build cannot run inside one.
+   *
+   * @public
+   * @type {import('mongodb').Collection | undefined}
+   */
+  outbox
+
+  /**
+   * The calls this component has made good on, absent unless it declares `once` anywhere.
+   * Created eagerly beside the entity collection, for the reason the outbox's is.
+   *
+   * @public
+   * @type {import('mongodb').Collection | undefined}
+   */
+  inbox
+
+  /**
+   * Whether this deployment can run transactions at all. A standalone mongod cannot, and an
+   * outbox without atomicity is worse than none, so the storage falls back to inline emission.
+   *
+   * @public
+   * @type {boolean}
+   */
+  transactional = false
+
+  /**
+   * The database this component's collections live in, which is where the migration state
+   * is kept as well.
+   *
+   * @public
+   * @type {import('mongodb').Db}
+   */
+  db
+
   /**
    * @private
    * @type {Locator}
@@ -41,13 +80,30 @@ class Client extends Connector {
    */
   key
 
+  /**
+   * @private
+   * @type {boolean}
+   */
+  publishes
+
+  /**
+   * @private
+   * @type {boolean}
+   */
+  claims
+
   /**
    * @param {Locator} locator
+   * @param {boolean} [publishes] whether this component publishes anything
+   * @param {boolean} [claims] whether any of its operations declares `once`
    */
-  constructor (locator) {
+  constructor(locator, publishes = false, claims = false) {
     super()
 
     this.locator = locator
+    this.name = locator.lowercase
+    this.publishes = publishes
+    this.claims = claims
   }
 
   /**
@@ -55,29 +111,66 @@ class Client extends Connector {
    * @override
    * @return {Promise<void>}
    */
-  async open () {
+  async open() {
     const urls = await this.resolveURLs()
     const dbname = this.resolveDB()
-    const collname = this.locator.lowercase
 
     this.key = getKey(dbname, urls)
 
-    INSTANCES[this.key] ??= this.createInstance(urls)
+    try {
+      INSTANCES[this.key] ??= this.createInstance(urls)
+    } catch (error) {
+      console.error('Failed to connect to MongoDB', { urls, error })
+    }
 
     this.instance = await INSTANCES[this.key]
     this.instance.count++
 
     const db = this.instance.client.db(dbname)
 
-    try {
-      this.collection = await db.createCollection(collname)
-    } catch (e) {
-      if (e.code !== ALREADY_EXISTS) {
-        throw e
-      }
+    this.db = db
+    this.collection = await collection(db, this.name)
+    this.transactional = await transactional(db)
 
-      this.collection = db.collection(collname)
+    /*
+     * The outbox may fall back and this may not: inline emission still delivers, where a call
+     * that is not recorded is a call that will be made twice, which is the opposite of what was
+     * asked for. So this refuses rather than warns.
+     */
+    if (this.claims) {
+      if (!this.transactional)
+        throw new Error(
+          `Component '${this.name}' declares 'once', which needs a MongoDB replica set ` +
+            'or a sharded cluster to commit a call with the entity it changed'
+        )
+
+      this.inbox = await collection(db, this.name + INBOX)
     }
+
+    if (!this.publishes) return
+
+    if (this.transactional) this.outbox = await collection(db, this.name + OUTBOX)
+    else
+      console.warn(
+        'MongoDB is not a replica set; events are emitted inline, without an outbox',
+        { collection: this.name }
+      )
+  }
+
+  /**
+   * Runs `fn` in a transaction and answers what it returned. The driver may call `fn` more
+   * than once, so it must not hold state of its own — an outbox row is built by the caller
+   * and reused, and a rolled back attempt leaves nothing behind.
+   *
+   * @public
+   * @template T
+   * @param {(session: import('mongodb').ClientSession) => Promise<T>} fn
+   * @return {Promise<T>}
+   */
+  async transaction(fn) {
+    return this.instance.client.withSession(async (session) =>
+      session.withTransaction(async () => fn(session))
+    )
   }
 
   /**
@@ -85,14 +178,26 @@ class Client extends Connector {
    * @override
    * @return {Promise<void>}
    */
-  async close () {
-    const instance = await INSTANCES[this.key]
+  async close() {
+    /*
+     * What was never counted is not discounted. An `open` that threw between taking the
+     * instance and incrementing it leaves the count one high, and a client nothing ever
+     * closes — which a process that is taken down and built again, as a halt does, would
+     * otherwise leak once per cycle.
+     */
+    if (this.instance === undefined) return
+
+    const instance = this.instance
+
+    this.instance = undefined
 
     instance.count--
 
     if (instance.count === 0) {
       await instance.client.close()
-      delete INSTANCES[this.key]
+
+      // another `open` may have taken it in the meantime, and that one is not this one
+      if ((await INSTANCES[this.key]) === instance) delete INSTANCES[this.key]
     }
   }
 
@@ -101,11 +206,11 @@ class Client extends Connector {
    * @param {string[]} urls
    * @return {Promise<Instance>}
    */
-  async createInstance (urls) {
+  async createInstance(urls) {
     const client = new MongoClient(urls.join(','), OPTIONS)
     const hosts = urls.map((str) => new URL(str).host)
 
-    console.info('Connecting to MongoDB:', hosts.join(', '))
+    console.info('Connecting to MongoDB', { address: hosts.join(', ') })
 
     await client.connect()
 
@@ -119,9 +224,11 @@ class Client extends Connector {
    * @private
    * @return {Promise<string[]>}
    */
-  async resolveURLs () {
-    if (process.env.TOA_DEV === '1') {
-      return ['mongodb://developer:secret@localhost']
+  async resolveURLs() {
+    // Toa's own development stack is not on the conventional ports: the applications built on
+    // Toa are, and they share the machine. See CONTRIBUTING.md.
+    if (environment.get('TOA_DEV') === '1') {
+      return ['mongodb://developer:secret@localhost:31020']
     } else {
       return await resolve(ID, this.locator.id)
     }
@@ -131,29 +238,63 @@ class Client extends Connector {
    * @private
    * @return {string}
    */
-  resolveDB () {
-    if (process.env.TOA_CONTEXT !== undefined) {
-      return process.env.TOA_CONTEXT
-    }
+  resolveDB() {
+    const scope = environment.scope()
+    const length = Buffer.byteLength(scope)
 
-    if (process.env.TOA_DEV === '1') {
-      return 'toa-dev'
-    }
+    // MongoDB refuses the name only when something is first written, far from what caused it
+    if (length > MAX_DB_LENGTH)
+      throw new Error(
+        `Database name '${scope}' is ${length} bytes, and MongoDB takes no more than ` +
+          `${MAX_DB_LENGTH}: shorten TOA_CONTEXT or TOA_SUFFIX`
+      )
 
-    throw new Error('Environment variable TOA_CONTEXT is not defined')
+    return scope
   }
 }
 
-function getKey (db, urls) {
+/** what MongoDB takes for a database name, in bytes */
+const MAX_DB_LENGTH = 63
+
+function getKey(db, urls) {
   return db + ':' + urls.sort().join(' ')
 }
 
+/**
+ * Concurrent pods race to create the same collection, and losing that race is not an error.
+ */
+async function collection(db, name) {
+  try {
+    return await db.createCollection(name)
+  } catch (e) {
+    if (e.code !== ALREADY_EXISTS) throw e
+
+    return db.collection(name)
+  }
+}
+
+async function transactional(db) {
+  try {
+    const hello = await db.admin().command({ hello: 1 })
+
+    return hello.setName !== undefined || hello.msg === 'isdbgrid'
+  } catch (e) {
+    console.warn('MongoDB transaction support could not be determined', { error: e })
+
+    return false
+  }
+}
+
+/**
+ * `monitorCommands` is deliberately absent. It makes the driver materialize every reply
+ * eagerly to populate the monitoring event (`CommandSucceededEvent`), which defeats the
+ * lazy per-document deserialization a cursor exists for — a 100-document batch is then
+ * deserialized twice. `Storage` times its own calls instead.
+ */
 const OPTIONS = {
-  ignoreUndefined: true,
-  connectTimeoutMS: 0,
-  serverSelectionTimeoutMS: 0
+  ignoreUndefined: true
 }
 
 const ALREADY_EXISTS = 48
-
-exports.Client = Client
+const OUTBOX = '_outbox'
+const INBOX = '_inbox'

package/src/factory.js

@@ -1,16 +1,10 @@
-'use strict'
+import { Client } from './client.js'
+import { Storage } from './storage.js'
 
-const { Client } = require('./client')
-const { Collection } = require('./collection')
-const { Storage } = require('./storage')
+export class Factory {
+  storage(locator, entity, options = {}) {
+    const client = new Client(locator, options.outbox === true, options.inbox === true)
 
-class Factory {
-  storage (locator, entity) {
-    const client = new Client(locator)
-    const connection = new Collection(client)
-
-    return new Storage(connection, entity)
+    return new Storage(client, entity)
   }
 }
-
-exports.Factory = Factory