@winkjs/composer 0.7.0 → 0.8.0

package/README.md

@@ -123,7 +123,7 @@ Reproduce them from a repo clone. `node benchmark/compare.js 10 500` reports the
 
 Write a flow once and run it anywhere Node.js does — an industrial-grade Raspberry Pi, a production server, a Kubernetes cluster. Each asset's state is isolated, so a fault in one stays in one. Messages [queue locally](docs/handbook/resilience.md) when the network drops and drain cleanly on reconnect. Shutdown is ordered and deterministic: sources close first, storage last. A misconfigured flow fails when you define it, not in production — unknown options, output collisions, and bad triggers are all caught at definition time.
 
-The test suite holds over 6,500 tests behind a 99.5% coverage gate. Integration tests run against real Mosquitto and QuestDB services, not mocks. Every npm release carries SLSA provenance — a public, verifiable link from the package back to the exact source commit and the build that produced it. `npm audit signatures` verifies it.
+The test suite holds over 7,000 tests behind a 100% coverage gate on statements, branches, functions, and lines. Integration tests run against real Mosquitto and QuestDB services, not mocks. Every npm release carries SLSA provenance — a public, verifiable link from the package back to the exact source commit and the build that produced it. `npm audit signatures` verifies it.
 
 The documentation also relates winkComposer's concepts to two industrial standards: [ISO 13374](https://composer.winkjs.org/docs/reference/iso-13374-mapping) for condition monitoring, the [NIST AI RMF](https://composer.winkjs.org/docs/reference/nist-ai-rmf-mapping) for AI risk management.
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@winkjs/composer",
-    "version": "0.7.0",
+    "version": "0.8.0",
     "description": "Composable Streaming Intelligence — turn streaming data into actionable insights, from edge to cloud",
     "keywords": [
         "streaming",

package/src/core/env-vars.js

@@ -99,23 +99,7 @@ const ENV_VARS = {
     // outright, so a deployment that sets it stops here, at import.
     questdbIlpUrl: ( process.env.QUESTDB_ILP_URL ?? '127.0.0.1:9000' ).trim(),
     questdbPgUrl: ( process.env.QUESTDB_PG_URL ?? '127.0.0.1:8812' ).trim(),
-    // The five legacy flush variables (deprecated, ADR-029; removed in
-    // 0.8.0). Each carries a value only when set, so the adapter's
-    // DEPRECATED_OPTION line names only what an operator actually set.
-    // QUESTDB_IDLE_FLUSH_CHECK_MS maps to QUESTDB_FLUSH_INTERVAL_MS and
-    // QUESTDB_AUTO_FLUSH_ROWS to QUESTDB_FLUSH_ROWS; the other three
-    // are accepted and ignored.
-    questdbFlushMode: process.env.QUESTDB_FLUSH_MODE ?
-        process.env.QUESTDB_FLUSH_MODE.trim() : undefined,
-    questdbIdleFlushAfterMs: process.env.QUESTDB_IDLE_FLUSH_AFTER_MS ?
-        parseInt( process.env.QUESTDB_IDLE_FLUSH_AFTER_MS, 10 ) : undefined,
-    questdbIdleFlushCheckMs: process.env.QUESTDB_IDLE_FLUSH_CHECK_MS ?
-        parseInt( process.env.QUESTDB_IDLE_FLUSH_CHECK_MS, 10 ) : undefined,
     // Optional QuestDB settings (undefined if not set)
-    questdbAutoFlushRows: process.env.QUESTDB_AUTO_FLUSH_ROWS ?
-        parseInt( process.env.QUESTDB_AUTO_FLUSH_ROWS, 10 ) : undefined,
-    questdbAutoFlushIntervalMs: process.env.QUESTDB_AUTO_FLUSH_INTERVAL_MS ?
-        parseInt( process.env.QUESTDB_AUTO_FLUSH_INTERVAL_MS, 10 ) : undefined,
     questdbMaxBufSize: process.env.QUESTDB_MAX_BUF_SIZE ?
         parseInt( process.env.QUESTDB_MAX_BUF_SIZE, 10 ) : undefined,
     questdbRetryTimeout: process.env.QUESTDB_RETRY_TIMEOUT ?
@@ -218,15 +202,6 @@ const validators = {
         return null;
     },
 
-    questdbFlushMode: function ( value ) {
-        if ( value === undefined ) return null;
-        const validModes = [ 'auto', 'manual' ];
-        if ( !validModes.includes( value ) ) {
-            return `Must be one of ${validModes.join( ', ' )}, got: "${value}"`;
-        }
-        return null;
-    },
-
     questdbStdlibHttp: function ( value ) {
         if ( value === undefined ) return null;
         const validWords = [ 'on', 'off' ];
@@ -251,14 +226,6 @@ const validators = {
         return null;
     },
 
-    nonNegativeIntOrUndefined: function ( value, originalEnv ) {
-        if ( value === undefined ) return null;
-        if ( isNaN( value ) || value < 0 ) {
-            return `Must be non-negative integer, got: "${originalEnv}"`;
-        }
-        return null;
-    },
-
     nonNegativeNumberOrInfinity: function ( value, originalEnv ) {
         if ( isNaN( value ) || value < 0 ) {
             return `Must be a non-negative number or Infinity, got: "${originalEnv}"`;
@@ -313,11 +280,6 @@ const validationConfig = [
     // QuestDB Configuration
     { field: 'questdbIlpUrl', validator: validators.hostPort, label: 'QUESTDB_ILP_URL' },
     { field: 'questdbPgUrl', validator: validators.hostPort, label: 'QUESTDB_PG_URL' },
-    { field: 'questdbFlushMode', validator: validators.questdbFlushMode, label: 'QUESTDB_FLUSH_MODE' },
-    { field: 'questdbIdleFlushAfterMs', validator: validators.nonNegativeIntOrUndefined, originalEnv: 'QUESTDB_IDLE_FLUSH_AFTER_MS' },
-    { field: 'questdbIdleFlushCheckMs', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_IDLE_FLUSH_CHECK_MS' },
-    { field: 'questdbAutoFlushRows', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_AUTO_FLUSH_ROWS' },
-    { field: 'questdbAutoFlushIntervalMs', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_AUTO_FLUSH_INTERVAL_MS' },
     { field: 'questdbMaxBufSize', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_MAX_BUF_SIZE' },
     { field: 'questdbRetryTimeout', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_RETRY_TIMEOUT' },
     { field: 'questdbFlushRows', validator: validators.positiveIntOrUndefined, originalEnv: 'QUESTDB_FLUSH_ROWS' },
@@ -333,12 +295,32 @@ const validationConfig = [
     // questdbPassword: no validation — allows empty for passwordless auth
 ];
 
+// The five QuestDB flush variables 0.7.0 deprecated and 0.8.0 removed
+// (ADR-029 item 10). A deployment that still sets one stops here, at
+// import, and the failure line names what to do instead. Ignoring the
+// variable would let an operator believe a setting took effect.
+const REMOVED_ENV_VARS = [
+    { name: 'QUESTDB_FLUSH_MODE', action: 'delete it, composer owns every flush' },
+    { name: 'QUESTDB_IDLE_FLUSH_AFTER_MS', action: 'delete it, composer owns every flush' },
+    { name: 'QUESTDB_IDLE_FLUSH_CHECK_MS', action: 'use QUESTDB_FLUSH_INTERVAL_MS' },
+    { name: 'QUESTDB_AUTO_FLUSH_ROWS', action: 'use QUESTDB_FLUSH_ROWS' },
+    { name: 'QUESTDB_AUTO_FLUSH_INTERVAL_MS', action: 'delete it, composer owns every flush' }
+];
+
 // ============================================================================
 // VALIDATION RUNNER
 // ============================================================================
 const validate = function () {
     const errors = [];
 
+    // A removed variable is refused by name, even when empty, so a
+    // leftover line in an env file is found.
+    for ( const removed of REMOVED_ENV_VARS ) {
+        if ( process.env[ removed.name ] !== undefined ) {
+            errors.push( `${removed.name}: Removed in 0.8.0; ${removed.action}` );
+        }
+    }
+
     // Run each validator
     for ( const config of validationConfig ) {
         const value = ENV_VARS[ config.field ];

package/src/core/storage-manager/questdb/index.js

@@ -116,12 +116,13 @@
  * `consecutiveFlushFailures`, `lastFlushAt`, and `lastFlushError`.
  * `flush-engine.js` documents the derivation.
  *
- * Deprecated options (ADR-029, removed in 0.8.0). `autoFlushRows` maps
- * to `flushRows` and `idleFlushCheckMs` maps to `flushIntervalMs`.
- * `flushMode`, `idleFlushAfterMs` and `autoFlushIntervalMs` are
- * accepted and ignored. The same holds for their `QUESTDB_*`
- * variables. Setup prints one `DEPRECATED_OPTION` line naming every
- * legacy key in use and what happened to it.
+ * The five legacy flush keys of 0.7.0 (`flushMode`, `idleFlushAfterMs`,
+ * `idleFlushCheckMs`, `autoFlushRows`, `autoFlushIntervalMs`) are gone
+ * since 0.8.0 (ADR-029 item 10). The schema refuses each one as an
+ * unknown key at flow definition, with `INVALID_CONFIG`. `env-vars.js`
+ * refuses their `QUESTDB_*` variables at import. `autoFlushRows`
+ * became `flushRows` and `idleFlushCheckMs` became `flushIntervalMs`.
+ * The other three have no replacement: composer owns every flush.
  *
  * `err.code` vocabulary (per-adapter; ADR-018 has each adapter document
  * its own codes in this header):
@@ -258,8 +259,6 @@
  *   interval. Remediation: the endpoint is not taking rows; read the
  *   `CIRCUIT_OPEN` and `DELIVERY_HEALTH` lines beside it, and raise
  *   `bufferCeilingRows` when a longer outage must be ridden through.
- * - `DEPRECATED_OPTION` — one `logger.warn` line at setup naming the
- *   legacy keys in use (see Deprecated options above).
  * - `ADDRESS_IS_NAME`  — `ilpUrl` or `pgUrl` is a name other than
  *   `localhost`. One `logger.warn` line per field at setup, before any
  *   socket opens (ADR-030). A name is allowed and the adapter
@@ -356,13 +355,12 @@ import { Sender } from '@questdb/nodejs-client';
 import pg from 'pg';
 
 import { ENV_VARS } from '../../env-vars.js';
-import { logger } from '../../logger/index.js';
 import { validators } from '../../utils/validate/index.js';
 import { probeAddress, describeProbe } from '../../utils/address/probe.js';
 import { buildPersistPlans } from './persist-plan.js';
 import { ensureTables } from './ensure-tables.js';
 import { assertColumnFacts } from './assert-columns.js';
-import { resolveOptions, deprecationMessage } from './resolve-options.js';
+import { resolveOptions } from './resolve-options.js';
 import {
     isAllowedIlpUrl,
     isAllowedPgUrl,
@@ -505,9 +503,8 @@ const openTransport = async function ( { SenderClass, senderConfig, ilpUrl, stdl
  * Create QuestDB storage adapter.
  *
  * The options are resolved by `resolve-options.js` (config over
- * environment over default; legacy aliases mapped or ignored). Only the
- * keys that reach the engine are listed here; the resolver documents
- * the rest.
+ * environment over default). Only the keys that reach the engine are
+ * listed here; the resolver documents the rest.
  *
  * @param {Object} assetClass - Asset class definition with columns and insightTypes
  * @param {string} tablePrefix - Prefix for table names (typically assetClass.name)
@@ -536,17 +533,12 @@ const openTransport = async function ( { SenderClass, senderConfig, ilpUrl, stdl
 const createQuestDBStorage = async function ( assetClass, tablePrefix, options, deps = {} ) {
     // Options become settings here, once (ADR-029). A ceiling below the
     // threshold fails setup inside the resolver with INVALID_CONFIG.
-    const { settings, deprecations } = resolveOptions( options, ENV_VARS );
+    const settings = resolveOptions( options, ENV_VARS );
     const {
         ilpUrl, pgUrl, stdlibHttp, requestTimeout, retryTimeout, initBufSize, maxBufSize,
         partitionBy, onWarning, onDeliveryFailure
     } = settings;
 
-    // One line names every legacy key in use and what happened to it.
-    if ( deprecations.length > 0 ) {
-        logger.warn( deprecationMessage( deprecations ) );
-    }
-
     // Runtime validation — required from either DSL config or ENV_VARS.
     // Per ADR-018, setup-time throws carry classified err.code.
     if ( !ilpUrl ) {
@@ -716,11 +708,6 @@ const configSchema = {
         'ilpUrl',
         'pgUrl',
         'tablePrefix',
-        'flushMode',
-        'idleFlushAfterMs',
-        'idleFlushCheckMs',
-        'autoFlushRows',
-        'autoFlushIntervalMs',
         'flushRows',
         'flushIntervalMs',
         'bufferCeilingRows',
@@ -760,40 +747,6 @@ const configSchema = {
         error: 'tablePrefix must use letters, digits, _ and $ only, and not start with a digit ' +
             '(defaults to assetClass.name when omitted)'
     },
-    // The five legacy keys (deprecated, ADR-029; removed in 0.8.0). They
-    // keep their old validation so an existing flow still passes the
-    // schema. The option resolver maps or ignores them and setup prints
-    // one DEPRECATED_OPTION line.
-    flushMode: {
-        type: 'string',
-        required: false,
-        validator: validators.oneOf( [ 'auto', 'manual' ] ),
-        error: 'flushMode must be "auto" or "manual" (deprecated: composer owns every flush)'
-    },
-    idleFlushAfterMs: {
-        type: 'number',
-        required: false,
-        validator: validators.positiveInteger,
-        error: 'idleFlushAfterMs must be a positive integer (deprecated: ignored)'
-    },
-    idleFlushCheckMs: {
-        type: 'number',
-        required: false,
-        validator: validators.positiveInteger,
-        error: 'idleFlushCheckMs must be a positive integer (deprecated: use flushIntervalMs)'
-    },
-    autoFlushRows: {
-        type: 'number',
-        required: false,
-        validator: validators.positiveInteger,
-        error: 'autoFlushRows must be a positive integer (deprecated: use flushRows)'
-    },
-    autoFlushIntervalMs: {
-        type: 'number',
-        required: false,
-        validator: validators.positiveInteger,
-        error: 'autoFlushIntervalMs must be a positive integer (deprecated: ignored)'
-    },
     // The flush settings composer owns (ADR-029). The relation between
     // the ceiling and the threshold is checked by the option resolver
     // at setup, where both values are known.

package/src/core/storage-manager/questdb/resolve-options.js

@@ -36,23 +36,15 @@
  * 50000-row catch-up send gets about 9 minutes. When the operator
  * sets `flushDeadlineMs`, that fixed value is used for every flush.
  *
- * Precedence, highest first (ADR-018 §10, extended for the legacy
- * aliases of ADR-029 item 10):
+ * Precedence, highest first (ADR-018 §10): the key in the config
+ * (`flushRows`), then the environment variable (`QUESTDB_FLUSH_ROWS`),
+ * then the default.
  *
- *   1. the new key in the config          (`flushRows`)
- *   2. the legacy key in the config       (`autoFlushRows`)
- *   3. the new environment variable       (`QUESTDB_FLUSH_ROWS`)
- *   4. the legacy environment variable    (`QUESTDB_AUTO_FLUSH_ROWS`)
- *   5. the default
- *
- * Two legacy keys map to a new one: `autoFlushRows` to `flushRows`, and
- * `idleFlushCheckMs` to `flushIntervalMs`. The other three,
- * `flushMode`, `idleFlushAfterMs` and `autoFlushIntervalMs`, have no
- * meaning once composer owns every flush (ADR-029). They are accepted
- * and ignored. Every supplied legacy key is named in the deprecation
- * report, with what happened to it, so the adapter can print one
- * `DEPRECATED_OPTION` line at setup. All five keys are removed in
- * 0.8.0.
+ * The five legacy keys of 0.7.0 (`flushMode`, `idleFlushAfterMs`,
+ * `idleFlushCheckMs`, `autoFlushRows`, `autoFlushIntervalMs`) are gone
+ * since 0.8.0 (ADR-029 item 10). The adapter's schema refuses each one
+ * as an unknown key, and `env-vars.js` refuses their `QUESTDB_*`
+ * variables at import. Nothing here looks for them.
  *
  * The one relation enforced here: the ceiling is never below twice
  * the threshold. Rows in flight count against the ceiling, so a
@@ -170,105 +162,10 @@ const CLIENT_MAX_RETRY_BACKOFF_MS = 1000;
  */
 const FLUSH_DEADLINE_MARGIN_MS = 5000;
 
-/**
- * The release that removes the five legacy keys.
- * @type {string}
- */
-const REMOVAL_RELEASE = '0.8.0';
-
-/**
- * The five legacy keys in report order. `envField` is the property on
- * the `ENV_VARS` object, `envVar` the variable name an operator sees.
- * `target` names the new key a legacy value maps to; a key without one
- * is accepted and ignored.
- * @type {Array<{key: string, envField: string, envVar: string, target?: string}>}
- */
-const LEGACY_KEYS = [
-    { key: 'flushMode', envField: 'questdbFlushMode', envVar: 'QUESTDB_FLUSH_MODE' },
-    { key: 'idleFlushAfterMs', envField: 'questdbIdleFlushAfterMs', envVar: 'QUESTDB_IDLE_FLUSH_AFTER_MS' },
-    {
-        key: 'idleFlushCheckMs',
-        envField: 'questdbIdleFlushCheckMs',
-        envVar: 'QUESTDB_IDLE_FLUSH_CHECK_MS',
-        target: 'flushIntervalMs'
-    },
-    { key: 'autoFlushRows', envField: 'questdbAutoFlushRows', envVar: 'QUESTDB_AUTO_FLUSH_ROWS', target: 'flushRows' },
-    { key: 'autoFlushIntervalMs', envField: 'questdbAutoFlushIntervalMs', envVar: 'QUESTDB_AUTO_FLUSH_INTERVAL_MS' }
-];
-
 // ============================================================================
 // RESOLUTION
 // ============================================================================
 
-/**
- * Resolves one setting that has a legacy alias, walking the five layers
- * in precedence order. Returns the value and which layer supplied it,
- * so the deprecation report can say whether a legacy value took effect.
- *
- * @param {Object} options - The raw storage options
- * @param {Object} envVars - The `ENV_VARS` object
- * @param {Object} spec - `{ newKey, legacyKey, newEnvField, legacyEnvField, fallback }`
- * @returns {{value: *, winner: string}} The value and its layer
- */
-const resolveWithAlias = function ( options, envVars, spec ) {
-    if ( options[ spec.newKey ] !== undefined ) {
-        return { value: options[ spec.newKey ], winner: 'config' };
-    }
-    if ( options[ spec.legacyKey ] !== undefined ) {
-        return { value: options[ spec.legacyKey ], winner: 'legacyConfig' };
-    }
-    if ( envVars[ spec.newEnvField ] !== undefined ) {
-        return { value: envVars[ spec.newEnvField ], winner: 'env' };
-    }
-    if ( envVars[ spec.legacyEnvField ] !== undefined ) {
-        return { value: envVars[ spec.legacyEnvField ], winner: 'legacyEnv' };
-    }
-    return { value: spec.fallback, winner: 'default' };
-}; // resolveWithAlias()
-
-/**
- * Describes one supplied legacy key for the report. A key with a target
- * is `mapped` when its layer is the one that won that target, and
- * `ignored` otherwise. A key without a target is always `ignored`.
- *
- * @param {string} name - The config key or the environment variable name
- * @param {Object} legacy - The `LEGACY_KEYS` entry
- * @param {Object} winners - Winning layer per target setting
- * @param {string} layer - `'legacyConfig'` or `'legacyEnv'`
- * @returns {{name: string, effect: string, target?: string}} One report entry
- */
-const describeLegacy = function ( name, legacy, winners, layer ) {
-    if ( ( legacy.target !== undefined ) && ( winners[ legacy.target ] === layer ) ) {
-        return { name, effect: 'mapped', target: legacy.target };
-    }
-    return { name, effect: 'ignored' };
-}; // describeLegacy()
-
-/**
- * Builds the deprecation report: every supplied legacy config key in
- * order, then every supplied legacy environment variable in the same
- * order.
- *
- * @param {Object} options - The raw storage options
- * @param {Object} envVars - The `ENV_VARS` object
- * @param {Object} winners - Winning layer per target setting
- * @returns {Array<Object>} The report entries, possibly empty
- */
-const collectDeprecations = function ( options, envVars, winners ) {
-    const report = [];
-    for ( const legacy of LEGACY_KEYS ) {
-        if ( options[ legacy.key ] !== undefined ) {
-            report.push( describeLegacy( legacy.key, legacy, winners, 'legacyConfig' ) );
-        }
-    }
-    for ( const legacy of LEGACY_KEYS ) {
-        if ( envVars[ legacy.envField ] !== undefined ) {
-            report.push( describeLegacy( legacy.envVar, legacy, winners, 'legacyEnv' ) );
-        }
-    }
-    return report;
-}; // collectDeprecations()
-
 /**
  * Maps the words of `QUESTDB_STDLIB_HTTP` to a boolean. The words are
  * the client's own for `stdlib_http`, `on` and `off`, so an operator
@@ -286,55 +183,41 @@ const onOffToBoolean = function ( word ) {
 }; // onOffToBoolean()
 
 /**
- * Resolves the storage options into the adapter's settings and the
- * deprecation report. `settings.flushDeadlineMs` is undefined unless the
- * operator set it; the engine then derives each flush's deadline with
- * `flushDeadlineFor`.
+ * Resolves the storage options into the adapter's settings.
+ * `flushDeadlineMs` is undefined unless the operator set it; the engine
+ * then derives each flush's deadline with `flushDeadlineFor`.
  *
  * @param {Object} options - The raw storage options from `.storage()` config
  * @param {Object} envVars - The `ENV_VARS` object (injected, for purity)
- * @returns {{settings: Object, deprecations: Array<Object>}} Settings and report
+ * @returns {Object} The resolved settings
  * @throws {Error} INVALID_CONFIG when the ceiling is below twice the threshold
  */
 const resolveOptions = function ( options, envVars ) {
-    const rows = resolveWithAlias( options, envVars, {
-        newKey: 'flushRows',
-        legacyKey: 'autoFlushRows',
-        newEnvField: 'questdbFlushRows',
-        legacyEnvField: 'questdbAutoFlushRows',
-        fallback: DEFAULT_FLUSH_ROWS
-    } );
-    const interval = resolveWithAlias( options, envVars, {
-        newKey: 'flushIntervalMs',
-        legacyKey: 'idleFlushCheckMs',
-        newEnvField: 'questdbFlushIntervalMs',
-        legacyEnvField: 'questdbIdleFlushCheckMs',
-        fallback: DEFAULT_FLUSH_INTERVAL_MS
-    } );
-
+    const flushRows = options.flushRows ?? envVars.questdbFlushRows ?? DEFAULT_FLUSH_ROWS;
+    const flushIntervalMs = options.flushIntervalMs ?? envVars.questdbFlushIntervalMs ?? DEFAULT_FLUSH_INTERVAL_MS;
     const bufferCeilingRows = options.bufferCeilingRows ??
         envVars.questdbBufferCeilingRows ??
-        ( rows.value * DEFAULT_CEILING_MULTIPLIER );
+        ( flushRows * DEFAULT_CEILING_MULTIPLIER );
 
     // Rows in flight count against the ceiling. A ceiling equal to the
     // threshold would refuse every row that arrives during a
     // row-triggered flush, because the batch in flight fills it. Twice
     // the threshold is the least that holds one batch in flight and
     // one batch buffering.
-    if ( bufferCeilingRows < ( 2 * rows.value ) ) {
+    if ( bufferCeilingRows < ( 2 * flushRows ) ) {
         const err = new Error(
-            `winkComposer/questdb: bufferCeilingRows ${bufferCeilingRows} is below twice flushRows ${rows.value} [INVALID_CONFIG]: ` +
+            `winkComposer/questdb: bufferCeilingRows ${bufferCeilingRows} is below twice flushRows ${flushRows} [INVALID_CONFIG]: ` +
             'the ceiling must hold one batch in flight and one batch buffering; raise bufferCeilingRows or lower flushRows'
         );
         err.code = 'INVALID_CONFIG';
         throw err;
     }
 
-    const settings = {
+    return {
         ilpUrl: options.ilpUrl ?? envVars.questdbIlpUrl,
         pgUrl: options.pgUrl ?? envVars.questdbPgUrl,
-        flushRows: rows.value,
-        flushIntervalMs: interval.value,
+        flushRows,
+        flushIntervalMs,
         bufferCeilingRows,
         flushDeadlineMs: options.flushDeadlineMs ?? envVars.questdbFlushDeadlineMs,
         stdlibHttp: options.stdlibHttp ?? onOffToBoolean( envVars.questdbStdlibHttp ) ?? DEFAULT_STDLIB_HTTP,
@@ -346,9 +229,6 @@ const resolveOptions = function ( options, envVars ) {
         onWarning: options.onWarning,
         onDeliveryFailure: options.onDeliveryFailure
     };
-
-    const winners = { flushRows: rows.winner, flushIntervalMs: interval.winner };
-    return { settings, deprecations: collectDeprecations( options, envVars, winners ) };
 }; // resolveOptions()
 
 /**
@@ -379,25 +259,8 @@ const flushDeadlineFor = function ( rows, settings ) {
     return retryTimeout + ( 2 * attemptMs ) + CLIENT_MAX_RETRY_BACKOFF_MS + FLUSH_DEADLINE_MARGIN_MS;
 }; // flushDeadlineFor()
 
-/**
- * Formats the deprecation report as the one console line the adapter
- * prints at setup, in the ADR-028 message grammar.
- *
- * @param {Array<Object>} deprecations - A non-empty report from `resolveOptions`
- * @returns {string} The console line
- */
-const deprecationMessage = function ( deprecations ) {
-    const parts = deprecations.map( function ( entry ) {
-        return ( entry.effect === 'mapped' ) ?
-            `${entry.name} maps to ${entry.target}` :
-            `${entry.name} is ignored`;
-    } );
-    return 'winkComposer/questdb: deprecated storage options in use [DEPRECATED_OPTION]: ' +
-        `${parts.join( '; ' )}; all five deprecated keys are removed in ${REMOVAL_RELEASE}`;
-}; // deprecationMessage()
-
 // ============================================================================
 // EXPORTS
 // ============================================================================
 
-export { resolveOptions, flushDeadlineFor, deprecationMessage, CLIENT_MAX_RETRY_BACKOFF_MS };
+export { resolveOptions, flushDeadlineFor, CLIENT_MAX_RETRY_BACKOFF_MS };