@toa.io/storages.mongodb 1.0.0-alpha.26 → 1.0.0-alpha.263

package/CHANGELOG.md

@@ -0,0 +1,32 @@
+# 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.263](https://github.com/toa-io/toa/compare/v1.0.0-alpha.262...v1.0.0-alpha.263) (2026-08-29)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.262](https://github.com/toa-io/toa/compare/v1.0.0-alpha.261...v1.0.0-alpha.262) (2026-08-28)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.259](https://github.com/toa-io/toa/compare/v1.0.0-alpha.258...v1.0.0-alpha.259) (2026-08-24)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb
+
+
+
+
+
+# [1.0.0-alpha.257](https://github.com/toa-io/toa/compare/v1.0.0-alpha.256...v1.0.0-alpha.257) (2026-08-24)
+
+**Note:** Version bump only for package @toa.io/storages.mongodb

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toa.io/storages.mongodb",
-  "version": "1.0.0-alpha.26",
+  "version": "1.0.0-alpha.263",
   "description": "Toa MongoDB Storage Connector",
   "author": "temich <tema.gurtovoy@gmail.com>",
   "homepage": "https://github.com/toa-io/toa#readme",
@@ -19,13 +19,13 @@
     "test": "echo \"Error: run tests from root\" && exit 1"
   },
   "dependencies": {
-    "@toa.io/console": "1.0.0-alpha.26",
-    "@toa.io/conveyor": "1.0.0-alpha.26",
-    "@toa.io/core": "1.0.0-alpha.26",
-    "@toa.io/generic": "1.0.0-alpha.26",
-    "@toa.io/pointer": "1.0.0-alpha.26",
-    "mongodb": "6.3.0",
+    "@toa.io/conveyor": "1.0.0-alpha.254",
+    "@toa.io/core": "1.0.0-alpha.262",
+    "@toa.io/generic": "1.0.0-alpha.254",
+    "@toa.io/pointer": "1.0.0-alpha.263",
+    "mongodb": "7.2.0",
+    "openspan": "1.0.0-alpha.254",
     "saslprep": "1.0.3"
   },
-  "gitHead": "0ddd9260762d1fe2b8be88eb7b9725e3f7838299"
+  "gitHead": "165daa284bb7a75907a593fde359e3553b083417"
 }

package/readme.md

@@ -0,0 +1,16 @@
+# MongoDB Storage
+
+## Tracing
+
+Commands are recorded as `client` spans within the trace of the current invocation,
+using the [driver command monitoring events](https://www.mongodb.com/docs/drivers/node/current/monitoring/command-monitoring/).
+
+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.

package/src/client.js

@@ -6,10 +6,12 @@
  * @typedef {import('@toa.io/core').Locator} Locator
  */
 
+const { console } = require('openspan')
 const { Connector } = require('@toa.io/core')
 const { resolve } = require('@toa.io/pointer')
 const { ID } = require('./deployment')
 const { MongoClient } = require('mongodb')
+const { monitor } = require('./monitoring')
 
 /**
  * @type {Record<string, Promise<Instance>>}
@@ -17,6 +19,8 @@ const { MongoClient } = require('mongodb')
 const INSTANCES = {}
 
 class Client extends Connector {
+  name
+
   /**
    * @public
    * @type {import('mongodb').Collection}
@@ -48,6 +52,7 @@ class Client extends Connector {
     super()
 
     this.locator = locator
+    this.name = locator.lowercase
   }
 
   /**
@@ -58,11 +63,14 @@ class Client extends Connector {
   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++
@@ -70,13 +78,13 @@ class Client extends Connector {
     const db = this.instance.client.db(dbname)
 
     try {
-      this.collection = await db.createCollection(collname)
+      this.collection = await db.createCollection(this.name)
     } catch (e) {
       if (e.code !== ALREADY_EXISTS) {
         throw e
       }
 
-      this.collection = db.collection(collname)
+      this.collection = db.collection(this.name)
     }
   }
 
@@ -105,7 +113,9 @@ class Client extends Connector {
     const client = new MongoClient(urls.join(','), OPTIONS)
     const hosts = urls.map((str) => new URL(str).host)
 
-    console.info('Connecting to MongoDB:', hosts.join(', '))
+    monitor(client)
+
+    console.info('Connecting to MongoDB', { address: hosts.join(', ') })
 
     await client.connect()
 
@@ -150,8 +160,7 @@ function getKey (db, urls) {
 
 const OPTIONS = {
   ignoreUndefined: true,
-  connectTimeoutMS: 0,
-  serverSelectionTimeoutMS: 0
+  monitorCommands: true
 }
 
 const ALREADY_EXISTS = 48

package/src/factory.js

@@ -1,15 +1,13 @@
 'use strict'
 
 const { Client } = require('./client')
-const { Collection } = require('./collection')
 const { Storage } = require('./storage')
 
 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)
   }
 }
 

package/src/monitoring.js

@@ -0,0 +1,87 @@
+'use strict'
+
+const { create, current, record } = require('openspan')
+
+/**
+ * Records MongoDB commands as client spans using the driver command monitoring events.
+ * Requires the client to be created with `monitorCommands: true`.
+ *
+ * Commands executed outside of a sampled trace context are not recorded.
+ *
+ * @param {import('mongodb').MongoClient} client
+ */
+function monitor (client) {
+  /** @type {Map<number, { context: object, time: number, attributes: object }>} */
+  const pending = new Map()
+
+  client.on('commandStarted', (event) => {
+    if (IGNORE.has(event.commandName))
+      return
+
+    const parent = current()
+
+    if (parent === undefined || !parent.sampled)
+      return
+
+    if (pending.size >= CAPACITY) // should never happen
+      pending.delete(pending.keys().next().value)
+
+    // https://opentelemetry.io/docs/specs/semconv/database/mongodb/
+    const attributes = {
+      'db.system': 'mongodb',
+      'db.namespace': event.databaseName,
+      'db.operation.name': event.commandName
+    }
+
+    const collection = event.command?.[event.commandName]
+
+    if (typeof collection === 'string')
+      attributes['db.collection.name'] = collection
+
+    pending.set(event.requestId, {
+      context: create(parent),
+      time: Date.now(),
+      attributes
+    })
+  })
+
+  client.on('commandSucceeded', (event) => complete(event))
+  client.on('commandFailed', (event) => complete(event, 'error'))
+
+  function complete (event, status) {
+    const started = pending.get(event.requestId)
+
+    if (started === undefined)
+      return
+
+    pending.delete(event.requestId)
+
+    const collection = started.attributes['db.collection.name']
+
+    const span = {
+      name: collection === undefined
+        ? event.commandName
+        : `${event.commandName} ${collection}`,
+      traceId: started.context.traceId,
+      spanId: started.context.spanId,
+      parentId: started.context.parentId,
+      kind: 'client',
+      time: started.time,
+      duration: event.duration,
+      attributes: started.attributes,
+      service: started.context.service
+    }
+
+    if (status !== undefined)
+      span.status = status
+
+    record(span)
+  }
+}
+
+// internal commands, including authentication and connection management
+const IGNORE = new Set(['hello', 'ismaster', 'ping', 'endSessions', 'saslStart', 'saslContinue'])
+
+const CAPACITY = 1024
+
+exports.monitor = monitor

package/src/record.js

@@ -1,29 +1,16 @@
 'use strict'
 
-/**
- * @param {toa.core.storages.Record} entity
- * @returns {toa.mongodb.Record}
- */
-const to = (entity) => {
-  const {
-    id,
-    ...rest
-  } = entity
+function to (entity) {
+  const { id, ...rest } = entity
 
   return /** @type {toa.mongodb.Record} */ { _id: id, ...rest }
 }
 
-/**
- * @param {toa.mongodb.Record} record
- * @returns {toa.core.storages.Record}
- */
-const from = (record) => {
-  if (record === undefined || record === null) return null
+function from (record) {
+  if (record === undefined || record === null)
+    return null
 
-  const {
-    _id,
-    ...rest
-  } = record
+  const { _id, ...rest } = record
 
   return { id: _id, ...rest }
 }

package/src/storage.js

@@ -1,58 +1,94 @@
 'use strict'
 
-const {
-  Connector,
-  exceptions
-} = require('@toa.io/core')
-
+const { Connector, exceptions } = require('@toa.io/core')
+const { console } = require('openspan')
 const { translate } = require('./translate')
-const {
-  to,
-  from
-} = require('./record')
+const { to, from } = require('./record')
+const { ReturnDocument } = require('mongodb')
 
 class Storage extends Connector {
-  #connection
+  #client
+
+  /** @type {import('mongodb').Collection} */
+  #collection
   #entity
 
-  constructor (connection, entity) {
+  constructor (client, entity) {
     super()
 
-    this.#connection = connection
+    this.#client = client
     this.#entity = entity
 
-    this.depends(connection)
+    this.depends(client)
+  }
+
+  get raw () {
+    return this.#collection
   }
 
   async open () {
+    this.#collection = this.#client.collection
+
     await this.index()
   }
 
   async get (query) {
-    const {
-      criteria,
-      options
-    } = translate(query)
+    const { criteria, options } = translate(query)
+
+    // identity lookups must return deleted records, so that callers
+    // can tell a deleted entity from a missing one
+    if (query?.id === undefined && query?.options?.deleted !== true)
+      criteria._deleted = null
 
-    const record = await this.#connection.get(criteria, options)
+    this.debug('findOne', { criteria, options })
+
+    const record = await this.#collection.findOne(criteria, options)
 
     return from(record)
   }
 
   async find (query) {
-    const {
-      criteria,
-      options
-    } = translate(query)
+    const { criteria, options, sample } = translate(query)
+
+    if (query?.options?.deleted !== true)
+      criteria._deleted = null
+
+    let cursor
+
+    if (sample === undefined) {
+      this.debug('find', { criteria, options })
+
+      cursor = this.#collection.find(criteria, options)
+    } else {
+      const pipeline = toPipeline(criteria, options, sample)
 
-    const recordset = await this.#connection.find(criteria, options)
+      this.debug('aggregate', { pipeline })
+
+      cursor = this.#collection.aggregate(pipeline)
+    }
+
+    const recordset = await cursor.toArray()
 
     return recordset.map((item) => from(item))
   }
 
+  async stream (query = undefined) {
+    const { criteria, options } = translate(query)
+
+    if (query?.options?.deleted !== true)
+      criteria._deleted = null
+
+    this.debug('find (stream)', { criteria, options })
+
+    return this.#collection.find(criteria, options).stream({ transform: from })
+  }
+
   async add (entity) {
     const record = to(entity)
-    const result = await this.#connection.add(record)
+
+    this.debug('insertOne', { record })
+
+    const result = await this.#collection.insertOne(record)
 
     return result.acknowledged
   }
@@ -62,41 +98,92 @@ class Storage extends Connector {
       _id: entity.id,
       _version: entity._version - 1
     }
-    const result = await this.#connection.replace(criteria, to(entity))
+
+    const record = to(entity)
+
+    this.debug('findOneAndReplace', { criteria, record })
+
+    const result = await this.#collection.findOneAndReplace(criteria, record)
 
     return result !== null
   }
 
-  async store (entity) {
+  async store (entity, attempt = 0) {
     try {
-      if (entity._version === 1) {
+      if (entity._version === 1)
         return await this.add(entity)
-      } else {
+      else
         return await this.set(entity)
-      }
     } catch (error) {
-      if (error.code === ERR_DUPLICATE_KEY) {
+      console.error('MongoDB error', error)
+
+      const retry = await retriable(error, attempt)
+
+      if (retry)
+        return await this.store(entity, attempt + 1)
+      else
+        return false
+    }
+  }
+
+  async massStore (entities, attempt = 0) {
+    if (entities.length === 0)
+      return true
 
-        const id = error.keyPattern === undefined
-          ? error.message.includes(' index: _id_ ') // AWS DocumentDB
-          : error.keyPattern._id === 1
+    const operations = entities.map((entity) => {
+      const record = to(entity)
 
-        if (id) {
-          return false
-        } else {
-          throw new exceptions.DuplicateException()
+      if (entity._version === 1) {
+        const { _version, ...rest } = record
+
+        return { // upsert in required when document is deleted
+          updateOne: {
+            filter: { _id: entity.id },
+            update: {
+              $set: {    
+                ...rest,
+                _deleted: null
+              },
+              $inc: { _version: 1 },
+            },
+            upsert: true
+          } 
         }
-      } else {
-        throw error
-      }
+      } else
+        return {  
+          replaceOne: { 
+            filter: { _id: entity.id, _version: entity._version - 1 },
+            replacement: record
+          }
+        }
+    })
+
+    const client = this.#client.instance.client
+
+    try {
+      await client.withSession(async (session) => {
+        await session.withTransaction(async () => {
+          this.debug('bulkWrite', { operations: operations.length })
+
+          await this.#collection.bulkWrite(operations, { session })
+        })
+      })
+
+      return true
+    } catch (error) {
+      console.error('MongoDB error', error)
+
+      const retry = await retriable(error, attempt)
+
+      if (retry)
+        return await this.massStore(entities, attempt + 1)
+      else
+        return false
     }
   }
 
   async upsert (query, changeset) {
-    const {
-      criteria,
-      options
-    } = translate(query)
+    const { criteria, options } = translate(query)
 
     if (!('_deleted' in changeset) || changeset._deleted === null) {
       delete criteria._deleted
@@ -108,20 +195,50 @@ class Storage extends Connector {
       $inc: { _version: 1 }
     }
 
-    options.returnDocument = 'after'
+    options.returnDocument = ReturnDocument.AFTER
+
+    this.debug('findOneAndUpdate', { criteria, update, options })
 
-    const result = await this.#connection.update(criteria, update, options)
+    const result = await this.#collection.findOneAndUpdate(criteria, update, options)
 
     return from(result)
   }
 
+  async ensure (query, properties, state) {
+    let { criteria, options } = translate(query)
+
+    if (query === undefined)
+      criteria = properties
+
+    const update = { $setOnInsert: to(state) }
+
+    options.upsert = true
+    options.returnDocument = ReturnDocument.AFTER
+
+    console.debug('Database query', { collection: this.#collection.collectionName, method: 'findOneAndUpdate', criteria, update, options })
+
+    try {
+      const result = await this.#collection.findOneAndUpdate(criteria, update, options)
+
+      if (result._deleted !== undefined && result._deleted !== null)
+        return null
+      else
+        return from(result)
+    } catch (error) {
+      if (error.code === ERR_DUPLICATE_KEY)
+        throw new exceptions.DuplicateException(this.#client.name)
+      else
+        throw error
+    }
+  }
+
   async index () {
     const indexes = []
 
     if (this.#entity.unique !== undefined) {
       for (const [name, fields] of Object.entries(this.#entity.unique)) {
-        const sparse = this.checkFields(fields)
-        const unique = await this.uniqueIndex(name, fields, sparse)
+        const optional = this.getOptional(fields)
+        const unique = await this.uniqueIndex(name, fields, optional)
 
         indexes.push(unique)
       }
@@ -131,23 +248,24 @@ class Storage extends Connector {
       for (const [suffix, declaration] of Object.entries(this.#entity.index)) {
         const name = 'index_' + suffix
         const fields = Object.fromEntries(Object.entries(declaration)
-          .map(([name, type]) => [name, INDEX_TYPES[type]]))
+          .map(([name, type]) => [name, INDEX_TYPES[type] ?? type]))
 
-        const sparse = this.checkFields(Object.keys(fields))
+        const optional = this.getOptional(Object.keys(fields))
+        const options = { name, sparse: optional.length > 0 }
 
-        await this.#connection.index(fields, {
-          name,
-          sparse
-        })
+        console.info('Creating index', { fields, options })
+
+        await this.#collection.createIndex(fields, options)
+          .catch((e) => console.warn('MongoDB index creation failed', { collection: this.#collection.collectionName, name, fields, error: e }))
 
         indexes.push(name)
       }
     }
 
-    await this.removeObsolete(indexes)
+    await this.removeObsoleteIndexes(indexes)
   }
 
-  async uniqueIndex (name, properties, sparse = false) {
+  async uniqueIndex (name, properties, optional) {
     const fields = properties.reduce((acc, property) => {
       acc[property] = 1
       return acc
@@ -155,48 +273,80 @@ class Storage extends Connector {
 
     name = 'unique_' + name
 
-    await this.#connection.index(fields, {
-      name,
-      unique: true,
-      sparse
-    })
+    const options = { name, unique: true }
+
+    if (optional.length > 0)
+      options.partialFilterExpression = Object.fromEntries(optional.map((field) => [field, { $exists: true }]))
+
+    console.info('Creating unique index', { name, fields, options })
+
+    await this.#collection.createIndex(fields, options)
+      .catch((e) => console.warn('MongoDB unique index creation failed', 
+        { collection: this.#collection.collectionName, name, fields, error: e }))
 
     return name
   }
 
-  async removeObsolete (desired) {
-    const current = await this.#connection.indexes()
+  async removeObsoleteIndexes (desired) {
+    const current = await this.getCurrentIndexes()
     const obsolete = current.filter((name) => !desired.includes(name))
 
     if (obsolete.length > 0) {
-      console.info(`Remove obsolete indexes: [${obsolete.join(', ')}]`)
+      console.info('Removing obsolete indexes', { collection: this.#collection.collectionName, indexes: obsolete.join(', ') })
 
-      await this.#connection.dropIndexes(obsolete)
+      await Promise.all(obsolete.map((name) => this.#collection.dropIndex(name)))
     }
   }
 
-  checkFields (fields) {
+  async getCurrentIndexes () {
+    try {
+      const array = await this.#collection.listIndexes().toArray()
+
+      return array.map(({ name }) => name).filter((name) => name !== '_id_')
+    } catch {
+      return []
+    }
+  }
+
+  getOptional (fields) {
     const optional = []
 
-    for (const field of fields) {
-      if (!(field in this.#entity.schema.properties)) {
+    for (const field of fields) {      
+      if (!field.includes('.') && !(field in this.#entity.schema.properties))
         throw new Error(`Index field '${field}' is not defined.`)
-      }
 
-      if (!this.#entity.schema.required?.includes(field)) {
+      if (!this.#entity.schema.required?.includes(field))
         optional.push(field)
-      }
     }
 
-    if (optional.length > 0) {
-      console.info(`Index fields [${optional.join(', ')}] are optional, creating sparse index.`)
+    return optional
+  }
 
-      return true
-    } else {
-      return false
-    }
+  debug (method, attributes) {
+    console.debug('MongoDB query', {
+      collection: this.#collection.collectionName,
+      method,
+      ...attributes
+    })
   }
+}
+
+function toPipeline (criteria, options, sample) {
+  const pipeline = []
+
+  if (criteria !== undefined)
+    pipeline.push({ $match: criteria })
 
+  if (sample !== undefined)
+    pipeline.push({ $sample: { size: sample } })
+
+  if (options?.sort !== undefined)
+    pipeline.push({ $sort: options.sort })
+
+  if (options?.projection !== undefined)
+    pipeline.push({ $project: options.projection })
+
+  return pipeline
 }
 
 const INDEX_TYPES = {
@@ -207,4 +357,29 @@ const INDEX_TYPES = {
 
 const ERR_DUPLICATE_KEY = 11000
 
+async function retriable (error, attempt) {
+  if (error.code === ERR_DUPLICATE_KEY) {
+    const id = error.keyPattern === undefined
+      ? error.message.includes(' index: _id_ ') // AWS DocumentDB
+      : error.keyPattern._id === 1
+
+    if (id)
+      return false
+    else
+      throw new exceptions.DuplicateException()
+  } else if (error.cause?.code === 'ECONNREFUSED') {
+    if (attempt === LAST_ATTEMPT)
+      throw error
+
+    const timeout = 1000 + 500 * attempt
+
+    await new Promise((resolve) => setTimeout(resolve, timeout))
+
+    return true
+  } else
+    throw error
+}
+
+const LAST_ATTEMPT = 9
+
 exports.Storage = Storage

package/src/translate.js

@@ -9,18 +9,21 @@ const parse = { ...require('./translate/criteria'), ...require('./translate/opti
 const translate = (query) => {
   const result = {
     criteria: query?.criteria === undefined ? {} : parse.criteria(query.criteria),
-    options: query?.options === undefined ? {} : parse.options(query.options)
+    options: query?.options === undefined ? {} : parse.options(query.options),
+    sample: query?.options?.sample
   }
 
-  if (query?.id !== undefined) {
+  if (query?.id !== undefined)
     result.criteria._id = query.id
-  }
 
-  if (query?.version !== undefined) {
+  if (query?.ids !== undefined)
+    result.criteria._id = { $in: query.ids }
+
+  if (query?.version !== undefined)
     result.criteria._version = query.version
-  }
 
-  result.criteria._deleted = null
+  if (query?.search !== undefined)
+    result.criteria.$text = { $search: query.search }
 
   return result
 }

package/test/monitoring.test.js

@@ -0,0 +1,99 @@
+'use strict'
+
+const { EventEmitter } = require('node:events')
+const { create, exporting, run, traces } = require('openspan')
+const { monitor } = require('../src/monitoring')
+
+let client
+let spans
+
+beforeEach(() => {
+  client = new EventEmitter()
+  spans = []
+
+  exporting([{ export: (span) => spans.push(span) }])
+  monitor(client)
+})
+
+afterEach(() => {
+  traces()
+})
+
+const started = {
+  requestId: 1,
+  commandName: 'find',
+  databaseName: 'toa-dev',
+  command: { find: 'pots', filter: {} }
+}
+
+const succeeded = { requestId: 1, commandName: 'find', duration: 1.5 }
+const failed = { requestId: 1, commandName: 'find', duration: 0.5, failure: 'boom' }
+
+it('should record a command as a client span', () => {
+  const context = create()
+
+  run(context, () => client.emit('commandStarted', started))
+  client.emit('commandSucceeded', succeeded)
+
+  expect(spans).toHaveLength(1)
+
+  expect(spans[0]).toMatchObject({
+    name: 'find pots',
+    traceId: context.traceId,
+    parentId: context.spanId,
+    kind: 'client',
+    duration: 1.5,
+    attributes: {
+      'db.system': 'mongodb',
+      'db.namespace': 'toa-dev',
+      'db.operation.name': 'find',
+      'db.collection.name': 'pots'
+    }
+  })
+
+  expect(spans[0].status).toBeUndefined()
+})
+
+it('should record failed commands with error status', () => {
+  run(create(), () => client.emit('commandStarted', started))
+  client.emit('commandFailed', failed)
+
+  expect(spans).toHaveLength(1)
+  expect(spans[0].status).toBe('error')
+})
+
+it('should not record commands outside of a trace context', () => {
+  client.emit('commandStarted', started)
+  client.emit('commandSucceeded', succeeded)
+
+  expect(spans).toHaveLength(0)
+})
+
+it('should not record commands of unsampled traces', () => {
+  const context = { ...create(), sampled: false }
+
+  run(context, () => client.emit('commandStarted', started))
+  client.emit('commandSucceeded', succeeded)
+
+  expect(spans).toHaveLength(0)
+})
+
+it('should ignore internal commands', () => {
+  run(create(), () => {
+    client.emit('commandStarted', { ...started, commandName: 'hello', command: { hello: 1 } })
+  })
+
+  client.emit('commandSucceeded', { ...succeeded, commandName: 'hello' })
+
+  expect(spans).toHaveLength(0)
+})
+
+it('should name spans without a collection by the command', () => {
+  const start = { ...started, commandName: 'listIndexes', command: { listIndexes: 1 } }
+
+  run(create(), () => client.emit('commandStarted', start))
+  client.emit('commandSucceeded', { ...succeeded, commandName: 'listIndexes' })
+
+  expect(spans).toHaveLength(1)
+  expect(spans[0].name).toBe('listIndexes')
+})

package/test/record.test.js

@@ -47,19 +47,4 @@ describe('from', () => {
       _version: 0
     })
   })
-
-  it('should not modify argument', () => {
-    /** @type {toa.mongodb.Record} */
-    const record = {
-      _id: '1',
-      _version: 0
-    }
-
-    from(record)
-
-    expect(record).toStrictEqual({
-      _id: '1',
-      _version: 0
-    })
-  })
 })

package/test/storage.test.js

@@ -0,0 +1,73 @@
+'use strict'
+
+const { Storage } = require('../src/storage')
+
+let collection
+let storage
+
+beforeEach(async () => {
+  collection = {
+    collectionName: 'test',
+    findOne: jest.fn(async () => null),
+    find: jest.fn(() => ({ stream: () => null }))
+  }
+
+  const client = {
+    collection,
+    link: () => null
+  }
+
+  storage = new Storage(client, { schema: { properties: {} } })
+
+  await storage.open()
+})
+
+describe('get', () => {
+  it('should filter deleted', async () => {
+    await storage.get({})
+
+    expect(collection.findOne).toHaveBeenCalledWith({ _deleted: null }, {})
+  })
+
+  it('should filter deleted with sort', async () => {
+    await storage.get({ options: { sort: [['_created', 'desc']] } })
+
+    expect(collection.findOne)
+      .toHaveBeenCalledWith({ _deleted: null }, { sort: [['_created', -1]] })
+  })
+
+  it('should not filter deleted if queried by id', async () => {
+    const id = 'bcb6780f50e243348cad40ed6b5ef575'
+
+    await storage.get({ id })
+
+    expect(collection.findOne).toHaveBeenCalledWith({ _id: id }, {})
+  })
+
+  it('should not filter deleted if requested', async () => {
+    await storage.get({ options: { deleted: true } })
+
+    expect(collection.findOne).toHaveBeenCalledWith({}, {})
+  })
+})
+
+describe('stream', () => {
+  it('should filter deleted', async () => {
+    await storage.stream()
+
+    expect(collection.find).toHaveBeenCalledWith({ _deleted: null }, {})
+  })
+
+  it('should filter deleted with sort', async () => {
+    await storage.stream({ options: { sort: [['_created', 'desc']] } })
+
+    expect(collection.find)
+      .toHaveBeenCalledWith({ _deleted: null }, { sort: [['_created', -1]] })
+  })
+
+  it('should not filter deleted if requested', async () => {
+    await storage.stream({ options: { deleted: true } })
+
+    expect(collection.find).toHaveBeenCalledWith({}, {})
+  })
+})

package/src/collection.js

@@ -1,69 +0,0 @@
-'use strict'
-
-const { Connector } = require('@toa.io/core')
-
-class Collection extends Connector {
-  #client
-  #collection
-
-  constructor (client) {
-    super()
-
-    this.#client = client
-
-    this.depends(client)
-  }
-
-  async open () {
-    this.#collection = this.#client.collection
-  }
-
-  /** @hot */
-  async get (query, options) {
-    return /** @type {toa.mongodb.Record} */ this.#collection.findOne(query, options)
-  }
-
-  /** @hot */
-  async find (query, options) {
-    const cursor = this.#collection.find(query, options)
-
-    return cursor.toArray()
-  }
-
-  /** @hot */
-  async add (record) {
-    return await this.#collection.insertOne(record)
-  }
-
-  /** @hot */
-  async replace (query, record, options) {
-    return await this.#collection.findOneAndReplace(query, record, options)
-  }
-
-  /** @hot */
-  async update (query, update, options) {
-    return this.#collection.findOneAndUpdate(query, update, options)
-  }
-
-  async index (keys, options) {
-    return this.#collection.createIndex(keys, options)
-  }
-
-  async indexes () {
-    try {
-      const array = await this.#collection.listIndexes().toArray()
-
-      return array.map(({ name }) => name).filter((name) => name !== '_id_')
-    } catch {
-      return []
-    }
-  }
-
-  async dropIndexes (names) {
-    const all = names.map((name) => this.#collection.dropIndex(name))
-
-    return Promise.all(all)
-  }
-}
-
-exports.Collection = Collection