@reefclaw/openclaw-plugin 0.1.3 → 0.1.5

package/ccxt/binance-public.d.ts

@@ -1,6 +1,7 @@
 import type { CcxtTicker, CcxtOHLCV } from '../types.js';
 import type { OrderBookDepth } from '../simulator/types.js';
-export declare class BinancePublicApi {
+import type { PublicMarketDataApi } from './public-market-data-api.js';
+export declare class BinancePublicApi implements PublicMarketDataApi {
     private exchange;
     constructor();
     /** Fetch raw CCXT ticker (includes funding rate, OI, info). Returns null on error. */

package/ccxt/intel-public.d.ts

@@ -0,0 +1,25 @@
+import { type IntelApiDeps } from '../tools/intel-api.js';
+import type { CcxtTicker, CcxtOHLCV } from '../types.js';
+import type { OrderBookDepth } from '../simulator/types.js';
+import type { PublicMarketDataApi } from './public-market-data-api.js';
+type RawCcxt = Record<string, any>;
+export declare class IntelPublicApi implements PublicMarketDataApi {
+    private readonly deps;
+    constructor(deps: IntelApiDeps);
+    /** Latest price for `symbol` as a CcxtTicker (last=bid=ask=intel 1m close;
+     *  zero modeled spread — the paper fill engine models slippage itself).
+     *  Returns null on any intel error, matching BinancePublicApi's contract. */
+    fetchTicker(symbol: string): Promise<CcxtTicker | null>;
+    /** Intel has no raw-ticker/funding/OI HTTP route yet — null (same as a 451). */
+    fetchTickerRaw(): Promise<RawCcxt | null>;
+    fetchFundingRate(): Promise<RawCcxt | null>;
+    fetchOpenInterest(): Promise<RawCcxt | null>;
+    /** No intel depth route yet — null. The paper fill engine falls back to
+     *  random slippage when the book is absent (see helpers.ts fetchOrderBook). */
+    fetchOrderBook(): Promise<OrderBookDepth | null>;
+    /** No intel candle route yet — null (candle-consuming analysis tools degrade
+     *  exactly as they do under a 451; the agent's decisioning reads intel
+     *  signals/scan tools, not raw candles). */
+    fetchOHLCV(): Promise<CcxtOHLCV[] | null>;
+}
+export {};

package/ccxt/intel-public.js

@@ -0,0 +1,80 @@
+// Intel-backed public market-data source for PAPER mode on hosts that Binance
+// geo-blocks (HTTP 451). Paper execution is fully simulated; it only needs a
+// price, which the intel service serves from its own (permitted-region) feeds
+// via GET /api/price/:symbol (latest 1m close, ≤~60s stale). See
+// docs/AGENT_READINESS_GATE_PLAN.md sibling + the wisekid 451 findings.
+//
+// Selected only when plugin-config `paperMarketDataSource: 'intel'` AND paper
+// mode (index.ts). Live trading NEVER uses this — live needs the real exchange.
+//
+// Increment 1 serves PRICE only (fetchTicker), which is the trading-critical
+// path: create_order / close_position price + the PaperMarketFeed mark loop.
+// Candles / order book / funding / OI have no intel HTTP route yet, so those
+// return null here — the SAME shape a 451 produces today (no regression), and
+// order-book absence is already handled by the fill engine's random-slippage
+// fallback. Candle/book/OI intel routes are a follow-up.
+import { logger } from '../logger.js';
+import { fetchIntelApi } from '../tools/intel-api.js';
+const TAG = 'intel-public';
+/** CCXT symbol (`BTC/USDT`, `BTC/USDT:USDT`) → intel format (`BTCUSDT`). */
+function toIntelSymbol(symbol) {
+    return symbol.replace('/', '').replace(/:.*$/, '');
+}
+export class IntelPublicApi {
+    deps;
+    constructor(deps) {
+        this.deps = deps;
+        logger.info(TAG, 'Intel public market-data source active (paper mode, Binance reads routed to intel)');
+    }
+    /** Latest price for `symbol` as a CcxtTicker (last=bid=ask=intel 1m close;
+     *  zero modeled spread — the paper fill engine models slippage itself).
+     *  Returns null on any intel error, matching BinancePublicApi's contract. */
+    async fetchTicker(symbol) {
+        const res = await fetchIntelApi(`/api/price/${toIntelSymbol(symbol)}`, this.deps);
+        if ('error' in res) {
+            logger.warn(TAG, `fetchTicker(${symbol}) via intel failed: ${res.error}`);
+            return null;
+        }
+        const body = res;
+        const price = Number(body.price);
+        if (!Number.isFinite(price) || price <= 0) {
+            logger.warn(TAG, `fetchTicker(${symbol}) via intel returned a bad price: ${body.price}`);
+            return null;
+        }
+        const timestamp = Number.isFinite(body.time) ? body.time : Date.now();
+        return {
+            // Key on the ORIGINAL ccxt symbol so the simulator's per-symbol maps line up.
+            symbol,
+            last: price,
+            bid: price,
+            ask: price,
+            baseVolume: 0,
+            quoteVolume: 0,
+            change: 0,
+            percentage: 0,
+            timestamp,
+            datetime: new Date(timestamp).toISOString(),
+        };
+    }
+    /** Intel has no raw-ticker/funding/OI HTTP route yet — null (same as a 451). */
+    async fetchTickerRaw() {
+        return null;
+    }
+    async fetchFundingRate() {
+        return null;
+    }
+    async fetchOpenInterest() {
+        return null;
+    }
+    /** No intel depth route yet — null. The paper fill engine falls back to
+     *  random slippage when the book is absent (see helpers.ts fetchOrderBook). */
+    async fetchOrderBook() {
+        return null;
+    }
+    /** No intel candle route yet — null (candle-consuming analysis tools degrade
+     *  exactly as they do under a 451; the agent's decisioning reads intel
+     *  signals/scan tools, not raw candles). */
+    async fetchOHLCV() {
+        return null;
+    }
+}

package/ccxt/public-market-data-api.d.ts

@@ -0,0 +1,12 @@
+import type { CcxtTicker, CcxtOHLCV } from '../types.js';
+import type { OrderBookDepth } from '../simulator/types.js';
+type RawCcxt = Record<string, any>;
+export interface PublicMarketDataApi {
+    fetchTickerRaw(symbol: string): Promise<RawCcxt | null>;
+    fetchTicker(symbol: string): Promise<CcxtTicker | null>;
+    fetchFundingRate(symbol: string): Promise<RawCcxt | null>;
+    fetchOpenInterest(symbol: string): Promise<RawCcxt | null>;
+    fetchOrderBook(symbol: string, limit?: number): Promise<OrderBookDepth | null>;
+    fetchOHLCV(symbol: string, timeframe?: string, limit?: number): Promise<CcxtOHLCV[] | null>;
+}
+export {};

package/ccxt/public-market-data-api.js

@@ -0,0 +1,9 @@
+// The public market-data surface paper mode reads (prices, candles, book,
+// funding, OI). Both BinancePublicApi (direct Binance) and IntelPublicApi
+// (via the intel service, for Binance-451-geo-blocked hosts) implement it, so
+// the paper wiring can select a source without any consumer knowing which.
+//
+// Deliberately EXCLUDES probeReachability: that is Binance-specific and only
+// the readiness reporter uses it — it must always probe the REAL Binance host
+// to detect the 451, so it keeps a concrete BinancePublicApi, never this.
+export {};

package/config/plugin-config-io.d.ts

@@ -21,8 +21,23 @@ export interface PluginConfigFile {
      *  prod runs the bridge under systemd and a supervisor there would
      *  double-connect the relay room. Kill-switch: RC_CONNECTOR_SUPERVISOR=off. */
     connectorSupervisor?: 'on' | 'off';
+    /** Where PAPER mode reads market-data prices from. 'binance' (default) hits
+     *  Binance public endpoints directly; 'intel' routes them through the intel
+     *  service (GET /api/price/:symbol) so a host that Binance geo-blocks (HTTP
+     *  451) can still paper-trade. PAPER-ONLY + advisory: live trading always
+     *  uses the real exchange, and readiness still probes real Binance to detect
+     *  the 451. Kill-switch: RC_PAPER_MARKET_DATA=binance forces the default. */
+    paperMarketDataSource?: 'binance' | 'intel';
     apiBaseUrl?: string;
     intelligenceUrl?: string;
+    /** Exchange credentials + venue selection. `exchange.venue` picks the
+     *  trading venue ('binance' default | 'hyperliquid'); it is LOCAL mechanism
+     *  (never central-pushable) per docs/HYPERLIQUID_INTEGRATION_PLAN.md §5.2.
+     *  Until Phase 3 ships the hyperliquid adapter, a non-PAPER mode on
+     *  venue='hyperliquid' falls back to PAPER at boot. NOTE: updatePluginConfig
+     *  replaces this block wholesale (the documented `exchange: null` clears
+     *  credentials), so any writer (set_exchange_credentials, SSH edits) must
+     *  carry `venue` through or it resets to the binance default. */
     exchange?: ExchangeConfig;
     tradingMode?: TradingMode;
     microLive?: {

package/index.js

@@ -11,6 +11,7 @@ import { homedir } from 'node:os';
 import { join, dirname } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { BinancePublicApi } from './ccxt/binance-public.js';
+import { IntelPublicApi } from './ccxt/intel-public.js';
 import { BinancePrivateApi } from './ccxt/binance-private.js';
 import { ExchangeSimulator } from './simulator/exchange-simulator.js';
 import { ShadowTracker } from './shadow/shadow-tracker.js';
@@ -18,7 +19,8 @@ import { StateManager } from './persistence/state-manager.js';
 import { logger, setLogLevel, formatError } from './logger.js';
 import { DEFAULT_CONFIG } from './types.js';
 import { PaperAdapter } from './paper-adapter.js';
-import { LiveAdapter } from './live/live-adapter.js';
+import { createLiveAdapter, fillExchangeId, isLiveVenueSupported, parseVenue, venueQuoteCurrency, } from './venues/registry.js';
+import { HyperliquidPublicApi } from './venues/hyperliquid/hl-public.js';
 import { loadBracketMode } from './config/brackets-config.js';
 import { loadUserDataStreamMode, loadUserDataStreamTunables, loadUserDataStreamDbWrite, getUserDataStreamIngestBaseUrl, resolveIngestToken, resolveReefclawUserId, } from './config/user-data-stream-config.js';
 import { TradeStoreClient } from './ingest/trade-store-client.js';
@@ -874,11 +876,24 @@ const paperTradingPlugin = {
             return;
         }
         logger.info(TAG, 'Initializing paper trading plugin...');
-        // Resolve config from plugin settings
+        // Resolve config from plugin settings. The paper wallet's quote currency
+        // follows the VENUE (USDC on hyperliquid, USDT on binance — issue #174:
+        // a hardcoded USDT wallet reads as $0 equity on the USDC venue). The
+        // authoritative plugin-config read happens further down, AFTER the
+        // simulator exists — peek only the venue here; same file, same parse
+        // rules, and an unreadable config falls back to the binance default
+        // exactly like the main read does.
+        let paperQuoteCurrency = DEFAULT_CONFIG.quoteCurrency;
+        try {
+            paperQuoteCurrency = venueQuoteCurrency(parseVenue(readPluginConfig().exchange?.venue).venue);
+        }
+        catch {
+            /* unreadable plugin-config → binance default, matching the main read */
+        }
         const pluginConfig = {
             startingBalance: DEFAULT_CONFIG.startingBalance,
             symbol: DEFAULT_CONFIG.symbol,
-            quoteCurrency: DEFAULT_CONFIG.quoteCurrency,
+            quoteCurrency: paperQuoteCurrency,
         };
         logger.info(TAG, `Config: ${pluginConfig.startingBalance} ${pluginConfig.quoteCurrency}, symbol: ${pluginConfig.symbol}`);
         // Load or initialize state (sync — OpenClaw ignores async register())
@@ -912,6 +927,14 @@ const paperTradingPlugin = {
         let intelligenceUrl = 'https://intel.reefclaw.com';
         let exchangeConfig = null;
         let tradingMode = 'PAPER';
+        // Trading venue (multi-venue Phase 0 — docs/HYPERLIQUID_INTEGRATION_PLAN.md).
+        // Absent config field = 'binance', so every pre-venue install is untouched.
+        let venue = 'binance';
+        // exchange.testnet as written on disk — needed independently of
+        // exchangeConfig because the hyperliquid venue never builds a Binance
+        // credential config (Phase 0 gate) but its keyless public API still needs
+        // the mainnet/testnet routing.
+        let venueTestnet = false;
         let signalsEvaluator = 'central';
         let signalsSymbols = [];
         try {
@@ -928,9 +951,18 @@ const paperTradingPlugin = {
                 if (Array.isArray(rcConfig?.signals?.symbols)) {
                     signalsSymbols = rcConfig.signals.symbols.map((s) => String(s).toUpperCase());
                 }
-                // Read exchange credentials (Phase 9b)
+                // Read exchange credentials (Phase 9b) + venue (multi-venue Phase 0)
                 const exchange = rcConfig?.exchange;
-                if (exchange?.apiKey && exchange?.secret) {
+                const venueParse = parseVenue(exchange?.venue);
+                venue = venueParse.venue;
+                venueTestnet = exchange?.testnet === true;
+                if (venueParse.unrecognized) {
+                    logger.warn(TAG, `Unrecognized exchange.venue '${venueParse.unrecognized}' in plugin-config — treating as 'binance'`);
+                }
+                // apiKey/secret are a BINANCE credential pair; never build a Binance
+                // client config for another venue (a hyperliquid block carries
+                // walletAddress/agentPrivateKey instead — consumed from Phase 3 on).
+                if (venue === 'binance' && exchange?.apiKey && exchange?.secret) {
                     exchangeConfig = {
                         apiKey: exchange.apiKey,
                         secret: exchange.secret,
@@ -964,6 +996,14 @@ const paperTradingPlugin = {
         else {
             logger.warn(TAG, 'No connection token found — get_market_intel will return errors');
         }
+        // Venue support gate (multi-venue Phase 0). Checked BEFORE the credentials
+        // gate so a hyperliquid config gets the accurate diagnosis ("venue not
+        // supported yet"), not a misleading "missing API keys". Fallback-to-PAPER,
+        // never a throw — register() must stay non-crashing.
+        if (tradingMode !== 'PAPER' && !isLiveVenueSupported(venue)) {
+            logger.warn(TAG, `Trading mode ${tradingMode} on venue '${venue}' is not supported in this build — falling back to PAPER (live support arrives in Phase 3 of docs/HYPERLIQUID_INTEGRATION_PLAN.md)`);
+            tradingMode = 'PAPER';
+        }
         // Validate trading mode vs credentials
         if (tradingMode !== 'PAPER' && !exchangeConfig) {
             logger.warn(TAG, `Trading mode ${tradingMode} requires exchange API keys — falling back to PAPER`);
@@ -1063,6 +1103,9 @@ const paperTradingPlugin = {
             // closure — only invoked on a fill, long after `runtime` is built), so it
             // follows a live<->paper reconnect.
             resolveMode: () => (runtime.adapter.isLive ? 'live' : 'paper'),
+            // Venue tag (positions.exchange, migration 0058) — static per process;
+            // a venue change requires config edit + restart.
+            venue,
         };
         if (positionDecisionsClient) {
             logger.info(TAG, `Journal close-on-reduce-only-fill ${closeOnReduceOnlyFill ? 'ENABLED' : 'disabled'}`);
@@ -1146,6 +1189,11 @@ const paperTradingPlugin = {
                     tradeIngest = {
                         client: new TradeStoreClient({ baseUrl: ingestBaseUrl, ingestToken }),
                         userId: reefclawUserId,
+                        // Venue-derived FillEvent.exchange — 'binance_futures' for the
+                        // binance venue, i.e. byte-identical to the pre-venue literal
+                        // WsIngest defaulted to. Half of the (exchange, exchange_trade_id)
+                        // audit-trail idempotency key; never a fresh string literal.
+                        exchange: fillExchangeId(venue),
                     };
                     logger.info(TAG, `User-data stream dbWrite=on — WS audit-trail ingest wired to ${ingestBaseUrl} (userId=${reefclawUserId.slice(0, 8)}…)`);
                 }
@@ -1203,7 +1251,12 @@ const paperTradingPlugin = {
                     logger.warn(TAG, `approval mode=${approvalModeForTool} but WEBAPP_INGEST_TOKEN or REEFCLAW_USER_ID env missing — proposal path disabled`);
                     approvalModeForTool = 'off';
                 }
-                const liveAdapter = new LiveAdapter(exchangeConfig, tradingMode, microLiveConfig, bracketMode, userDataStreamMode, userDataStreamTunables, tradeIngest, autoCapture);
+                // Venue-dispatched construction (multi-venue Phase 0). For 'binance'
+                // this is a pure pass-through to `new LiveAdapter(...)` — identical
+                // args, identical behavior; the unsupported-venue arm is unreachable
+                // here because the isLiveVenueSupported() gate above already fell back
+                // to PAPER. Phase 3 adds the hyperliquid adapter inside the factory.
+                const liveAdapter = createLiveAdapter(venue, exchangeConfig, tradingMode, microLiveConfig, bracketMode, userDataStreamMode, userDataStreamTunables, tradeIngest, autoCapture);
                 adapter = liveAdapter;
                 // ---- Approval-mode Phase B — start ProposalDecisionListener ----
                 // Started only when approval.mode='per_trade' AND proposalManagerCtx
@@ -1320,11 +1373,64 @@ const paperTradingPlugin = {
         else {
             adapter = new PaperAdapter(simulator);
         }
+        // PAPER market-data source selection.
+        //
+        // Venue precedence (Hyperliquid Phase 1 — docs/HYPERLIQUID_INTEGRATION_PLAN.md):
+        // the configured VENUE decides which exchange prices paper mode, full stop.
+        // venue='hyperliquid' → every public read (PaperMarketFeed marks, the
+        // simulator's fill prices, fetch_ticker/fetch_ohlcv/get_orderbook tools)
+        // comes from Hyperliquid's keyless /info endpoints; the Binance-only
+        // `paperMarketDataSource:'intel'` escape hatch does not apply (intel has
+        // no HL rows until Phase 2 — silently serving Binance prices for a
+        // Hyperliquid book would be a lie, the exact class the symbol-translation
+        // rule forbids).
+        //
+        // Binance venue (default) is unchanged: a host Binance geo-blocks (HTTP
+        // 451) can route PRICE reads through the intel service by setting
+        // plugin-config `paperMarketDataSource:'intel'` (GET /api/price/:symbol).
+        // LIVE always uses the real exchange, and the readiness reporter probes
+        // the CONFIGURED venue. Kill-switch RC_PAPER_MARKET_DATA=binance forces
+        // the Binance-direct default (binance venue only).
+        //
+        // hlPublicApi is constructed once per process when the venue is
+        // hyperliquid — shared by the paper data path and the readiness probe.
+        const hlPublicApi = venue === 'hyperliquid' ? new HyperliquidPublicApi({ testnet: venueTestnet }) : null;
+        let configuredPaperSource;
+        try {
+            configuredPaperSource = readPluginConfig().paperMarketDataSource;
+        }
+        catch {
+            configuredPaperSource = undefined;
+        }
+        const paperDataSource = process.env.RC_PAPER_MARKET_DATA === 'binance'
+            ? 'binance'
+            : configuredPaperSource ?? 'binance';
+        const useIntelPaperData = venue === 'binance' &&
+            !adapter.isLive && paperDataSource === 'intel' && connectionToken.length > 0;
+        const marketDataApi = !adapter.isLive && hlPublicApi
+            ? hlPublicApi
+            : useIntelPaperData
+                ? new IntelPublicApi({ connectionToken, intelligenceUrl })
+                : binanceApi;
+        if (!adapter.isLive && hlPublicApi) {
+            logger.info(TAG, `PAPER market data sourced from Hyperliquid${venueTestnet ? ' TESTNET' : ''} (venue=hyperliquid) — use USDC pairs (e.g. BTC/USDC)`);
+            if (paperDataSource === 'intel') {
+                logger.warn(TAG, 'paperMarketDataSource=intel is a Binance-venue option — ignored on venue=hyperliquid (intel has no Hyperliquid rows until Phase 2)');
+            }
+        }
+        else if (useIntelPaperData) {
+            logger.warn(TAG, 'PAPER market data routed through intel (paperMarketDataSource=intel) — prices from GET /api/price; live + readiness still use Binance directly');
+        }
+        else if (venue === 'binance' && !adapter.isLive && paperDataSource === 'intel' && connectionToken.length === 0) {
+            logger.warn(TAG, 'paperMarketDataSource=intel requested but no connection token — falling back to Binance direct (which will 451 on a geo-blocked host)');
+        }
         // Build tool dependencies
         // `simDeps` for tools that need the simulator directly (ticker, market structure, risk scenario)
         // `adapterDeps` for the 7 adapter-based trading tools
-        const simDeps = { binanceApi, simulator };
-        const adapterDeps = { binanceApi, adapter };
+        // Both carry `marketDataApi` (Binance direct, or intel in paper mode) so
+        // every paper market-data read follows the selected source at once.
+        const simDeps = { binanceApi: marketDataApi, simulator };
+        const adapterDeps = { binanceApi: marketDataApi, adapter };
         // ---- Plugin runtime (mutable holder) ----
         // Owns the adapter + mode + stop-watcher and exposes `reconnect()` so the
         // operator can swap credentials / trading mode from the dashboard without
@@ -1357,7 +1463,7 @@ const paperTradingPlugin = {
         // resting limits fill. Paper-only — started here when the boot adapter is
         // paper; PluginRuntime.reconnect() stops it on a switch to live and
         // restarts it on a switch back to paper. Reuses the ban-gated fetchTicker.
-        const marketFeed = new PaperMarketFeed(binanceApi, simulator);
+        const marketFeed = new PaperMarketFeed(marketDataApi, simulator);
         if (!adapter.isLive) {
             marketFeed.start();
         }
@@ -2022,16 +2128,19 @@ const paperTradingPlugin = {
         pluginInitialised = true;
         logger.info(TAG, `Registered ${gatedTools.length} tools (gate mode=${toolGate.getMode()}): ${toolNames.join(', ')}. Plugin v3.8.0 (${runtime.mode} mode)`);
         // Agent-readiness reporter (docs/AGENT_READINESS_GATE_PLAN.md Phase 1):
-        // probe host→Binance reachability (HTTP 451 geo-block) + clock drift on the
-        // host and POST a plain-English report to the webapp, so a silently-broken
-        // agent shows an actionable dashboard alert instead of a false green.
-        // Advisory + fire-and-forget; no token → no-op. Runs here once (guarded by
-        // the pluginInitialised early-return → once per process) + on an unref'd
-        // interval inside the reporter.
+        // probe host→venue reachability (Binance HTTP 451 geo-block / Hyperliquid
+        // /info) + clock drift on the host and POST a plain-English report to the
+        // webapp, so a silently-broken agent shows an actionable dashboard alert
+        // instead of a false green. Probes ONLY the configured venue — on a
+        // Binance-451 host trading Hyperliquid a Binance probe would be a
+        // permanent false alarm. Advisory + fire-and-forget; no token → no-op.
+        // Runs here once (guarded by the pluginInitialised early-return → once per
+        // process) + on an unref'd interval inside the reporter.
         startReadinessReporter({
             apiBaseUrl,
             token: resolveIngestToken({ connectionToken }),
-            binanceApi,
+            venue,
+            publicApi: hlPublicApi ?? binanceApi,
             toolCount: toolNames.length,
         });
         maybeStartConnectorSupervisor();

package/ingest/position-auto-capture.d.ts

@@ -21,6 +21,11 @@ export interface AutoCaptureContext {
      *  from the current adapter (so it follows a runtime reconnect). Tags the
      *  journal position row so paper and live entries can be segregated. */
     resolveMode?: () => 'paper' | 'live';
+    /** Venue this gateway trades ('binance' | 'hyperliquid'). Static per process
+     *  — changing venue requires a config edit + restart (plan §5.2), so unlike
+     *  `resolveMode` this is a plain value, not a resolver. Tags journal rows
+     *  (positions.exchange, migration 0058). */
+    venue?: 'binance' | 'hyperliquid';
 }
 export interface CreateOrderInputs {
     symbol: string;

package/ingest/position-auto-capture.js

@@ -85,6 +85,7 @@ export async function onCreateOrderFilled(ctx, inputs, order) {
         currentSize: filledQty,
         avgEntryPrice: fillPrice,
         mode: ctx.resolveMode?.(),
+        exchange: ctx.venue,
     };
     // postPosition is awaited — we need the UUID before posting the entry row.
     const positionId = await ctx.decisionsClient.postPosition(ctx.userId, upsert);
@@ -253,6 +254,7 @@ export async function onWsFillObserved(ctx, fill) {
         currentSize: fill.fillSize,
         avgEntryPrice: fill.fillPrice,
         mode: ctx.resolveMode?.(),
+        exchange: ctx.venue,
     };
     const positionId = await ctx.decisionsClient.postPosition(ctx.userId, upsert);
     if (!positionId) {
@@ -332,8 +334,12 @@ async function handleReduceOnlyExit(ctx, fill) {
         regimeConfidence: 0.5,
         fillPrice: fill.fillPrice,
         fillSize: Math.abs(fill.fillSize),
-        // realizedPnl is Binance-exact (summed across exit fills). When it's a true
-        // break-even 0 the webapp recomputes from entries server-side — harmless.
+        // realizedPnl is Binance-exact (summed across exit fills); the zeros below
+        // are filled in PER-METRIC by the webapp close route (price-based R from
+        // the pinned invalidation_price, mfe/give-back from the last review — see
+        // webapp lib/api/close-metrics.ts). The route's old all-four-zero gate
+        // meant this non-zero PnL used to suppress that fill-in entirely, persisting
+        // realized_r=0 on every bracket_fill close (the 2026-07-11 zero-hole).
         realizedPnl,
         realizedR: 0,
         mfeRAtClose: 0,

package/ingest/position-decisions-client.d.ts

@@ -17,6 +17,10 @@ export interface PositionUpsertPayload {
      *  journal row so paper and live positions can be segregated. Omitted only
      *  if the active adapter can't be resolved (treated as legacy/NULL). */
     mode?: 'paper' | 'live';
+    /** Venue the position was opened on ('binance' | 'hyperliquid') — journal /
+     *  analytics segmentation (fees + funding cadence differ per venue).
+     *  Absent → NULL (legacy binance), mirroring `mode`. Migration 0058. */
+    exchange?: string;
 }
 export interface PositionEntryPayload {
     positionId: string;

package/ingest/readiness-reporter.d.ts

@@ -1,9 +1,23 @@
-import { type ReadinessReport } from '@reefclaw/shared';
-import type { BinancePublicApi } from '../ccxt/binance-public.js';
+import { type ReadinessReport, type VenueId } from '@reefclaw/shared';
+/** Venue-agnostic reachability probe — BinancePublicApi.probeReachability and
+ *  HyperliquidPublicApi.probeReachability both return exactly this shape. */
+export interface VenueReachabilityProbe {
+    probeReachability(): Promise<{
+        outcome: 'reachable' | 'geo_blocked' | 'unreachable' | 'unknown';
+        driftMs: number | null;
+    }>;
+}
 export interface ReadinessReporterOptions {
     apiBaseUrl: string;
     token: string;
-    binanceApi: Pick<BinancePublicApi, 'probeReachability'>;
+    /** Which venue this agent trades — decides the reachability check id
+     *  (binance_reachable vs hyperliquid_reachable) and which host `publicApi`
+     *  probes. The reporter probes ONLY the configured venue: on a
+     *  Binance-451-blocked host trading Hyperliquid, a Binance probe would be
+     *  a permanent false alarm (plan §5.11). */
+    venue: VenueId;
+    /** Public API of the CONFIGURED venue (probe only). */
+    publicApi: VenueReachabilityProbe;
     /** Number of trading tools registered (a health signal). */
     toolCount: number;
     fetchImpl?: typeof fetch;
@@ -11,8 +25,17 @@ export interface ReadinessReporterOptions {
     requestTimeoutMs?: number;
 }
 /** Run every connect-phase check once and assemble the report. Exported for
- *  unit tests. */
-export declare function collectReadiness(opts: Pick<ReadinessReporterOptions, 'binanceApi' | 'toolCount'>): Promise<ReadinessReport>;
+ *  unit tests.
+ *
+ *  `bootWarmup` marks the FIRST cycle, which fires during gateway boot while
+ *  the event loop is congested (WS start, snapshot, seed all racing). That
+ *  delay between the probe's server-time response and the local `Date.now()`
+ *  read inflates the apparent clock drift — observed −4112ms at boot
+ *  self-correcting to −108ms on the next 5-min cycle. During warm-up a
+ *  warn/fail drift is reported 'unknown' (not amber/red) so the readiness
+ *  banner doesn't cry-wolf for ~5 min after every restart; a genuinely
+ *  skewed clock still surfaces on cycle 2. */
+export declare function collectReadiness(opts: Pick<ReadinessReporterOptions, 'venue' | 'publicApi' | 'toolCount'>, bootWarmup?: boolean): Promise<ReadinessReport>;
 /** Test-only — reset the singleton guard between unit tests. */
 export declare function __resetReadinessReporterForTests(): void;
 /** Fire the readiness report once at boot then on an unref'd interval. */

package/ingest/readiness-reporter.js

@@ -24,10 +24,24 @@ function resolveIntervalMs(explicit) {
     return Math.max(MIN_INTERVAL_MS, raw);
 }
 /** Run every connect-phase check once and assemble the report. Exported for
- *  unit tests. */
-export async function collectReadiness(opts) {
+ *  unit tests.
+ *
+ *  `bootWarmup` marks the FIRST cycle, which fires during gateway boot while
+ *  the event loop is congested (WS start, snapshot, seed all racing). That
+ *  delay between the probe's server-time response and the local `Date.now()`
+ *  read inflates the apparent clock drift — observed −4112ms at boot
+ *  self-correcting to −108ms on the next 5-min cycle. During warm-up a
+ *  warn/fail drift is reported 'unknown' (not amber/red) so the readiness
+ *  banner doesn't cry-wolf for ~5 min after every restart; a genuinely
+ *  skewed clock still surfaces on cycle 2. */
+export async function collectReadiness(opts, bootWarmup = false) {
     const now = Date.now();
     const checks = [];
+    // The venue decides which reachability check this report carries; the copy
+    // for both ids lives in shared/src/readiness.ts. Clock drift comes from the
+    // same probe on both venues (Binance fapi serverTime / Hyperliquid
+    // exchangeStatus.time — live-verified 2026-07-11).
+    const reachId = opts.venue === 'hyperliquid' ? 'hyperliquid_reachable' : 'binance_reachable';
     // plugin_loaded — trivially true (this code runs inside the loaded plugin),
     // but a positive row is what proves the report path is alive at all.
     checks.push(makeReadinessCheck('plugin_loaded', 'pass', { checkedAt: now }));
@@ -36,37 +50,46 @@ export async function collectReadiness(opts) {
         detail: `${opts.toolCount} tools`,
         checkedAt: now,
     }));
-    // binance_reachable (+ clock drift from the same probe response)
-    const probe = await opts.binanceApi.probeReachability();
+    // venue reachability (+ clock drift from the same probe response)
+    const probe = await opts.publicApi.probeReachability();
     if (probe.outcome === 'reachable') {
-        checks.push(makeReadinessCheck('binance_reachable', 'pass', { checkedAt: now }));
+        checks.push(makeReadinessCheck(reachId, 'pass', { checkedAt: now }));
         const drift = probe.driftMs;
         if (drift == null) {
             checks.push(makeReadinessCheck('clock_in_sync', 'unknown', { checkedAt: now }));
         }
         else {
             const abs = Math.abs(drift);
-            const status = abs <= 1000 ? 'pass' : abs <= 5000 ? 'warn' : 'fail';
+            const rawStatus = abs <= 1000 ? 'pass' : abs <= 5000 ? 'warn' : 'fail';
+            // Boot warm-up: a non-pass reading on the first cycle is untrustworthy
+            // (event-loop congestion inflates the measured drift) — report 'unknown'
+            // and let the next cycle confirm, rather than flash the banner amber/red.
+            const status = bootWarmup && rawStatus !== 'pass' ? 'unknown' : rawStatus;
             checks.push(makeReadinessCheck('clock_in_sync', status, {
-                detail: `drift ${Math.round(drift)}ms`,
+                detail: bootWarmup && rawStatus !== 'pass'
+                    ? `drift ${Math.round(drift)}ms (boot warm-up — rechecking)`
+                    : `drift ${Math.round(drift)}ms`,
                 checkedAt: now,
             }));
         }
     }
     else if (probe.outcome === 'geo_blocked') {
-        checks.push(makeReadinessCheck('binance_reachable', 'fail', { detail: 'HTTP 451', checkedAt: now }));
+        checks.push(makeReadinessCheck(reachId, 'fail', {
+            detail: opts.venue === 'binance' ? 'HTTP 451' : 'blocked (HTTP 451/403)',
+            checkedAt: now,
+        }));
         checks.push(makeReadinessCheck('clock_in_sync', 'unknown', { checkedAt: now }));
     }
     else if (probe.outcome === 'unreachable') {
         // Network/DNS/timeout — could be transient, so warn (amber) rather than
         // asserting a definitive failure. A persistent problem stays amber across
-        // re-checks; only the 451 geo-block is a hard red.
-        checks.push(makeReadinessCheck('binance_reachable', 'warn', { detail: 'unreachable', checkedAt: now }));
+        // re-checks; only the geo-block is a hard red.
+        checks.push(makeReadinessCheck(reachId, 'warn', { detail: 'unreachable', checkedAt: now }));
         checks.push(makeReadinessCheck('clock_in_sync', 'unknown', { checkedAt: now }));
     }
     else {
         // 'unknown' — the ban/weight gate paused the probe; don't assert anything.
-        checks.push(makeReadinessCheck('binance_reachable', 'unknown', { checkedAt: now }));
+        checks.push(makeReadinessCheck(reachId, 'unknown', { checkedAt: now }));
         checks.push(makeReadinessCheck('clock_in_sync', 'unknown', { checkedAt: now }));
     }
     return {
@@ -74,7 +97,7 @@ export async function collectReadiness(opts) {
         generatedAt: now,
         overall: deriveOverallReadiness(checks),
         checks,
-        agent: { pluginVersion: PLUGIN_VERSION, toolCount: opts.toolCount },
+        agent: { pluginVersion: PLUGIN_VERSION, toolCount: opts.toolCount, venue: opts.venue },
     };
 }
 async function postReadiness(apiBaseUrl, token, report, fetchImpl, timeoutMs) {
@@ -119,12 +142,9 @@ export function startReadinessReporter(opts) {
     const fetchImpl = opts.fetchImpl ?? fetch;
     const intervalMs = resolveIntervalMs(opts.intervalMs);
     const timeoutMs = opts.requestTimeoutMs ?? 10_000;
-    const cycle = async () => {
+    const cycle = async (bootWarmup) => {
         try {
-            const report = await collectReadiness({
-                binanceApi: opts.binanceApi,
-                toolCount: opts.toolCount,
-            });
+            const report = await collectReadiness({ venue: opts.venue, publicApi: opts.publicApi, toolCount: opts.toolCount }, bootWarmup);
             await postReadiness(opts.apiBaseUrl, opts.token, report, fetchImpl, timeoutMs);
             if (report.overall === 'fail') {
                 const failing = report.checks.filter((c) => c.status === 'fail').map((c) => c.id).join(', ');
@@ -135,8 +155,9 @@ export function startReadinessReporter(opts) {
             logger.warn(TAG, `readiness cycle failed: ${formatError(err)}`);
         }
     };
-    void cycle();
-    const timer = setInterval(() => void cycle(), intervalMs);
+    // First fire is the boot cycle (warm-up); the interval cycles are steady-state.
+    void cycle(true);
+    const timer = setInterval(() => void cycle(false), intervalMs);
     timer.unref();
     logger.info(TAG, `readiness reporter started (interval ${Math.round(intervalMs / 1000)}s)`);
 }

package/live/bracket-id.d.ts

@@ -1,3 +1,4 @@
+import type { VenueId } from '@reefclaw/shared';
 import type { BracketId, BracketRole } from './bracket-types.js';
 /** Generate a fresh 16-char hex bracketId. */
 export declare function generateBracketId(): BracketId;
@@ -16,3 +17,11 @@ export declare function parseBracketCid(cid: string): {
 /** Cheap type-guard for "is this a reefclaw-managed bracket clientOrderId"
  *  — true for both the current and legacy schemes. */
 export declare function isBracketCid(cid: string): boolean;
+/** Venue-aware parseBracketCid. Binance delegates to the existing dual-scheme
+ *  parser; other venues return null until their scheme ships. */
+export declare function parseBracketClientId(venue: VenueId, cid: string): {
+    bracketId: BracketId;
+    role: BracketRole;
+} | null;
+/** Venue-aware isBracketCid. */
+export declare function isBracketClientId(venue: VenueId, cid: string): boolean;