@reefclaw/openclaw-plugin 0.1.11 → 0.1.12

package/bridge/gateway/event-parser.d.ts

@@ -155,6 +155,29 @@ export declare function extractBracketField(pos: {
     tpPrice?: number;
     state: string;
 } | undefined;
+/**
+ * Narrow the agent's PLANNED protective levels off a plugin-decorated position.
+ *
+ * `bracket` above is exchange truth and only exists where a venue-side ledger
+ * row does. In PAPER (either venue) there are no exchange legs — the levels
+ * live in the position's metadata and the stop-watcher enforces them — so
+ * without these fields the dashboard had NO target at all and drew the
+ * frozen-at-fill `originalStopPrice` as the stop even after a modify_stop.
+ * Both adapters already emit `stopPrice` / `targetPrice` on the CCXT position
+ * (ExchangeSimulator.getPositions, LiveAdapter.decoratePositionsWithMetadata);
+ * we only re-validate the shape here because CcxtPosition's index signature
+ * is `unknown`.
+ *
+ * ★ `plannedStopPrice` is the CURRENT stop (modify_stop moves it), NOT the
+ * frozen `originalStopPrice` R-denominator — never conflate the two. Display
+ * precedence is bracket (exchange truth) → planned → original.
+ */
+export declare function extractPlannedLevels(pos: {
+    [key: string]: unknown;
+}): {
+    plannedStopPrice?: number;
+    plannedTargetPrice?: number;
+};
 /** The payload shape received from GatewayWsClient 'agent' event */
 export interface AgentEventPayload {
     runId: string;

package/bridge/gateway/event-parser.js

@@ -304,6 +304,42 @@ export function extractBracketField(pos) {
         out.tpPrice = tpPrice;
     return out;
 }
+/**
+ * Narrow the agent's PLANNED protective levels off a plugin-decorated position.
+ *
+ * `bracket` above is exchange truth and only exists where a venue-side ledger
+ * row does. In PAPER (either venue) there are no exchange legs — the levels
+ * live in the position's metadata and the stop-watcher enforces them — so
+ * without these fields the dashboard had NO target at all and drew the
+ * frozen-at-fill `originalStopPrice` as the stop even after a modify_stop.
+ * Both adapters already emit `stopPrice` / `targetPrice` on the CCXT position
+ * (ExchangeSimulator.getPositions, LiveAdapter.decoratePositionsWithMetadata);
+ * we only re-validate the shape here because CcxtPosition's index signature
+ * is `unknown`.
+ *
+ * ★ `plannedStopPrice` is the CURRENT stop (modify_stop moves it), NOT the
+ * frozen `originalStopPrice` R-denominator — never conflate the two. Display
+ * precedence is bracket (exchange truth) → planned → original.
+ */
+export function extractPlannedLevels(pos) {
+    const num = (v) => typeof v === 'number' && Number.isFinite(v) && v > 0 ? v : undefined;
+    // A `fixed_target` realization rule carries the same number; use it only as a
+    // fallback for entries that pinned the rule without a top-level targetPrice.
+    const rule = pos.realizationRule;
+    const ruleTarget = rule && typeof rule === 'object'
+        ? rule.type === 'fixed_target'
+            ? num(rule.targetPrice)
+            : undefined
+        : undefined;
+    const stop = num(pos.stopPrice);
+    const target = num(pos.targetPrice) ?? ruleTarget;
+    const out = {};
+    if (stop !== undefined)
+        out.plannedStopPrice = stop;
+    if (target !== undefined)
+        out.plannedTargetPrice = target;
+    return out;
+}
 /**
  * Build the wallet/available/locked tuple from Binance Futures `info.assets[]`
  * when present. Returns null when the field is absent or unusable so the

package/bridge/providers/gateway.js

@@ -9,7 +9,7 @@ import { isTradingMode } from '../types.js';
 import { GatewayHttpClient } from '../gateway/gateway-http-client.js';
 import { GatewayWsClient } from '../gateway/gateway-ws-client.js';
 import { discoverTools } from '../gateway/tool-discovery.js';
-import { EventParser, mapCcxtBalance, mapCcxtOrder, extractLiquidationFields, extractBracketField, } from '../gateway/event-parser.js';
+import { EventParser, mapCcxtBalance, mapCcxtOrder, extractLiquidationFields, extractBracketField, extractPlannedLevels, } from '../gateway/event-parser.js';
 import { Poller } from '../gateway/poller.js';
 import { ensureHeartbeatCron } from '../gateway/heartbeat-cron.js';
 import { computeEquity as _computeEquity, computePositionNotional as _computePositionNotional, computeRiskMetrics as _computeRiskMetrics, DEFAULT_RISK_LIMITS, } from './risk-calculator.js';
@@ -766,6 +766,8 @@ export class GatewayProvider {
                 giveBackRatio: typeof pos.giveBackRatio === 'number' ? pos.giveBackRatio : undefined,
                 originalStopPrice: typeof pos.originalStopPrice === 'number' ? pos.originalStopPrice : undefined,
                 ...(bracketField !== undefined ? { bracket: bracketField } : {}),
+                // Agent's planned stop/target — the ONLY SL/TP source in paper mode.
+                ...extractPlannedLevels(pos),
                 ...liqFields,
             });
         }
@@ -2063,6 +2065,8 @@ export class GatewayProvider {
                 giveBackRatio: typeof p.giveBackRatio === 'number' ? p.giveBackRatio : undefined,
                 originalStopPrice: typeof p.originalStopPrice === 'number' ? p.originalStopPrice : undefined,
                 ...(bracketField !== undefined ? { bracket: bracketField } : {}),
+                // Agent's planned stop/target — the ONLY SL/TP source in paper mode.
+                ...extractPlannedLevels(p),
                 ...liqFields,
             };
         });

package/bridge/types.d.ts

@@ -186,6 +186,8 @@ export interface RiskUpdatePayload {
             tpPrice?: number;
             state: string;
         };
+        plannedStopPrice?: number;
+        plannedTargetPrice?: number;
     }>;
     balance?: {
         total: number;
@@ -239,6 +241,8 @@ export interface ReconciliationSnapshot {
             tpPrice?: number;
             state: string;
         };
+        plannedStopPrice?: number;
+        plannedTargetPrice?: number;
     }>;
     balance: {
         currency: string;

package/openclaw.plugin.json

@@ -1,7 +1,7 @@
 {
   "id": "reefclaw-paper-trading",
   "name": "ReefClaw Trading",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "Supervised trading plugin for the ReefClaw dashboard: paper trading with real market data (no API keys required), and optional live trading on Binance or Hyperliquid behind explicit operator opt-in, exchange API credentials, and always-on protective stop brackets. Includes the dashboard connector bridge, heartbeat automation, and remote SKILL.md instruction updates from the ReefClaw webapp.",
   "author": "ReefClaw",
   "activation": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reefclaw/openclaw-plugin",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "ReefClaw supervised trading plugin for OpenClaw \u2014 paper trading with real market data, optional live trading on Binance or Hyperliquid (operator opt-in, API keys, always-on protective brackets), plus the ReefClaw dashboard connector with heartbeat automation and remote SKILL.md updates from the ReefClaw webapp. Install: /plugins install clawhub:@reefclaw/openclaw-plugin",
   "type": "module",
   "main": "index.js",

package/venues/hyperliquid/hl-live-adapter.d.ts

@@ -79,6 +79,14 @@ export declare class HyperliquidLiveAdapter extends EventEmitter implements IExc
      */
     closePosition(symbol: string, _closeReason?: CloseReason): Promise<CcxtOrder>;
     getBalance(): Promise<CcxtBalance>;
+    /** The live SL/TP trigger prices for `symbol`, or undefined when no
+     *  non-terminal ledger row exists. Mirrors LiveAdapter's Binance-side
+     *  `lookupBracket` so the dashboard's protective-level surfaces read the same
+     *  shape on both venues. Reads `_coordinator` directly rather than through
+     *  the lazy getter — merely displaying positions must not construct the
+     *  coordinator (which writes a ledger file); with no coordinator there are
+     *  no brackets to report anyway. */
+    private lookupBracket;
     /** Display contract (`?? []`) — the 20+ KPI/display callers. */
     getPositions(symbol?: string): Promise<CcxtPosition[]>;
     /** ★ Decision contract — null means UNKNOWN, and destructive paths must not act. */

package/venues/hyperliquid/hl-live-adapter.js

@@ -120,6 +120,11 @@ export class HyperliquidLiveAdapter extends EventEmitter {
         // Prime the ADDRESS action budget (the starvation guard, §5.6).
         await this.api.refreshAddressBudget();
         // ---- Bracket wiring (issue #209) ----
+        // Build the coordinator here, not on first bracket action: getPositions
+        // reads its ledger to surface live SL/TP, and until the first truth-check
+        // fired (60s) a fresh boot would otherwise show every open position as
+        // unprotected on the dashboard. Construction is idempotent + cheap.
+        this.getHlBracketCoordinator();
         // Fast path: the user stream (fills drive attach for resting limits;
         // orderUpdates is the authoritative leg-lifecycle signal — the HL analog
         // of Binance's ALGO_UPDATE). Truth path: T-5 proved the WS replays
@@ -497,9 +502,29 @@ export class HyperliquidLiveAdapter extends EventEmitter {
         enriched.equity = round4(nav.equity);
         return enriched;
     }
+    /** The live SL/TP trigger prices for `symbol`, or undefined when no
+     *  non-terminal ledger row exists. Mirrors LiveAdapter's Binance-side
+     *  `lookupBracket` so the dashboard's protective-level surfaces read the same
+     *  shape on both venues. Reads `_coordinator` directly rather than through
+     *  the lazy getter — merely displaying positions must not construct the
+     *  coordinator (which writes a ledger file); with no coordinator there are
+     *  no brackets to report anyway. */
+    lookupBracket(symbol) {
+        const row = this._coordinator?.getLedger().getBySymbol(symbol);
+        if (!row)
+            return undefined;
+        if (row.state !== 'active' && row.state !== 'partial' && row.state !== 'attaching') {
+            return undefined;
+        }
+        return { slPrice: row.stopPrice, tpPrice: row.targetPrice, state: row.state };
+    }
     /** Display contract (`?? []`) — the 20+ KPI/display callers. */
     async getPositions(symbol) {
-        return (await this.api.fetchPositions(symbol)) ?? [];
+        const positions = (await this.api.fetchPositions(symbol)) ?? [];
+        // HL live keeps no per-symbol metadata map, so the bracket ledger is the
+        // ONLY protective-level source here — without this the dashboard drew
+        // neither a stop nor a target for an HL live position.
+        return positions.map((p) => ({ ...p, bracket: this.lookupBracket(p.symbol) }));
     }
     /** ★ Decision contract — null means UNKNOWN, and destructive paths must not act. */
     async getPositionsOrNull(symbol) {