perp-cli 0.14.0 → 0.14.1

package/dist/cli-program.js

@@ -195,8 +195,25 @@ export function buildProgram(opts) {
     function getExchange() {
         return resolveVenueId(program.opts().exchange);
     }
+    /**
+     * The `--dex` value for the single `-e` venue. HIP-3 dexes exist only on Hyperliquid, so a
+     * `--dex` with another `-e` venue is refused instead of being dropped (the command would
+     * otherwise read / trade that venue's own markets while the user asked for a HIP-3 dex).
+     * Multi-venue commands (arb, portfolio) apply `--dex` to their Hyperliquid leg only and do
+     * not go through this check.
+     */
+    function dexForExchange() {
+        const dex = program.opts().dex ?? null;
+        if (dex !== null && getExchange() !== "hyperliquid") {
+            const venue = program.getOptionValueSource("exchange") === "cli" ? `-e ${getExchange()}` : `the default exchange (${getExchange()})`;
+            throw new PerpError("INVALID_PARAMS", `--dex ${dex} is a Hyperliquid HIP-3 dex; ${venue} has no HIP-3 dexes`, {
+                remediation: `Use -e hyperliquid --dex ${dex}, or drop --dex for ${getExchange()}.`,
+            });
+        }
+        return dex;
+    }
     async function getAdapter() {
-        _adapter = await createVenueAdapter(getExchange(), program.opts().dex ?? null);
+        _adapter = await createVenueAdapter(getExchange(), dexForExchange());
         return _adapter;
     }
     // Sync wrapper for commands that need adapter (lazy init)
@@ -220,7 +237,7 @@ export function buildProgram(opts) {
     }
     // Public market-data adapters (read-only; no credential resolution).
     async function getPublicAdapter() {
-        return createVenueAdapter(getExchange(), program.opts().dex ?? null, true);
+        return createVenueAdapter(getExchange(), dexForExchange(), true);
     }
     async function getPublicAdapterForExchange(rawExchange) {
         return createVenueAdapter(rawExchange, program.opts().dex ?? null, true);

package/dist/commands/funds-onchain.d.ts

@@ -1,5 +1,5 @@
 /**
- * `funds deposit aster|ondo`, `funds withdraw aster`, `funds address-book add ondo` — fund movements
+ * `funds deposit aster|ondo`, `funds withdraw aster` — fund movements
  * whose on-chain leg is signed by the OWS master wallet (src/evm-master-tx.ts).
  *
  * Every command: `--dry-run` prints the plan and returns before any key, adapter or network is

package/dist/commands/funds-onchain.js

@@ -1,5 +1,5 @@
 /**
- * `funds deposit aster|ondo`, `funds withdraw aster`, `funds address-book add ondo` — fund movements
+ * `funds deposit aster|ondo`, `funds withdraw aster` — fund movements
  * whose on-chain leg is signed by the OWS master wallet (src/evm-master-tx.ts).
  *
  * Every command: `--dry-run` prints the plan and returns before any key, adapter or network is
@@ -12,7 +12,7 @@ import { cliCredentialOptsFrom } from "../config.js";
 import { PerpError, parsePositiveDecimal } from "../errors.js";
 import { logExecution } from "../execution-log.js";
 import { unsupported } from "../exchanges/trading-v2.js";
-import { getVenue, networkConfig } from "../exchanges/venue.js";
+import { getVenue } from "../exchanges/venue.js";
 import { ASTER_FUNDS_CHAINS } from "../exchanges/venues/aster.js";
 import { ONDO_DEPOSIT_CHAINS } from "../exchanges/venues/ondo.js";
 import { jsonOk, printJson } from "../utils.js";
@@ -235,52 +235,14 @@ export function registerOnchainFundsCommands(ctx) {
         }
     });
     // ═══ Address book (Ondo withdrawal destinations) ═══
-    const addressBook = funds.command("address-book").description("Withdrawal address book (Ondo Perps only withdraws to listed addresses)");
+    // Ondo Perps withdrawals are web-app only (the API does not allow them), so an address-book
+    // entry made here would serve nothing. The command stays so the refusal is explicit.
+    const addressBook = funds.command("address-book").description("Not available: the Ondo Perps withdrawal address book is managed in the web app");
     supportsDryRun(addressBook.command("add <venue> <address>"))
-        .description("Add the master wallet's own address to the Ondo Perps withdrawal address book (SIWE challenge signed by the OWS master wallet; supports --dry-run)")
-        .option("--label <text>", "Label of the address book entry")
-        .action(async (venue, address, opts) => {
+        .description("Not available: Ondo Perps withdrawals and their address book are web-app only (refused, also with --dry-run)")
+        .option("--label <text>", "Ignored")
+        .action(async (venue) => {
         const def = getVenue(venue);
-        if (def.id !== "ondo") {
-            throw unsupported(def.displayName, "address-book", "only Ondo Perps has a withdrawal address book", "perp funds address-book add ondo <address>");
-        }
-        const net = getNetwork();
-        if (!/^0x[0-9a-fA-F]{40}$/.test(address))
-            throw new PerpError("INVALID_PARAMS", `"${address}" is not an EVM address`);
-        const plan = {
-            venue: "ondo", network: net, address, label: opts.label === undefined ? null : opts.label,
-            steps: "POST /v1/auth/erc-4361/address_book/get_challenge → personal_sign with the OWS master wallet → POST …/complete_challenge",
-        };
-        if (isDryRun()) {
-            if (isJson())
-                return printJson(jsonOk({ dryRun: true, sent: false, action: "address-book add", ...plan }));
-            console.log(chalk.yellow(`\n  [dry-run] Would add ${address} to the Ondo Perps address book (${net})`));
-            console.log(`  steps: ${plan.steps}`);
-            console.log(chalk.gray("  Nothing was signed or sent.\n"));
-            return;
-        }
-        const { loadMasterEvmSigner } = await import("../evm-master-tx.js");
-        const signer = await loadMasterEvmSigner("ondo", creds(), "Adding an Ondo address book entry");
-        if (signer.getAddress().toLowerCase() !== address.toLowerCase()) {
-            throw new PerpError("INVALID_PARAMS", `The CLI only adds the master wallet's own address (${signer.getAddress()}) to the Ondo address book; ${address} is a different address`, {
-                remediation: "Add other addresses in the Ondo Perps web app (Withdraw → address book)",
-            });
-        }
-        const { OndoAdapter } = await import("../exchanges/ondo/adapter.js");
-        const adapter = await getAdapterForExchange("ondo");
-        if (!(adapter instanceof OndoAdapter))
-            throw new Error("ondo venue did not produce an OndoAdapter");
-        const { addOndoAddressBookEntry } = await import("../exchanges/ondo/funds.js");
-        const res = await addOndoAddressBookEntry(adapter.restClient, {
-            wallet: signer.getAddress(), restUrl: networkConfig(def, net).restUrl, label: plan.label, signMessage: m => signer.signMessage(m),
-        });
-        if (isJson())
-            return printJson(jsonOk({ venue: "ondo", network: net, ...res }));
-        if (res.alreadyListed)
-            console.log(chalk.green(`\n  ${res.address} is already on the Ondo Perps address book.\n`));
-        else if (res.listed)
-            console.log(chalk.green(`\n  Added ${res.address} to the Ondo Perps address book.\n`));
-        else
-            console.log(chalk.yellow(`\n  Ondo accepted the challenge, but ${res.address} is not listed yet (a cooldown may apply). Check the Ondo Perps app before withdrawing.\n`));
+        throw unsupported(def.displayName, "address-book", "the Ondo Perps API does not allow withdrawals, so the CLI keeps no withdrawal address book", "Manage withdrawals and the address book in the Ondo Perps web app");
     });
 }

package/dist/commands/funds.js

@@ -10,8 +10,7 @@ import { cliCredentialOptsFrom, requireMasterWallet } from "../config.js";
 import { getVenue, requireCapabilities, READ_ONLY_CREDENTIALS } from "../exchanges/venue.js";
 import { unsupported } from "../exchanges/trading-v2.js";
 import { registerOnchainFundsCommands } from "./funds-onchain.js";
-import { ONDO_WITHDRAW_NETWORK } from "../exchanges/venues/ondo.js";
-import { QFEX_BANKER_NETWORK, QFEX_WITHDRAW_NETWORK } from "../exchanges/venues/qfex.js";
+import { QFEX_BANKER_NETWORK, QFEX_WITHDRAW_DISABLED, QFEX_WITHDRAW_ENABLED, QFEX_WITHDRAW_NETWORK } from "../exchanges/venues/qfex.js";
 import { loadSettings } from "../settings.js";
 import { assertWithdrawAllowed, forgetWithdrawAttempt, recordWithdrawAttempt } from "../withdraw-gate.js";
 import "../exchanges/venues/index.js";
@@ -105,10 +104,9 @@ function withdrawAssetOf(def) {
 }
 /** Withdrawal chain of venues that settle off-chain and pay out on a public chain. */
 const OFFCHAIN_WITHDRAW_CHAIN = {
-    ondo: ONDO_WITHDRAW_NETWORK,
     qfex: QFEX_WITHDRAW_NETWORK,
 };
-/** Chain a venue's withdrawals arrive on (Ondo / QFEX settle off-chain and pay out on Ethereum / Arbitrum). */
+/** Chain a venue's withdrawals arrive on (QFEX settles off-chain and pays out on Arbitrum). */
 function withdrawChainOf(def, net) {
     const payout = OFFCHAIN_WITHDRAW_CHAIN[def.id];
     if (payout !== undefined)
@@ -1044,14 +1042,20 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
             throw err;
         }
     });
-    // ── Ondo Perps / Arcus — uniform surface (design §13.1): same <amount> + --to,
+    // ── Arcus — uniform surface (design §13.1): same <amount> + --to,
     //    same jsonOk(result), global --dry-run plans without sending. ──
-    for (const venueId of ["ondo", "arcus"]) {
+    // Ondo Perps: the API does not allow withdrawals (address-book destinations only; adding one is
+    // refused for API keys). The command stays so the refusal is explicit — also under --dry-run.
+    supportsDryRun(withdraw.command("ondo [amount]"))
+        .description("Not available: Ondo Perps withdrawals are web-app only (refused, also with --dry-run)")
+        .option("--to <address>", "Ignored — the API cannot withdraw")
+        .action(async () => {
+        throw unsupported("Ondo Perps", "withdraw", "the Ondo Perps API does not allow withdrawals", "Withdraw from the Ondo Perps web app");
+    });
+    for (const venueId of ["arcus"]) {
         const def = getVenue(venueId);
         const asset = withdrawAssetOf(def);
-        const toHelp = venueId === "ondo"
-            ? "Destination EVM address (required; must be on the Ondo address book — see: perp funds address-book add ondo <address>)"
-            : "Destination EVM address (must equal the Arcus account owner — the default)";
+        const toHelp = "Destination EVM address (must equal the Arcus account owner — the default)";
         supportsDryRun(withdraw.command(`${venueId} <amount>`))
             .description(`Withdraw ${asset} from ${def.displayName} to your ${chainLabel(withdrawChainOf(def, "mainnet"))} wallet (supports --dry-run)`)
             .option("--to <address>", toHelp)
@@ -1065,27 +1069,16 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
             const adapter = await getAdapterForExchange(venueId);
             if (!isWithdrawCapable(adapter))
                 throw new Error(`${def.id} adapter has no withdraw()`);
-            let destination;
-            if (venueId === "ondo") {
-                if (opts.to === undefined) {
-                    throw new PerpError("INVALID_PARAMS", "Ondo Perps withdraw needs a destination address (the API key has no owner address)", {
-                        remediation: `perp funds withdraw ondo ${amount} --to <0x address on your Ondo address book>`,
-                    });
-                }
-                destination = opts.to;
-            }
-            else {
-                const { ArcusAdapter } = await import("../exchanges/arcus/adapter.js");
-                if (!(adapter instanceof ArcusAdapter))
-                    throw new Error("arcus venue did not produce an ArcusAdapter");
-                const owner = adapter.withdrawDestination;
-                if (opts.to !== undefined && opts.to.toLowerCase() !== owner.toLowerCase()) {
-                    throw new PerpError("INVALID_PARAMS", `Arcus only withdraws to the account owner ${owner}`, {
-                        remediation: `Omit --to (defaults to ${owner})`,
-                    });
-                }
-                destination = owner;
+            const { ArcusAdapter } = await import("../exchanges/arcus/adapter.js");
+            if (!(adapter instanceof ArcusAdapter))
+                throw new Error("arcus venue did not produce an ArcusAdapter");
+            const owner = adapter.withdrawDestination;
+            if (opts.to !== undefined && opts.to.toLowerCase() !== owner.toLowerCase()) {
+                throw new PerpError("INVALID_PARAMS", `Arcus only withdraws to the account owner ${owner}`, {
+                    remediation: `Omit --to (defaults to ${owner})`,
+                });
             }
+            const destination = owner;
             const plan = { venue: def.id, network: net, asset, amount: String(amountNum), destination, chain };
             if (isDryRun()) {
                 logExecution({
@@ -1100,16 +1093,6 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
                 console.log(chalk.gray("  Nothing was sent.\n"));
                 return;
             }
-            if (venueId === "ondo") {
-                // Address-book preflight (read-only): Ondo refuses destinations that are not listed.
-                const { OndoAdapter } = await import("../exchanges/ondo/adapter.js");
-                if (!(adapter instanceof OndoAdapter))
-                    throw new Error("ondo venue did not produce an OndoAdapter");
-                const book = await adapter.addressBook();
-                if (!book.some(a => a.toLowerCase() === destination.toLowerCase())) {
-                    throw (await import("../exchanges/ondo/funds.js")).ondoAddressNotOnBook(destination, book);
-                }
-            }
             if (!isJson()) {
                 console.log(chalk.cyan(`\n  Withdrawing ${formatUsd(amountNum)} ${asset} from ${def.displayName}...\n`));
                 console.log(`  Destination: ${destination} (${chain})`);
@@ -1139,7 +1122,7 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
         });
     }
     // ── QFEX (design §14.6, §14.12 #8) — custodial; deposit = address display, withdraw = banker POST
-    //    gated by settings.withdraw.qfex inside QfexAdapter.withdraw(). ──
+    //    gated by settings.withdraw.qfex inside QfexAdapter.withdraw(); switched off (QFEX_WITHDRAW_ENABLED). ──
     /** QFEX adapter for the active network — the default imported key, or the one named by --agent. */
     const qfexAdapterFor = async (agentName) => {
         const { QfexAdapter: Cls } = await import("../exchanges/qfex/adapter.js");
@@ -1164,11 +1147,15 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
             throw new Error("qfex venue did not produce a QfexAdapter");
         return adapter;
     };
-    supportsDryRun(withdraw.command("qfex <amount>"))
-        .description("Withdraw USDC from QFEX to an allowlisted Arbitrum One address (needs settings withdraw.qfex and a key with the \"Deposit and withdraw\" permission; supports --dry-run)")
+    supportsDryRun(withdraw.command(QFEX_WITHDRAW_ENABLED ? "qfex <amount>" : "qfex [amount]"))
+        .description(QFEX_WITHDRAW_ENABLED
+        ? "Withdraw USDC from QFEX to an allowlisted Arbitrum One address (needs settings withdraw.qfex and a key with the \"Deposit and withdraw\" permission; supports --dry-run)"
+        : "Not available: QFEX withdrawals are disabled in perp-cli for now — use the QFEX web app (refused, also with --dry-run)")
         .option("--to <address>", "Destination EVM address on Arbitrum One (required; must be on the settings withdraw.qfex allowlist)")
         .option("--agent <name>", "Imported QFEX key to use (a key with the \"Deposit and withdraw\" permission, bound to the primary account)")
         .action(async (amount, opts) => {
+        if (!QFEX_WITHDRAW_ENABLED)
+            throw unsupported("QFEX", "withdraw", QFEX_WITHDRAW_DISABLED.reason, QFEX_WITHDRAW_DISABLED.remediation);
         const def = getVenue("qfex");
         const net = getNetwork();
         if (!/^\d+(\.\d+)?$/.test(amount) || !(Number(amount) > 0)) {
@@ -1247,7 +1234,7 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
         console.log(chalk.yellow(`\n  ${info.warning}`));
         console.log(chalk.gray("  perp-cli did not send anything — transfer from your own wallet or exchange.\n"));
     });
-    // ── Aster deposit / withdraw, Ondo deposit, Ondo address book (OWS-master on-chain legs) ──
+    // ── Aster deposit / withdraw, Ondo deposit (OWS-master on-chain legs; address book refused) ──
     registerOnchainFundsCommands({
         program, funds, deposit, withdraw, isJson, isDryRun, getNetwork, getAdapterForExchange,
         requireMaster, printFundsPlan, failStatus: fundsFailStatus,
@@ -1361,8 +1348,8 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
                     hyperliquid: { chain: "Arbitrum", token: "USDC", speed: "minutes", fee: "$1" },
                     lighter: { chain: "Ethereum L1", token: "USDC", speed: "~12 hours (standard), minutes (fast)", fee: "varies" },
                     "lighter-rh": { chain: "Robinhood Chain", token: "USDG", command: "perp -e lighter-rh funds withdraw lighter <amount>" },
-                    ondo: { chain: "Ethereum", token: "USDC", command: "perp funds withdraw ondo <amount> --to <address>", addressBook: "destination must be listed: perp funds address-book add ondo <master address>" },
-                    qfex: { chain: "Arbitrum", token: "USDC", command: "perp funds withdraw qfex <amount> --to <allowlisted address>", gate: "settings withdraw.qfex (allowlist, maxPerTx, maxPerDay)", note: "needs a key with the \"Deposit and withdraw\" permission (All accounts access)" },
+                    ondo: { supported: false, note: "the Ondo Perps API does not allow withdrawals — withdraw in the Ondo Perps web app" },
+                    qfex: { supported: false, note: "QFEX withdrawals are disabled in perp-cli for now — withdraw in the QFEX web app" },
                     arcus: { chain: "Robinhood Chain", token: "USDG", command: "perp funds withdraw arcus <amount>" },
                     aster: { chain: "BNB Chain (default) / Arbitrum", token: "USDT (default) / USDC", command: "perp funds withdraw aster <amount> [--chain bsc|arbitrum] [--asset USDT|USDC]", fee: "POST /fapi/v3/aster/user-withdraw-info (taken from the amount)", requires: "mainnet; API wallet approved with --can-withdraw --ip-whitelist; destination = the account's own address" },
                 },
@@ -1411,9 +1398,8 @@ export function registerFundsCommands(program, getAdapter, isJson, getNetwork, g
         console.log(`  Command: ${chalk.green("perp -e lt funds withdraw lighter <amount>")}`);
         console.log(chalk.white.bold("\n  Lighter RH") + chalk.gray(" → Robinhood Chain (USDG)"));
         console.log(`  Command: ${chalk.green("perp -e lighter-rh funds withdraw lighter <amount>")}`);
-        console.log(chalk.white.bold("\n  Ondo Perps") + chalk.gray(" → Ethereum (USDC)"));
-        console.log(`  Command: ${chalk.green("perp funds withdraw ondo <amount> --to <address>")}`);
-        console.log(`  Address book: ${chalk.green("perp funds address-book add ondo <your master address>")}`);
+        console.log(chalk.white.bold("\n  Ondo Perps") + chalk.gray(" — not available over the API"));
+        console.log(chalk.gray("  Withdraw in the Ondo Perps web app."));
         console.log(chalk.white.bold("\n  Arcus") + chalk.gray(" → Robinhood Chain (USDG, to the account owner)"));
         console.log(`  Command: ${chalk.green("perp funds withdraw arcus <amount>")}`);
         console.log(chalk.white.bold("\n  Aster") + chalk.gray(" → BNB Chain / Arbitrum (USDT, to the account's own address; mainnet only)"));

package/dist/commands/history.js

@@ -647,24 +647,42 @@ function registerAnalyticsSubcommands(parent, getAdapterForExchange, isJson) {
             let totalEquity = 0;
             let totalUPnl = 0;
             const symbolSet = new Set();
+            // A venue whose read failed is reported here and left out of the totals — never counted as 0.
+            const errors = {};
+            const errText = (e) => (e instanceof Error ? e.message : String(e));
             await Promise.allSettled(exchanges.map(async (ex) => {
+                let adapter;
                 try {
-                    const adapter = await getAdapterForExchange(ex);
-                    const [balance, trades, funding] = await Promise.all([
-                        adapter.getBalance(),
-                        adapter.getTradeHistory(50).catch(() => []),
-                        adapter.getFundingPayments(50).catch(() => []),
-                    ]);
+                    adapter = await getAdapterForExchange(ex);
+                }
+                catch (e) {
+                    errors[`${ex}.adapter`] = errText(e);
+                    return;
+                }
+                try {
+                    const balance = await adapter.getBalance();
                     totalEquity += Number(balance.equity);
                     totalUPnl += Number(balance.unrealizedPnl);
-                    for (const t of trades) {
+                }
+                catch (e) {
+                    errors[`${ex}.balance`] = errText(e);
+                    return;
+                }
+                const [trades, funding] = await Promise.allSettled([adapter.getTradeHistory(50), adapter.getFundingPayments(50)]);
+                if (trades.status === "fulfilled") {
+                    for (const t of trades.value) {
                         totalFees += Number(t.fee);
                         symbolSet.add(t.symbol);
                     }
-                    for (const f of funding)
+                }
+                else
+                    errors[`${ex}.trades`] = errText(trades.reason);
+                if (funding.status === "fulfilled") {
+                    for (const f of funding.value)
                         totalFunding += Number(f.payment);
                 }
-                catch { /* skip */ }
+                else
+                    errors[`${ex}.funding`] = errText(funding.reason);
             }));
             const report = {
                 period: opts.since ?? "all time",
@@ -691,6 +709,7 @@ function registerAnalyticsSubcommands(parent, getAdapterForExchange, isJson) {
                         ? (stats.totalTrades / Math.max(1, (Date.now() - new Date(records[records.length - 1]?.timestamp ?? Date.now()).getTime()) / 86400000)).toFixed(1)
                         : "0",
                 },
+                errors,
             };
             if (isJson())
                 return printJson(jsonOk(report));
@@ -707,6 +726,8 @@ function registerAnalyticsSubcommands(parent, getAdapterForExchange, isJson) {
             console.log(`    Trading Fees:    ${chalk.red(`-$${formatUsd(totalFees)}`)}`);
             console.log(`    Funding Income:  ${formatPnl(totalFunding)}`);
             console.log(`    Net:             ${formatPnl(totalFunding - totalFees)}`);
+            for (const [k, msg] of Object.entries(errors))
+                console.log(chalk.yellow(`    ⚠ ${k} unavailable (left out of the totals): ${msg}`));
             console.log();
         });
     });