@sap/cds 6.6.1 → 6.7.0

package/libx/_runtime/audit/Service.js

@@ -1,13 +1,11 @@
 const cds = require('../cds')
 const OutboxService = require('../messaging/Outbox')
-
 const v2utils = require('./utils/v2')
-
 const ANONYMOUS = 'anonymous'
 
 const _getTenantAndUser = () => ({
-  user: (cds.context && cds.context.user && cds.context.user.id) || ANONYMOUS,
-  tenant: cds.context && cds.context.tenant
+  user: cds.context?.user?.id ?? ANONYMOUS,
+  tenant: cds.context?.tenant
 })
 
 module.exports = class AuditLogService extends OutboxService {
@@ -82,11 +80,8 @@ module.exports = class AuditLogService extends OutboxService {
     }
 
     // write the logs
-    await Promise.all(
-      entries.map(entry => {
-        v2utils.sendDataAccessLog(entry).catch(err => errors.push(err))
-      })
-    )
+    await Promise.all(entries.map(entry => v2utils.sendDataAccessLog(entry).catch(err => errors.push(err))))
+
     if (errors.length) {
       throw errors.length === 1 ? errors[0] : Object.assign(new Error('MULTIPLE_ERRORS'), { details: errors })
     }
@@ -108,6 +103,7 @@ module.exports = class AuditLogService extends OutboxService {
         v2utils.sendDataModificationLog(entry).catch(err => errors.push(err))
       })
     )
+
     if (errors.length) {
       throw errors.length === 1 ? errors[0] : Object.assign(new Error('MULTIPLE_ERRORS'), { details: errors })
     }
@@ -124,10 +120,12 @@ module.exports = class AuditLogService extends OutboxService {
           tenant = parsed.tenant
           delete parsed.tenant
         }
+
         if (parsed.user && typeof parsed.user === 'string') {
           user = parsed.user
           delete parsed.user
         }
+
         data = JSON.stringify(parsed)
       } catch (e) {
         // ignore
@@ -157,6 +155,7 @@ module.exports = class AuditLogService extends OutboxService {
         v2utils.sendConfigChangeLog(entry).catch(err => errors.push(err))
       })
     )
+
     if (errors.length) {
       throw errors.length === 1 ? errors[0] : Object.assign(new Error('MULTIPLE_ERRORS'), { details: errors })
     }

package/libx/_runtime/audit/generic/personal/index.js

@@ -8,7 +8,7 @@ const {
 const { auditAccessHandler } = require('./access')
 
 exports.impl = cds.service.impl(function () {
-  if (!cds.db) return cds.on('connect', srv => srv instanceof cds.DatabaseService && exports.impl.call(this))
+  if (!cds.db) return cds.on('connect', srv => srv.isDatabaseService && exports.impl.call(this))
   // REVISIT: diff() doesn't work in srv after phase but foreign key propagation has not yet taken place in srv before phase
   //          -> calc diff in db layer and store in audit data structure at context
   //          -> REVISIT for GA: clear req._.partialPersistentState?

package/libx/_runtime/audit/generic/personal/utils.js

@@ -7,7 +7,7 @@ const WRITE = { CREATE: 1, UPDATE: 1, DELETE: 1 }
 const getMapKeyForCurrentRequest = req => {
   // running in srv or db layer? -> srv's req.query used as key of diff and logs maps at req.context
   // REVISIT: req._tx should not be used like that!
-  return req.tx instanceof cds.DatabaseService ? req._.query : req.query
+  return req.tx.isDatabaseService ? req._.query : req.query
 }
 
 const getRootEntity = element => {

package/libx/_runtime/audit/utils/v2.js

@@ -3,26 +3,23 @@ const LOG = cds.log('audit-log')
 
 const { getObjectAndDataSubject, getAttributeToLog } = require('./log')
 
-function connect(credentials) {
-  return new Promise((resolve, reject) => {
-    let auditLogging
-    try {
-      // eslint-disable-next-line cds/no-missing-dependencies -- needs to be added by app dev
-      auditLogging = require('@sap/audit-logging')
-    } catch (e) {
-      // not able to require lib -> no audit logging ootb
-      return resolve()
-    }
-    try {
-      auditLogging.v2(credentials, function (err, auditLog) {
-        if (err) return reject(err)
-        resolve(auditLog)
-      })
-    } catch (e) {
-      LOG._warn && LOG.warn('Unable to initialize audit-logging client with error:', e)
-      return resolve()
-    }
-  })
+async function connect(credentials) {
+  let auditLogging
+
+  try {
+    // eslint-disable-next-line cds/no-missing-dependencies -- needs to be added by app dev
+    auditLogging = require('@sap/audit-logging')
+  } catch (error) {
+    // not able to require lib -> no audit logging ootb
+    return Promise.resolve()
+  }
+
+  try {
+    return await auditLogging.v2(credentials)
+  } catch (error) {
+    LOG._warn && LOG.warn('Unable to initialize audit-logging client with error:', error)
+    return Promise.resolve()
+  }
 }
 
 function sendDataAccessLog(entry) {

package/libx/_runtime/cds-services/adapter/odata-v4/ODataRequest.js

@@ -255,6 +255,8 @@ class ODataRequest extends cds.Request {
       }
     })
 
+    Object.defineProperty(this, '_isOData', { value: true })
+
     /*
      * req.validateEtag()
      */

package/libx/_runtime/cds-services/adapter/odata-v4/handlers/read.js

@@ -21,7 +21,8 @@ const { isStreaming, getStreamProperties } = require('../utils/stream')
 const { resolveStructuredName } = require('../utils/handlerUtils')
 const getError = require('../../../../common/error')
 const { getSapMessages } = require('../../../../common/error/frontend')
-const { getMaxPageSize } = require('../../../../common/utils/page')
+const { getPageSize, commonGenericPaging } = require('../../../../common/generic/paging')
+const { handler: commonGenericSorting } = require('../../../../common/generic/sorting')
 
 /**
  * Checks whether a bound function or function import is invoked.
@@ -255,8 +256,12 @@ const _reliablePagingPossible = req => {
   if (req.target._isDraftEnabled) return false
   if (cds.context?.http.req.query.$apply) return false
   if (req.query.SELECT.limit.offset?.val ?? req.query.SELECT.limit.offset > 0) return false
-  if (req.query.SELECT.orderBy.some(o => !o.ref)) return false
-  return req.query.SELECT.orderBy.every(o => req.query.SELECT.columns.some(c => o.ref[0] === c.ref[0]))
+  if (req.query.SELECT.orderBy?.some(o => !o.ref)) return false
+  return (
+    !req.query.SELECT.columns ||
+    req.query.SELECT.columns.some(c => c === '*' || c.ref?.[0] === '*') ||
+    req.query.SELECT.orderBy?.every(o => req.query.SELECT.columns?.some(c => o.ref[0] === c.ref?.[0]))
+  )
 }
 
 /**
@@ -270,6 +275,8 @@ const _reliablePagingPossible = req => {
  */
 // eslint-disable-next-line complexity
 const _readCollection = async (tx, req, odataReq) => {
+  commonGenericPaging(req)
+  commonGenericSorting(req)
   const result = (await tx.dispatch(req)) || []
   if (Array.isArray(req.query)) {
     const adjustedResult = []
@@ -288,7 +295,7 @@ const _readCollection = async (tx, req, odataReq) => {
   } else if (req.query.SELECT.count && !('$count' in result)) result.$count = 0
 
   const limit = Array.isArray(req.query)
-    ? getMaxPageSize(req.query[0]._target)
+    ? getPageSize(req.query[0]._target).max
     : req.query.SELECT.limit && req.query.SELECT.limit.rows && req.query.SELECT.limit.rows.val
   const top = odataReq.getUriInfo().getQueryOption(QueryOptions.TOP)
   if (limit && limit === result.length && limit !== top && !('$nextLink' in result)) {

package/libx/_runtime/cds-services/adapter/odata-v4/handlers/update.js

@@ -11,7 +11,6 @@ const { isReturnMinimal } = require('../utils/handlerUtils')
 const { readAfterWrite } = require('../utils/readAfterWrite')
 const { toODataResult, postProcess, postProcessMinimal } = require('../utils/result')
 const { hasOmitValuesPreference } = require('../utils/omitValues')
-const { isStreaming } = require('../utils/stream')
 
 const { getSapMessages } = require('../../../../common/error/frontend')
 

package/libx/_runtime/cds-services/adapter/odata-v4/odata-to-cqn/readToCQN.js

@@ -36,7 +36,7 @@ const expandToCQN = require('./expandToCQN')
 const { resolveStructuredName } = require('../utils/handlerUtils')
 const { isStreaming } = require('../utils/stream')
 
-const { getMaxPageSize, getDefaultPageSize } = require('../../../../common/utils/page')
+const { getPageSize } = require('../../../../common/generic/paging')
 const { isAsteriskColumn } = require('../../../../common/utils/rewriteAsterisks')
 
 const { ensureUnlocalized } = require('../../../../fiori/utils/handler')
@@ -139,9 +139,9 @@ const _apply = (uriInfo, queryOptions, entity, model) => {
 
 const _topSkip = (queryOptions, target, cqn) => {
   if (queryOptions && (queryOptions.$top || queryOptions.$skip)) {
-    const top = queryOptions.$top ? parseInt(queryOptions.$top) : getDefaultPageSize(target)
+    const top = queryOptions.$top ? parseInt(queryOptions.$top) : getPageSize(target).default
     const skip = parseInt(queryOptions.$skip || 0)
-    cqn.limit(Math.min(top, getMaxPageSize(target)), skip)
+    cqn.limit(Math.min(top, getPageSize(target).max), skip)
   }
 }
 

package/libx/_runtime/cds-services/adapter/odata-v4/utils/data.js

@@ -9,7 +9,7 @@ const { deepCopy } = require('../../../../common/utils/copy')
 const { getSegmentKeyValue } = require('../odata-to-cqn/utils')
 
 const _isFunctionInvocation = req =>
-  req.getUriInfo().getLastSegment().getFunction || req.getUriInfo().getLastSegment().getFunctionImport
+  req.getUriInfo().getLastSegment().getFunction() || req.getUriInfo().getLastSegment().getFunctionImport()
 
 const _addStructuredProperties = ([structName, property, ...nestedProperties], paramData, value) => {
   paramData[structName] = paramData[structName] || {}

package/libx/_runtime/cds-services/adapter/odata-v4/utils/metaInfo.js

@@ -94,14 +94,14 @@ const _columnsFromQuery = (columns, target, options) => {
 }
 
 const _processFn = columns => {
-  return ({ row, key, element, pathSegments }) => {
+  return ({ row, key, element, pathSegmentsInfo }) => {
     if (!(key in row) || row[key] === null) return
     let cur = columns
     if (element.parent._isStructured) {
-      const prefix = pathSegments.join('/')
+      const prefix = pathSegmentsInfo.join('/')
       key = `${prefix}/${key}`
     } else {
-      for (let p of pathSegments) {
+      for (let p of pathSegmentsInfo) {
         if (!cur[p]) cur[p] = {}
         cur = cur[p]
       }
@@ -116,7 +116,7 @@ const _columnsFromData = (data, definition, service) => {
   if (!template || !template.elements.size) return ''
   const arrayData = Array.isArray(data) ? data : data ? [data] : []
   for (const row of arrayData) {
-    templateProcessor({ processFn: _processFn(columns), row, template, pathOptions: { pathSegments: [] } })
+    templateProcessor({ processFn: _processFn(columns), row, template, pathOptions: { pathSegmentsInfo: [] } })
   }
   return _stringifyColumnsFromData(columns)
 }

package/libx/_runtime/cds-services/adapter/odata-v4/utils/oDataConfiguration.js

@@ -1,4 +1,4 @@
-const { getMaxPageSize } = require('../../../../common/utils/page')
+const { getPageSize } = require('../../../../common/generic/paging')
 const { findCsnTargetFor } = require('../../../../common/utils/csn')
 
 const _getEntitySets = (edm, namespace) => {
@@ -37,7 +37,7 @@ const oDataConfiguration = (edm, csn) => {
     const e = findCsnTargetFor(entitySet, csn, namespace)
 
     configuration[entitySet] = {
-      maxPageSize: getMaxPageSize(e),
+      maxPageSize: getPageSize(e).max,
       isConcurrent: !!e._etag
     }
 

package/libx/_runtime/cds-services/adapter/odata-v4/utils/result.js

@@ -273,7 +273,7 @@ const _pick = options => (element, target) => {
 
   if (element['@odata.etag']) categories.push('@odata.etag')
   if (element._type === 'cds.Decimal') categories.push('@cds.Decimal')
-  if (cds.db?.kind === 'better-sqlite' && element._type === 'cds.Boolean') categories.push('@cds.Boolean')
+  if (cds.db?.cqn2sql && element._type === 'cds.Boolean') categories.push('@cds.Boolean') // REVISIT: violates modularization -> do we still need that?
 
   categories.push(..._assocs(element, target))
 

package/libx/_runtime/cds-services/services/Service.js

@@ -53,7 +53,7 @@ class ApplicationService extends cds.Service {
   }
 
   registerFioriHandlers() {
-    if (cds.env.features.lean_draft) {
+    if (cds.env.fiori.lean_draft) {
       const { onNew, onPrepare, onEdit, onCancel } = require('../../fiori/lean-draft')
 
       for (const each of this.entities)

package/libx/_runtime/cds-services/util/assert.js

@@ -1,5 +1,6 @@
 const cds = require('../../cds')
 const LOG = cds.log('app')
+const templatePathSerializer = require('../../common/utils/templateProcessorPathSerializer')
 
 // REVISIT: replace with cds.Request
 const getEntry = require('../../common/error/entry')
@@ -42,7 +43,7 @@ const _enumValues = element => {
 }
 
 // REVISIT: this needs a cleanup!
-const assertError = (code, element, value, key, pathSegments = []) => {
+const assertError = (code, element, value, key, path) => {
   let args
 
   if (typeof code === 'object') {
@@ -51,14 +52,12 @@ const assertError = (code, element, value, key, pathSegments = []) => {
   }
 
   const { name, type, precision, scale } = element
-  const path = `${pathSegments.join('/')}${pathSegments.length ? '/' : ''}${name || key}`
-
   const error = new Error()
   const errorEntry = {
     code,
     message: code,
-    target: path,
-    args: args || [name || key]
+    target: path ?? element.name ?? key,
+    args: args ?? [name ?? key]
   }
 
   const assertError = Object.assign(error, getEntry(errorEntry))
@@ -81,13 +80,9 @@ const assertError = (code, element, value, key, pathSegments = []) => {
   return assertError
 }
 
-const _checkString = value => {
-  return typeof value === 'string'
-}
+const _checkString = value => typeof value === 'string'
 
-const _checkNumber = value => {
-  return typeof value === 'number'
-}
+const _checkNumber = value => typeof value === 'number'
 
 const _checkDecimal = (value, element) => {
   const [left, right] = String(value).split('.')
@@ -98,38 +93,24 @@ const _checkDecimal = (value, element) => {
   )
 }
 
-const _checkInteger = value => {
-  return _checkNumber(value) && parseInt(value, 10) === value
-}
+const _checkInteger = value => _checkNumber(value) && parseInt(value, 10) === value
 
-const _checkBoolean = value => {
-  return typeof value === 'boolean'
-}
+const _checkBoolean = value => typeof value === 'boolean'
 
-const _checkBuffer = value => {
-  // REVISIT: Extension parameter in push is an object with buffer data
-  return Buffer.isBuffer(value) || value.type === 'Buffer'
-}
+// REVISIT: Extension parameter in push is an object with buffer data
+const _checkBuffer = value => Buffer.isBuffer(value) || value.type === 'Buffer'
 
 const _checkUUID = value => {
   return _checkString(value) && UUID_REGEX.test(value)
 }
 
-const _checkISODate = value => {
-  return (_checkString(value) && ISO_DATE_REGEX.test(value)) || value instanceof Date
-}
+const _checkISODate = value => (_checkString(value) && ISO_DATE_REGEX.test(value)) || value instanceof Date
 
-const _checkISOTime = value => {
-  return _checkString(value) && ISO_TIME_REGEX.test(value)
-}
+const _checkISOTime = value => _checkString(value) && ISO_TIME_REGEX.test(value)
 
-const _checkISODateTime = value => {
-  return (_checkString(value) && ISO_DATE_TIME_REGEX.test(value)) || value instanceof Date
-}
+const _checkISODateTime = value => (_checkString(value) && ISO_DATE_TIME_REGEX.test(value)) || value instanceof Date
 
-const _checkISOTimestamp = value => {
-  return (_checkString(value) && ISO_TIMESTAMP_REGEX.test(value)) || value instanceof Date
-}
+const _checkISOTimestamp = value => (_checkString(value) && ISO_TIMESTAMP_REGEX.test(value)) || value instanceof Date
 
 const _checkInRange = (val, range) => {
   return _checkISODate(val)
@@ -137,10 +118,9 @@ const _checkInRange = (val, range) => {
     : (val - range[0]) * (val - range[1]) <= 0
 }
 
-const _checkRegExpFormat = (val, format) => {
-  // process.env.CDS_ASSERT_FORMAT_FLAGS not official!
-  return _checkString(val) && val.match(new RegExp(format, process.env.CDS_ASSERT_FORMAT_FLAGS || 'u'))
-}
+// process.env.CDS_ASSERT_FORMAT_FLAGS not official!
+const _checkRegExpFormat = (val, format) =>
+  _checkString(val) && val.match(new RegExp(format, process.env.CDS_ASSERT_FORMAT_FLAGS || 'u'))
 
 const CDS_TYPE_CHECKS = {
   'cds.UUID': _checkUUID,
@@ -215,28 +195,23 @@ const checkStaticElementByKey = (definition, key, value, result = [], ignoreNonM
   return result
 }
 
-const _isNotFilled = value => {
-  return value === null || value === undefined || (typeof value === 'string' && value.trim() === '')
-}
+const _isNotFilled = value =>
+  value === null || value === undefined || (typeof value === 'string' && value.trim() === '')
 
-const _checkMandatoryElement = (element, value, errors, key, pathSegments) => {
+const _checkMandatoryElement = (element, value, errors, key, pathSegmentsInfo) => {
   if (element.parent?.query?.SELECT?.columns?.find(col => _isNavigationColumn(col, element.name))) return
   if (element._isMandatory && !element.default && _isNotFilled(value)) {
-    errors.push(assertError(ASSERT_NOT_NULL, element, value, key, pathSegments))
+    errors.push(assertError(ASSERT_NOT_NULL, element, value, key, pathSegmentsInfo))
   }
 }
 
-const _isNavigationColumn = (column, searched) => {
-  return (
-    column.ref && column.ref.length > 1 && (column.as === searched || column.ref[column.ref.length - 1] === searched)
-  )
-}
+const _isNavigationColumn = (column, searched) =>
+  column.ref?.length > 1 && (column.as === searched || column.ref[column.ref.length - 1] === searched)
 
-const _getEnumElement = element => {
-  return (element['@assert.range'] && element.enum) || element['@assert.enum'] ? element.enum : undefined
-}
+const _getEnumElement = element =>
+  (element['@assert.range'] && element.enum) || element['@assert.enum'] ? element.enum : undefined
 
-const _checkEnumElement = (element, value, errors, key, pathSegments) => {
+const _checkEnumElement = (element, value, errors, key, pathSegmentsInfo) => {
   const enumElements = _getEnumElement(element)
   const enumValues = enumElements && _enumValues(enumElements)
 
@@ -246,15 +221,15 @@ const _checkEnumElement = (element, value, errors, key, pathSegments) => {
         ? ['"' + value + '"', enumValues.map(ele => '"' + ele + '"').join(', ')]
         : [value, enumValues.join(', ')]
 
-    errors.push(assertError({ code: ASSERT_ENUM, args }, element, value, key, pathSegments))
+    errors.push(assertError({ code: ASSERT_ENUM, args }, element, value, key, pathSegmentsInfo))
   }
 }
 
-const _checkRangeElement = (element, value, errors, key, pathSegments) => {
+const _checkRangeElement = (element, value, errors, key, pathSegmentsInfo) => {
   const rangeElements = element['@assert.range'] && !_getEnumElement(element) ? element['@assert.range'] : undefined
   if (rangeElements && !_checkInRange(value, rangeElements)) {
     const args = [value, ...element['@assert.range']]
-    errors.push(assertError({ code: ASSERT_RANGE, args }, element, value, key, pathSegments))
+    errors.push(assertError({ code: ASSERT_RANGE, args }, element, value, key, pathSegmentsInfo))
   }
 }
 
@@ -268,19 +243,18 @@ const _checkFormatElement = (element, value, errors, key, pathSegments) => {
 /**
  * @param {import('../../types/api').InputConstraints} constraints
  */
-const checkInputConstraints = ({ element, value, errors, key, pathSegments }) => {
+const checkInputConstraints = ({ element, value, errors, key, pathSegmentsInfo }) => {
   if (!element) return errors
+  let path
 
-  _checkMandatoryElement(element, value, errors, key, pathSegments)
+  if (pathSegmentsInfo?.length) path = templatePathSerializer(element.name || key, pathSegmentsInfo)
+  _checkMandatoryElement(element, value, errors, key, path)
 
   if (value == null) return errors
 
-  _checkEnumElement(element, value, errors, key, pathSegments)
-
-  _checkRangeElement(element, value, errors, key, pathSegments)
-
-  _checkFormatElement(element, value, errors, key, pathSegments)
-
+  _checkEnumElement(element, value, errors, key, path)
+  _checkRangeElement(element, value, errors, key, path)
+  _checkFormatElement(element, value, errors, key, path)
   return errors
 }
 
@@ -323,7 +297,7 @@ const assertNotNullError = element => assertError(ASSERT_NOT_NULL, element)
  *
  * @param {import('../../types/api').assertTargetMap} assertMap
  * @param {array} errors An array to appends the possible errors.
- * @see {@link https://pages.github.tools.sap/cap/docs/guides/providing-services#assert-target @assert.target} for
+ * @see {@link https://cap.cloud.sap/docs/guides/providing-services#assert-target @assert.target} for
  * further information.
  */
 const assertTargets = async (assertMap, errors) => {
@@ -358,9 +332,11 @@ const assertTargets = async (assertMap, errors) => {
     allTargets
       .filter(t => t.key === target.key)
       .forEach(target => {
-        const { row, pathSegments } = target.assocInfo
+        const { row, pathSegmentsInfo } = target.assocInfo
         const key = target.foreignKey.name
-        const error = assertError('ASSERT_TARGET', target.foreignKey, row[key], key, pathSegments)
+        let path
+        if (pathSegmentsInfo?.length) path = templatePathSerializer(key, pathSegmentsInfo)
+        const error = assertError('ASSERT_TARGET', target.foreignKey, row[key], key, path)
         errors.push(error)
       })
   })

package/libx/_runtime/common/code-ext/WorkerPool.js

@@ -0,0 +1,90 @@
+const cds = require('../../cds')
+const os = require('os')
+const { Worker } = require('worker_threads')
+
+class ExtensionWorker extends Worker {
+  constructor(id, workerPath, options) {
+    super(workerPath, options)
+    this.id = id
+    this.tasksAssigned = 0
+  }
+}
+
+class WorkerPool {
+  static instances = []
+  constructor(workerPath, options) {
+    this.workerPath = workerPath
+    this.options = options
+    this.size = options.size ?? Math.max(os.cpus().length, 1)
+    this.idleWorkers = new Set()
+    this.workers = []
+    WorkerPool.instances.push(this)
+  }
+
+  #createWorker() {
+    const id = cds.utils.uuid()
+    const worker = new ExtensionWorker(id, this.workerPath, {
+      workerData: { id },
+      resourceLimits: this.options.resourceLimits
+    })
+    worker.on('exit', this.#onWorkerExit.bind(this, worker))
+    return worker
+  }
+
+  #onWorkerExit(worker) {
+    this.idleWorkers.delete(worker)
+    this.workers.splice(this.workers.indexOf(worker), 1)
+    worker.tasksAssigned = 0
+  }
+
+  adquire() {
+    if (this.idleWorkers.size === 0 && this.workers.length < this.size) {
+      const worker = this.#createWorker()
+      this.idleWorkers.add(worker)
+      this.workers.push(worker)
+    }
+
+    const worker = this.idleWorkers.values().next().value
+
+    if (worker) {
+      this.idleWorkers.delete(worker)
+      worker.tasksAssigned++
+      return worker
+    }
+
+    const randomWorkerIndex = Math.floor(Math.random() * this.workers.length)
+    const busyWorker = this.workers[randomWorkerIndex]
+    busyWorker.tasksAssigned++
+    return busyWorker
+  }
+
+  release(worker) {
+    if (worker.tasksAssigned === 0) return
+
+    worker.tasksAssigned--
+    if (worker.tasksAssigned === 0) {
+      this.idleWorkers.add(worker)
+    }
+  }
+
+  async destroy() {
+    if (this.workers.length === 0) return
+
+    const workers = Array.from(this.workers)
+    const iterable = workers.map(worker => {
+      worker.removeAllListeners()
+      return worker.terminate()
+    })
+
+    await Promise.all(iterable)
+    this.idleWorkers = new Set()
+    this.workers = []
+    WorkerPool.instances = []
+  }
+
+  static async destroyAll() {
+    for (const workerPool of WorkerPool.instances) await workerPool.destroy()
+  }
+}
+
+module.exports = WorkerPool

package/libx/_runtime/common/code-ext/WorkerReq.js

@@ -71,10 +71,6 @@ class WorkerReq {
       prop: 'reject',
       args
     })
-
-    let error = Responses.get(4, ...args)
-    if (!error.stack) Error.captureStackTrace((error = Object.assign(new Error(), error)), this.reject)
-    throw error
   }
 }