@lifi/perps-sdk 0.0.0-preview-34f3bb2 → 0.0.0-preview-2d69491

package/src/services/getMarket.ts

@@ -1,4 +1,6 @@
 import type { Market, MarketsResponse } from '@lifi/perps-types'
+import { PerpsErrorCode } from '@lifi/perps-types'
+import { PerpsError } from '../errors/PerpsError.js'
 import { buildUrl, request } from '../transport/request.js'
 import type { SDKRequestOptions } from '../types/config.js'
 import type { PerpsSDKClient } from '../types/provider.js'
@@ -21,10 +23,11 @@ export interface GetMarketParams {
 /**
  * Get a specific market by its opaque marketId. Requests the single market
  * from the backend `/markets` route (filtered by `marketIds`) and returns the
- * first match; the backend responds 404 (thrown as a {@link PerpsError}) when
- * nothing matches.
+ * first match; throws a `MarketNotFound` {@link PerpsError} both when the
+ * backend responds 404 and when it responds 200 with an empty list, matching
+ * `MarketRegistry.require`'s miss behaviour.
  *
- * @throws {PerpsError} On backend (e.g. 404), network, or parsing errors.
+ * @throws {PerpsError} On backend (e.g. 404 or empty match), network, or parsing errors.
  * @example
  * ```ts
  * const client = createPerpsClient({ integrator: 'my-app' })
@@ -48,5 +51,14 @@ export async function getMarket(
     {},
     options
   )
-  return markets[0]
+  const market = markets[0]
+  if (market === undefined) {
+    const error = new PerpsError(
+      PerpsErrorCode.MarketNotFound,
+      `No ${params.provider} market found for marketId '${params.marketId}'`
+    )
+    error.tool = params.provider
+    throw error
+  }
+  return market
 }

package/src/version.ts

@@ -1,2 +1,2 @@
 export const name = '@lifi/perps-sdk'
-export const version = '0.0.0-preview-34f3bb2'
+export const version = '0.0.0-preview-2d69491'

package/src/websocket/ReconnectingWebSocket.ts

@@ -238,16 +238,21 @@ export class ReconnectingWebSocket {
 
   /**
    * Send `data`, buffering it (bounded) for replay when the socket is not yet
-   * open.
+   * open. Throws once {@link ReconnectingWebSocket.close} has been called —
+   * there is no socket left to buffer against.
    *
    * @public
    */
   send(data: string) {
+    if (this.closed) {
+      throw new Error(CLOSED_ERROR)
+    }
     this.socket.send(data)
   }
 
   /**
-   * Permanently close the socket and suppress further reconnection.
+   * Permanently close the socket, suppress further reconnection, and set
+   * status to terminal `disconnected`.
    *
    * @public
    */
@@ -256,6 +261,7 @@ export class ReconnectingWebSocket {
     this.stopPing()
     this.clearWatchdog()
     this.socket.close()
+    this.setStatus('disconnected')
     for (const { reject } of this.readyResolvers) {
       reject(new Error(CLOSED_ERROR))
     }
@@ -300,8 +306,9 @@ export class ReconnectingWebSocket {
 
   /**
    * Current connection health. `reconnecting` until the first open;
-   * `disconnected` once auto-reconnect is abandoned (terminal until
-   * {@link ReconnectingWebSocket.reconnect} is called).
+   * `disconnected` once auto-reconnect is abandoned or {@link
+   * ReconnectingWebSocket.close} is called (terminal until {@link
+   * ReconnectingWebSocket.reconnect} is called).
    *
    * @public
    */

package/src/websocket/WsProviderBase.ts

@@ -94,7 +94,12 @@ export abstract class WsProviderBase<TSub = unknown> implements WsProvider {
       onStatus(this.rws.getStatus())
     }
 
+    let released = false
     return () => {
+      if (released) {
+        return
+      }
+      released = true
       if (onStatus) {
         const remaining = (this.statusListeners.get(onStatus) ?? 0) - 1
         if (remaining > 0) {
@@ -188,7 +193,15 @@ export abstract class WsProviderBase<TSub = unknown> implements WsProvider {
   }
 
   private async replaySubs(): Promise<void> {
-    for (const [key, state] of this.wireSubs) {
+    // Snapshot at replay start, and re-check the live registry before each
+    // send. An async sendSubscribe (e.g. Lighter's auth-token fetch) suspends
+    // the loop; status is already 'connected' while open listeners run, so a
+    // subscribe landing mid-replay is sent inline by registerSub — it must not
+    // be revisited here, and an entry unregistered mid-replay must be skipped.
+    for (const [key, state] of [...this.wireSubs]) {
+      if (this.wireSubs.get(key) !== state) {
+        continue
+      }
       // Isolate each resubscribe: a send can reject (e.g. an auth-token fetch
       // after reconnect), and one sub's failure must not abort the rest of
       // the loop or escape as an unhandled rejection.

package/src/websocket/types.ts

@@ -28,9 +28,9 @@ export type SubscriptionListener = (event: SubscriptionEvent) => void
  * - `connected` — socket open, live data flowing.
  * - `reconnecting` — socket dropped; auto-reconnect with backoff in progress.
  *   Data may be stale until it returns to `connected`.
- * - `disconnected` — reconnection abandoned after the retry cap. The
- *   connection is permanently dead and will not recover on its own; the
- *   subscription's data is stale. Terminal.
+ * - `disconnected` — reconnection abandoned after the retry cap, or the
+ *   connection was explicitly closed. The connection is permanently dead and
+ *   will not recover on its own; the subscription's data is stale. Terminal.
  *
  * @public
  */