@reefclaw/openclaw-plugin 0.1.7 → 0.1.8

package/index.js

@@ -1181,9 +1181,16 @@ const paperTradingPlugin = {
         // ingest credentials. See docs/APPROVAL_MODE_DESIGN.md §2 + §7.3.
         let proposalDecisionListener;
         if (tradingMode === 'MICRO_LIVE' || tradingMode === 'LIVE') {
-            if (!exchangeConfig) {
+            // Per-VENUE credential presence (rehearsal find #4): this safety check
+            // predated the venue seam and tested only the Binance shape, so a
+            // PERSISTED HL MICRO_LIVE (tradingMode written by the runtime flip)
+            // silently fell back to PAPER on every reboot — while the same config
+            // flipped live fine at runtime. The per-venue gate above is the real
+            // validation; this remains the defense-in-depth backstop.
+            const liveCredsPresent = venue === 'hyperliquid' ? hlCredentials !== null : exchangeConfig !== null;
+            if (!liveCredsPresent) {
                 // Already handled above (falls back to PAPER), but safety check
-                logger.error(TAG, `${tradingMode} mode requires exchange config — this should not happen`);
+                logger.error(TAG, `${tradingMode} mode requires ${venue} exchange credentials — this should not happen`);
                 adapter = new PaperAdapter(simulator);
                 tradingMode = 'PAPER';
             }

package/onboarding/runtime.d.ts

@@ -3,6 +3,8 @@ import type { ExchangeConfig, TradingMode } from '../types.js';
 import type { ExchangeSimulator } from '../simulator/exchange-simulator.js';
 import type { PaperMarketFeed } from '../simulator/paper-market-feed.js';
 import { LiveAdapter } from '../live/live-adapter.js';
+import type { HlCredentials } from '../venues/hyperliquid/hl-private.js';
+import { type VenueId } from '../venues/registry.js';
 import { PositionWatcher } from '../live/stop-watcher.js';
 import type { TradingOperationLock } from '../lifecycle/trading-operation-lock.js';
 export interface MicroLiveConfig {
@@ -14,6 +16,14 @@ export interface BuildAdapterInput {
     exchange: ExchangeConfig | null;
     microLive?: MicroLiveConfig;
     simulator: ExchangeSimulator;
+    /** Venue seam for the RUNTIME transition path (issue #217). Absent →
+     *  'binance' (every pre-venue caller byte-identical). The BOOT path got
+     *  this seam in Phase 3 (createLiveAdapter in index.ts); the runtime flip
+     *  — set_trading_mode, the product's go-live moment — missed it and
+     *  constructed a Binance LiveAdapter unconditionally. */
+    venue?: VenueId;
+    /** Required for a live-mode build on the hyperliquid venue. */
+    hlCredentials?: HlCredentials | null;
 }
 /** Wave 9 safety wiring is created only after its durable ledger is loaded.
  *  Runtime reapplies these hooks to both the bootstrap objects and every
@@ -81,6 +91,8 @@ export declare class PluginRuntime {
         mode: TradingMode;
         exchange: ExchangeConfig | null;
         microLive?: MicroLiveConfig;
+        venue?: VenueId;
+        hlCredentials?: HlCredentials | null;
     }, deps: {
         adapterDeps: {
             adapter: IExchangeAdapter;

package/onboarding/runtime.js

@@ -13,9 +13,11 @@
 // instant — no gateway restart required.
 import { PaperAdapter } from '../paper-adapter.js';
 import { LiveAdapter } from '../live/live-adapter.js';
+import { HyperliquidLiveAdapter } from '../venues/hyperliquid/hl-live-adapter.js';
+import { createLiveAdapter } from '../venues/registry.js';
 import { PositionWatcher } from '../live/stop-watcher.js';
 import { loadBracketMode } from '../config/brackets-config.js';
-import { loadStopWatcherIntervalMs } from '../config/plugin-config-io.js';
+import { loadStopWatcherIntervalMs, readPluginConfig } from '../config/plugin-config-io.js';
 import { logger, formatError } from '../logger.js';
 const TAG = 'plugin-runtime';
 /** Pure-ish factory: builds an adapter for the requested mode.
@@ -29,6 +31,23 @@ export function buildAdapter(input) {
         // ShadowTracker wraps a BinancePrivateApi for real-balance comparison.
         return new PaperAdapter(simulator);
     }
+    // Hyperliquid live (issue #217): mirror the boot path's construction —
+    // per-venue credential shape, the SAME factory (createLiveAdapter), and
+    // the same fall-back-to-paper defense when credentials are absent.
+    if (input.venue === 'hyperliquid') {
+        if (!input.hlCredentials) {
+            logger.warn(TAG, `${mode} on hyperliquid requested without walletAddress+agentPrivateKey — falling back to PAPER`);
+            return new PaperAdapter(simulator);
+        }
+        return createLiveAdapter({
+            venue: 'hyperliquid',
+            args: {
+                credentials: input.hlCredentials,
+                mode: mode,
+                marketSlippagePct: readPluginConfig().hl?.marketSlippagePct,
+            },
+        });
+    }
     if (!exchange) {
         logger.warn(TAG, `${mode} requested without credentials — falling back to PAPER`);
         return new PaperAdapter(simulator);
@@ -131,12 +150,24 @@ export class PluginRuntime {
                 logger.warn(TAG, `old adapter shutdown failed: ${formatError(err)}`);
             }
         }
+        else if (old instanceof HyperliquidLiveAdapter) {
+            // Stops the HL user stream + the 60s REST truth-check timer — without
+            // this, a flip AWAY from HL live leaks both (issue #217).
+            try {
+                old.stop();
+            }
+            catch (err) {
+                logger.warn(TAG, `old HL adapter stop failed: ${formatError(err)}`);
+            }
+        }
         // 3. Build the new adapter.
         const fresh = buildAdapter({
             mode: next.mode,
             exchange: next.exchange,
             microLive: next.microLive,
             simulator: this.simulator,
+            venue: next.venue,
+            hlCredentials: next.hlCredentials,
         });
         // Install autonomous protection callbacks before initialization can emit
         // user-data or bracket-reconciler events.
@@ -145,7 +176,7 @@ export class PluginRuntime {
         }
         // 4. Fire async init for live adapters (non-blocking — readiness flips
         //    INIT_PENDING → READY/DEGRADED/BLOCKED on its own).
-        if (fresh instanceof LiveAdapter) {
+        if (fresh instanceof LiveAdapter || fresh instanceof HyperliquidLiveAdapter) {
             fresh.initialize().catch(err => {
                 logger.error(TAG, `New adapter init failed: ${formatError(err)}`);
             });

package/openclaw.plugin.json

@@ -1,7 +1,7 @@
 {
   "id": "reefclaw-paper-trading",
   "name": "ReefClaw Paper Trading",
-  "version": "0.1.0",
+  "version": "0.1.8",
   "description": "Paper trading plugin with real Binance market data and simulated execution. No API keys required.",
   "author": "ReefClaw",
   "activation": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@reefclaw/openclaw-plugin",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "ReefClaw trading plugin for OpenClaw \u2014 paper trading with real Binance market data, plus the ReefClaw dashboard connector (supervised by OpenClaw, no service manager needed). Install: /plugins install clawhub:@reefclaw/openclaw-plugin",
   "type": "module",
   "main": "index.js",

package/scripts/assemble.mjs

@@ -66,16 +66,32 @@ if (!existsSync(pluginDist)) {
   console.log('[assemble] plugin/dist -> package root');
 }
 
-// 2. Manifest, with the `skills` declaration injected (npm package only).
+// 2. Manifest, with the `skills` declaration injected (npm package only) and the
+//    release version stamped from THIS package's package.json.
+//
+//    package.json#version is the authoritative release version — that is what
+//    ClawHub's `package-manifest-version-drift` rule compares against, and what
+//    OpenClaw's own `openclaw plugins build` writes into the manifest. The repo
+//    manifest (plugin/openclaw.plugin.json) carries a workspace-local version
+//    nobody bumps at release time, so copying it verbatim published
+//    manifest:0.1.0 against package:0.1.7 and tripped the validator. Deriving it
+//    here makes that drift structurally impossible instead of a bump to remember.
 const manifestSrc = join(repoRoot, 'plugin', 'openclaw.plugin.json');
+const pkgVersion = JSON.parse(readFileSync(join(pkgRoot, 'package.json'), 'utf-8')).version;
 if (!existsSync(manifestSrc)) {
   console.error('[assemble] missing plugin/openclaw.plugin.json.');
   missing += 1;
+} else if (typeof pkgVersion !== 'string' || pkgVersion.length === 0) {
+  console.error('[assemble] package.json has no version — cannot stamp the manifest.');
+  missing += 1;
 } else {
   const manifest = JSON.parse(readFileSync(manifestSrc, 'utf-8'));
+  manifest.version = pkgVersion;
   manifest.skills = ['./skills'];
   writeFileSync(join(pkgRoot, 'openclaw.plugin.json'), JSON.stringify(manifest, null, 2) + '\n', 'utf-8');
-  console.log(`[assemble] manifest (+skills decl), ${manifest.contracts?.tools?.length ?? 0} contract tools`);
+  console.log(
+    `[assemble] manifest v${pkgVersion} (+skills decl), ${manifest.contracts?.tools?.length ?? 0} contract tools`,
+  );
 }
 
 // 3. The connector, bundled — this is what flips the supervisor's auto-start.

package/tools/create-order.d.ts

@@ -10,6 +10,13 @@ import { type Wave9LiveSymbolOwnershipCheck } from '../wave9/live-symbol-ownersh
 import { type Wave9LiveResidualProtectionResult } from '../wave9/live-residual-protection.js';
 import { type Wave9PaperAdmissionGuard } from '../wave9/paper-admission-guard.js';
 type Wave9LiveResidualProtector = (adapter: IExchangeAdapter, ledger: Wave9LiveExecutionLedger, candidateId: string, symbol: string) => Promise<Wave9LiveResidualProtectionResult>;
+/** Venue-aware idempotency cid for the live entry stash. HL client-order-ids
+ *  are 128-bit hex (0x + 32 hex chars) — a dashed randomUUID() fails
+ *  hl-private's cloid validation and the ENTRY IS REJECTED before reaching
+ *  the exchange (found live by the go-live rehearsal's first dust trade).
+ *  buildHlOrderCloid() mints the 0x0d… ORDER prefix, which the bracket parser
+ *  deliberately never recognises (cross-scheme cancels are destructive). */
+export declare function mintEntryStashCid(adapter: IExchangeAdapter): string;
 export declare function createOrderTool(args: {
     symbol: string;
     side: string;

package/tools/create-order.js

@@ -11,6 +11,8 @@ import { onCreateOrderFilled } from '../ingest/position-auto-capture.js';
 import { wave9ClientOrderId, } from '../wave9/live-execution-ledger.js';
 import { inspectWave9LiveSymbolOwnership, } from '../wave9/live-symbol-ownership.js';
 import { confirmWave9LivePositionFlat } from '../wave9/live-position-confirmation.js';
+import { HyperliquidLiveAdapter } from '../venues/hyperliquid/hl-live-adapter.js';
+import { buildHlOrderCloid } from '../venues/hyperliquid/hl-cloid.js';
 import { reprotectWave9LiveResidual, } from '../wave9/live-residual-protection.js';
 import { captureWave9PaperAccountSnapshot, isWave9ManagedPosition, WAVE9_BUNDLE_SETUP_TYPE, } from '../wave9/paper-admission-guard.js';
 function currentWave9ExecutionMode(deps) {
@@ -231,6 +233,15 @@ function candidateBoundFoundOrder(resolution, clientOrderId, symbol, side) {
         return undefined;
     return order;
 }
+/** Venue-aware idempotency cid for the live entry stash. HL client-order-ids
+ *  are 128-bit hex (0x + 32 hex chars) — a dashed randomUUID() fails
+ *  hl-private's cloid validation and the ENTRY IS REJECTED before reaching
+ *  the exchange (found live by the go-live rehearsal's first dust trade).
+ *  buildHlOrderCloid() mints the 0x0d… ORDER prefix, which the bracket parser
+ *  deliberately never recognises (cross-scheme cancels are destructive). */
+export function mintEntryStashCid(adapter) {
+    return adapter instanceof HyperliquidLiveAdapter ? buildHlOrderCloid() : randomUUID();
+}
 async function resolveWave9EntryCid(adapter, clientOrderId, symbol) {
     if (!adapter.resolveOrderByClientId) {
         return { status: 'unknown', detail: 'deterministic client-order resolver is unavailable' };
@@ -1133,7 +1144,7 @@ export async function createOrderTool(args, deps) {
                 ? wave9ClientOrderId(args.candidate_id)
                 : undefined;
             if (deps.autoCapture?.pendingEntries && metadata) {
-                stashCid ??= randomUUID();
+                stashCid ??= mintEntryStashCid(deps.adapter);
                 deps.autoCapture.pendingEntries.put({
                     orderId: stashCid,
                     clientOrderId: stashCid,

package/tools/set-trading-mode.js

@@ -9,6 +9,7 @@
 // because the caller chain is currently controlled end-to-end by the skill.
 import { readPluginConfig } from '../config/plugin-config-io.js';
 import { validateModeTransition, modeRequiresCredentials } from '../onboarding/mode-ladder.js';
+import { parseVenue } from '../venues/registry.js';
 import { logger } from '../logger.js';
 import { recordModeTransition } from '../audit/mode-transition-audit.js';
 const TAG = 'set-trading-mode';
@@ -49,12 +50,26 @@ export async function setTradingModeTool(args, deps) {
             readiness: deps.runtime.adapter.readiness,
         };
     }
-    // 3. Credential requirement check.
+    // 3. Credential requirement check — PER VENUE (issue #217: this gate was
+    //    Binance-only, so the go-live flip on the hyperliquid venue was rejected
+    //    despite valid walletAddress+agentPrivateKey in plugin-config).
     let exchange = null;
+    let hlCredentials = null;
+    let venue = 'binance';
     if (modeRequiresCredentials(target)) {
         try {
             const cfg = readPluginConfig(deps.configPath);
-            if (cfg.exchange?.apiKey && cfg.exchange?.secret) {
+            venue = parseVenue(cfg.exchange?.venue).venue;
+            if (venue === 'hyperliquid') {
+                if (cfg.exchange?.walletAddress && cfg.exchange?.agentPrivateKey) {
+                    hlCredentials = {
+                        walletAddress: String(cfg.exchange.walletAddress),
+                        agentPrivateKey: String(cfg.exchange.agentPrivateKey),
+                        testnet: cfg.exchange.testnet === true,
+                    };
+                }
+            }
+            else if (cfg.exchange?.apiKey && cfg.exchange?.secret) {
                 exchange = {
                     apiKey: cfg.exchange.apiKey,
                     secret: cfg.exchange.secret,
@@ -74,11 +89,13 @@ export async function setTradingModeTool(args, deps) {
                 reason: 'config_read_error',
             };
         }
-        if (!exchange) {
+        if (!exchange && !hlCredentials) {
             recordModeTransition({ previousMode, targetMode: target, acknowledged, ok: false, reason: 'missing_credentials' });
             return {
                 ok: false,
-                message: `${target} mode requires exchange credentials. Run set_exchange_credentials first.`,
+                message: venue === 'hyperliquid'
+                    ? `${target} mode on Hyperliquid requires exchange.walletAddress (master) + exchange.agentPrivateKey (agent wallet) in plugin-config.`
+                    : `${target} mode requires exchange credentials. Run set_exchange_credentials first.`,
                 previousMode,
                 mode: previousMode,
                 readiness: deps.runtime.adapter.readiness,
@@ -116,14 +133,14 @@ export async function setTradingModeTool(args, deps) {
         };
     }
     // 6. Swap the adapter.
-    await deps.runtime.reconnect({ mode: target, exchange }, { adapterDeps: deps.adapterDeps });
+    await deps.runtime.reconnect({ mode: target, exchange, venue, hlCredentials }, { adapterDeps: deps.adapterDeps });
     logger.info(TAG, `Transitioned ${previousMode} → ${target}`);
     recordModeTransition({
         previousMode,
         targetMode: target,
         acknowledged,
         ok: true,
-        testnet: exchange?.testnet,
+        testnet: exchange?.testnet ?? hlCredentials?.testnet,
     });
     return {
         ok: true,