Skip to content

feat(trade): --take-profit / --stop-loss perp flags #63

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
psmiratisu wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(trade): --take-profit / --stop-loss perp flags #63

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions src/commands/trade.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,8 @@ export function registerTradeCommands(program: Command): void {
" acp trade withdraw-from-hl --amount 25 # withdraw to Arbitrum\n" +
" acp trade --side long --token BTC --size 0.01 --leverage 5\n" +
" acp trade --side long --token BTC --size 0.01 --leverage 5 --dry-run # preview only\n" +
" acp trade --side long --token BTC --amount-usdc 100 --take-profit 130000 --stop-loss 80000 # open with TP/SL\n" +
" acp trade --side long --token BTC --stop-loss 80000 # set a stop on an existing long\n" +
" acp trade --amount-in 25 --chain-out hyperliquid # deposit (alias)\n" +
" acp trade --token AAPL --amount-usdc 50 # buy tokenized AAPL with USDC on Ethereum\n" +
" acp trade --token AAPL --token-in eth --chain-in 8453 --amount-in 0.02 # buy AAPL, funded by ETH on Base\n" +
Expand Down Expand Up @@ -281,6 +283,16 @@ export function registerTradeCommands(program: Command): void {
.option("--leverage <n>", "Set leverage for this token before a perp order")
.option("--isolated", "Use isolated margin when setting leverage", false)
.option("--reduce-only", "Only reduce an existing perp position", false)
.option(
"--take-profit <price>",
"Perp take-profit trigger price. With --size/--amount-usdc it attaches to the entry; alone it attaches to an existing position. Market close unless --take-profit-limit is set."
)
.option("--take-profit-limit <price>", "Rest the take-profit as a limit at this price instead of a market close")
.option(
"--stop-loss <price>",
"Perp stop-loss trigger price. With --size/--amount-usdc it attaches to the entry; alone it attaches to an existing position. Market close unless --stop-loss-limit is set."
)
.option("--stop-loss-limit <price>", "Rest the stop-loss as a limit at this price instead of a market close")
.option("--dry-run", "Preview the trade (route, size, margin, fees) without signing or submitting anything", false)
.action(async (opts, cmd) => {
const json = isJson(cmd);
Expand Down Expand Up @@ -393,6 +405,12 @@ async function runTrade(opts: Record<string, unknown>, json: boolean): Promise<v
fwd("amountShares", opts.amountShares);
fwd("protocol", opts.protocol);
fwd("chain", opts.chain);
// Perp TP/SL triggers. commander camelCases --take-profit → takeProfit, etc.
// The backend names them *Price / *LimitPrice.
fwd("takeProfitPrice", opts.takeProfit);
fwd("takeProfitLimitPrice", opts.takeProfitLimit);
fwd("stopLossPrice", opts.stopLoss);
fwd("stopLossLimitPrice", opts.stopLossLimit);
if (opts.postOnly) body.postOnly = true;
if (opts.reduceOnly) body.reduceOnly = true;
if (opts.isolated) body.isolated = true;
Expand Down