@cap-js/db-service 1.1.0 → 1.2.1

package/lib/cqn2sql.js

@@ -2,6 +2,8 @@ const cds = require('@sap/cds/lib')
 const cds_infer = require('./infer')
 const cqn4sql = require('./cqn4sql')
 
+const { Readable } = require('stream')
+
 const DEBUG = (() => {
   let DEBUG = cds.debug('sql-json')
   if (DEBUG) return DEBUG
@@ -19,35 +21,37 @@ class CQN2SQLRenderer {
    * @constructor
    * @param {import('@sap/cds/apis/services').ContextProperties} context the cds.context of the request
    */
-  constructor(context) {
-    /**
-     * @type {import('@sap/cds/apis/services').ContextProperties}
-     */
-    this.context = cds.context || context
-    // REVISIT: find a way to make CQN2SQLRenderer work in SQLService as well
-    /** @type {CQN2SQLRenderer|unknown} */
+  constructor(srv) {
+    this.context = srv?.context || cds.context // Using srv.context is required due to stakeholders doing unmanaged txs without cds.context being set
     this.class = new.target // for IntelliSense
     this.class._init() // is a noop for subsequent calls
   }
 
+  static _add_mixins (aspect, mixins) {
+    const fqn = this.name + aspect
+    const types = cds.builtin.types
+    for (let each in mixins) {
+      const def = types[each]
+      if (!def) continue
+      Object.defineProperty(def, fqn, { value: mixins[each] })
+    }
+    return fqn
+  }
+
   /**
    * Initializes the class one first creation to link types to data converters
    */
   static _init() {
-    const _add_mixins = (aspect, mixins) => {
-      const fqn = this.name + aspect
-      const types = cds.builtin.types
-      for (let each in mixins) {
-        const def = types[each]
-        if (!def) continue
-        Object.defineProperty(def, fqn, { value: mixins[each] })
-      }
-      return fqn
+    this._localized = this._add_mixins(':localized', this.localized)
+    this._convertInput = this._add_mixins(':convertInput', this.InputConverters)
+    this._convertOutput = this._add_mixins(':convertOutput', this.OutputConverters)
+    this._sqlType = this._add_mixins(':sqlType', this.TypeMap)
+    // Have all-uppercase all-lowercase, and capitalized keywords to speed up lookups
+    for (let each in this.ReservedWords) {
+      // ORDER
+      this.ReservedWords[each[0] + each.slice(1).toLowerCase()] = 1 // Order
+      this.ReservedWords[each.toLowerCase()] = 1 // order
     }
-    this._localized = _add_mixins(':localized', this.localized)
-    this._convertInput = _add_mixins(':convertInput', this.InputConverters)
-    this._convertOutput = _add_mixins(':convertOutput', this.OutputConverters)
-    this._sqlType = _add_mixins(':sqlType', this.TypeMap)
     this._init = () => {} // makes this a noop for subsequent calls
   }
 
@@ -189,23 +193,22 @@ class CQN2SQLRenderer {
    */
   SELECT(q) {
     let { from, expand, where, groupBy, having, orderBy, limit, one, distinct, localized } = q.SELECT
-    if (!expand) expand = q.SELECT.expand = has_expands(q) || has_arrays(q)
     // REVISIT: When selecting from an entity that is not in the model the from.where are not normalized (as cqn4sql is skipped)
     if (!where && from?.ref?.length === 1 && from.ref[0]?.where) where = from.ref[0]?.where
     let columns = this.SELECT_columns(q)
-    let x,
-      sql = `SELECT`
+    let sql = `SELECT`
     if (distinct) sql += ` DISTINCT`
-    if (!_empty((x = columns))) sql += ` ${x}`
-    if (!_empty((x = from))) sql += ` FROM ${this.from(x)}`
-    if (!_empty((x = where))) sql += ` WHERE ${this.where(x)}`
-    if (!_empty((x = groupBy))) sql += ` GROUP BY ${this.groupBy(x)}`
-    if (!_empty((x = having))) sql += ` HAVING ${this.having(x)}`
-    if (!_empty((x = orderBy))) sql += ` ORDER BY ${this.orderBy(x, localized)}`
-    if (one) sql += ` LIMIT ${this.limit({ rows: { val: 1 } })}`
-    else if ((x = limit)) sql += ` LIMIT ${this.limit(x)}`
+    if (!_empty(columns)) sql += ` ${columns}`
+    if (!_empty(from)) sql += ` FROM ${this.from(from)}`
+    if (!_empty(where)) sql += ` WHERE ${this.where(where)}`
+    if (!_empty(groupBy)) sql += ` GROUP BY ${this.groupBy(groupBy)}`
+    if (!_empty(having)) sql += ` HAVING ${this.having(having)}`
+    if (!_empty(orderBy)) sql += ` ORDER BY ${this.orderBy(orderBy, localized)}`
+    if (one) limit = Object.assign({}, limit, { rows: { val: 1 } })
+    if (limit) sql += ` LIMIT ${this.limit(limit)}`
     // Expand cannot work without an inferred query
     if (expand) {
+      // REVISIT: Why don't we handle that as an error in SELECT_expand?
       if (!q.elements) cds.error`Query was not inferred and includes expand. For which the metadata is missing.`
       sql = this.SELECT_expand(q, sql)
     }
@@ -217,13 +220,8 @@ class CQN2SQLRenderer {
    * @param {import('./infer/cqn').SELECT} param0
    * @returns {string} SQL
    */
-  SELECT_columns({ SELECT }) {
-    // REVISIT: We don't have to run x.as through this.column_name(), do we?
-    if (!SELECT.columns) return '*'
-    return SELECT.columns.map(x => {
-      if (x === '*') return x
-      return this.column_expr(x) + (typeof x.as === 'string' ? ' as ' + this.quote(x.as) : '')
-    })
+  SELECT_columns(q) {
+    return (q.SELECT.columns ?? ['*']).map(x => this.column_expr(x, q))
   }
 
   /**
@@ -234,28 +232,23 @@ class CQN2SQLRenderer {
    */
   SELECT_expand({ SELECT, elements }, sql) {
     if (!SELECT.columns) return sql
-    if (!elements) return sql
-    let cols = !SELECT.columns
-      ? ['*']
-      : SELECT.columns.map(x => {
-          const name = this.column_name(x)
-          // REVISIT: can be removed when alias handling is resolved properly
-          const d = elements[name] || elements[name.substring(1, name.length - 1)]
-          let col = `'$."${name}"',${this.output_converter4(d, this.quote(name))}`
-
-          if (x.SELECT?.count) {
-            // Return both the sub select and the count for @odata.count
-            const qc = cds.ql.clone(x, { columns: [{ func: 'count' }], one: 1, limit: 0, orderBy: 0 })
-            col += `, '$."${name}@odata.count"',${this.expr(qc)}`
-          }
-          return col
-        })
+    if (!elements) return sql // REVISIT: Above we say this is an error condition, but here we say it's ok?
+    let cols = SELECT.columns.map(x => {
+      const name = this.column_name(x)
+      let col = `'${name}',${this.output_converter4(x.element, this.quote(name))}`
+      if (x.SELECT?.count) {
+        // Return both the sub select and the count for @odata.count
+        const qc = cds.ql.clone(x, { columns: [{ func: 'count' }], one: 1, limit: 0, orderBy: 0 })
+        return [col, `'${name}@odata.count',${this.expr(qc)}`]
+      }
+      return col
+    }).flat()
 
     // Prevent SQLite from hitting function argument limit of 100
-    let colsLength = cols.length
-    let obj = "'{}'"
-    for (let i = 0; i < colsLength; i += 48) {
-      obj = `json_insert(${obj},${cols.slice(i, i + 48)})`
+    let obj = ''
+    for (let i = 0; i < cols.length; i += 50) {
+      const n =  `json_object(${cols.slice(i, i + 50)})`
+      obj = obj ? `json_patch(${obj},${n})` : n
     }
     return `SELECT ${SELECT.one || SELECT.expand === 'root' ? obj : `json_group_array(${obj})`} as _json_ FROM (${sql})`
   }
@@ -265,34 +258,41 @@ class CQN2SQLRenderer {
    * @param {import('./infer/cqn').col} x
    * @returns {string} SQL
    */
-  column_expr(x) {
-    if (x.func && !x.as) x.as = x.func
+  column_expr(x, q) {
+    if (x === '*') return '*'
+    ///////////////////////////////////////////////////////////////////////////////////////
+    // REVISIT: that should move out of here!
     if (x?.element?.['@cds.extension']) {
-      x.as = x.as || x.element.name
-      return `extensions__->${this.string('$."' + x.element.name + '"')}`
+      return `extensions__->${this.string('$."' + x.element.name + '"')} as ${x.as || x.element.name}`
     }
+    ///////////////////////////////////////////////////////////////////////////////////////
     let sql = this.expr(x)
+    let alias = this.column_alias4(x, q)
+    if (alias) sql += ' as ' + this.quote(alias)
     return sql
   }
 
+  /**
+   * Extracts the column alias from a SELECT column expression
+   * @param {import('./infer/cqn').col} x
+   * @returns {string}
+   */
+  column_alias4(x) {
+    return typeof x.as === 'string' ? x.as : x.func
+  }
+
   /**
    * Renders a FROM clause into generic SQL
    * @param {import('./infer/cqn').source} from
    * @returns {string} SQL
    */
   from(from) {
-    const { ref, as } = from,
-      _aliased = as ? s => s + ` as ${this.quote(as)}` : s => s
+    const { ref, as } = from
+    const _aliased = as ? s => s + ` as ${this.quote(as)}` : s => s
     if (ref) return _aliased(this.quote(this.name(ref[0])))
     if (from.SELECT) return _aliased(`(${this.SELECT(from)})`)
-    if (from.join) {
-      const {
-        join,
-        args: [left, right],
-        on,
-      } = from
-      return `${this.from(left)} ${join} JOIN ${this.from(right)} ON ${this.xpr({ xpr: on })}`
-    }
+    if (from.join)
+      return `${this.from(from.args[0])} ${from.join} JOIN ${this.from(from.args[1])} ON ${this.where(from.on)}`
   }
 
   /**
@@ -414,8 +414,10 @@ class CQN2SQLRenderer {
       .filter(a => a)
       .map(c => c.sql)
 
-    this.entries = [[JSON.stringify(INSERT.entries)]]
-    return (this.sql = `INSERT INTO ${entity}${alias ? ' as ' + this.quote(alias) : ''} (${
+    // Include this.values for placeholders
+    /** @type {unknown[][]} */
+    this.entries = [[...this.values, JSON.stringify(INSERT.entries)]]
+    return (this.sql = `INSERT INTO ${this.quote(entity)}${alias ? ' as ' + this.quote(alias) : ''} (${
       this.columns
     }) SELECT ${extraction} FROM json_each(?)`)
   }
@@ -433,14 +435,11 @@ class CQN2SQLRenderer {
     if (!INSERT.columns && !elements) {
       throw cds.error`Cannot insert rows without columns or elements`
     }
-    let columns = INSERT.columns || (elements && ObjectKeys(elements))
-    if (elements) {
-      columns = columns.filter(c => c in elements && !elements[c].virtual && !elements[c].isAssociation)
-    }
+    let columns = INSERT.columns || (elements && ObjectKeys(elements).filter(c => !elements[c].virtual && !elements[c].isAssociation))
     this.columns = columns.map(c => this.quote(c))
 
     const inputConverterKey = this.class._convertInput
-    const extraction = columns.map((c, i) => {
+    const extraction = columns.map((c,i) => {
       const element = elements?.[c] || {}
       const extract = `value->>'$[${i}]'`
       const converter = element[inputConverterKey] || (e => e)
@@ -448,7 +447,7 @@ class CQN2SQLRenderer {
     })
 
     this.entries = [[JSON.stringify(INSERT.rows)]]
-    return (this.sql = `INSERT INTO ${entity}${alias ? ' as ' + this.quote(alias) : ''} (${
+    return (this.sql = `INSERT INTO ${this.quote(entity)}${alias ? ' as ' + this.quote(alias) : ''} (${
       this.columns
     }) SELECT ${extraction} FROM json_each(?)`)
   }
@@ -536,30 +535,26 @@ class CQN2SQLRenderer {
    * @returns {string} SQL
    */
   UPDATE(q) {
-    const {
-        UPDATE: { entity, with: _with, data, where },
-      } = q,
-      elements = q.target?.elements
+    const { entity, with: _with, data, where } = q.UPDATE
+    const elements = q.target?.elements
     let sql = `UPDATE ${this.name(entity.ref?.[0] || entity)}`
     if (entity.as) sql += ` AS ${entity.as}`
+
     let columns = []
-    if (data)
-      for (let c in data)
-        if (!elements || (c in elements && !elements[c].virtual)) {
-          columns.push({ name: c, sql: this.val({ val: data[c] }) })
-        }
-    if (_with)
-      for (let c in _with)
+    if (data) _add (data, val => this.val({val}))
+    if (_with) _add (_with, x => this.expr(x))
+    function _add (data, sql4) {
+      for (let c in data) {
         if (!elements || (c in elements && !elements[c].virtual)) {
-          columns.push({ name: c, sql: this.expr(_with[c]) })
+          columns.push({ name: c, sql: sql4(data[c]) })
         }
+      }
+    }
 
     columns = columns.map(c => {
-      if (q.elements?.[c.name]?.['@cds.extension']) {
-        return {
-          name: 'extensions__',
-          sql: `json_set(extensions__,${this.string('$."' + c.name + '"')},${c.sql})`,
-        }
+      if (q.elements?.[c.name]?.['@cds.extension']) return {
+        name: 'extensions__',
+        sql: `json_set(extensions__,${this.string('$."' + c.name + '"')},${c.sql})`,
       }
       return c
     })
@@ -592,24 +587,65 @@ class CQN2SQLRenderer {
    * @returns {string} SQL
    */
   STREAM(q) {
-    let { from, into, where, column, data } = q.STREAM
-    let x, sql
-    // reading stream
-    if (from) {
-      sql = `SELECT`
-      if (!_empty((x = column))) sql += ` ${this.quote(x)}`
-      if (!_empty((x = from))) sql += ` FROM ${this.from(x)}`
+    const { STREAM } = q
+    return STREAM.from
+      ? this.STREAM_from(q)
+      : STREAM.into
+      ? this.STREAM_into(q)
+      : cds.error`Missing .form or .into in ${q}`
+  }
+
+  /**
+   * Renders a STREAM.into query into generic SQL
+   * @param {import('./infer/cqn').STREAM} q
+   * @returns {string} SQL
+   */
+  STREAM_into(q) {
+    const { into, column, where, data } = q.STREAM
+
+    let sql
+    if (!_empty(column)) {
+      data.type = 'binary'
+      const update = UPDATE(into)
+        .with({ [column]: data })
+        .where(where)
+      Object.defineProperty(update, 'target', { value: q.target })
+      sql = this.UPDATE(update)
     } else {
-      // writing stream
-      const entity = this.name(q.target?.name || into.ref[0])
-      sql = `UPDATE ${this.quote(entity)}${into.as ? ` AS ${into.as}` : ``} SET ${this.quote(column)}=?`
-      this.entries = [data]
+      data.type = 'json'
+      // REVISIT: decide whether dataset streams should behave like INSERT or UPSERT
+      sql = this.UPSERT(UPSERT([{}]).into(into).forSQL())
+      this.values = [data]
     }
-    if (!_empty((x = where))) sql += ` WHERE ${this.where(x)}`
-    if (from) sql += ` LIMIT ${this.limit({ rows: { val: 1 } })}`
+
     return (this.sql = sql)
   }
 
+  /**
+   * Renders a STREAM.from query into generic SQL
+   * @param {import('./infer/cqn').STREAM} q
+   * @returns {string} SQL
+   */
+  STREAM_from(q) {
+    const { column, from, where, columns } = q.STREAM
+
+    const select = cds.ql
+      .SELECT(column ? [column] : columns)
+      .where(where)
+      .limit(column ? 1 : undefined)
+
+    // SELECT.from() does not accept joins
+    select.SELECT.from = from
+
+    if (column) {
+      this.one = true
+    } else {
+      select.SELECT.expand = 'root'
+      this.one = !!from.SELECT?.one
+    }
+    return this.SELECT(select.forSQL())
+  }
+
   // Expression Clauses ---------------------------------------------
 
   /**
@@ -655,11 +691,35 @@ class CQN2SQLRenderer {
    * @returns {string} The correct operator string
    */
   operator(x, i, xpr) {
-    if (x === '=' && xpr[i + 1]?.val === null) return 'is'
-    if (x === '!=') return 'is not'
+
+    // Translate = to IS NULL for rhs operand being NULL literal
+    if (x === '=')  return xpr[i+1]?.val === null ? 'is' : '='
+
+    // Translate == to IS NOT NULL for rhs operand being NULL literal, otherwise ...
+    // Translate == to IS NOT DISTINCT FROM, unless both operands cannot be NULL
+    if (x === '==') return xpr[i+1]?.val === null ? 'is' : _not_null(i-1) && _not_null(i+1) ? '=' : this.is_not_distinct_from_
+
+    // Translate != to IS NULL for rhs operand being NULL literal, otherwise...
+    // Translate != to IS DISTINCT FROM, unless both operands cannot be NULL
+    if (x === '!=') return xpr[i+1]?.val === null ? 'is not' : _not_null(i-1) && _not_null(i+1) ? '<>' : this.is_distinct_from_
+
     else return x
+
+    /** Checks if the operand at xpr[i+-1] can be NULL. @returns true if not */
+    function _not_null(i) {
+      const operand = xpr[i]
+      if (!operand) return false
+      if (operand.val != null) return true // non-null values are not null
+      let element = operand.element
+      if (!element) return false
+      if (element.key) return true // primary keys usually should not be null
+      if (element.notNull) return true // not null elements cannot be null
+    }
   }
 
+  get is_distinct_from_() { return 'is distinct from' }
+  get is_not_distinct_from_() { return 'is not distinct from' }
+
   /**
    * Renders an argument place holder into the SQL for prepared statements
    * @param {import('./infer/cqn').ref} param0
@@ -677,7 +737,12 @@ class CQN2SQLRenderer {
    * @returns {string} SQL
    */
   ref({ ref }) {
-    return ref.map(r => this.quote(r)).join('.')
+    switch (ref[0]) {
+      case '$now': return this.func({ func: 'session_context', args: [{ val: '$now' }]})
+      case '$user':
+      case '$user.id': return this.func({ func: 'session_context', args: [{ val: '$user.id' }]})
+      default: return ref.map(r => this.quote(r)).join('.')
+    }
   }
 
   /**
@@ -687,22 +752,21 @@ class CQN2SQLRenderer {
    */
   val({ val }) {
     switch (typeof val) {
-      case 'function':
-        throw new Error('Function values not supported.')
-      case 'undefined':
-        return 'NULL'
-      case 'boolean':
-        return val
-      case 'number':
-        return val // REVISIT for HANA
+      case 'function': throw new Error('Function values not supported.')
+      case 'undefined': return 'NULL'
+      case 'boolean': return `${val}`
+      case 'number': return `${val}` // REVISIT for HANA
       case 'object':
         if (val === null) return 'NULL'
         if (val instanceof Date) return `'${val.toISOString()}'`
-        if (Buffer.isBuffer(val)) val = val.toString('base64')
-        else val = this.regex(val) || this.json(val)
+        if (val instanceof Readable) ; // go on with default below
+        else if (Buffer.isBuffer(val)) val = val.toString('base64')
+        else if (is_regexp(val)) val = val.source
+        else val = JSON.stringify(val)
+      case 'string': // eslint-disable-line no-fallthrough
     }
     if (!this.values) return this.string(val)
-    this.values.push(val)
+    else this.values.push(val)
     return '?'
   }
 
@@ -727,25 +791,7 @@ class CQN2SQLRenderer {
   }
 
   /**
-   * Renders a Regular Expression into its string representation
-   * @param {RegExp} o
-   * @returns {string} SQL
-   */
-  regex(o) {
-    if (is_regexp(o)) return o.source
-  }
-
-  /**
-   * Renders the object as a JSON string in generic SQL
-   * @param {object} o
-   * @returns {string} SQL
-   */
-  json(o) {
-    return this.string(JSON.stringify(o))
-  }
-
-  /**
-   * Renders a javascript string into a generic SQL string
+   * Renders a javascript string into a SQL string literal
    * @param {string} s
    * @returns {string} SQL
    */
@@ -760,8 +806,9 @@ class CQN2SQLRenderer {
    */
   column_name(col) {
     if (col === '*')
+      // REVISIT: When could this ever happen? I think this is only about that irrealistic test whech uses column_name to implement SELECT_columns. We should eliminate column_name as its only used and designed for use in SELECT_expand, isn't it?
       cds.error`Query was not inferred and includes '*' in the columns. For which there is no column name available.`
-    return (typeof col.as === 'string' && col.as) || ('val' in col && col.val + '') || col.ref[col.ref.length - 1]
+    return (typeof col.as === 'string' && col.as) || ('val' in col && col.val + '') || col.func || col.ref.at(-1)
   }
 
   /**
@@ -783,7 +830,8 @@ class CQN2SQLRenderer {
   quote(s) {
     if (typeof s !== 'string') return '"' + s + '"'
     if (s.includes('"')) return '"' + s.replace(/"/g, '""') + '"'
-    if (s.toUpperCase() in this.class.ReservedWords || /^\d|[$' ?@./\\]/.test(s)) return '"' + s + '"'
+    // Column names like "Order" clash with "ORDER" keyword so toUpperCase is required
+    if (s in this.class.ReservedWords || /^\d|[$' ?@./\\]/.test(s)) return '"' + s + '"'
     return s
   }
 
@@ -796,46 +844,39 @@ class CQN2SQLRenderer {
    */
   managed(columns, elements, isUpdate = false) {
     const annotation = isUpdate ? '@cds.on.update' : '@cds.on.insert'
-    const inputConverterKey = this.class._convertInput
+    const { _convertInput } = this.class
     // Ensure that missing managed columns are added
     const requiredColumns = !elements
       ? []
       : Object.keys(elements)
           .filter(
             e =>
-              (elements[e]?.[annotation] || (!isUpdate && elements[e]?.default && !elements[e].virtual)) &&
+              (elements[e]?.[annotation] || (!isUpdate && elements[e]?.default && !elements[e].virtual && !elements[e].isAssociation)) &&
               !columns.find(c => c.name === e),
           )
           .map(name => ({ name, sql: 'NULL' }))
 
     return [...columns, ...requiredColumns].map(({ name, sql }) => {
-      const element = elements?.[name] || {}
-      let extract = sql ?? `value->>'$."${name}"'`
-      const converter = element[inputConverterKey] || (e => e)
-      let managed = element[annotation]?.['=']
-      switch (managed) {
-        case '$user.id':
-        case '$user':
-          managed = this.string(this.context.user.id)
-          break
-        case '$now':
-          managed = this.string(this.context.timestamp.toISOString())
-          break
-        default:
-          managed = undefined
-      }
-      if (!isUpdate) {
+      let element = elements?.[name] || {}
+      if (!sql) sql = `value->>'$."${name}"'`
+
+      let converter = element[_convertInput]
+      if (converter && sql[0] !== '$') sql = converter(sql, element)
+
+      let val = _managed[element[annotation]?.['=']]
+      if (val) sql = `coalesce(${sql}, ${this.func({ func: 'session_context', args: [{ val }] })})`
+
+      else if (!isUpdate && element.default) {
         const d = element.default
-        if (d && (d.val !== undefined || d.ref?.[0] === '$now')) {
-          extract = `(CASE WHEN json_type(value,'$."${name}"') IS NULL THEN ${this.defaultValue(
-            d.val,
-          )} ELSE ${extract} END)`
+        if (d.val !== undefined || d.ref?.[0] === '$now') {
+          // REVISIT: d.ref is not used afterwards
+          sql = `(CASE WHEN json_type(value,'$."${name}"') IS NULL THEN ${
+            this.defaultValue(d.val) // REVISIT: this.defaultValue is a strange function
+          } ELSE ${sql} END)`
         }
       }
-      return {
-        name,
-        sql: converter(managed === undefined ? extract : `coalesce(${extract}, ${managed})`, element),
-      }
+
+      return { name, sql }
     })
   }
 
@@ -844,6 +885,7 @@ class CQN2SQLRenderer {
    * @param {string} defaultValue
    * @returns {string}
    */
+  // REVISIT: This is a strange method, also overridden inconsistently in postgres
   defaultValue(defaultValue = this.context.timestamp.toISOString()) {
     return typeof defaultValue === 'string' ? this.string(defaultValue) : defaultValue
   }
@@ -855,8 +897,11 @@ Buffer.prototype.toJSON = function () {
 }
 
 const ObjectKeys = o => (o && [...ObjectKeys(o.__proto__), ...Object.keys(o)]) || []
-const has_expands = q => q.SELECT.columns?.some(c => c.SELECT?.expand)
-const has_arrays = q => q.elements && Object.values(q.elements).some(e => e.items)
+const _managed = {
+  '$user.id': '$user.id',
+  $user: '$user.id',
+  $now: '$now',
+}
 
 const is_regexp = x => x?.constructor?.name === 'RegExp' // NOTE: x instanceof RegExp doesn't work in repl
 const _empty = a => !a || a.length === 0