drop liquidated field from PaperOrderResult

Mirrors the real SignerClient shape where order submission returns only
fill/tx info and post-order state is queried separately. Callers detect
liquidation via get_position() returning None or get_health().has_been_liquidated.

Fixes AttributeError in paper_trading_health.py when an aggressive order
is liquidated on the same tick as the fill and get_position(0) returns None.

Author: kutay25

examples/paper_trading_health.py

@@ -87,13 +87,17 @@ async def main():
     # Show the contrast
     cons_liq = conservative.get_liquidation_price(0)
     aggr_liq = aggressive.get_liquidation_price(0)
-    aggr_mark = aggressive.get_position(0).mark_price
+    aggr_pos = aggressive.get_position(0)
     print()
     print("-" * 60)
     print("COMPARISON")
     cons_liq_str = "n/a (can't be liquidated)" if cons_liq == 0 else f"${cons_liq:.2f}"
     print(f"  Conservative liq price: {cons_liq_str}")
-    print(f"  Aggressive liq price:   ${aggr_liq:.2f}  (${aggr_mark - aggr_liq:.2f} below mark)")
+    if aggr_pos is None:
+        reason = "already liquidated" if aggressive.get_health().has_been_liquidated else "no open position"
+        print(f"  Aggressive position:    {reason}")
+    else:
+        print(f"  Aggressive liq price:   ${aggr_liq:.2f}  (${aggr_pos.mark_price - aggr_liq:.2f} below mark)")
     print("-" * 60)
 
     await api_client.close()

lighter/paper_client/client.py

@@ -184,7 +184,7 @@ async def create_paper_order(
                 avg_price = (
                     total_quote / total_filled_size if total_filled_size > 0 else 0.0
                 )
-                liquidated_markets = self._check_liquidation_and_update_metrics()
+                self._check_liquidation_and_update_metrics()
 
                 return PaperOrderResult(
                     order_type=request.order_type,
@@ -197,7 +197,6 @@ async def create_paper_order(
                     quote_amount=total_quote,
                     unfilled=unfilled,
                     timestamp=utc_now(),
-                    liquidated=request.market_id in liquidated_markets,
                 )
 
     def get_health(self) -> PaperAccountHealth:

lighter/paper_client/types.py

@@ -87,7 +87,6 @@ class PaperOrderResult:
     quote_amount: float
     unfilled: float
     timestamp: datetime
-    liquidated: bool = False
 
 
 @dataclass

test/paper_client/test_client.py

@@ -109,7 +109,7 @@ async def test_health_uses_cross_margin_across_markets(self) -> None:
         self.assertGreater(health.leverage, 0)
         self.assertLess(health.leverage, 2)
 
-    async def test_liquidated_result_is_scoped_to_request_market(self) -> None:
+    async def test_order_can_trigger_cross_market_liquidation(self) -> None:
         order_api = FakeOrderApi()
         order_api.details[1] = default_detail(1, "BTC")
         order_api.details[1].maintenance_margin_fraction = 500
@@ -140,11 +140,11 @@ async def test_liquidated_result_is_scoped_to_request_market(self) -> None:
             )
         )
 
-        self.assertFalse(result.liquidated)
+        self.assertIsNotNone(result)
         self.assertIsNone(client.get_position(0))
         self.assertIsNone(client.get_position(1))
 
-    async def test_request_market_liquidation_sets_result_flag(self) -> None:
+    async def test_request_market_liquidation_clears_position(self) -> None:
         order_api = FakeOrderApi()
         order_api.details[0].maintenance_margin_fraction = 500
         order_api.details[0].closeout_margin_fraction = 250
@@ -161,8 +161,9 @@ async def test_request_market_liquidation_sets_result_flag(self) -> None:
             PaperOrderRequest(0, PaperOrderSide.BUY, 0.1)
         )
 
-        self.assertTrue(result.liquidated)
+        self.assertIsNotNone(result)
         self.assertIsNone(client.get_position(0))
+        self.assertTrue(client.get_health().has_been_liquidated)
 
     async def test_mark_price_fallback_to_last_trade_price(self) -> None:
         order_api = FakeOrderApi()