@reefclaw/openclaw-plugin 0.1.27 → 0.1.29

package/plugin-version.js

@@ -0,0 +1,58 @@
+// Which ReefClaw plugin RELEASE is running, and how it was installed — the two
+// facts the dashboard needs to (a) tell the trader an update exists and (b)
+// offer the right update path (one click for npx installs, ClawHub steps for
+// ClawHub installs, silence for the operator's own source/dist deploys).
+//
+// Version source: the `openclaw.plugin.json` that ships NEXT TO index.js in
+// every installed layout. Both release channels stamp it with the package
+// release version — installer/scripts/bundle-assets.mjs for `npx
+// @reefclaw/connect`, plugin-package/scripts/assemble.mjs for ClawHub. The
+// repo's own manifest carries the unstamped 0.1.0, which is what tells a
+// source-tree / script-deployed box apart from a packaged install.
+//
+// History: the readiness report used to send a hardcoded internal constant
+// ('3.8.0') that lived in a different namespace from the release versions
+// ('0.1.x'), so the dashboard's update banner compared apples to oranges and
+// never fired (memory feedback_plugin_update_banner_inert_version_namespace).
+// The webapp still recognises that legacy value and nudges those boxes once.
+import { existsSync, readFileSync } from 'node:fs';
+import { join, resolve, sep } from 'node:path';
+import { homedir } from 'node:os';
+/** The repo manifest's placeholder version. A box reporting it was not
+ *  installed from a release package (no release will ever be 0.1.0 — the
+ *  release line passed it long ago). */
+export const UNSTAMPED_VERSION = '0.1.0';
+const RELEASE_VERSION_RE = /^\d+\.\d+\.\d+$/;
+export function resolvePluginInstallFacts(input) {
+    let version;
+    try {
+        const raw = readFileSync(join(input.pluginRoot, 'openclaw.plugin.json'), 'utf-8');
+        const v = JSON.parse(raw).version;
+        if (typeof v === 'string' && RELEASE_VERSION_RE.test(v.trim()))
+            version = v.trim();
+    }
+    catch {
+        // No (readable) manifest next to index.js — legacy deploy layout. Report
+        // nothing rather than a guess; the dashboard fails open (no banner).
+    }
+    return { version, channel: detectChannel(input, version) };
+}
+function detectChannel(input, version) {
+    // Unstamped or absent manifest → the repo's own: source tree or a dist deploy
+    // (the operator's rigs, updated by deploy scripts — never by the installer).
+    if (version === undefined || version === UNSTAMPED_VERSION)
+        return 'source';
+    // ClawHub packages carry the bootstrap skill at the package root
+    // (plugin-package/scripts/assemble.mjs); the npx layout deliberately does
+    // not. Checked FIRST: a box that once ran the npx installer and later
+    // installed from ClawHub loads the ClawHub copy.
+    if (existsSync(join(input.pluginRoot, 'skills')))
+        return 'clawhub';
+    const home = resolve(input.reefclawHome ?? join(homedir(), '.reefclaw'));
+    const root = resolve(input.pluginRoot);
+    if (input.connectorSupervisor === 'on' || root.startsWith(home + sep))
+        return 'npx';
+    // Stamped release package without an installer marker (placed by hand).
+    // The installer still updates it in place, so it gets the one-click path.
+    return 'npx';
+}

package/release-notes.json

@@ -0,0 +1,6 @@
+{
+  "_comment": "One line per released plugin version, shown in the dashboard's update banner. Baked into the webapp at build by webapp/scripts/generate-skill-content.mjs (LATEST_PLUGIN_NOTES = the entry for plugin-package/package.json#version). Add a line in the same PR that bumps the version.",
+  "0.1.27": "Fixes paper positions showing as closed at $0 in the Journal while still open, and adds a 2-minute grace to the position reconciler.",
+  "0.1.28": "Fixes the connector's gateway handshake on OpenClaw 2026.9+ (it now connects as OpenClaw's local-backend client) and adds in-app update notices with one-click updates.",
+  "0.1.29": "Paper stop and target fills now use the price the decision was made on when the cached order book is stale (paper P&L gets more honest); on a gateway with several agents the connector shows only its own agent's runs; one-click updates can be switched off per box."
+}

package/simulator/exchange-simulator.js

@@ -9,6 +9,7 @@ import { randomUUID } from 'node:crypto';
 import { logger, formatError } from '../logger.js';
 import { MAX_TRADE_HISTORY, DEFAULT_SIMULATION_CONFIG } from './types.js';
 import { fillMarketOrder, fillLimitOrder, parseSymbol } from './fill-engine.js';
+import { MAX_BOOK_DRIFT_BPS } from './realistic-fills.js';
 import { updateMfe } from '../mfe.js';
 import { computeInvalidationHit } from '../pinned-plan.js';
 const TAG = 'simulator';
@@ -805,7 +806,10 @@ export class ExchangeSimulator extends EventEmitter {
             logger.info(TAG, `Market order filled: ${order.side} ${order.amount} ${order.symbol} @ ${result.order.average}` +
                 ` (decision: ${eq.decisionPrice.toFixed(2)}, slippage: ${eq.slippageBps.toFixed(2)}bps` +
                 `, latency: ${eq.latencyMs.toFixed(0)}ms, fee: ${eq.feeRate * 100}%` +
-                `, book: ${eq.bookDepthAvailable ? `${eq.bookLevelsConsumed} levels` : 'unavailable'})`);
+                `, book: ${eq.bookDepthAvailable ? `${eq.bookLevelsConsumed} levels` : 'unavailable'}` +
+                `${eq.bookDriftBps !== undefined && Math.abs(eq.bookDriftBps) > MAX_BOOK_DRIFT_BPS
+                    ? `, stale book ${eq.bookDriftBps.toFixed(0)}bps off the decision price: fill anchored`
+                    : ''})`);
         }
         else {
             logger.info(TAG, `Market order filled: ${order.side} ${order.amount} ${order.symbol} @ ${result.order.average}`);

package/simulator/realistic-fills.d.ts

@@ -1,4 +1,18 @@
 import type { OrderBookDepth, SimulationConfig, ExecutionQuality } from './types.js';
+/**
+ * How far (bps) the cached book's mid may sit from the decision price before
+ * the fill is anchored to the decision price instead of the raw book VWAP.
+ *
+ * The book is refreshed only when a tool fetches it (tools/helpers.ts
+ * fetchOrderBook, typically at entry), so a stop or target hours later used to
+ * fill against an hours-old book: an ADA/USDT short stop with a 0.2123 breach
+ * mark filled at 0.207103, 245 bps in the trader's favour, booking a -1.04R
+ * loss as -0.02R (2026-09-16 audit). Inside the band the book is fresh enough
+ * to price the fill directly (unchanged behaviour); outside it the book only
+ * shapes the impact (VWAP vs mid), applied to the price the decision was
+ * actually made on.
+ */
+export declare const MAX_BOOK_DRIFT_BPS = 10;
 /**
  * Walk the order book to compute a volume-weighted average fill price.
  *
@@ -45,6 +59,8 @@ export declare function getFeeRate(orderType: 'market' | 'limit', config: Simula
  *
  * Combines: orderbook VWAP + latency drift + appropriate fee rate.
  * Falls back to simple random slippage if no orderbook is available.
+ * A book whose mid has drifted more than MAX_BOOK_DRIFT_BPS from the decision
+ * price is stale: its impact is kept, the level is re-anchored.
  *
  * @returns fillPrice and ExecutionQuality metrics
  */

package/simulator/realistic-fills.js

@@ -1,6 +1,20 @@
 // Phase 9a: Realistic fill simulation.
 // Order book-aware VWAP fills, latency modeling, maker/taker fees.
 import { DEFAULT_SIMULATION_CONFIG, priceToBps } from './types.js';
+/**
+ * How far (bps) the cached book's mid may sit from the decision price before
+ * the fill is anchored to the decision price instead of the raw book VWAP.
+ *
+ * The book is refreshed only when a tool fetches it (tools/helpers.ts
+ * fetchOrderBook, typically at entry), so a stop or target hours later used to
+ * fill against an hours-old book: an ADA/USDT short stop with a 0.2123 breach
+ * mark filled at 0.207103, 245 bps in the trader's favour, booking a -1.04R
+ * loss as -0.02R (2026-09-16 audit). Inside the band the book is fresh enough
+ * to price the fill directly (unchanged behaviour); outside it the book only
+ * shapes the impact (VWAP vs mid), applied to the price the decision was
+ * actually made on.
+ */
+export const MAX_BOOK_DRIFT_BPS = 10;
 /**
  * Walk the order book to compute a volume-weighted average fill price.
  *
@@ -92,6 +106,8 @@ export function getFeeRate(orderType, config) {
  *
  * Combines: orderbook VWAP + latency drift + appropriate fee rate.
  * Falls back to simple random slippage if no orderbook is available.
+ * A book whose mid has drifted more than MAX_BOOK_DRIFT_BPS from the decision
+ * price is stale: its impact is kept, the level is re-anchored.
  *
  * @returns fillPrice and ExecutionQuality metrics
  */
@@ -103,6 +119,7 @@ export function computeRealisticMarketFill(side, amount, decisionPrice, orderboo
     let latencyImpactBps;
     let bookLevelsConsumed;
     let bookDepthAvailable;
+    let bookDriftBps;
     if (orderbook && orderbook.asks.length > 0 && orderbook.bids.length > 0) {
         // Book-aware VWAP fill
         const { vwap, levelsConsumed } = computeBookAwareFillPrice(side, amount, orderbook);
@@ -113,8 +130,14 @@ export function computeRealisticMarketFill(side, amount, decisionPrice, orderboo
         marketImpactBps = priceToBps(vwap, midPrice);
         // For sells, impact is negative (received less), so take absolute for the metric
         // but keep signed for the actual price
-        // Apply latency drift on top of VWAP
-        const latencyResult = applyLatencyDrift(vwap, side, latencyMs, volFactor);
+        // A book that has drifted from the decision price is stale: keep its
+        // impact, re-anchor the level (see MAX_BOOK_DRIFT_BPS).
+        bookDriftBps = priceToBps(midPrice, decisionPrice);
+        const level = Math.abs(bookDriftBps) > MAX_BOOK_DRIFT_BPS
+            ? decisionPrice * (1 + marketImpactBps / 10_000)
+            : vwap;
+        // Apply latency drift on top of the (possibly re-anchored) VWAP
+        const latencyResult = applyLatencyDrift(level, side, latencyMs, volFactor);
         fillPrice = latencyResult.adjustedPrice;
         latencyImpactBps = latencyResult.latencyImpactBps;
     }
@@ -147,6 +170,7 @@ export function computeRealisticMarketFill(side, amount, decisionPrice, orderboo
         feePaid,
         bookLevelsConsumed,
         bookDepthAvailable,
+        ...(bookDriftBps !== undefined ? { bookDriftBps } : {}),
     };
     return { fillPrice, executionQuality };
 }

package/simulator/types.d.ts

@@ -39,6 +39,10 @@ export interface ExecutionQuality {
     feePaid: number;
     bookLevelsConsumed: number;
     bookDepthAvailable: boolean;
+    /** Signed bps between the cached book's mid and the decision price at fill
+     *  time. Beyond MAX_BOOK_DRIFT_BPS the fill was anchored to the decision
+     *  price (stale book). Absent when no book was available. */
+    bookDriftBps?: number;
     /** Age of the quote the fill priced against (fill time − ticker.timestamp).
      *  Surfaces feed staleness (issue #202); absent on records from before the
      *  field existed or when the ticker carried no usable timestamp. */