From 6f6c0400f2a21dd106a958f72319f640195fb479 Mon Sep 17 00:00:00 2001
From: duna-akin <tuna.takin@gmail.com>
Date: Sun, 9 Aug 2026 13:55:47 -0400
Subject: [PATCH] fix(ts-sdk): add missing fee field to UserTrade (#1983)

The core engine (core/src/types.ts) and the Python SDK both expose
UserTrade.fee, and the generated sdks/typescript/API_REFERENCE.md already
documents it, but the TypeScript interface omitted it.

The value already arrives at runtime -- convertUserTrade() spreads the raw
response -- so TypeScript users could not reach a field that was in fact
present. Hyperliquid populates it today.

Placed between marketId and txHash to match the field order in core and the
Python SDK.

Closes #1983
---
 sdks/typescript/pmxt/models.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/sdks/typescript/pmxt/models.ts b/sdks/typescript/pmxt/models.ts
index 5db743cb..19dd6ca3 100644
--- a/sdks/typescript/pmxt/models.ts
+++ b/sdks/typescript/pmxt/models.ts
@@ -236,6 +236,9 @@ export interface UserTrade {
     /** Market ID */
     marketId?: string;
 
+    /** Trading fee paid by the user for this fill, when the venue exposes it. */
+    fee?: number;
+
     /** On-chain transaction hash (populated in hosted mode when the trade was settled on-chain). */
     txHash?: string | null;