fix: stop order arg order + Pacifica trade dedup

- trade stop: changed CLI arg order from <stopPrice> <size> to <size> <stopPrice>
  for consistency with market/limit commands (size before price)
- pacifica: deduplicate recent trades (API returns duplicate entries)
- Updated 3 stop order tests to match new arg order

Verified: 941/941 tests pass, dry-run stop size/price correct

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Hiksang

src/__tests__/commands/trade-commands.test.ts

@@ -267,7 +267,7 @@ describe("trade stop -- parameter correctness", () => {
   it("calls adapter.stopOrder with correct args and options", async () => {
     const adapter = mockAdapter();
     await run(adapter, [
-      "trade", "stop", "btc", "sell", "60000", "0.1",
+      "trade", "stop", "btc", "sell", "0.1", "60000",
       "--limit-price", "59500", "--reduce-only",
     ]);
 
@@ -283,7 +283,7 @@ describe("trade stop -- parameter correctness", () => {
 
   it("without --limit-price and --reduce-only: options have undefined values", async () => {
     const adapter = mockAdapter();
-    await run(adapter, ["trade", "stop", "btc", "sell", "60000", "0.1"]);
+    await run(adapter, ["trade", "stop", "btc", "sell", "0.1", "60000"]);
 
     expect(adapter.stopOrder).toHaveBeenCalledTimes(1);
     expect(adapter.stopOrder).toHaveBeenCalledWith(
@@ -297,7 +297,7 @@ describe("trade stop -- parameter correctness", () => {
 
   it("uppercases symbol for stop orders", async () => {
     const adapter = mockAdapter();
-    await run(adapter, ["trade", "stop", "eth", "buy", "3500", "1"]);
+    await run(adapter, ["trade", "stop", "eth", "buy", "1", "3500"]);
 
     expect(adapter.stopOrder).toHaveBeenCalledWith(
       "ETH",
@@ -963,7 +963,7 @@ describe("no unexpected extra adapter calls", () => {
 
   it("stop order only calls stopOrder", async () => {
     const adapter = mockAdapter();
-    await run(adapter, ["trade", "stop", "btc", "sell", "60000", "0.1"]);
+    await run(adapter, ["trade", "stop", "btc", "sell", "0.1", "60000"]);
 
     expect(adapter.stopOrder).toHaveBeenCalledTimes(1);
     expect(adapter.marketOrder).not.toHaveBeenCalled();

src/commands/trade.ts

@@ -489,11 +489,11 @@ export function registerTradeCommands(
   // === Stop / Trigger orders — Pacifica + Hyperliquid ===
 
   trade
-    .command("stop <symbol> <side> <stopPrice> <size>")
+    .command("stop <symbol> <side> <size> <stopPrice>")
     .description("Place a stop order")
     .option("--limit-price <price>", "Limit price (makes it stop-limit)")
     .option("--reduce-only", "Reduce only order")
-    .action(async (symbol: string, side: string, stopPrice: string, size: string, opts: { limitPrice?: string; reduceOnly?: boolean }) => {
+    .action(async (symbol: string, side: string, size: string, stopPrice: string, opts: { limitPrice?: string; reduceOnly?: boolean }) => {
       const s = side.toLowerCase();
       if (s !== "buy" && s !== "sell") errorAndExit("Side must be buy or sell");
 

src/exchanges/pacifica.ts

@@ -286,7 +286,15 @@ export class PacificaAdapter implements ExchangeAdapter {
 
   async getRecentTrades(symbol: string, _limit = 20): Promise<ExchangeTrade[]> {
     const trades = await this.client.getTrades(symbol);
-    return trades.slice(0, _limit).map((t) => ({
+    // Deduplicate by time+price+size+side (API sometimes returns duplicate entries)
+    const seen = new Set<string>();
+    const unique = trades.filter((t) => {
+      const key = `${t.created_at}:${t.price}:${t.amount}:${t.side}`;
+      if (seen.has(key)) return false;
+      seen.add(key);
+      return true;
+    });
+    return unique.slice(0, _limit).map((t) => ({
       time: Number(t.created_at ?? 0),
       symbol,
       side: t.side === "bid" ? "buy" as const : "sell" as const,