fix: address Copilot review comments on PR #7

- bitmex-recipe-drawdown-circuit-breaker: fix intro to say "margin balance" not "wallet balance"
- bitmex-risk-operations: soften "always 0" realisedPnl claim to "may be 0"
- bitmex-order-types: fix "Behaviour" → "Behavior" for consistency
- bitmex-recipe-daily-pnl-report: fix currency grouping from .homeNotional (not a keyed object) to .currency
- bitmex-dca-strategy: fix avg_entry rounding from broken "round/100" to "(.*100|round)/100"
- bitmex-liquidation-guard: fix buffer_pct rounding same way — value was 100x too small
- bitmex-position-risk: fix gap_pct rounding same way

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

Author: yshing

skills/bitmex-dca-strategy/SKILL.md

@@ -100,7 +100,7 @@ bitmex execution trade-history --symbol XBTUSD --reverse --count 100 -o json 2>/
       avg_entry: (
         (map(.lastQty) | add) /
         (map(.lastQty / .lastPx) | add)
-        | round / 100
+        | (. * 100 | round) / 100
       )
     }
   '

skills/bitmex-liquidation-guard/SKILL.md

@@ -37,7 +37,7 @@ bitmex position list -o json 2>/dev/null | jq '
     symbol,
     markPrice,
     liquidationPrice,
-    buffer_pct: (((.markPrice - .liquidationPrice) / .markPrice * 100) | fabs | round / 100)
+    buffer_pct: (((.markPrice - .liquidationPrice) / .markPrice * 100) | fabs | (. * 100 | round) / 100)
   }]
 '
 ```

skills/bitmex-order-types/SKILL.md

@@ -126,7 +126,7 @@ bitmex order buy XBTUSD 100 --price 50000 --order-type Limit \
 
 BitMEX supports paired order logic via `contingencyType` and a shared `clOrdLinkID`. The CLI does not currently expose these as flags; use the REST API for these order types.
 
-| Type | Behaviour |
+| Type | Behavior |
 |---|---|
 | `OneCancelsTheOther` | OCO: when one order fills or triggers, the linked order is cancelled |
 | `OneTriggersTheOther` | OTO: when the first order fills, the linked order becomes active |

skills/bitmex-position-risk/SKILL.md

@@ -118,7 +118,7 @@ bitmex position list --symbol XBTUSD -o json 2>/dev/null | jq '
     {symbol, markPrice, liquidationPrice, gap_pct: null}
   else
     ((.markPrice - .liquidationPrice) / .markPrice * 100 | fabs) as $pct |
-    {symbol, markPrice, liquidationPrice, gap_pct: ($pct | round / 100)}
+    {symbol, markPrice, liquidationPrice, gap_pct: (($pct * 100 | round) / 100)}
   end
 '
 ```

skills/bitmex-recipe-daily-pnl-report/SKILL.md

@@ -32,8 +32,8 @@ bitmex execution trade-history --reverse --count 500 -o json \
 > P&L units depend on the instrument's settlement currency. XBt-settled instruments (e.g. XBTUSD) report in satoshis; divide by 1e8 for XBT. USDT-settled instruments (e.g. XBTUSDT) report in USDT units. Always group by `homeNotional` currency rather than summing across all symbols.
 
 ```bash
-jq 'group_by(.homeNotional | keys[0]) | map({
-  currency: (.[0].homeNotional | keys[0]),
+jq 'group_by(.currency) | map({
+  currency: .[0].currency,
   realisedPnl: ([.[].realisedPnl] | add // 0)
 })' /tmp/trades_today.json
 ```

skills/bitmex-recipe-drawdown-circuit-breaker/SKILL.md

@@ -5,7 +5,7 @@ description: Halt trading when portfolio drawdown exceeds threshold.
 
 # Drawdown Circuit Breaker
 
-Monitor wallet balance against a high-water mark and automatically flatten all positions when drawdown exceeds your configured threshold.
+Monitor margin balance against a high-water mark and automatically flatten all positions when drawdown exceeds your configured threshold.
 
 ## Prerequisites