@lifi/perps-sdk-provider-lighter 1.5.2 → 1.5.4

package/src/signers/signActions.ts

@@ -231,6 +231,12 @@ async function requireApiKey(
  * step (e.g. approve) having mined. A reverted leg throws and aborts the
  * remaining legs. Each step's `txParams` carries chainId, target, function
  * name, args, and a human-readable abi from the backend.
+ *
+ * The wallet MUST already be on the leg's `txParams.chainId`: this signer
+ * broadcasts on the wallet's active chain and never switches it (switching is
+ * the integrator's responsibility — the widget drives it before signing). A leg
+ * whose target chain differs from the wallet's chain throws before broadcasting,
+ * so a token approve/deposit can never land on an unintended chain.
  * @internal
  */
 export async function signEvmTxActions(
@@ -256,6 +262,16 @@ export async function signEvmTxActions(
       abi: readonly string[]
     }
 
+    if (walletSigner.chain?.id !== params.chainId) {
+      throw new PerpsError(
+        PerpsErrorCode.SDKError,
+        `EVM_TX leg '${step.action}' targets chain ${params.chainId}, but the ` +
+          `connected wallet is on chain ${walletSigner.chain?.id ?? 'unknown'}. ` +
+          `Switch the wallet to chain ${params.chainId} before signing — the ` +
+          "SDK broadcasts on the wallet's active chain and does not switch it."
+      )
+    }
+
     const txHash = await walletSigner.writeContract({
       address: params.to,
       abi: parseAbi(params.abi),

package/src/websocket/LighterWsProvider.ts

@@ -112,9 +112,19 @@ interface SubState {
   needsAuth: boolean
 }
 
+/**
+ * A maintained book level, keyed in {@link OrderbookState} by its price string.
+ * `priceNum` is the price parsed once on insert so the emit-time sort orders by
+ * a cached number instead of re-parsing every price on every comparison.
+ */
+interface BookLevel {
+  size: string
+  priceNum: number
+}
+
 interface OrderbookState {
-  bids: Map<string, string>
-  asks: Map<string, string>
+  bids: Map<string, BookLevel>
+  asks: Map<string, BookLevel>
   assetId: string
 }
 
@@ -511,8 +521,7 @@ export class LighterWsProvider extends WsProviderBase<SubState> {
       return
     }
     const raw = collectAuthChannelItems<LtOrder>(msg, 'orders')
-    // A registry miss warns once and schedules a cooldown-gated refetch (the
-    // id may have listed after the snapshot); the order is skipped.
+    // Unknown market id (absent from the synced snapshot); the order is skipped.
     const data = classifyAndMapOrders(raw, (marketIndex) =>
       this.registry?.get(String(marketIndex))
     )
@@ -665,14 +674,8 @@ export class LighterWsProvider extends WsProviderBase<SubState> {
       data: {
         provider: this.providerKey,
         marketId: assetId,
-        bids: mapToLevels(
-          state.bids,
-          (a, b) => Number(b.price) - Number(a.price)
-        ),
-        asks: mapToLevels(
-          state.asks,
-          (a, b) => Number(a.price) - Number(b.price)
-        ),
+        bids: mapToLevels(state.bids, true),
+        asks: mapToLevels(state.asks, false),
         timestamp: Date.now(),
       },
     })
@@ -761,25 +764,34 @@ function isMarketStatsEntry(
 }
 
 function applyLevels(
-  book: Map<string, string>,
+  book: Map<string, BookLevel>,
   levels: LtWsOrderBook['bids']
 ): void {
   for (const level of levels) {
     if (level.size === '0' || Number(level.size) === 0) {
       book.delete(level.price)
     } else {
-      book.set(level.price, level.size)
+      const existing = book.get(level.price)
+      if (existing) {
+        existing.size = level.size
+      } else {
+        book.set(level.price, {
+          size: level.size,
+          priceNum: Number(level.price),
+        })
+      }
     }
   }
 }
 
 function mapToLevels(
-  book: Map<string, string>,
-  compare: (a: { price: string }, b: { price: string }) => number
+  book: Map<string, BookLevel>,
+  descending: boolean
 ): Array<{ price: string; size: string }> {
-  const levels = [...book].map(([price, size]) => ({ price, size }))
-  levels.sort(compare)
-  return levels
+  const entries = [...book].sort(([, a], [, b]) =>
+    descending ? b.priceNum - a.priceNum : a.priceNum - b.priceNum
+  )
+  return entries.map(([price, { size }]) => ({ price, size }))
 }
 
 /**