suzi-cli 0.1.26 → 0.1.28

package/dist/commands/portfolio.js

@@ -11,6 +11,8 @@ const config_1 = require("../lib/config");
 const api_1 = require("../lib/api");
 const ui_1 = require("../utils/ui");
 const tty_1 = require("../utils/tty");
+const agent_picker_1 = require("../utils/agent-picker");
+const PORTFOLIO_ORDERS_INCLUDES = new Set(['fills']);
 function formatPnlPercent(pct) {
     if (pct == null)
         return chalk_1.default.gray('—');
@@ -35,17 +37,230 @@ function exposureLabel(pos) {
             return '—';
     }
 }
+function formatPortfolioOrderStatus(status) {
+    if (status === 'filled')
+        return chalk_1.default.green('Filled');
+    if (status === 'partial')
+        return chalk_1.default.yellow('Partial');
+    if (status === 'cancelled')
+        return chalk_1.default.red('Cancelled');
+    if (status === 'expired')
+        return chalk_1.default.gray('Expired');
+    return chalk_1.default.cyan('Open');
+}
+function renderPortfolioPositionsSection(positions) {
+    console.log(chalk_1.default.bold('  Positions'));
+    const byProtocol = new Map();
+    for (const pos of positions) {
+        const existing = byProtocol.get(pos.protocol) ?? [];
+        existing.push(pos);
+        byProtocol.set(pos.protocol, existing);
+    }
+    for (const [protocol, protocolPositions] of byProtocol) {
+        console.log();
+        console.log(ui_1.colors.secondary(`  ${protocol}`));
+        const hasPerps = protocolPositions.some((p) => p.detail?.kind === 'perps');
+        const posHead = [
+            chalk_1.default.gray('Instrument'),
+            chalk_1.default.gray('Type'),
+            chalk_1.default.gray('Exposure'),
+            chalk_1.default.gray('Value'),
+            chalk_1.default.gray('PnL%'),
+            ...(hasPerps ? [chalk_1.default.gray('Funding')] : []),
+            chalk_1.default.gray('Status'),
+        ];
+        const posTable = new cli_table3_1.default({
+            head: posHead,
+            style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
+        });
+        for (const pos of protocolPositions) {
+            const fundingCell = pos.detail?.kind === 'perps' && pos.detail.fundingRate != null
+                ? (pos.detail.fundingRate >= 0
+                    ? chalk_1.default.green(`${(pos.detail.fundingRate * 100).toFixed(4)}%`)
+                    : chalk_1.default.red(`${(pos.detail.fundingRate * 100).toFixed(4)}%`))
+                : chalk_1.default.gray('—');
+            posTable.push([
+                chalk_1.default.white(pos.instrument),
+                pos.type,
+                exposureLabel(pos),
+                (0, ui_1.formatUsd)(pos.valueUsd),
+                formatPnlPercent(pos.pnlPercent),
+                ...(hasPerps ? [fundingCell] : []),
+                pos.status,
+            ]);
+        }
+        console.log(posTable.toString());
+    }
+}
+function renderPortfolioOrdersSection(orders) {
+    console.log(chalk_1.default.bold('  Orders'));
+    const ordersTable = new cli_table3_1.default({
+        head: [
+            chalk_1.default.gray('Instrument'),
+            chalk_1.default.gray('Side'),
+            chalk_1.default.gray('Type'),
+            chalk_1.default.gray('Amount'),
+            chalk_1.default.gray('Price'),
+            chalk_1.default.gray('Status'),
+            chalk_1.default.gray('Protocol'),
+        ],
+        style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
+    });
+    for (const o of orders) {
+        const sideColor = o.side === 'buy' || o.side === 'long' ? chalk_1.default.green : chalk_1.default.red;
+        const filledInfo = o.amountFilled != null && o.amountFilled > 0
+            ? ` (${o.amountFilled.toFixed(2)} filled)`
+            : '';
+        ordersTable.push([
+            chalk_1.default.white(o.instrument),
+            sideColor(o.side.toUpperCase()),
+            o.type,
+            `${(o.amount ?? 0).toFixed(4)} ${o.amountSymbol}${filledInfo}`,
+            o.price,
+            formatPortfolioOrderStatus(o.status),
+            ui_1.colors.muted(o.protocol),
+        ]);
+    }
+    console.log(ordersTable.toString());
+}
+function attributedOrderStatus(status) {
+    if (status === 'filled')
+        return chalk_1.default.green(status);
+    if (status === 'open' || status === 'partially_filled')
+        return chalk_1.default.yellow(status);
+    if (status === 'cancelled')
+        return chalk_1.default.gray(status);
+    return status;
+}
+function renderAttributedOrdersSection(orders) {
+    console.log(chalk_1.default.bold('  Orders'));
+    const table = new cli_table3_1.default({
+        head: [
+            chalk_1.default.gray('Order ID'),
+            chalk_1.default.gray('Proto'),
+            chalk_1.default.gray('Status'),
+            chalk_1.default.gray('Side'),
+            chalk_1.default.gray('Asset'),
+            chalk_1.default.gray('Size'),
+            chalk_1.default.gray('Filled'),
+            chalk_1.default.gray('Price'),
+            chalk_1.default.gray('Time'),
+        ],
+        style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
+    });
+    for (const order of orders) {
+        table.push([
+            order.orderId.length > 14 ? `${order.orderId.slice(0, 13)}…` : order.orderId,
+            order.protocol === 'hyperliquid' ? 'HL' : order.protocol === 'polymarket' ? 'PM' : order.protocol,
+            attributedOrderStatus(order.status),
+            order.side === 'buy' ? chalk_1.default.green(order.side) : chalk_1.default.red(order.side),
+            order.asset.length > 10 ? `${order.asset.slice(0, 6)}…` : order.asset,
+            order.size,
+            order.filledSize,
+            order.price ?? '-',
+            order.createdAt ? new Date(order.createdAt).toLocaleString() : '-',
+        ]);
+    }
+    console.log(table.toString());
+}
+function renderAttributedPositionsSection(positions) {
+    console.log(chalk_1.default.bold('  Positions'));
+    const table = new cli_table3_1.default({
+        head: [
+            chalk_1.default.gray('Proto'),
+            chalk_1.default.gray('Asset'),
+            chalk_1.default.gray('Net Size'),
+            chalk_1.default.gray('Bought'),
+            chalk_1.default.gray('Sold'),
+            chalk_1.default.gray('Status'),
+        ],
+        style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
+    });
+    for (const position of positions) {
+        const keyTail = position.positionKey.split(':').pop() || position.positionKey;
+        table.push([
+            position.protocol === 'hyperliquid' ? 'HL' : position.protocol === 'polymarket' ? 'PM' : position.protocol,
+            keyTail.length > 10 ? `${keyTail.slice(0, 6)}…` : keyTail,
+            position.netSize,
+            position.totalBought,
+            position.totalSold,
+            position.status,
+        ]);
+    }
+    console.log(table.toString());
+}
+function renderPortfolioFillsSection(fills) {
+    console.log(chalk_1.default.bold('  Fills'));
+    const table = new cli_table3_1.default({
+        head: [
+            chalk_1.default.gray('Proto'),
+            chalk_1.default.gray('Side'),
+            chalk_1.default.gray('Asset'),
+            chalk_1.default.gray('Size'),
+            chalk_1.default.gray('Price'),
+            chalk_1.default.gray('Fee'),
+            chalk_1.default.gray('Role'),
+            chalk_1.default.gray('Time'),
+        ],
+        style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
+    });
+    for (const fill of fills) {
+        const protocol = fill.protocol === 'hyperliquid' ? 'HL' : fill.protocol === 'polymarket' ? 'PM' : fill.protocol;
+        const side = fill.side === 'buy' ? chalk_1.default.green(fill.side) : chalk_1.default.red(fill.side);
+        const asset = fill.asset.length > 10 ? `${fill.asset.slice(0, 6)}…` : fill.asset;
+        table.push([
+            protocol,
+            side,
+            asset,
+            fill.fillSize,
+            fill.fillPrice,
+            fill.fee ?? '-',
+            fill.liquidityRole ?? '-',
+            new Date(fill.filledAt).toLocaleString(),
+        ]);
+    }
+    console.log(table.toString());
+}
+function renderPaginationHint(meta, shownCount, labelText) {
+    if (meta.total <= meta.offset + shownCount)
+        return;
+    (0, ui_1.info)(`Showing ${meta.offset + 1}-${meta.offset + shownCount} of ${meta.total} ${labelText}. Use --limit/--offset to page.`);
+}
+function renderAgentPortfolioDisplay(agentId, positions, orders, protocol) {
+    (0, ui_1.header)('Portfolio');
+    console.log();
+    (0, ui_1.label)('Agent', agentId);
+    if (protocol) {
+        (0, ui_1.label)('Protocol', protocol);
+    }
+    console.log();
+    if (positions.length > 0) {
+        (0, ui_1.divider)();
+        renderAttributedPositionsSection(positions);
+    }
+    else {
+        (0, ui_1.info)('No attributed positions found.');
+    }
+    console.log();
+    if (orders.length > 0) {
+        (0, ui_1.divider)();
+        renderAttributedOrdersSection(orders);
+    }
+    else {
+        (0, ui_1.info)('No attributed orders found.');
+    }
+    console.log();
+    (0, ui_1.divider)();
+}
 function renderPortfolioDisplay(snapshot) {
     (0, ui_1.header)('Portfolio');
     console.log();
-    // === Totals ===
     const t = snapshot.totals;
     (0, ui_1.label)('Total Value', (0, ui_1.formatUsd)(t.totalValueUsd));
     (0, ui_1.label)('Exposure', (0, ui_1.formatUsd)(t.totalExposure));
     (0, ui_1.label)('Spot', (0, ui_1.formatUsd)(t.totalSpot));
     (0, ui_1.label)('Limit Orders', (0, ui_1.formatUsd)(t.totalLimitOrders));
     console.log();
-    // === Holdings (dust already filtered server-side) ===
     if (snapshot.holdings && snapshot.holdings.length > 0) {
         (0, ui_1.divider)();
         console.log(chalk_1.default.bold('  Holdings'));
@@ -67,102 +282,19 @@ function renderPortfolioDisplay(snapshot) {
     else if (snapshot.holdings !== null) {
         (0, ui_1.info)('No spot balances found.');
     }
-    // === Positions ===
     if (snapshot.positions && snapshot.positions.length > 0) {
         console.log();
         (0, ui_1.divider)();
-        console.log(chalk_1.default.bold('  Positions'));
-        // Group by protocol
-        const byProtocol = new Map();
-        for (const pos of snapshot.positions) {
-            const existing = byProtocol.get(pos.protocol) ?? [];
-            existing.push(pos);
-            byProtocol.set(pos.protocol, existing);
-        }
-        for (const [protocol, positions] of byProtocol) {
-            console.log();
-            console.log(ui_1.colors.secondary(`  ${protocol}`));
-            const hasPerps = positions.some((p) => p.detail?.kind === 'perps');
-            const posHead = [
-                chalk_1.default.gray('Instrument'),
-                chalk_1.default.gray('Type'),
-                chalk_1.default.gray('Exposure'),
-                chalk_1.default.gray('Value'),
-                chalk_1.default.gray('PnL%'),
-                ...(hasPerps ? [chalk_1.default.gray('Funding')] : []),
-                chalk_1.default.gray('Status'),
-            ];
-            const posTable = new cli_table3_1.default({
-                head: posHead,
-                style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
-            });
-            for (const pos of positions) {
-                const fundingCell = pos.detail?.kind === 'perps' && pos.detail.fundingRate != null
-                    ? (pos.detail.fundingRate >= 0
-                        ? chalk_1.default.green(`${(pos.detail.fundingRate * 100).toFixed(4)}%`)
-                        : chalk_1.default.red(`${(pos.detail.fundingRate * 100).toFixed(4)}%`))
-                    : chalk_1.default.gray('—');
-                posTable.push([
-                    chalk_1.default.white(pos.instrument),
-                    pos.type,
-                    exposureLabel(pos),
-                    (0, ui_1.formatUsd)(pos.valueUsd),
-                    formatPnlPercent(pos.pnlPercent),
-                    ...(hasPerps ? [fundingCell] : []),
-                    pos.status,
-                ]);
-            }
-            console.log(posTable.toString());
-        }
+        renderPortfolioPositionsSection(snapshot.positions);
     }
     else if (snapshot.positions !== null) {
         console.log();
         (0, ui_1.info)('No open positions.');
     }
-    // === Orders ===
     if (snapshot.orders && snapshot.orders.length > 0) {
         console.log();
         (0, ui_1.divider)();
-        console.log(chalk_1.default.bold('  Orders'));
-        const ordersTable = new cli_table3_1.default({
-            head: [
-                chalk_1.default.gray('Instrument'),
-                chalk_1.default.gray('Side'),
-                chalk_1.default.gray('Type'),
-                chalk_1.default.gray('Amount'),
-                chalk_1.default.gray('Price'),
-                chalk_1.default.gray('Status'),
-                chalk_1.default.gray('Protocol'),
-            ],
-            style: { head: [], border: chalk_1.default.level > 0 ? ['gray'] : [] },
-        });
-        for (const o of snapshot.orders) {
-            const sideColor = o.side === 'buy' || o.side === 'long' ? chalk_1.default.green : chalk_1.default.red;
-            let statusLabel;
-            if (o.status === 'filled')
-                statusLabel = chalk_1.default.green('Filled');
-            else if (o.status === 'partial')
-                statusLabel = chalk_1.default.yellow('Partial');
-            else if (o.status === 'cancelled')
-                statusLabel = chalk_1.default.red('Cancelled');
-            else if (o.status === 'expired')
-                statusLabel = chalk_1.default.gray('Expired');
-            else
-                statusLabel = chalk_1.default.cyan('Open');
-            const filledInfo = o.amountFilled != null && o.amountFilled > 0
-                ? ` (${o.amountFilled.toFixed(2)} filled)`
-                : '';
-            ordersTable.push([
-                chalk_1.default.white(o.instrument),
-                sideColor(o.side.toUpperCase()),
-                o.type,
-                `${(o.amount ?? 0).toFixed(4)} ${o.amountSymbol}${filledInfo}`,
-                o.price,
-                statusLabel,
-                ui_1.colors.muted(o.protocol),
-            ]);
-        }
-        console.log(ordersTable.toString());
+        renderPortfolioOrdersSection(snapshot.orders);
     }
     else if (snapshot.orders !== null) {
         console.log();
@@ -171,34 +303,183 @@ function renderPortfolioDisplay(snapshot) {
     console.log();
     (0, ui_1.divider)();
 }
+function parseSectionLimit(limit, fallback) {
+    if (!limit)
+        return fallback;
+    const parsed = parseInt(limit, 10);
+    return Number.isNaN(parsed) || parsed < 1 ? fallback : Math.min(parsed, 200);
+}
+function parseSectionOffset(offset) {
+    if (!offset)
+        return 0;
+    const parsed = parseInt(offset, 10);
+    return Number.isNaN(parsed) || parsed < 0 ? 0 : parsed;
+}
+function buildSectionQuery(opts, defaults) {
+    const params = new URLSearchParams();
+    if (opts.agent)
+        params.set('agentId', opts.agent);
+    if (opts.protocol)
+        params.set('protocol', opts.protocol);
+    if (opts.status)
+        params.set('status', opts.status);
+    if (opts.include)
+        params.set('include', opts.include);
+    params.set('limit', String(parseSectionLimit(opts.limit, defaults.limit)));
+    params.set('offset', String(parseSectionOffset(opts.offset)));
+    return params;
+}
+function parseOrdersIncludeOption(include) {
+    if (!include)
+        return { ok: true };
+    const includes = include
+        .split(',')
+        .map((token) => token.trim())
+        .filter((token) => token.length > 0);
+    if (includes.length === 0)
+        return { ok: true };
+    const invalid = includes.find((token) => !PORTFOLIO_ORDERS_INCLUDES.has(token));
+    if (invalid) {
+        return { ok: false, invalid };
+    }
+    return { ok: true, value: Array.from(new Set(includes)).join(',') };
+}
+function getApiErrorBody(resp) {
+    if (resp == null ||
+        typeof resp !== 'object' ||
+        !Object.prototype.hasOwnProperty.call(resp, 'data')) {
+        return undefined;
+    }
+    const tmp = resp;
+    return tmp.data;
+}
+function formatProtocolFlag(protocol) {
+    return protocol ? ` --protocol ${protocol}` : '';
+}
+async function fetchPortfolioSection(path, labelText, jsonMode) {
+    const spinner = (0, tty_1.createSpinner)(`Loading ${labelText}...`, jsonMode);
+    spinner?.start();
+    try {
+        const resp = await (0, api_1.get)(path);
+        spinner?.stop();
+        if (!resp.ok) {
+            const errorBody = getApiErrorBody(resp);
+            const msg = `Failed to load ${labelText}: ${errorBody?.message || errorBody?.error || 'Unknown error'}`;
+            if (jsonMode)
+                (0, tty_1.outputJson)({ success: false, error: msg });
+            else
+                (0, ui_1.error)(msg);
+            return null;
+        }
+        return resp.data;
+    }
+    catch (err) {
+        spinner?.stop();
+        const msg = `Failed to load ${labelText}: ${err instanceof Error ? err.message : 'Unknown error'}`;
+        if (jsonMode)
+            (0, tty_1.outputJson)({ success: false, error: msg });
+        else
+            (0, ui_1.error)(msg);
+        return null;
+    }
+}
 function registerPortfolioCommand(program) {
     const portfolioCmd = program
         .command('portfolio')
         .description('View your complete portfolio — balances, positions, P&L across all protocols')
         .enablePositionalOptions()
         .passThroughOptions();
+    const ensureActiveAccount = (jsonMode) => {
+        const account = (0, config_1.getActiveAccount)();
+        if (account)
+            return true;
+        const msg = 'No active account.';
+        if (jsonMode)
+            (0, tty_1.outputJson)({ success: false, error: msg });
+        else
+            (0, ui_1.error)(msg);
+        return false;
+    };
+    const resolveAgentOption = async (agentOpt, jsonMode) => {
+        if (agentOpt == null)
+            return null;
+        return (0, agent_picker_1.fetchAndPickAgent)(typeof agentOpt === 'string' ? agentOpt : undefined, jsonMode);
+    };
     // Default action — runs when no subcommand is given (existing behavior)
     portfolioCmd
         .option('--json', 'Output as JSON')
         .option('--all-balances', 'Show low/dust spot balances (default hides tiny balances)')
         .option('--svm-wallet <address>', 'Override SVM wallet address')
         .option('--evm-wallet <address>', 'Override EVM wallet address')
+        .option('--agent [agentIdOrName]', 'Filter to a specific agent; omit value to pick interactively')
+        .option('-p, --protocol <protocol>', 'Filter positions/orders by protocol in agent summary mode')
         .action(async (opts) => {
         const jsonMode = opts?.json || false;
         if (!(0, ui_1.requireAuth)(jsonMode))
             return;
-        const showAllBalances = opts?.allBalances || false;
-        const account = (0, config_1.getActiveAccount)();
-        if (!account) {
-            const msg = 'No active account.';
-            if (jsonMode) {
-                (0, tty_1.outputJson)({ success: false, error: msg });
+        if (!ensureActiveAccount(jsonMode))
+            return;
+        const protocol = opts?.protocol;
+        const resolvedAgent = await resolveAgentOption(opts?.agent, jsonMode);
+        if (opts?.agent && !resolvedAgent)
+            return;
+        if (resolvedAgent) {
+            const spinner = (0, tty_1.createSpinner)('Loading attributed portfolio...', jsonMode);
+            spinner?.start();
+            try {
+                const params = new URLSearchParams();
+                params.set('agentId', resolvedAgent.id);
+                params.set('limit', '20');
+                params.set('offset', '0');
+                if (protocol)
+                    params.set('protocol', protocol);
+                const qs = params.toString();
+                const [ordersResp, positionsResp] = await Promise.all([
+                    (0, api_1.get)(`/api/portfolio/agent-orders?${qs}`),
+                    (0, api_1.get)(`/api/portfolio/agent-positions?${qs}`),
+                ]);
+                spinner?.stop();
+                if (!ordersResp.ok || !positionsResp.ok) {
+                    const errorBody = getApiErrorBody(!ordersResp.ok ? ordersResp : positionsResp);
+                    const msg = `Failed to load attributed portfolio: ${errorBody?.message || errorBody?.error || 'Unknown error'}`;
+                    if (jsonMode)
+                        (0, tty_1.outputJson)({ success: false, error: msg });
+                    else
+                        (0, ui_1.error)(msg);
+                    return;
+                }
+                if (jsonMode) {
+                    (0, tty_1.outputJson)({
+                        success: true,
+                        data: {
+                            agentId: resolvedAgent.id,
+                            protocol: protocol ?? null,
+                            orders: ordersResp.data,
+                            positions: positionsResp.data,
+                        },
+                    });
+                    return;
+                }
+                renderAgentPortfolioDisplay(resolvedAgent.title || resolvedAgent.id, positionsResp.data.positions, ordersResp.data.orders, protocol);
+                if (positionsResp.data.pagination.total > positionsResp.data.positions.length) {
+                    (0, ui_1.info)(`Use ${chalk_1.default.cyan(`suzi portfolio positions --agent ${resolvedAgent.id}${formatProtocolFlag(protocol)}`)} for full position pagination.`);
+                }
+                if (ordersResp.data.pagination.total > ordersResp.data.orders.length) {
+                    (0, ui_1.info)(`Use ${chalk_1.default.cyan(`suzi portfolio orders --agent ${resolvedAgent.id}${formatProtocolFlag(protocol)}`)} for full order pagination.`);
+                }
+                return;
             }
-            else {
-                (0, ui_1.error)(msg);
+            catch (err) {
+                spinner?.stop();
+                const errorMessage = err instanceof Error ? err.message : 'Unknown error';
+                if (jsonMode)
+                    (0, tty_1.outputJson)({ success: false, error: `Failed to load attributed portfolio: ${errorMessage}` });
+                else
+                    (0, ui_1.error)(`Failed to load attributed portfolio: ${errorMessage}`);
+                return;
             }
-            return;
         }
+        const showAllBalances = opts?.allBalances || false;
         const spinner = (0, tty_1.createSpinner)('Loading portfolio...', jsonMode);
         spinner?.start();
         try {
@@ -214,14 +495,12 @@ function registerPortfolioCommand(program) {
             const resp = await (0, api_1.get)(endpoint);
             if (!resp.ok) {
                 spinner?.stop();
-                const errorBody = resp.data;
+                const errorBody = getApiErrorBody(resp);
                 const msg = `Failed to load portfolio: ${errorBody?.message || errorBody?.error || 'Unknown error'}`;
-                if (jsonMode) {
+                if (jsonMode)
                     (0, tty_1.outputJson)({ success: false, error: msg });
-                }
-                else {
+                else
                     (0, ui_1.error)(msg);
-                }
                 return;
             }
             spinner?.stop();
@@ -238,6 +517,150 @@ function registerPortfolioCommand(program) {
             (0, ui_1.error)(`Failed to load portfolio: ${errorMessage}`);
         }
     });
+    portfolioCmd
+        .command('orders')
+        .description('View account-level or agent-attributed orders')
+        .option('--json', 'Output as JSON')
+        .option('--agent [agentIdOrName]', 'Filter to a specific agent; omit value to pick interactively')
+        .option('-p, --protocol <protocol>', 'Filter by protocol')
+        .option('--include <sections>', 'Include extra sections (currently: fills)')
+        .option('-s, --status <status>', 'Status filter (supports aliases like active/inactive or exact statuses like filled)')
+        .option('-n, --limit <limit>', 'Max rows to return', '50')
+        .option('--offset <offset>', 'Pagination offset', '0')
+        .action(async (opts) => {
+        const jsonMode = opts.json || false;
+        if (!(0, ui_1.requireAuth)(jsonMode))
+            return;
+        if (!ensureActiveAccount(jsonMode))
+            return;
+        const resolvedAgent = await resolveAgentOption(opts.agent, jsonMode);
+        if (opts.agent && !resolvedAgent)
+            return;
+        const includeResult = parseOrdersIncludeOption(opts.include);
+        if (!includeResult.ok) {
+            const msg = `Invalid include section: "${includeResult.invalid}". Valid values: fills`;
+            if (jsonMode)
+                (0, tty_1.outputJson)({ success: false, error: msg });
+            else
+                (0, ui_1.error)(msg);
+            return;
+        }
+        const params = buildSectionQuery({ ...opts, agent: resolvedAgent?.id, include: includeResult.value }, { limit: 50 });
+        const qs = params.toString();
+        const path = resolvedAgent
+            ? `/api/portfolio/agent-orders?${qs}`
+            : `/api/portfolio/orders?${qs}`;
+        const result = await fetchPortfolioSection(path, 'portfolio orders', jsonMode);
+        if (!result)
+            return;
+        if (jsonMode) {
+            (0, tty_1.outputJson)({ success: true, data: result });
+            return;
+        }
+        (0, ui_1.header)('Portfolio Orders');
+        console.log();
+        if (resolvedAgent) {
+            const data = result;
+            if (data.orders.length === 0 && (!data.fills || data.fills.length === 0)) {
+                (0, ui_1.info)('No attributed orders found.');
+                return;
+            }
+            if (data.orders.length > 0) {
+                renderAttributedOrdersSection(data.orders);
+                renderPaginationHint(data.pagination, data.orders.length, 'orders');
+            }
+            else {
+                (0, ui_1.info)('No attributed orders found.');
+            }
+            if (includeResult.value?.includes('fills')) {
+                console.log();
+                if (data.fills && data.fills.length > 0) {
+                    renderPortfolioFillsSection(data.fills);
+                    if (data.fillsPagination) {
+                        renderPaginationHint(data.fillsPagination, data.fills.length, 'fills');
+                    }
+                }
+                else {
+                    (0, ui_1.info)('No attributed fills found.');
+                }
+            }
+            return;
+        }
+        const data = result;
+        if (data.orders.length === 0 && (!data.fills || data.fills.length === 0)) {
+            (0, ui_1.info)('No orders found.');
+            return;
+        }
+        if (data.orders.length > 0) {
+            renderPortfolioOrdersSection(data.orders);
+            renderPaginationHint(data.pagination, data.orders.length, 'orders');
+        }
+        else {
+            (0, ui_1.info)('No orders found.');
+        }
+        if (includeResult.value?.includes('fills')) {
+            console.log();
+            if (data.fills && data.fills.length > 0) {
+                renderPortfolioFillsSection(data.fills);
+                if (data.fillsPagination) {
+                    renderPaginationHint(data.fillsPagination, data.fills.length, 'fills');
+                }
+            }
+            else {
+                (0, ui_1.info)('No fills found.');
+            }
+        }
+    });
+    portfolioCmd
+        .command('positions')
+        .description('View account-level or agent-attributed positions')
+        .option('--json', 'Output as JSON')
+        .option('--agent [agentIdOrName]', 'Filter to a specific agent; omit value to pick interactively')
+        .option('-p, --protocol <protocol>', 'Filter by protocol')
+        .option('-s, --status <status>', 'Status filter (supports aliases like open/closed or exact statuses like live)')
+        .option('-n, --limit <limit>', 'Max rows to return', '50')
+        .option('--offset <offset>', 'Pagination offset', '0')
+        .action(async (opts) => {
+        const jsonMode = opts.json || false;
+        if (!(0, ui_1.requireAuth)(jsonMode))
+            return;
+        if (!ensureActiveAccount(jsonMode))
+            return;
+        const resolvedAgent = await resolveAgentOption(opts.agent, jsonMode);
+        if (opts.agent && !resolvedAgent)
+            return;
+        const params = buildSectionQuery({ ...opts, agent: resolvedAgent?.id }, { limit: 50 });
+        const qs = params.toString();
+        const path = resolvedAgent
+            ? `/api/portfolio/agent-positions?${qs}`
+            : `/api/portfolio/positions?${qs}`;
+        const result = await fetchPortfolioSection(path, 'portfolio positions', jsonMode);
+        if (!result)
+            return;
+        if (jsonMode) {
+            (0, tty_1.outputJson)({ success: true, data: result });
+            return;
+        }
+        (0, ui_1.header)('Portfolio Positions');
+        console.log();
+        if (resolvedAgent) {
+            const data = result;
+            if (data.positions.length === 0) {
+                (0, ui_1.info)('No attributed positions found.');
+                return;
+            }
+            renderAttributedPositionsSection(data.positions);
+            renderPaginationHint(data.pagination, data.positions.length, 'positions');
+            return;
+        }
+        const data = result;
+        if (data.positions.length === 0) {
+            (0, ui_1.info)('No positions found.');
+            return;
+        }
+        renderPortfolioPositionsSection(data.positions);
+        renderPaginationHint(data.pagination, data.positions.length, 'positions');
+    });
     // Snapshots subcommand
     portfolioCmd
         .command('snapshots')
@@ -270,7 +693,7 @@ function registerPortfolioCommand(program) {
             const resp = await (0, api_1.get)(endpoint);
             if (!resp.ok) {
                 spinner?.stop();
-                const errorBody = resp.data;
+                const errorBody = getApiErrorBody(resp);
                 const msg = `Failed to load snapshots: ${errorBody?.message || errorBody?.error || 'Unknown error'}`;
                 if (jsonMode) {
                     (0, tty_1.outputJson)({ success: false, error: msg });