dexbot 1.4.11 → 1.4.13

package/dist/modules/dexbot_maintenance_runtime.js

@@ -1,21 +1,31 @@
-"use strict";
 /** Maintenance runtime - periodic sync loops, grid health checks, rebalance */
-const { createHash } = require('./crypto/sync');
-const fs = require('fs');
-const { path } = require('./path_api');
-const { spawn } = require('child_process');
-const chainOrders = require('./chain_orders');
-const { readOpenOrdersGuarded } = require('./chain_orders');
-const grid = require('./order/grid');
-const { ORDER_STATES, ORDER_TYPES, TIMING, BTS_PRECISION, NATIVE_CLIENT } = require('./constants');
-const { PATHS } = require('./paths');
-const Format = require('./order/format');
-const { getStorage } = require('./storage');
+import { createRequire } from 'node:module';
+import { fileURLToPath } from 'node:url';
+import { dirname as _esmDirname } from 'node:path';
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = _esmDirname(__filename);
+const require = createRequire(import.meta.url);
+import fs from 'node:fs';
+import { spawn } from 'node:child_process';
+import { createHash } from './crypto/sync.js';
+import { path } from './path_api.js';
+import * as chainOrders from './chain_orders.js';
+import * as grid from './order/grid.js';
+import { ORDER_STATES, ORDER_TYPES, TIMING, BTS_PRECISION, NATIVE_CLIENT } from './constants.js';
+import { acquireIfNotHeld } from './order/async_lock.js';
+const { readOpenOrdersGuarded } = chainOrders;
+import { PATHS } from './paths.js';
+import * as Format from './order/format.js';
+import { getStorage } from './storage/index.js';
 const storage = getStorage();
-const fundRegistry = require('./fund_registry');
-const { BitShares } = require('./bitshares_client');
-const { BroadcastUncertainError } = require('./dexbot_credential_client');
-const { Config } = require('./config');
+const { ensureDir, unlink: safeUnlink } = storage;
+import * as fundRegistry from './fund_registry.js';
+import * as bitsharesModule from './bitshares_client.js';
+const { BitShares } = bitsharesModule;
+import { BroadcastUncertainError } from './dexbot_credential_client.js';
+import * as configModule from './config.js';
+const { Config } = configModule;
+import { getErrorMessage } from './utils/errors.js';
 function hasOpenOrdersSyncLoopMsSet(...args) { return require('./config').hasOpenOrdersSyncLoopMsSet(...args); }
 function getOpenOrdersSyncLoopMs(...args) { return require('./config').getOpenOrdersSyncLoopMs(...args); }
 function isGridBloated(...args) { return grid.isGridBloated(...args); }
@@ -47,8 +57,6 @@ function getActiveOrdersTotal(config) { return require('./order/utils/order').ge
 function correctAllPriceMismatches(...args) { return require('./order/utils/order').correctAllPriceMismatches(...args); }
 function isOrderOnChain(...args) { return require('./order/utils/order').isOrderOnChain(...args); }
 function parseChainOrder(...args) { return require('./order/utils/order').parseChainOrder(...args); }
-const { ensureDir, safeUnlink } = require('./utils/fs_utils');
-const { getErrorMessage } = require('./utils/errors');
 const CODE_ROOT = path.join(__dirname, '..');
 const PROFILES_DIR = PATHS.PROFILES_DIR;
 const PROFILES_BOTS_FILE = PATHS.PROFILES.BOTS_JSON;
@@ -102,7 +110,7 @@ function usesAmaGridPrice(bot) {
 /**
  * Find a bot entry in the bots config snapshot that matches a runtime config.
  * Matches by botKey or name.
- * @param {import('./types').BotsConfigSnapshot} snapshot - Bots configuration snapshot
+ * @param {import('./types.js').BotsConfigSnapshot} snapshot - Bots configuration snapshot
  * @param {Object} config - Runtime bot configuration
  * @returns {Object|null} Matched bot entry or null
  */
@@ -124,7 +132,7 @@ function findSnapshotBotForRuntimeConfig(snapshot, config) {
 }
 /**
  * Check if a runtime bot configuration requires the market adapter.
- * @param {import('./types').BotsConfigSnapshot} snapshot - Bots configuration snapshot
+ * @param {import('./types.js').BotsConfigSnapshot} snapshot - Bots configuration snapshot
  * @param {Object} config - Runtime bot configuration
  * @returns {boolean} True if the bot uses AMA grid pricing
  */
@@ -196,7 +204,7 @@ async function maybeRunTargetedDriftReconciliation(bot, context) {
     const now = Date.now();
     const cooldownMs = Number.isFinite(Number(bot._targetedDriftSyncCooldownMs))
         ? Number(bot._targetedDriftSyncCooldownMs)
-        : 60_000;
+        : TIMING.TARGETED_DRIFT_SYNC_COOLDOWN_MS;
     const lastSyncAt = Number(bot._lastTargetedDriftSyncAt || 0);
     if (lastSyncAt > 0 && now - lastSyncAt < cooldownMs) {
         bot._log(`[TARGETED-SYNC] Deferring ${context} reconciliation for ${Math.ceil((cooldownMs - (now - lastSyncAt)) / TIMING.MILLISECONDS_PER_SECOND)}s: ${trigger.reason}`, 'debug');
@@ -245,7 +253,7 @@ async function maybeRunTargetedDriftReconciliation(bot, context) {
 }
 /**
  * Load and fingerprint the bots.json configuration file.
- * @returns {import('./types').BotsConfigSnapshot} Snapshot with exists flag, fingerprint, active bots list, and adapter requirement
+ * @returns {import('./types.js').BotsConfigSnapshot} Snapshot with exists flag, fingerprint, active bots list, and adapter requirement
  */
 function loadBotsConfigSnapshot() {
     if (!storage.exists(PROFILES_BOTS_FILE)) {
@@ -363,9 +371,9 @@ async function stopMarketAdapterPm2() {
 /**
  * Synchronize market adapter state based on periodic config checks.
  * Starts or stops the market adapter based on whether any active bot uses AMA grid pricing.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} [context='periodic'] - Context label for logging
- * @returns {Promise<import('./types').MarketAdapterSyncResult>}
+ * @returns {Promise<import('./types.js').MarketAdapterSyncResult>}
  */
 async function syncMarketAdapterOnPeriodicConfigCheck(bot, context = 'periodic') {
     if (bot._marketAdapterWatchdogInFlight) {
@@ -499,9 +507,9 @@ async function syncMarketAdapterOnPeriodicConfigCheck(bot, context = 'periodic')
 /**
  * Refresh the dynamic weight distribution from the AMA center snapshot.
  * Applies live dynamic weights if the bot is whitelisted and weights are ready.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} [context='runtime'] - Context label for logging
- * @returns {import('./types').DynamicWeightRefreshResult}
+ * @returns {import('./types.js').DynamicWeightRefreshResult}
  */
 function refreshDynamicWeightDistribution(bot, context = 'runtime') {
     const baseWeights = cloneWeightDistribution(bot._baseWeightDistribution, bot.config?.weightDistribution || bot.manager?.config?.weightDistribution);
@@ -557,7 +565,7 @@ function refreshDynamicWeightDistribution(bot, context = 'runtime') {
  * Read and parse a trigger file's metadata payload.
  * Determines whether the trigger originated from the market adapter or was manual.
  * @param {string} triggerFile - Path to the trigger file
- * @returns {import('./types').GridResyncMetadata} Parsed trigger metadata
+ * @returns {import('./types.js').GridResyncMetadata} Parsed trigger metadata
  */
 function readTriggerMetadata(triggerFile) {
     const manualTriggerMetadata = (payload = null) => ({
@@ -591,7 +599,7 @@ function readTriggerMetadata(triggerFile) {
  * Build grid resync metadata from a reason string.
  * Maps known reason strings to structured metadata with refresh flags.
  * @param {string} reason - Resync reason identifier (e.g. 'manual_grid_resync', 'rms_structural_grid_resync')
- * @returns {import('./types').GridResyncMetadata}
+ * @returns {import('./types.js').GridResyncMetadata}
  */
 function buildGridResyncMetadata(reason) {
     const resetSource = String(reason || '').trim() || 'dexbot_grid_resync';
@@ -615,8 +623,8 @@ function buildGridResyncMetadata(reason) {
 }
 /**
  * Build grid resync options from a reason string or metadata object.
- * @param {string|import('./types').GridResyncMetadata} reasonOrMetadata - Reason string or metadata object
- * @returns {import('./types').GridResyncOptions}
+ * @param {string|import('./types.js').GridResyncMetadata} reasonOrMetadata - Reason string or metadata object
+ * @returns {import('./types.js').GridResyncOptions}
  */
 function buildGridResyncOptions(reasonOrMetadata) {
     const metadata = typeof reasonOrMetadata === 'string'
@@ -703,8 +711,8 @@ function updateBotGridResetMetadata(botKey, options = {}) {
 /**
  * Perform a full grid resync: reload config, optionally refresh center price,
  * recalculate the grid, persist, and record reset metadata.
- * @param {import('./dexbot_class').DEXBot} bot
- * @param {import('./types').GridResyncOptions} [options] - Grid resync options
+ * @param {import('./dexbot_class.js').DEXBot} bot
+ * @param {import('./types.js').GridResyncOptions} [options] - Grid resync options
  * @param {boolean} [options.skipIdle=false] - Skip the idle-cooldown deferral
  * @returns {Promise<boolean>} True if resync succeeded
  */
@@ -843,7 +851,7 @@ function performGridResync(bot, options = {}) {
 /**
  * Handle a pending trigger file detected at startup or during runtime.
  * Processes the trigger and performs a grid resync if the trigger file exists.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Promise<boolean>} True if reset was handled successfully
  */
 async function handlePendingTriggerReset(bot) {
@@ -864,7 +872,7 @@ async function handlePendingTriggerReset(bot) {
 /**
  * Set up a file watcher on the profiles directory to detect trigger file creation.
  * When a trigger file appears, debounces and processes the grid resync.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Promise<void>}
  */
 async function setupTriggerFileDetection(bot) {
@@ -923,7 +931,7 @@ async function setupTriggerFileDetection(bot) {
 /**
  * Perform periodic grid health checks (divergence, spread condition, dust detection).
  * Called as part of the periodic blockchain fetch interval.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Promise<void>}
  */
 async function performPeriodicGridChecks(bot) {
@@ -936,7 +944,7 @@ async function performPeriodicGridChecks(bot) {
 }
 /**
  * Check if the continuous open-orders sync loop is enabled.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {boolean} True if the sync loop is enabled in TIMING config
  */
 function isOpenOrdersSyncLoopEnabled(bot) {
@@ -948,7 +956,7 @@ function isOpenOrdersSyncLoopEnabled(bot) {
 /**
  * Start the continuous open-orders sync loop.
  * Periodically reads on-chain orders and synchronizes with the grid manager.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 function startOpenOrdersSyncLoop(bot) {
     if (bot._mainLoopPromise)
@@ -1022,7 +1030,7 @@ function startOpenOrdersSyncLoop(bot) {
 }
 /**
  * Stop the continuous open-orders sync loop.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Promise<void>}
  */
 async function stopOpenOrdersSyncLoop(bot) {
@@ -1034,7 +1042,7 @@ async function stopOpenOrdersSyncLoop(bot) {
 /**
  * Set up the periodic blockchain fetch interval.
  * Periodically fetches account totals and syncs open orders from the blockchain.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 function setupBlockchainFetchInterval(bot) {
     let intervalMin = bot.config?.timing?.BLOCKCHAIN_FETCH_INTERVAL_MIN;
@@ -1159,7 +1167,7 @@ function setupBlockchainFetchInterval(bot) {
 }
 /**
  * Stop the periodic blockchain fetch interval.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 function stopBlockchainFetchInterval(bot) {
     if (bot._blockchainFetchInterval !== null && bot._blockchainFetchInterval !== undefined) {
@@ -1171,10 +1179,10 @@ function stopBlockchainFetchInterval(bot) {
 /**
  * Release the market adapter runtime for a bot.
  * In PM2 mode this is a no-op; in direct mode it calls the shared runtime's releaseBot.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} botId - Bot identifier
  * @param {string} [context='shutdown'] - Context label for logging
- * @returns {Promise<import('./types').MarketAdapterReleaseResult>}
+ * @returns {Promise<import('./types.js').MarketAdapterReleaseResult>}
  */
 async function releaseMarketAdapterRuntime(_bot, botId, context = 'shutdown') {
     if (isPm2Runtime()) {
@@ -1195,25 +1203,14 @@ async function releaseMarketAdapterRuntime(_bot, botId, context = 'shutdown') {
 // Dust is cancelled immediately on detection — no timer infrastructure needed.
 /**
  * Check if an error message indicates that an order does not exist on the blockchain.
+ * Delegates to the canonical implementation in order/utils/order.ts (single source
+ * of truth shared with the correction/reconcile-cancel paths).
  * @param {string} message - Error message to check
  * @param {string} [orderId] - Optional order ID for context-aware matching
  * @returns {boolean} True if the message indicates a nonexistent order
  */
 function isOrderDoesNotExistError(message, orderId) {
-    if (typeof message !== 'string' || message.length === 0)
-        return false;
-    const normalized = message.toLowerCase();
-    if (/\border\b.*\bdoes not exist\b/i.test(message))
-        return true;
-    if (/\bdoes not exist\b.*\border\b/i.test(message))
-        return true;
-    if (orderId && normalized.includes(String(orderId).toLowerCase())) {
-        return /\bdoes not exist\b/i.test(message)
-            || /\bcould not find object\b/i.test(message)
-            || /\bunable to find object\b/i.test(message)
-            || /\bobject\b.*\bnot found\b/i.test(message);
-    }
-    return false;
+    return require('./order/utils/order').isOrderGoneErrorMessage(message, orderId);
 }
 /**
  * Calculate the remaining idle delay (ms) before grid maintenance can proceed.
@@ -1224,7 +1221,7 @@ function isOrderDoesNotExistError(message, orderId) {
 function getMaintenanceIdleDelayMs(ctx) {
     const settleDelayMs = Number.isFinite(TIMING.BLOCKCHAIN_SETTLE_DELAY_MS)
         ? Math.max(0, TIMING.BLOCKCHAIN_SETTLE_DELAY_MS)
-        : 6_000;
+        : TIMING.BLOCKCHAIN_SETTLE_DELAY_MS;
     if (settleDelayMs <= 0)
         return 0;
     if (ctx?._incomingFillQueue?.length > 0)
@@ -1265,7 +1262,7 @@ function scheduleMaintenanceAfterIdle(ctx, context, options = {}) {
 /**
  * Schedule a deferred grid resync after idle delay elapses.
  * @param {Object} ctx - Bot context
- * @param {import('./types').GridResyncOptions} [options] - Grid resync options
+ * @param {import('./types.js').GridResyncOptions} [options] - Grid resync options
  */
 function scheduleDeferredGridResync(ctx, options = {}) {
     if (!ctx ||
@@ -1278,7 +1275,7 @@ function scheduleDeferredGridResync(ctx, options = {}) {
     const triggerFileWasPresent = !!(ctx.triggerFile && storage.exists(ctx.triggerFile));
     const settleDelayMs = Number.isFinite(TIMING.BLOCKCHAIN_SETTLE_DELAY_MS)
         ? Math.max(0, TIMING.BLOCKCHAIN_SETTLE_DELAY_MS)
-        : 6_000;
+        : TIMING.BLOCKCHAIN_SETTLE_DELAY_MS;
     const delayMs = idleDelayMs + settleDelayMs;
     if (!(delayMs > 0))
         return;
@@ -1309,7 +1306,7 @@ function scheduleDeferredGridResync(ctx, options = {}) {
  * Execute the core maintenance logic: recalculate funds, check pipeline,
  * refresh dynamic weights, check grid health, cancel dust orders,
  * apply divergence corrections, and fix spread conditions.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} context - Context label for logging (e.g. 'periodic', 'dust-timer')
  * @returns {Promise<void>}
  */
@@ -1564,8 +1561,8 @@ async function executeMaintenanceLogic(bot, context) {
  * broadcast), so a BROADCAST_DEADLINE reply cannot be caused by an expired
  * session — re-using the original signing token is correct.
  *
- * @param {import('./dexbot_class').DEXBot} bot
- * @param {import('./types').Order} order
+ * @param {import('./dexbot_class.js').DEXBot} bot
+ * @param {import('./types.js').Order} order
  * @returns {Promise<*>} Result from chainOrders.cancelOrder
  */
 async function cancelOrderDeferredOnUncertain(bot, order) {
@@ -1584,10 +1581,10 @@ async function cancelOrderDeferredOnUncertain(bot, order) {
  * Each dust order is cancelled on chain and its slot is rotated through
  * the normal synthetic-fill pipeline. Failures are logged and retried on
  * the next detection cycle (next fill batch or 5-min health check).
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {Object} [options] - Dust cancellation options
- * @param {import('./types').Order[]} [options.buy=[]] - Buy-side dust orders
- * @param {import('./types').Order[]} [options.sell=[]] - Sell-side dust orders
+ * @param {import('./types.js').Order[]} [options.buy=[]] - Buy-side dust orders
+ * @param {import('./types.js').Order[]} [options.sell=[]] - Sell-side dust orders
  * @returns {Promise<{cancelledCount: number, batchResult: {aborted: boolean}|null}>}
  */
 async function cancelDustOrders(bot, { buy: buyDust = [], sell: sellDust = [] } = {}) {
@@ -1653,7 +1650,7 @@ async function cancelDustOrders(bot, { buy: buyDust = [], sell: sellDust = [] }
 /**
  * Run grid maintenance with idle detection and lock acquisition.
  * Checks if the bot is idle before proceeding, and acquires the fill processing lock.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} [context='periodic'] - Context label for logging
  * @param {Object} [options] - Maintenance options
  * @param {boolean} [options.skipIdle=false] - Skip idle delay check
@@ -1696,7 +1693,7 @@ const _lastBtsAcquisitionTimestamps = new Map();
  * Check if the bot's BTS balance is below the minimum threshold and trigger acquisition.
  * Only applies to non-BTS pairs. Uses hysteresis: triggers at 1× min_BTS_value,
  * fills to BTS_ACQUIRE_TARGET_MULTIPLIER × min_BTS_value.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Promise<void>}
  */
 async function checkBtsBalanceAndAcquire(bot) {
@@ -1745,7 +1742,7 @@ async function checkBtsBalanceAndAcquire(bot) {
 /**
  * Acquire BTS by swapping one of the trading pair assets through an AMM pool.
  * Tries both assets for a BTS pool, picks the best (lowest price impact).
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {number} deficit - Amount of BTS needed (float)
  * @returns {Promise<void>}
  */
@@ -1833,7 +1830,7 @@ async function acquireBts(bot, deficit) {
 }
 /**
  * Run a single dust health check cycle.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 async function runDustHealthCheck(bot) {
     if (bot._shuttingDown || !bot.manager)
@@ -1855,11 +1852,15 @@ async function runDustHealthCheck(bot) {
                 }, { timeout: TIMING.DUST_CANCEL_TIMEOUT_MS });
             }
             else {
-                bot._warn('[DUST] Fill lock unavailable — cancelling dust without lock (potential race)');
-                await bot._cancelDustOrders({
-                    buy: health.buyDustOrders,
-                    sell: health.sellDustOrders,
-                });
+                // Lock-bypass guard: never cancel dust without the fill lock.
+                // _cancelDustOrders rotates slots through the synthetic-fill
+                // pipeline (mutating grid + broadcasting), which races the fill
+                // consumer if run concurrently. When the lock is unavailable
+                // (shutdown/teardown or a partial manager), defer instead — the
+                // dust orders stay on the book and are picked up by the next
+                // periodic maintenance tick or fill batch, both of which run
+                // under the lock.
+                bot._warn('[DUST] Fill lock unavailable — deferring dust cancel to the next locked maintenance cycle (skipped this tick to avoid racing fill processing)');
             }
         }
     }
@@ -1874,7 +1875,7 @@ async function runDustHealthCheck(bot) {
 }
 /**
  * Set up the periodic dust health check interval.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 function setupDustHealthCheckInterval(bot) {
     bot._dustHealthCheckTimer = setInterval(() => {
@@ -1886,7 +1887,7 @@ function setupDustHealthCheckInterval(bot) {
 }
 /**
  * Request a full grid reset from fresh on-chain state.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} [reason='structural change']
  * @param {{refreshCenterPrice?: boolean, skipIdle?: boolean, skipFillLock?: boolean}} [options={}]
  * @returns {Promise<Object>}
@@ -1904,10 +1905,13 @@ async function requestGridReset(bot, reason = 'structural change', options = {})
     // Structural recovery resyncs are chain-read/rebuild operations that do not
     // mutate order state in place; running them behind the fill-processing lock
     // lets a fill storm starve recovery indefinitely. Skip the lock when the
-    // caller explicitly requests it (structural resync wiring). While the
-    // rebuild runs unlocked, raise _recoverySyncInFlight so the fill consumer
-    // defers fills entirely (no concurrent mutation race) instead of racing
-    // them behind a lock.
+    // caller explicitly requests it (structural resync wiring), when there is
+    // no lock, or when re-entrant (already under the fill lock — acquire()
+    // would just re-run the callback and we want the in-flight guard active
+    // so the fill consumer defers fills entirely). While the rebuild runs
+    // unlocked, raise _recoverySyncInFlight so the fill consumer defers fills
+    // entirely (no concurrent mutation race) instead of racing them behind a
+    // lock.
     if (options.skipFillLock === true || !bot.manager._fillProcessingLock || bot.manager._fillProcessingLock.isReentrant()) {
         bot._recoverySyncInFlight = (bot._recoverySyncInFlight || 0) + 1;
         try {
@@ -1921,7 +1925,7 @@ async function requestGridReset(bot, reason = 'structural change', options = {})
 }
 /**
  * Wire the structural grid resync request handler on the manager.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  */
 function wireStructuralGridResyncRequest(bot) {
     if (!bot.manager || bot.manager.requestStructuralGridResync)
@@ -2002,7 +2006,7 @@ function wireStructuralGridResyncRequest(bot) {
 }
 /**
  * Get current pipeline signal state for congestion checks.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Object}
  */
 function getPipelineSignals(bot) {
@@ -2017,7 +2021,7 @@ function getPipelineSignals(bot) {
 }
 /**
  * Mark that grid activity occurred (updates idle timer).
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} [reason='activity']
  */
 function markGridActivity(bot, reason = 'activity') {
@@ -2026,7 +2030,7 @@ function markGridActivity(bot, reason = 'activity') {
 }
 /**
  * Get current metrics for monitoring and debugging.
- * @param {import('./dexbot_class').DEXBot} bot
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @returns {Object}
  */
 function getMetrics(bot) {
@@ -2043,11 +2047,27 @@ function getMetrics(bot) {
 }
 /**
  * Read open orders from chain, sync with local state, and process any fills found.
- * @param {import('./dexbot_class').DEXBot} bot
+ * Self-guarding wrapper: every path here mutates grid state and can broadcast a
+ * rebalance (via _processFillsWithBatching), so it must never run concurrently
+ * with the fill consumer. acquireIfNotHeld serializes an unlocked caller through
+ * _fillProcessingLock while letting a caller that already holds the lock (e.g.
+ * the fill pipeline or grid maintenance) run the impl directly — a future caller
+ * that forgets to take the lock is serialized instead of silently bypassing it.
+ * @param {import('./dexbot_class.js').DEXBot} bot
  * @param {string} tag - Context label for logging
  * @returns {Promise<Object>}
  */
 async function syncOpenOrdersAndProcessFills(bot, tag) {
+    return acquireIfNotHeld(bot.manager?._fillProcessingLock, () => syncOpenOrdersAndProcessFillsImpl(bot, tag));
+}
+/**
+ * Read open orders from chain, sync with local state, and process any fills found.
+ * Implementation — callers MUST already hold _fillProcessingLock (see wrapper).
+ * @param {import('./dexbot_class.js').DEXBot} bot
+ * @param {string} tag - Context label for logging
+ * @returns {Promise<Object>}
+ */
+async function syncOpenOrdersAndProcessFillsImpl(bot, tag) {
     if (!bot.accountId || bot.config?.dryRun) {
         return { syncResult: null, aborted: false, hasUnmatched: 0, openOrders: null };
     }
@@ -2097,7 +2117,8 @@ async function syncOpenOrdersAndProcessFills(bot, tag) {
         return { syncResult: null, aborted: true, hasUnmatched: -1, openOrders: null };
     }
 }
-module.exports = {
+export { loadBotsConfigSnapshot, refreshDynamicWeightDistribution, performGridResync, updateBotGridResetMetadata, handlePendingTriggerReset, setupTriggerFileDetection, performPeriodicGridChecks, isOpenOrdersSyncLoopEnabled, startOpenOrdersSyncLoop, stopOpenOrdersSyncLoop, setupBlockchainFetchInterval, stopBlockchainFetchInterval, executeMaintenanceLogic, cancelDustOrders, isOrderDoesNotExistError, runGridMaintenance, stopMarketAdapterPm2, releaseMarketAdapterRuntime, syncMarketAdapterOnPeriodicConfigCheck, findSnapshotBotForRuntimeConfig, runtimeConfigNeedsMarketAdapter, usesAmaGridPrice, checkBtsBalanceAndAcquire, acquireBts, runDustHealthCheck, setupDustHealthCheckInterval, requestGridReset, wireStructuralGridResyncRequest, getPipelineSignals, markGridActivity, getMetrics, syncOpenOrdersAndProcessFills };
+export default {
     loadBotsConfigSnapshot,
     refreshDynamicWeightDistribution,
     performGridResync,